Перейти к содержанию
JohnnyVega

Товары с "нулевой" ценой в конце списка

Рекомендуемые сообщения

Всех приветствую.
 
Есть такая проблема. Нужно, чтобы в категории при сортировке товаров по цене по возрастанию, товары с нулевой ценой были в конце списка.

 

Пробывал разные варианты, но, видимо, моих знаний MySql не хватает.

 

Буду благодарен, если кто-нибудь поможет решить проблему или направит на путь истинный.



 
На входе:
 
Opencart 1.5.3.1
 
Кусок кода из файла catalog/model/catalog/product.php с функцией getProducts($data = array())
 
Как Вы видите, код немного изменён. В начале функции изменён sql - запрос, чтобы вытягивать discount и special, и в конце запроса, там где сортировка, запрос тоже изменён, чтобы использовать discount и special. Собственно, изменения взяты из более поздних версий движка.
 
Ниже кусок кода. Комментом выделено то место, где скорее всего потребуются изменения.
      // Sorting by price with special and discount
      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";    
      }
      // Sorting by price with special and discount

Код функции getProducts($data = array())

 

public function getProducts($data = array()) {
        if ($this->customer->isLogged()) {
            $customer_group_id = $this->customer->getCustomerGroupId();
        } else {
            $customer_group_id = $this->config->get('config_customer_group_id');
        }    
        
        $cache = md5(http_build_query($data));
        
        $product_data = $this->cache->get('product.' . (int)$this->config->get('config_language_id') . '.' . (int)$this->config->get('config_store_id') . '.' . (int)$customer_group_id . '.' . $cache);
    
        if (!$product_data) {
            //Original
      //$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 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)";
            
      // price with special and discount
      $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)$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)$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 
   
        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)
      ";
      // price with special and discount
   
            if (!empty($data['filter_tag'])) {
                $sql .= " LEFT JOIN " . DB_PREFIX . "product_tag pt ON (p.product_id = pt.product_id)";            
            }
                        
            if (!empty($data['filter_category_id'])) {
                $sql .= " LEFT JOIN " . DB_PREFIX . "product_to_category p2c ON (p.product_id = p2c.product_id)";            
            }
            
            $sql .= " WHERE 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') . "'";
            
            if (!empty($data['filter_name']) || !empty($data['filter_tag'])) {
                $sql .= " AND (";
                                            
                if (!empty($data['filter_name'])) {
                    $implode = array();
                    
                    $words = explode(' ', trim(preg_replace('/\s\s+/', ' ', $data['filter_name'])));
                    
                    foreach ($words as $word) {
                        if (!empty($data['filter_description'])) {
                            $implode[] = "LCASE(pd.name) LIKE '%" . $this->db->escape(utf8_strtolower($word)) . "%' OR LCASE(pd.description) LIKE '%" . $this->db->escape(utf8_strtolower($word)) . "%'";
                        } else {
                            $implode[] = "LCASE(pd.name) LIKE '%" . $this->db->escape(utf8_strtolower($word)) . "%'";
                        }                
                    }
                    
                    if ($implode) {
                        $sql .= " " . implode(" OR ", $implode) . "";
                    }
                }
                
                if (!empty($data['filter_name']) && !empty($data['filter_tag'])) {
                    $sql .= " OR ";
                }
                
                if (!empty($data['filter_tag'])) {
                    $implode = array();
                    
                    $words = explode(' ', trim(preg_replace('/\s\s+/', ' ', $data['filter_tag'])));
                    
                    foreach ($words as $word) {
                        $implode[] = "LCASE(pt.tag) LIKE '%" . $this->db->escape(utf8_strtolower($word)) . "%'";
                    }
                    
                    if ($implode) {
                        $sql .= " " . implode(" OR ", $implode) . " AND pt.language_id = '" . (int)$this->config->get('config_language_id') . "'";
                    }
                }
            
                $sql .= ")";
            }
            
            if (!empty($data['filter_category_id'])) {
                if (!empty($data['filter_sub_category'])) {
                    $implode_data = array();
                    
                    $implode_data[] = "p2c.category_id = '" . (int)$data['filter_category_id'] . "'";
                    
                    $this->load->model('catalog/category');
                    
                    $categories = $this->model_catalog_category->getCategoriesByParentId($data['filter_category_id']);
                                        
                    foreach ($categories as $category_id) {
                        $implode_data[] = "p2c.category_id = '" . (int)$category_id . "'";
                    }
                                
                    $sql .= " AND (" . implode(' OR ', $implode_data) . ")";            
                } else {
                    $sql .= " AND p2c.category_id = '" . (int)$data['filter_category_id'] . "'";
                }
            }        
                    
            if (!empty($data['filter_manufacturer_id'])) {
                $sql .= " AND p.manufacturer_id = '" . (int)$data['filter_manufacturer_id'] . "'";
            }
            
            $sql .= " GROUP BY p.product_id";
            
            $sort_data = array(
                'pd.name',
                'p.model',
                'p.quantity',
                'p.price',
                'rating',
                'p.sort_order',
                'p.date_added'
            );    
            
      /*
      Old. Original
            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'] . ")";
                } else {
                    $sql .= " ORDER BY " . $data['sort'];
                }
            } else {
                $sql .= " ORDER BY p.sort_order";    
            }
      Old. Original
      */
      
      // Sorting by price with special and discount
      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";    
      }
      // Sorting by price with special and discount
            
            if (isset($data['order']) && ($data['order'] == 'DESC')) {
                $sql .= " DESC, LCASE(pd.name) DESC";
            } else {
                $sql .= " ASC, LCASE(pd.name) ASC";
            }
        
            if (isset($data['start']) || isset($data['limit'])) {
                if ($data['start'] < 0) {
                    $data['start'] = 0;
                }                
    
                if ($data['limit'] < 1) {
                    $data['limit'] = 20;
                }    
            
                $sql .= " LIMIT " . (int)$data['start'] . "," . (int)$data['limit'];
            }
            
            $product_data = array();
                    
            $query = $this->db->query($sql);
 
            foreach ($query->rows as $result) {
                $product_data[$result['product_id']] = $this->getProduct($result['product_id']);
            }
            
            $this->cache->set('product.' . (int)$this->config->get('config_language_id') . '.' . (int)$this->config->get('config_store_id') . '.' . (int)$customer_group_id . '.' . $cache, $product_data);
        }
        
        return $product_data;
    }

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Чтобы решить исходную задачу, надо сделать UNION результатов двух запросов:

