convead / joomla_convead Goto Github PK

При интеграции клиента столкнулся с 500-ками
JoomShopping 3.15.3
Joomla! 2.5.28

Подозрительные места, которые рождали ошибки:

1. JEventDispatcher::getInstance();
   https://github.com/Convead/joomla_convead/blob/master/convead.php#L76
   https://github.com/Convead/joomla_convead/blob/master/convead.php#L830
2. JFactory::getApplication()->getDocument()->addScriptDeclaration("
   вылечилось заменой на
   JFactory::getDocument()->addScriptDeclaration("
   https://github.com/Convead/joomla_convead/blob/master/convead.php#L104
   https://github.com/Convead/joomla_convead/blob/master/convead.php#L720
   https://github.com/Convead/joomla_convead/blob/master/convead.php#L751
   https://github.com/Convead/joomla_convead/blob/master/convead.php#L782
3. При оформлении заказа мне как гостю выдавался uid и все мои эвенты update_cart шли от зарегистрированного пользователя, а евенты link шли как от гостя (задваивание посетителей)
   Я думаю это происходит из-за того, что вот этот метод
   https://github.com/Convead/joomla_convead/blob/master/convead.php#L65
   вызывается внутри условия if($this->userId > 0)
   хотя он должен назначать юзеру uid вот тут
   https://github.com/Convead/joomla_convead/blob/master/convead.php#L654
   Наверное нужно его вынести из условия или еще что-то. Короче, надо разобраться. Тут какае-то архитектурная проблема.
4. При оформлении заказа не срабатывала функция
   https://github.com/Convead/joomla_convead/blob/master/convead.php#L535
   Пришлось ее переименовать на onAfterCreateOrder()
5. Этот метод вызывал ошибки
   https://github.com/Convead/joomla_convead/blob/master/convead.php#L644
   Быть может его можно заменить на JSFactory::getUserShop()