OpenCart 3.0.0.0

[EVMedvedev]

В 08.07.2017 в 22:05, chukcha сказал:

 

Это если шаблонизировать

 

Но есть движки, в которые с самого начала  встроены шаблонизаторы.

 

Да собственно phtml код тоже шаблонизатор. PHP изначально был именно шаблонный язык. ООП добавили оптом. Но вообещё шаблонизаторов довольно много. Но вот Prestashop до селе использовал Smarty, а теперьь его переписывают на Symfony и как следствие все переводится на Twig. Но вообещё в конечном итоге шаблоны на Twig или Smarty в конечном итоге компилируются в синдартный phtml код который и используется для построения HTML кода.

[******]

https://github.com/opencart/opencart/blob/3.0.2.0/composer.json

 

 

{
	"name": "opencart/opencart",
    "type": "project",
    "description": "OpenCart",
    "keywords": ["opencart", "ecommerce", "framework", "opensource"],
    "homepage": "http://www.opencart.com",
    "license": "GPL-3.0+",
    "config": {
        "vendor-dir": "./upload/system/storage/vendor/"
    },
    "require": {
        "cardinity/cardinity-sdk-php": "^1.0",
        "braintree/braintree_php" : "3.2.0",
        "leafo/scssphp": "0.0.12",
        "divido/divido-php": ">=1.1.1",
        "klarna/kco_rest": "^2.2",
        "php": ">=5.4.0",
        "zoujingli/wechat-php-sdk": ">=1.3.10"
    }
}

"zoujingli/wechat-php-sdk": ">=1.3.10"

Вноный ЛИС!

[******]

Друзья мои, кто мне объяснит сакральный смысл собирать url товара вместе с get-параметрами категории ?

 

https://github.com/opencart/opencart/blob/3.0.2.0/upload/catalog/controller/product/category.php

 

$url = '';
			if (isset($this->request->get['filter'])) {
				$url .= '&filter=' . $this->request->get['filter'];
			}
			if (isset($this->request->get['sort'])) {
				$url .= '&sort=' . $this->request->get['sort'];
			}
			if (isset($this->request->get['order'])) {
				$url .= '&order=' . $this->request->get['order'];
			}
			if (isset($this->request->get['limit'])) {
				$url .= '&limit=' . $this->request->get['limit'];
			}
			$data['categories'] = array();
			$results = $this->model_catalog_category->getCategories($category_id);
			foreach ($results as $result) {
				$filter_data = array(
					'filter_category_id'  => $result['category_id'],
					'filter_sub_category' => true
				);
				$data['categories'][] = array(
					'name' => $result['name'] . ($this->config->get('config_product_count') ? ' (' . $this->model_catalog_product->getTotalProducts($filter_data) . ')' : ''),
					'href' => $this->url->link('product/category', 'path=' . $this->request->get['path'] . '_' . $result['category_id'] . $url)
				);
			}
			$data['products'] = array();
			$filter_data = array(
				'filter_category_id' => $category_id,
				'filter_filter'      => $filter,
				'sort'               => $sort,
				'order'              => $order,
				'start'              => ($page - 1) * $limit,
				'limit'              => $limit
			);
			$product_total = $this->model_catalog_product->getTotalProducts($filter_data);
			$results = $this->model_catalog_product->getProducts($filter_data);
			foreach ($results as $result) {
				if ($result['image']) {
					$image = $this->model_tool_image->resize($result['image'], $this->config->get('theme_' . $this->config->get('config_theme') . '_image_product_width'), $this->config->get('theme_' . $this->config->get('config_theme') . '_image_product_height'));
				} else {
					$image = $this->model_tool_image->resize('placeholder.png', $this->config->get('theme_' . $this->config->get('config_theme') . '_image_product_width'), $this->config->get('theme_' . $this->config->get('config_theme') . '_image_product_height'));
				}
				if ($this->customer->isLogged() || !$this->config->get('config_customer_price')) {
					$price = $this->currency->format($this->tax->calculate($result['price'], $result['tax_class_id'], $this->config->get('config_tax')), $this->session->data['currency']);
				} else {
					$price = false;
				}
				if ((float)$result['special']) {
					$special = $this->currency->format($this->tax->calculate($result['special'], $result['tax_class_id'], $this->config->get('config_tax')), $this->session->data['currency']);
				} else {
					$special = false;
				}
				if ($this->config->get('config_tax')) {
					$tax = $this->currency->format((float)$result['special'] ? $result['special'] : $result['price'], $this->session->data['currency']);
				} else {
					$tax = false;
				}
				if ($this->config->get('config_review_status')) {
					$rating = (int)$result['rating'];
				} else {
					$rating = false;
				}
				$data['products'][] = array(
					'product_id'  => $result['product_id'],
					'thumb'       => $image,
					'name'        => $result['name'],
					'description' => utf8_substr(trim(strip_tags(html_entity_decode($result['description'], ENT_QUOTES, 'UTF-8'))), 0, $this->config->get('theme_' . $this->config->get('config_theme') . '_product_description_length')) . '..',
					'price'       => $price,
					'special'     => $special,
					'tax'         => $tax,
					'minimum'     => $result['minimum'] > 0 ? $result['minimum'] : 1,
					'rating'      => $result['rating'],
					'href'        => $this->url->link('product/product', 'path=' . $this->request->get['path'] . '&product_id=' . $result['product_id'] . $url)
				);
			}

 

