Jump to content
Sign in to follow this  
arshanskiyav

[OC 2.0.1.1] Неправильная сортировка

Recommended Posts

Доброго времени суток!

 

Заметил неприятную особенность (OC 2.0.1.1):

1. На сайте (список товаров в категории) неправильно работает сортировка, такое ощущение что сортировка идет по первым N символам (см.картинку).

2. При обновлении страницы (установлена сортировка по алфавиту ASC) порядок элементов меняется  :shock:

 

post-695214-0-14489300-1431547206_thumb.png

 

Сначала я подумал что это связано с кириллицей и латиницей в названии/модели, поэтому, как видно на рисунке, добавил товары с чисто цифровым названием/моделью/мета-тегом.

 

Результат тот же, а в админке сортировка работает правильно.

 

Подскажите где внести правки?

 

 

Спасибо.

Edited by arshanskiyav

Share this post


Link to post
Share on other sites

Сортировка идет по порядку

 

1103290730.png

Share this post


Link to post
Share on other sites

Попробуйте убрать LCASE

if ($data['sort'] == 'pd.name' || $data['sort'] == 'p.model') {
                $sql .= " ORDER BY LCASE(" . $data['sort'] . ")";

Share this post


Link to post
Share on other sites

Сортировка идет по порядку

 

 

Я правильно Вас понял, чтобы у меня сортировалось по Алфавиту, я должен вручную пронумеровать по этому полю?

Тогда логичный вопрос, зачем тогда в принципе возможность сортировки и почему в админ.панели все работает правильно?

Share this post


Link to post
Share on other sites

 

Попробуйте убрать LCASE

if ($data['sort'] == 'pd.name' || $data['sort'] == 'p.model') {
                $sql .= " ORDER BY LCASE(" . $data['sort'] . ")";

Я так полагаю, речь идет о функции getProducts в файле /catalog/model/catalog/product.php?

 

Если мне не изменяет память LCASE возвращает строку в нижнем регистре.

Share this post


Link to post
Share on other sites

я предположил, потому что в админке нет lcase

кроме того там еще интересная фишка

if (isset($data['sort']) && in_array($data['sort'], $sort_data)) {
			if ($data['sort'] == 'pd.name' || $data['sort'] == 'p.model') {
				$sql .= " ORDER BY LCASE(" . $data['sort'] . ")";
			} elseif ($data['sort'] == 'p.price') {
				$sql .= " ORDER BY (CASE WHEN special IS NOT NULL THEN special WHEN discount IS NOT NULL THEN discount ELSE p.price END)";
			} else {
				$sql .= " ORDER BY " . $data['sort'];
			}
		} else {
			$sql .= " ORDER BY p.sort_order";
		}

		if (isset($data['order']) && ($data['order'] == 'DESC')) {
			$sql .= " DESC, LCASE(pd.name) DESC";
		} else {
			$sql .= " ASC, LCASE(pd.name) ASC";

Т.е. получается что

 

ORDER BY LCASE (pd.name) DESC, LCASE (pd.name) DESC

или

ORDER BY LCASE (pd.name) ASC, LCASE (pd.name) ASC

 

поэтому нужно отсечь

if ($data['sort'] == 'pd.name' ) {
	if (isset($data['order']) && ($data['order'] == 'DESC') {
		$sql .= " DESC";
	} else {
		$sql .= " ASC";
	}
} else {
	if (isset($data['order']) && ($data['order'] == 'DESC')) {
		$sql .= " DESC, LCASE(pd.name) DESC";
	} else {
		$sql .= " ASC, LCASE(pd.name) ASC";
	}
}

Share this post


Link to post
Share on other sites

 

я предположил, потому что в админке нет lcase

кроме того там еще интересная фишка

if (isset($data['sort']) && in_array($data['sort'], $sort_data)) {
if ($data['sort'] == 'pd.name' || $data['sort'] == 'p.model') {
$sql .= " ORDER BY LCASE(" . $data['sort'] . ")";
} elseif ($data['sort'] == 'p.price') {
$sql .= " ORDER BY (CASE WHEN special IS NOT NULL THEN special WHEN discount IS NOT NULL THEN discount ELSE p.price END)";
} else {
$sql .= " ORDER BY " . $data['sort'];
}
} else {
$sql .= " ORDER BY p.sort_order";
}

if (isset($data['order']) && ($data['order'] == 'DESC')) {
$sql .= " DESC, LCASE(pd.name) DESC";
} else {
$sql .= " ASC, LCASE(pd.name) ASC";

Т.е. получается что

 

ORDER BY LCASE (pd.name) DESC, LCASE (pd.name) DESC

или

ORDER BY LCASE (pd.name) ASC, LCASE (pd.name) ASC

 

поэтому нужно отсечь

if ($data['sort'] == 'pd.name' ) {
	if (isset($data['order']) && ($data['order'] == 'DESC') {
		$sql .= " DESC";
	} else {
		$sql .= " ASC";
	}
} else {
	if (isset($data['order']) && ($data['order'] == 'DESC')) {
		$sql .= " DESC, LCASE(pd.name) DESC";
	} else {
		$sql .= " ASC, LCASE(pd.name) ASC";
	}
}

И все таки проблема не в LCASE (на всякий случай заменил весь блок), проблема, как мне кажется, кроется в конечном sql запросе:

 

/catalog/model/catalog/product.php:

$sql = "SELECT p.product_id, (SELECT AVG(rating) AS total FROM " . DB_PREFIX . "review r1 WHERE r1.product_id = p.product_id AND r1.status = '1' GROUP BY r1.product_id) AS rating, (SELECT price FROM " . DB_PREFIX . "product_discount pd2 WHERE pd2.product_id = p.product_id AND pd2.customer_group_id = '" . (int)$this->config->get('config_customer_group_id') . "' AND pd2.quantity = '1' AND ((pd2.date_start = '0000-00-00' OR pd2.date_start < NOW()) AND (pd2.date_end = '0000-00-00' OR pd2.date_end > NOW())) ORDER BY pd2.priority ASC, pd2.price ASC LIMIT 1) AS discount, (SELECT price FROM " . DB_PREFIX . "product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '" . (int)$this->config->get('config_customer_group_id') . "' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special";

и /admin/model/catalog/product.php:

$sql = "SELECT * FROM " . DB_PREFIX . "product p LEFT JOIN " . DB_PREFIX . "product_description pd ON (p.product_id = pd.product_id) WHERE pd.language_id = '" . (int)$this->config->get('config_language_id') . "'";

В админке нет выбора по категориям, поэтому запрос намного короче.

Я даже проверил количество символов в названии (увеличивал на один символ),  до 16 символов (цифры без пробелов) он правильно сортирует. Но я не вижу в запросе функций substr  или подобных.

Edited by arshanskiyav

Share this post


Link to post
Share on other sites

А что означает pd.name?

Точнее откуда он берется в функции getProducts?

В getProduct написано:

$query = $this->db->query("SELECT DISTINCT *, pd.name AS name ........ p.sort_order FROM " . DB_PREFIX . "product p LEFT JOIN " . DB_PREFIX . "product_description pd ON (p.product_id = pd.product_id) LEFT JOIN " . DB_PREFIX . "product_to_store p2s ON (p.product_id = p2s.product_id) LEFT JOIN " . DB_PREFIX . "manufacturer m ON (p.manufacturer_id = m.manufacturer_id) WHERE p.product_id = '" . (int)$product_id . "' AND pd.language_id = '" . (int)$this->config->get('config_language_id') . "' AND p.status = '1' AND p.date_available <= NOW() AND p2s.store_id = '" . (int)$this->config->get('config_store_id') . "'");

Выбрать pd.name как name из oc_product p(?), но я не вижу в таблице oc_product такого столбца, там даже нет столбца name.

Share this post


Link to post
Share on other sites

Это поле name из таблицы oc_product_description

Share this post


Link to post
Share on other sites

Это поле name из таблицы oc_product_description

Может я чего не догоняю.

В запросе написано поле pd.name из таблицы oc_product, почему тогда выборка идет из другой таблицы и по другому полю?

 

 

Я заметил много полей с различными префиксами, которых нет в таблицах.

Share this post


Link to post
Share on other sites

 pd.name из таблицы oc_product, почему тогда выборка идет из другой таблицы и по другому полю?

$sql .= " LEFT JOIN " . DB_PREFIX . "product_description pd ON ..."

Share this post


Link to post
Share on other sites

Кажется нашел:

		if (isset($data['sort']) && in_array($data['sort'], $sort_data)) {
			if ($data['sort'] == 'pd.name' || $data['sort'] == 'p.model') {
				//$sql .= " ORDER BY LCASE(" . $data['sort'] . ")";
			} elseif ($data['sort'] == 'p.price') {
				$sql .= " ORDER BY (CASE WHEN special IS NOT NULL THEN special WHEN discount IS NOT NULL THEN discount ELSE p.price END)";
			} else {
				$sql .= " ORDER BY " . $data['sort'];
			}
		} else {
			$sql .= " ORDER BY p.sort_order";
		}

		if (isset($data['order']) && ($data['order'] == 'DESC')) {
			$sql .= " DESC, LCASE(pd.name) DESC";
		} else {
			$sql .= " ASC, LCASE(pd.name) ASC";
		}

Строка номер 160, после комментирования все как-будто нормализовалось.

Соответственно сортировки по модели больше нет.

Share this post


Link to post
Share on other sites

Эм, ошибочка вышла, он сортировал правильно, только потому, что порядок добавления совпал с наименованием.

Share this post


Link to post
Share on other sites

Вопрос, я не силен в mysql, но можно ли сортировать таблицу по полю, которое не выбирается?

print_r($query);

stdClass Object ( 
[num_rows] => 15 [row] => Array ( [product_id] => 2439 [rating] => [discount] => [special] => ) 
[rows] => Array ( 
		[0] => Array ( [product_id] => 2439 [rating] => [discount] => [special] => ) 
		[1] => Array ( [product_id] => 2440 [rating] => [discount] => [special] => ) 
		[2] => Array ( [product_id] => 2441 [rating] => [discount] => [special] => ) 
		[3] => Array ( [product_id] => 1431 [rating] => [discount] => [special] => ) 
		[4] => Array ( [product_id] => 1464 [rating] => [discount] => [special] => ) 
		[5] => Array ( [product_id] => 1445 [rating] => [discount] => [special] => ) 
		[6] => Array ( [product_id] => 1458 [rating] => [discount] => [special] => ) 
		[7] => Array ( [product_id] => 1439 [rating] => [discount] => [special] => ) 
		[8] => Array ( [product_id] => 1452 [rating] => [discount] => [special] => ) 
		[9] => Array ( [product_id] => 1433 [rating] => [discount] => [special] => ) 
		[10] => Array ( [product_id] => 1465 [rating] => [discount] => [special] => ) 
		[11] => Array ( [product_id] => 1446 [rating] => [discount] => [special] => ) 
		[12] => Array ( [product_id] => 1459 [rating] => [discount] => [special] => ) 
		[13] => Array ( [product_id] => 1440 [rating] => [discount] => [special] => ) 
		[14] => Array ( [product_id] => 1453 [rating] => [discount] => [special] => ) 
		) 
)

Share this post


Link to post
Share on other sites

Да, конечно

Share this post


Link to post
Share on other sites

По ходу проблема в самой базе, выполнил:

SELECT * 
FROM  `oc_product_description` 
ORDER BY LCASE( `name` ) ASC 
LIMIT 30

см.скрин:

post-695214-0-00418200-1432020606_thumb.png

Share this post


Link to post
Share on other sites

Сделал дамп, скинул в базу Денвер (MySql 5.5.25), в ней сортировка была "правильной". Удалил базу у хостера, восстановил ее из дампа, и оно вроде как заработало.

Правда 4 по-прежнему больше 20 (такова логика MySQL).

Плюс выяснил, что у хостера криво работает ORDER BY LCASE(), пришлось использовать ORDER BY BINARY()

 

Я хочу прогнать pd.name через регулярное выражение, подставляющее лидирующие нули всем числам, в зависимости от длины этого числа (итоговая длина числа, допустим, 6 символов).

 

Кто-нибудь может подсказать, как правильно описать регулярное выражение, и возможно ли это?

Edited by arshanskiyav

Share this post


Link to post
Share on other sites

Правда 4 по-прежнему больше 20 (такова логика MySQL).

 

если поле текстовое, а не числовое - так и будет, все верно

чтобы 20 оказалось больше 4 - нужно преобразовать в число, например через int()

это конечно при условии, что у вас в поле только цифры

Share this post


Link to post
Share on other sites

если поле текстовое, а не числовое - так и будет, все верно

чтобы 20 оказалось больше 4 - нужно преобразовать в число, например через int()

это конечно при условии, что у вас в поле только цифры

Можно было бы и через:

ORDER BY (m_value+0)

Если бы число было в конце, или:

SELECT SUBSTRING_INDEX(a_title, ' ', -1) AS at FROM wp_5_posts ORDER BY at

Если бы число было одно, или во всех случаях известное/определенное количество раз. Но увы.

Share this post


Link to post
Share on other sites

Не самый лучший вариант решения, но все же.

 

При добавлении продукта (функция AddProduct,product.php,model,admin) добавляем лидирующие (ведущие) нули:

$pattern1="|([0-9][\d]*)|is";
$new_name=preg_replace_callback($pattern1,function($m) {return str_pad($m[0],5,"0", STR_PAD_LEFT);}, $value['name']);
$this->db->query("INSERT INTO " . DB_PREFIX . "product_description SET product_id = '" . (int)$product_id . "', language_id = '" . (int)$language_id . "', name = '" . $this->db->escape($new_name) . "',...

При выводе соответственно убираем (getProduct,product.php,model,catalog)

$pattern = "|\b[0]+([1-9][\d]*)|is";
$new_name=preg_replace($pattern,"\\1", $query->row['name']);
return array(
	'product_id'       => $query->row['product_id'],
	'name'             => $new_name,
...

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
You are posting as a guest. If you have an account, please sign in.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this  

  • Recently Browsing   0 members

    No registered users viewing this page.

×

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.