Jump to content
Sign in to follow this  
JohnnyVega

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

Recommended Posts

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

 

Пробывал разные варианты, но, видимо, моих знаний 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;
    }

Share this post


Link to post
Share on other sites

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

Share this post


Link to post
Share on other sites

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

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

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

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

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

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

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

Share this post


Link to post
Share on other sites

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

 

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

Share this post


Link to post
Share on other sites
order by price = 0, price

 

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

Share this post


Link to post
Share on other sites

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

Share this post


Link to post
Share on other sites

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

Share this post


Link to post
Share on other sites

Файл 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";
}

  • +1 1

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.