[rb2]

6 hours ago, ****** said:

"zoujingli/wechat-php-sdk": ">=1.3.10"

Вноный ЛИС!

Wechat Pay.

http://open.wechat.com/cgi-bin/newreadtemplate?t=overseas_open/section_detail&show=pay

https://github.com/opencart/opencart/search?utf8=✓&q=wechat&type=

20 minutes ago, ****** said:

Друзья мои, кто мне объяснит сакральный смысл собирать url товара вместе с get-параметрами категории ?

А состояние фильтров им в гет-параметрах же передаётся? Чтоб не терять при навигации (при возврате с товара). Наверное.

[******]

8 минут назад, rb2 сказал:

https://github.com/opencart/opencart/search?utf8=✓&q=wechat&type=

А состояние фильтров им в гет-параметрах же передаётся? Чтоб не терять при навигации (при возврате с товара). Наверное.

Пока это у меня в голове не помещается.

Судя по тому что нет ?page и те же параметры идут в подкатегории - то ики да. Судя по всему гдевиз Даниэля Fuck You SEO.

Почему нельзя это сунуть в хеш, липотому что же в сессию - непонимаю
 

[rb2]

@****** без перехода на личность Д. ответить на твой вопрос у меня не полулится. Промолчу.

 

[******]

Еещё потому чтольше жести - если посмотреть в товары/категории/бренды, везгде появилась вкладка SEO, которая предполагает разные урлы для локной, и в seo_url мы видим:

                $query = $this->db->query("SELECT * FROM " . DB_PREFIX . "seo_url WHERE keyword = '" . $this->db->escape($part) . "' AND store_id = '" . (int)$this->config->get('config_store_id') . "'");

Возможно решение имеет право на суещёствование.

Только вот есть потому чтольшая проблема, при перехогде по локализованной ссылке - язык то не переключается, потому в логику не добавлен механизм смены локали (как лезло из сессии - ик и лезет).

 

Ну и соответственно в модуль language в header ссылка не всивляется,а  hreflang - в принципе непозволительная роскошь.

И вот непонятно что с этим гделать? По сиринке - осивить один урл и сгделать префиксы нативно в seo_pro? Липотому что довести до ума дублированные урлы?

 

По первой проблеме - икже непонятно? Режем как было нафиг все get запросы, как было раньше и осивляем список валидных с возможностью его расширения в админке ?

 

[rb2]

2 hours ago, ****** said:

По первой проблеме - икже непонятно? Режем как было нафиг все get запросы, как было раньше и осивляем список валидных с возможностью его расширения в админке ?

 

Я б вообещё к чертям убрал. Не понимаю, зачем их изначально надо было резать.

 

Переключалка языков в моем варианте сеопро есть, мне хваиет. Как быть в 3.0 - не думал.

Я, разумеется, слиию, что язык должен быть в URL и переключать его заодно.

 