- с сортировкой части товаров по "finalprice ASC WHERE finalprice > 0"

- и оставшейся частью товаров "WHERE finalprice = 0"

(где `finalprice` -- это одна из трёх цен, price/special/discount; схема их использования есть в приведенном коде).

Поэтому малой кровью, вмешавшись лишь в одно место сборки длинного запроса, эту проблему не решить.

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

Хотя надо посмотреть: может быть не будет проблемой просто добавить в существующих код условие про цену больше нуля, а оставшиеся "нулевые" товары прицеплять в конце? Если такое упрощение прокатит, можно было бы в самом конце ф-ции формировать финальный запрос, не изменяя значительно весь код и не дублируя эту простыню.

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Чтобы решить исходную задачу, надо сделать UNION результатов двух запросов:

 

rb2, спасибо за ответ. Вариант с Union рассматривался, но пока отложился по таким же причинам трудоёмкости.

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
order by price = 0, price

 

Гуглить я тоже умею :-) . В боевых условиях пробовал конкретно на этом запросе? 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

код с рабочего магазина. а в чем проблема?

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Ну, наверное, в том, что этот вариант не работает в общем случае для указанного кода

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Не удалось решить вопрос?

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Файл catalog/model/catalog/product.php, метод getProducts(), строка 166

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['sort']) && in_array($data['sort'], $sort_data)) {
    if ($data['sort'] == 'pd.name' || $data['sort'] == 'p.model') {
        $sql .= " ORDER BY IF(p.price = '0', 1, 0), LCASE(" . $data['sort'] . ")";
    } elseif ($data['sort'] == 'p.price') {
        $sql .= " ORDER BY IF(p.price = '0', 1, 0), (CASE WHEN special IS NOT NULL THEN special WHEN discount IS NOT NULL THEN discount ELSE p.price END)";
    } else {
        $sql .= " ORDER BY IF(p.price = '0', 1, 0), " . $data['sort'];
    }
} else {
    $sql .= " ORDER BY IF(p.price = '0', 1, 0), p.sort_order";
}

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Для публикации сообщений создайте учётную запись или авторизуйтесь

Вы должны быть пользователем, чтобы оставить комментарий

Создать учетную запись

Зарегистрируйте новую учётную запись в нашем сообществе. Это очень просто!

Регистрация нового пользователя

Войти

Уже есть аккаунт? Войти в систему.

Войти

  • Последние посетители   0 пользователей онлайн

    Ни одного зарегистрированного пользователя не просматривает данную страницу

×

Важная информация

На нашем сайте используются файлы cookie и происходит обработка некоторых персональных данных пользователей, чтобы улучшить пользовательский интерфейс. Чтобы узнать для чего и какие персональные данные мы обрабатываем перейдите по ссылке. Если Вы нажмете «Я даю согласие», это означает, что Вы понимаете и принимаете все условия, указанные в этом Уведомлении о Конфиденциальности.