opencart 3.0.2.1b языки и требования

Тема в разделе "Обо всём", создана пользователем Chukcha, 18 дек 2017.

  1. Chukcha

    Chukcha Специалист

    Сообщения:
    3.013
    Симпатии:
    687
    гы...

    У меня сложилось впечатление , что Даниель смотрит в чужие модули,
    Одни из последних апдейтов вижу, что добавлено то, что должно было быть давно, на работу не влияло, но в некоторые моменты да.
     
  2. Chukcha

    Chukcha Специалист

    Сообщения:
    3.013
    Симпатии:
    687
    А это?
    getProducts

    Код:
    foreach ($query->rows as $result) {
                // for never get one more time with same product id
                if(!isset($product_data[$result['product_id']])){
                $product_data[$result['product_id']] = $this->getProduct($result['product_id']);
                }   
            }
    ЗАЧЕМ!!!!


    $sql .= " GROUP BY p.product_id";

    Этого недостаточно?
     
  3. Ravilr

    Ravilr Специалист

    Сообщения:
    4.209
    Симпатии:
    1.449
    Дело не в достаточности, а в нравится ему или нет )) Ну и скорей всего при каких то условиях не приводит к нужному результату.
     
  4. Chukcha

    Chukcha Специалист

    Сообщения:
    3.013
    Симпатии:
    687
    Я не смог придумать условий

    Такое возможно, если удалить group by
    Но тогда полетит пагинация..
     
  5. Ravilr

    Ravilr Специалист

    Сообщения:
    4.209
    Симпатии:
    1.449
    Возможно и удалит. Он же у себя тоже там придумывает и меняет. Пришла к нему мысль... сделал мерж и думает дальше.
     
  6. Chukcha

    Chukcha Специалист

    Сообщения:
    3.013
    Симпатии:
    687
    ЁЁЁЁ
    Это такая занимательная история смотреть за репозитарием
    Если кто понимает в керлинге меня поймет
     
  7. Vlad

    Vlad Эксперт

    Сообщения:
    4.161
    Симпатии:
    1.013
    :D
     
  8. Chukcha

    Chukcha Специалист

    Сообщения:
    3.013
    Симпатии:
    687
    Все чудейсатее и чудейсатее

    Явно эксперементирует
    Мы не ищем других решений
    Копипаст наше все..

    Модуле писателям посвящается

    $this->url->link(ROUTE, 'language=' . $this->config->get('config_language')),




    Код:
                 foreach ($parts as $part) {
    -                $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') . "'");
    +                $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') . "' AND language_id = '" . (int)$this->config->get('config_language_id') . "'");
    Т.е. вы посетили сайт на ру, у вас там кука сохранилась

    Затем, вы по ссылке с en пришли
    Опа! 404!!! Язык то не переключен, а из куки
    Как раз в этот момент есть возможность переключиться на язык по признаку языка. (url)

    Бум ждать когда отменит...
     
  9. Chukcha

    Chukcha Специалист

    Сообщения:
    3.013
    Симпатии:
    687
    Ура, закешировал

    Код:
            $query = $this->db->query("SELECT * FROM " . DB_PREFIX . "seo_regex ORDER BY sort_order ASC");
            $this->regex = $query->rows;
     
  10. Vlad

    Vlad Эксперт

    Сообщения:
    4.161
    Симпатии:
    1.013
    видимо
    :D
     
  11. Chukcha

    Chukcha Специалист

    Сообщения:
    3.013
    Симпатии:
    687
    фух, застыл, видимо много времени потратил на поиск/замену ссылок...
    Или пишет автоматический конвертер по всем файлам..

    А еще с преходом на новый формирователь

    Почувствуйте проблему переезда + еще один скрипт для переезда ссылок
     
  12. Chukcha

    Chukcha Специалист

    Сообщения:
    3.013
    Симпатии:
    687
    Печалько, таоарща с этими ЧПУ колбасит не по-деЦки..
    как было до сегодня
    Код:
            $url_data = $this->request->get;
    
            if (isset($url_data['route'])) {
                $route = $url_data['route'];
            } else {
                $route = 'common/home';
            }
    
            unset($url_data['_route_']);
    
            unset($url_data['route']);
    
            unset($url_data['language']);
    
            $url = '';
    
            if ($url_data) {
                $url = '&' . urldecode(http_build_query($url_data, '', '&'));
            }
    
            $data['languages'] = array();
    
            $this->load->model('localisation/language');
    
            $results = $this->model_localisation_language->getLanguages();
    
            foreach ($results as $result) {
                if ($result['status']) {
                    $data['languages'][] = array(
                        'name' => $result['name'],
                        'code' => $result['code'],
                        'href' => $this->url->link($route, 'language=' . $result['code'] . $url)
                    );
                }
            }
    
    Вроде все логично
    Т.е. в языковой переключателе - сразу ссылка на языковую

    Что сегодня?
    Код:
    'href' => $this->url->link('common/language/language', 'language=' . $result['code'] . '&redirect=' . urlencode($this->url->link($route, 'language=' . $result['code'] . $url)))
    Спрашивается! Зачем? Одно дело, когда это делалось POST, то теперь GET

    Ударьте по нему кто-нибудь.
     
  13. Ravilr

    Ravilr Специалист

    Сообщения:
    4.209
    Симпатии:
    1.449
    :D Хорошо сказано.

    Там с редиректом вроде муки какие то. Вот и перебирает.