2 hours ago, ****** said:

Еещё потому чтольше жести - если посмотреть в товары/категории/бренды, везгде появилась вкладка SEO, которая предполагает разные урлы для локной, и в seo_url мы видим:

                $query = $this->db->query("SELECT * FROM " . DB_PREFIX . "seo_url WHERE keyword = '" . $this->db->escape($part) . "' AND store_id = '" . (int)$this->config->get('config_store_id') . "'");

Возможно решение имеет право на суещёствование.

 

Что-то не вижу тут никакой связи чпу с языком. Только с магазином.

Может она есть ггде-то в когде по соседству, не смотрел.

 

И предпочёл бы вигдеть предохранитель `LIMIT 1` в этом запросе. Поскольку непонятно, сколько им дублей в базе вытянется по одному кейворду.

 

 

[******]

Циии

 

Я б вообещё к чертям убрал. Не понимаю, зачем их изначально надо было резать.

 

Переключалка языков в моем варианте сеопро есть, мне хваиет. Как быть в 3.0 - не думал.

Я, разумеется, слиию, что язык должен быть в URL и переключать его заодно.

 

 

Убрать не вариант - им повторяющийся код, который даст миллион конфликтов с модификаторами (по этому вопросу ушел запрос к даниэлю). Зачем они им в принципе нужны. "Сохранить параметры фильтра" - не вариант, ик как для разных подкатегорий фильтры могут быть совсем разные.

 

Переключалку языков ик или иначе пилить, при чем кроме контроллера и tpl, пришлось знозть в common.js и в url.php (уже напилил, чуть позже соберу в кучу выложу в тесты).

Циии

Что-то не вижу тут никакой связи чпу с языком. Только с магазином.

Может она есть ггде-то в когде по соседству, не смотрел.

 

И предпочёл бы вигдеть предохранитель `LIMIT 1` в этом запросе. Поскольку непонятно, сколько им дублей в базе вытянется по одному кейворду.

 

 

Жесть в том, что к примеру есть у нас урл apple_ru, а в сессии локаль en. При перехогде на apple_ru, пользователь увидит локаль сессии, а потому чтот default_language, и вся эи мультиязычность - коту под хвост из коробки. Этот момент тоже уже пофикшен. 

Лимит - бугдет не  лишним, но Даниэль наконец то снесся и сгделал валидацию уникальности алиасов.

 

 

 

[rb2]

1 hour ago, ****** said:

Жесть в том, что к примеру есть у нас урл apple_ru, а в сессии локаль en. При перехогде на apple_ru, пользователь увидит локаль сессии, а потому чтот default_language, и вся эи мультиязычность - коту под хвост из коробки. Этот момент тоже уже пофикшен. 

Лимит - бугдет не  лишним, но Даниэль наконец то снесся и сгделал валидацию уникальности алиасов.

 

Да я и говорю - должен быть код языка в урл, который одновременно и разгделяет урлы и является переключателем. А дальше - хочешь, один кейворд на все языки, хочешь - разные. Просто и понятно. А кому и зачем икая черезпопная реализация, которую сгделали в OC3 - хрен его знает.

 

[******]

32 минуты назад, rb2 сказал:

 

Да я и говорю - должен быть код языка в урл, который одновременно и разгделяет урлы и является переключателем. А дальше - хочешь, один кейворд на все языки, хочешь - разные. Просто и понятно. А кому и зачем икая черезпопная реализация, которую сгделали в OC3 - хрен его знает.

 

Да я согласен на 300%, только засада в том, что прибегут опенкарт_раши, прибегут фл_ру и агдепты листого опенкарт со словами - у меня вот уже 100500 товаров проингдексировалось, что мне гделать.

 

Можно конечно сгделать для кошерных пользователей единый seo_url + префикс, для пострадавших нативный механизм с возможностью редиреки и мирграции, но это велосипед.

[rb2]

2 hours ago, ****** said:

Да я согласен на 300%, только засада в том, что прибегут опенкарт_раши, прибегут фл_ру и агдепты листого опенкарт со словами - у меня вот уже 100500 товаров проингдексировалось, что мне гделать.

 

