yoomoney / cms-prestashop

YooKassa payment module for PrestaShop
https://yookassa.ru/developers
11 stars 9 forks source link

Конфликт с модулем Advanced SEO Friendly URLs #1

Closed nextonru closed 6 years ago

nextonru commented 6 years ago

При установке модуля выскакивает ошибка - Не получается установить override: Метод setRequestUri класса Dispatcher

этот метод уже есть в модуле Advanced SEO Friendly URLs (последняя версия 2.1.1) Prestashop 1.7.2.4

protected function setRequestUri()
{
    parent::setRequestUri();
    $remove_enabled = Configuration::get('FSAU_REMOVE_DEFAULT_LANG');
    $current_iso_lang = Tools::getValue('isolang');
    if ($this->use_routes && Language::isMultiLanguageActivated() && !$current_iso_lang && $remove_enabled) {
        $_GET['isolang'] = Language::getIsoById(Configuration::get('PS_LANG_DEFAULT'));
    }
}

как их можно совместить чтобы оба работали?

corsik commented 6 years ago

Только самому разбирать function setRequestUri

nextonru commented 6 years ago

Мне посоветовали удалить папку override/classes для устранения конфликта, после этого установить модуль, и модуль заработал

corsik commented 6 years ago

Вам дали очень плохой совет, вы удалили все остальные расширения классов которые были туда до этого скопированы другими модулями, в итоге рано или поздно вы наткнетесь на проблемы с работой тех модулей которым так же были необходимы эти расширения!

Listware commented 6 years ago

У вас просто конфликт функций. Если обозвать функцию по другому и везде заменить её вызов - нормально отрабатывает

corsik commented 6 years ago

@Listware не говорите ерунду. Папка override необходима специально, для расширения или изменения функционала действующий классов и их функций.

Listware commented 6 years ago

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

corsik commented 6 years ago

@Listware Опять же ты противоречишь тому, для чего нужен override

mossad-zika commented 6 years ago

@Listware скорее всего вообще не пользовался этим модулем, тем более на новой Presta