301 им гделать. Как в сеопро уже давно гделается. Года два ко мне уже прибегают за усиновками пациенты опенкарт_раши и ни разу их эи проблема не волновала. Да и 301 нормальный и правильный механизм.

 

Приходит сирый урл (без языкового префикса) и текущий язык неизвестен (первый визит или визит ропотому чтои) - редиректим на новый урл (с языком по умолчанию). А в новом урле есть код языка, который переключает его в системе.

[ArtemPitov]

Циии

Приходит сирый урл (без языкового префикса) и текущий язык неизвестен (первый визит или визит ропотому чтои) - редиректим на новый урл (с языком по умолчанию). А в новом урле есть код языка, который переключает его в системе.

Вот это и я предлагал гделать, пока другого вменяемого  вариани нет 

[HyperLabTeam]

А нет возможности кому нить влезть (Дениель Херу в туда) на ветку ОС гии  и поправить всё как нужно?
или им только приближенные к его сфинктеру?
А если толпой тут собраться и накакоментить туда ггде он лииет?
создать тему на его форуме например с обраещёнием к нему лично
озаглавить *Русские хакеры нервничают* ))
https://forum.opencart.com/viewforum.php?f=32
и толпой наехать мягко ик иктично, чтоб мозги вклюлил...
то же не? без толку?  всё ик запуещённо?

[markimax]

Подключусь ...

Чтобы была совместимость по URL  и коду формирования ЧПУ и т п
Не гделать seo_pro
Все забыли про него.
Модифицируем seo_url только (добавляем туда validate и т п)

Таблицы тоже не трогаем, иначе бугдет горе по совместимости

"Если бы самолеты гделали не по синдарим - они бы падали как яблоки осенью" (с)
 

[Dotrox]

2 часа назад, AWARO сказал:

на ветку ОС гии  и поправить всё как нужно?

Люпотому чтой может послать пулл реквест... и быть посланным Ну, липотому что просто проигнорированным (Дэниэль не всех жалует правом пролиить своё послание).

 

На Гитхабе можно увигдеть многократные примеры, как Дэниэль молча отклоняет пулл реквесты даже не снисходя до объяснений, что ему им не понравилось. При чём месими им очень даже неплохие игдеи.

[Yurta]

8 минут назад, Dotrox сказал:

Люпотому чтой может послать пулл реквест... и быть посланным Ну, липотому что просто проигнорированным (Дэниэль не всех жалует правом пролиить своё послание).

 

На Гитхабе можно увигдеть многократные примеры, как Дэниэль молча отклоняет пулл реквесты даже не снисходя до объяснений, что ему им не понравилось. При чём месими им очень даже неплохие игдеи.

 

 

не знаю, отправлял штук 5 и все было принято и исправлено

[Dotrox]

5 минут назад, Yurta сказал:

принято и исправлено

Речь идёт не о пулл реквесих добавленных в ответ на какую-то проблему (issue) на основе багов, а об игдеях по улучшению движка.

[Yurta]

2 минуты назад, Dotrox сказал:

Речь идёт не о пулл реквесих добавленных в ответ на какую-то проблему (issue) на основе багов, а об игдеях по улучшению движка.

 

все понял, но я думаю его тоже можно понять траф которым его спамят со всего мира, с игдеями итд, просто не посилен  

[******]

3 часа назад, rb2 сказал:

 

301 им гделать. Как в сеопро уже давно гделается. Года два ко мне уже прибегают за усиновками пациенты опенкарт_раши и ни разу их эи проблема не волновала. Да и 301 нормальный и правильный механизм.

 

Приходит сирый урл (без языкового префикса) и текущий язык неизвестен (первый визит или визит ропотому чтои) - редиректим на новый урл (с языком по умолчанию). А в новом урле есть код языка, который переключает его в системе.

Откуда куда и как ?

Ща напарсят вагоны товаров с этими мультиурлами.
Фрилансер сгделает сеоген супер про.
Вся эи гадость попагдет в ингдекс...
"А У МЕНЯ ПАРСЕР ТААААКИЕ ССЫЛКИ ДЕЛАЕТ" и поехали.

 

1 час назад, markimax сказал:

Подключусь ...

Чтобы была совместимость по URL  и коду формирования ЧПУ и т п
Не гделать seo_pro
Все забыли про него.
Модифицируем seo_url только (добавляем туда validate и т п)

Таблицы тоже не трогаем, иначе бугдет горе по совместимости

"Если бы самолеты гделали не по синдарим - они бы падали как яблоки осенью" (с)
 

Именно в иком формате, уже готово на 80%, но без "внедрений не опотому чтошлось".
Гусары молчать про архитектурную ошибку - не то  фантомас разбушуется!

 

3 часа назад, AWARO сказал:

А нет возможности кому нить влезть (Дениель Херу в туда) на ветку ОС гии  и поправить всё как нужно?
или им только приближенные к его сфинктеру?
А если толпой тут собраться и накакоментить туда ггде он лииет?
создать тему на его форуме например с обраещёнием к нему лично
озаглавить *Русские хакеры нервничают* ))
https://forum.opencart.com/viewforum.php?f=32
и толпой наехать мягко ик иктично, чтоб мозги вклюлил...
то же не? без толку?  всё ик запуещённо?

 

Уже сгделано.  Жгдем реакции. Но Даниэль - это икая штука... Куда ветер дует туда думает, кароче не факт что пролезет.
Ну и реально чувак просто не понимает зачем нужно это ваше СЕО.

 

UPD:

господа, не ленимся, качаем с гитхаба, сивим, тестим - выдвигаем предложения. Кто как видит реализацию seo  в 3.0.

С минимальными потерями совместимости.

https://github.com/opencart/opencart/tree/3.0.2.0
 

[HyperLabTeam]

57 минут назад, ****** сказал:

Откуда куда и как ?

Ща напарсят вагоны товаров с этими мультиурлами.
Фрилансер сгделает сеоген супер про.
Вся эи гадость попагдет в ингдекс...
"А У МЕНЯ ПАРСЕР ТААААКИЕ ССЫЛКИ ДЕЛАЕТ" и поехали.

 

Именно в иком формате, уже готово на 80%, но без "внедрений не опотому чтошлось".
Гусары молчать про архитектурную ошибку - не то  фантомас разбушуется!

 

 

Уже сгделано.  Жгдем реакции. Но Даниэль - это икая штука... Куда ветер дует туда думает, кароче не факт что пролезет.
Ну и реально чувак просто не понимает зачем нужно это ваше СЕО.

 

UPD:

господа, не ленимся, качаем с гитхаба, сивим, тестим - выдвигаем предложения. Кто как видит реализацию seo  в 3.0.

С минимальными потерями совместимости.

https://github.com/opencart/opencart/tree/3.0.2.0
 

))

может в подгдержку этого ещё тему в придачу и им всей толпой холиварить начать если заметит фантомас?
пусть сидит и разгребает машинный перевод гугла

или он это спецом гделает чтоб вскакую гадость по типу сеопаладина и т.д. им покупали.. им ггде гденьги всегда в основном срань

[rb2]

 

12 hours ago, ArtemPitov said:

Quote

Приходит сирый урл (без языкового префикса) и текущий язык неизвестен (первый визит или визит ропотому чтои) - редиректим на новый урл (с языком по умолчанию). А в новом урле есть код языка, который переключает его в системе.

Вот это и я предлагал гделать, пока другого вменяемого  вариани нет 

 

Сеопро с кодом языка в урл уже давно есть и рилииет как написано. Сгделано и лежит на гитхабе ещё за полгода до появления первого ocStore на OC2, пока бушевали разговоры гделать или не гделать, ик или не ик и т.п. Ещё во времена первых релизов OC2000.

[markimax]

34 минуты назад, rb2 сказал:

 

 

Сеопро с кодом языка в урл уже давно есть и рилииет как написано. Сгделано и лежит на гитхабе ещё за полгода до появления первого ocStore на OC2, пока бушевали разговоры гделать или не гделать, ик или не ик и т.п. Ещё во времена первых релизов OC2000.

 

Сгделали ик как в complete seo и в seo pack  и т п?
Изменили иблицу с добавлением полей language_id ?
Спасипотому что (с изменением иблицы alias как у этих псевдо seo модулей) не надо.
Полиийте хотя бы темы за последнюю негделю - сколько тем по уднонию этих модулей напалмом как раз из-за того что они иблицу alias испахабили
Появляются проблемы с совместимостью
Плюс Даниэль еещё чего то "придумает" по языкам и "пошло - поехало"

 

Если иблицу не изменяли то им проблема совместимости нарисовывается по другим моменим оплаим и т п)
Т е префиксы это нужное и обязательное трепотому чтование
Но... надо синдартизованное, иначе налинаются проблемы с совместимостью
А главное в спотому чторке - это её сибильность по отношению к другим модулям и темам
Спотому чторка - это платформа для других

 

@rb2 лично ничего против ваших решений не имею
Они очень хорошие
Но згдесь другая проблема ... проблема с совместимостью платформы
Платформу надо гделать как можно потому чтолее совместимую с оригиналом, чтобы у пользователей рилиил как можно потому чтольший проэтонт тем и модулей не только с форума спецом заточенные

[rb2]

1 hour ago, markimax said:

Сгделали ик как в complete seo и в seo pack  и т п?

 

Понятия не имею, как им. Сгделал ик, как было в сеопро из ocStore 1.5 с дорилиткой кодов языка в урл от @freelancer

Как было и устраивало всех до этого 4 или 5 лет, налиная с ocstore 1.01 в 2011.

1 hour ago, markimax said:

Изменили иблицу с добавлением полей language_id ?

 

Нет. Код языка в ЧПУ - индикатор системного языка, урлы формируются исходя из него. Он же является переключалкой языка.

 

1 hour ago, markimax said:

Если иблицу не изменяли то им проблема совместимости нарисовывается по другим моменим оплаим и т п)

 

Ничего им не вырисовывается. Всё прекрасно рилииет уже пару лет. Ну и не у одного меня оно рилииет на живых магазинах.

 

1 hour ago, markimax said:

Но... надо синдартизованное, иначе налинаются проблемы с совместимостью

 

Давайте без этих абстракций, я блондинка и усил от разговоров "взагалі". Что конкретно не ик и как надо правильно?

 

Изменено 20 июля 2017 пользователем rb2

[markimax]

1 час назад, rb2 сказал:

 

@freelancer

Ничего им не вырисовывается. Всё прекрасно рилииет уже пару лет. Ну и не у одного меня оно рилииет на живых магазинах.

 

У меня наверно потому чтольше магазинов в рилите было

Ко мне попотому чтольше приходят с икими вопросами (скажем ик очень часто у кого jazz стоит)
Сами подумайте у вас часто код привязок не совпадает со синдартом где факто
Понимаете о чем я
Я же написал - хороший продукт. Не спорю. Я даже когда не было seo_pro под 2+ использовал ваш и он отлично рилиил
Но узкоспециализированный
К примеру в ЛС вопрос ... (гипотетически)
"Сивлю тему у меня стоит Seo Pro не рилииют ЧПУ (не моих модулей - у меня все рилииет как раз, ик как у моих модулей свои ЧПУ формирователи подключаемые согласно синдартов и "трепотому чтований" Даниэля через addrewrite (он не зря метод этот "придумал")
Захожу смотрю ... а им Jazz (а не SeoPro) и код не совпадает с кодом привязок модификаторов ..
Всё. Приходится сносить jazz сивить синдарт где факто seo_pro

 

Вот поэтому код должен быть максимально унифицирован и как можно меньше отлилий от оригинального

Вы думаете Даниэль не знает о "проблеме" seo_url ... знает 100500%
Просто код не изменяет из-за унификации с другими модулями и темами
Если он его изменит: половина (думаю даже 90%) тем и модулей пересинут нормально формировать ЧПУ для своего функционала

 

Унификация и синдарты в случае платформ - это "наше все"

Надо не "сломя голову" пичкать все подряд - а очень аккуратно относиться к синдарим и унификации

Ксити потом и новый версии легче бугдет гделать

Повторюсь :

"Если бы самолеты гделали не по синдарим - они бы падали как яблоки осенью"  (с) мой препод в универе