Проблемы с соединением SOAP после обновления 1.9.3.0

12

Я обновил мой магазин Magento с 1.9.2.4 до 1.9.3.0

Мы используем программное обеспечение для доставки (Shipworks), которое подключается через пользователя SOAP / XML-RPC.

После обновления судовой регистрации логи показывают этот ответ в журналах:

<?xml version="1.0" encoding="UTF-8"?>
-<SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/">
-<SOAP-ENV:Body>
-<SOAP-ENV:Fault>
<faultcode>1</faultcode>
<faultstring>Internal Error. Please see log for details.</faultstring>
</SOAP-ENV:Fault>
</SOAP-ENV:Body>
</SOAP-ENV:Envelope>

Поэтому я пошел и включил регистрацию исключений в Magento и получил следующую ошибку:

2016-10-13T18:24:14+00:00 ERR (3): 
SoapFault exception: [1] Internal Error. Please see log for details. in /public_html/app/code/core/Mage/Api/Model/Server/Adapter/Soap.php:196
Stack trace:
#0 /public_html/app/code/core/Mage/Api/Model/Server/Handler/Abstract.php(140): Mage_Api_Model_Server_Adapter_Soap->fault('1', 'Internal Error....')
#1 /public_html/app/code/core/Mage/Api/Model/Server/Handler/Abstract.php(48): Mage_Api_Model_Server_Handler_Abstract->_fault('internal')
#2 /public_html/app/code/core/Mage/Api/Model/Server/Handler/Abstract.php(559): Mage_Api_Model_Server_Handler_Abstract->handlePhpError(4096, 'Argument 1 pass...', '/home/deepsix/p...', 559, Array)
#3 /public_html/app/code/core/Mage/Api/Model/Server/Handler/Abstract.php(299): Mage_Api_Model_Server_Handler_Abstract->processingMethodResult('<?xml version="...')
#4 [internal function]: Mage_Api_Model_Server_Handler_Abstract->call('ca4d34d100c92c8...', 'shipWorksApi.ge...', Array)
#5 /public_html/lib/Zend/Soap/Server.php(889): SoapServer->handle('<?xml version="...')
#6 /public_html/app/code/core/Mage/Api/Model/Server/Adapter/Soap.php(174): Zend_Soap_Server->handle()
#7 /public_html/app/code/core/Mage/Api/Model/Server.php(138): Mage_Api_Model_Server_Adapter_Soap->run()
#8 /public_html/app/code/core/Mage/Api/controllers/SoapController.php(40): Mage_Api_Model_Server->run()
#9 /public_html/app/code/core/Mage/Core/Controller/Varien/Action.php(418): Mage_Api_SoapController->indexAction()
#10 /public_html/app/code/core/Mage/Core/Controller/Varien/Router/Standard.php(254): Mage_Core_Controller_Varien_Action->dispatch('index')
#11 /public_html/app/code/core/Mage/Core/Controller/Varien/Front.php(172): Mage_Core_Controller_Varien_Router_Standard->match(Object(Mage_Core_Controller_Request_Http))
#12 /public_html/app/code/core/Mage/Core/Model/App.php(365): Mage_Core_Controller_Varien_Front->dispatch()
#13 /public_html/app/Mage.php(684): Mage_Core_Model_App->run(Array)
#14 /public_html/index.php(83): Mage::run('', 'store')
#15 {main}

Итак, я пошел и сделал diff между Abstract.php из 1.9.2.4 и 1.9.3.0 и получил следующее:

290a291
>                 $result = array();
292c293
<                     return $model->$method((is_array($args) ? $args : array($args)));
---
>                     $result = $model->$method((is_array($args) ? $args : array($args)));
294c295
<                     return $model->$method($args);
---
>                     $result = $model->$method($args);
296c297
<                     return call_user_func_array(array(&$model, $method), $args);
---
>                     $result = call_user_func_array(array(&$model, $method), $args);
297a299
>                 return $this->processingMethodResult($result);
403a406
>                     $callResult = array();
405c408
<                         $result[] = $model->$method((is_array($args) ? $args : array($args)));
---
>                         $callResult = $model->$method((is_array($args) ? $args : array($args)));
407c410
<                         $result[] = $model->$method($args);
---
>                         $callResult = $model->$method($args);
409c412
<                         $result[] = call_user_func_array(array(&$model, $method), $args);
---
>                         $callResult = call_user_func_array(array(&$model, $method), $args);
410a414
>                     $result[] = $this->processingMethodResult($callResult);
544a549,585
>     }
> 
>     /**
>      * Prepare Api data for XML exporting
>      * See allowed characters in XML:
>      * @link http://www.w3.org/TR/2000/REC-xml-20001006#NT-Char
>      *
>      * @param array $result
>      * @return mixed
>      */
>     public function processingMethodResult(array $result)
>     {
>         foreach ($result as &$row) {
>             if (!is_null($row) && !is_bool($row) && !is_numeric($row)) {
>                 $row = $this->processingRow($row);
>             }
>         }
>         return $result;
>     }
> 
>     /**
>      * Prepare Api row data for XML exporting
>      * Convert not allowed symbol to numeric character reference
>      *
>      * @param $row
>      * @return mixed
>      */
>     public function processingRow($row)
>     {
>         $row = preg_replace_callback(
>             '/[^\x{0009}\x{000a}\x{000d}\x{0020}-\x{D7FF}\x{E000}-\x{FFFD}\x{10000}-\x{10FFFF}]/u',
>             function ($matches) {
>                 return '&#' . Mage::helper('core/string')->uniOrd($matches[0]) . ';';
>             },
>             $row
>         );
>         return $row;

Любая помощь будет оценена.

LandonL
источник

Ответы:

14

Та же ошибка с другим расширением здесь. system.log говорит

Аргумент 1, передаваемый в Mage_Api_Model_Server_Handler_Abstract :: processingMethodResult (), должен иметь тип массива, заданная строка, вызываемая в app / code / core / Mage / Api / Model / Server / Handler / Abstract.php ...

Я думаю, что проблема в новом методе

Mage_Api_Model_Server_Handler_Abstract::processingMethodResult(array $result)

который принимает только массивы. Таким образом, каждая функция Api, возвращающая скалярное значение, выдаст эту ошибку. Для того, чтобы получить это происходит снова я скопировал app/code/core/Mage/Api/Model/Server/Handler/Abstract.phpв app/code/local/Mage/Api/Model/Server/Handler/Abstract.phpи исправлен processingMethodResult:

public function processingMethodResult($result)
{
    if (is_array($result)) {
        foreach ($result as &$row) {
            if (!is_null($row) && !is_bool($row) && !is_numeric($row)) {
                if (is_array($row)) {
                    $row = $this->processingMethodResult($row);
                } else {
                    $row = $this->processingRow($row);
                }
            }
        }
    } else {
        if (!is_null($result) && !is_bool($result) && !is_numeric($result)) {
            $result = $this->processingRow($result);
        }
    }
    return $result;
}
Белгор
источник
какую версию magento вы используете?
LandonL
Это решает проблему. Почему файл magento core в любом случае будет заменен на массив против строки?
LandonL
Отличное исправление, спасибо. Кто-нибудь поднимал это как ошибку в Magento? Многие люди будут укушены этим.
BlueC
Изменения касаются PHP7, см. Мой ответ ниже. Кроме того, stackoverflow.com/a/4103536/158325 это не совсем ошибка, но делает API-интерфейс PHP7 совместимым.
B00MER
1
@LandonL: я обновил с 1.9.2.4 до 1.9.3.0.
Белгор
1

Скорее всего, модуль ShipStation и / или версия PHP, с которой вы работаете, несовместимы:

Отгадывая усеченное сообщение об ошибке PHP, которое возвращается:

Argument 1 pass...' скорее всего Argument 1 passed to methodhere() must be an instance of string, string given

Какую версию PHP вы используете, и проконсультировались ли вы с ShipStation, чтобы узнать, есть ли у них какие-либо более новые версии и / или проблемы совместимости с последними версиями / исправлениями Magento.

Кроме того, вы можете добавить немного больше регистрации здесь: https://github.com/OpenMage/magento-mirror/blob/magento-1.9/lib/Zend/Soap/Server.php#L889, чтобы захватить больше усеченной ошибки PHP это возвращается, чтобы получить подтверждение, что это верная ошибка, которая возвращается.

Надеюсь это поможет.

B00MER
источник
Как бы я получил больше усеченной ошибки? Я не уверен, как увеличить длину линии.
LandonL
В настоящее время система работает под управлением php 5.6.25, сегодня я звонил на shipworks, но поскольку вчера вышло обновление magento 1.9.3.0, я не верю, что у них еще была возможность взглянуть на проблему.
LandonL
1

Ответ Бельгийцев действительно помог мне, но я немного изменил патч, чтобы добавить дополнительные объекты в репозиторий API.

Например, теперь вы получаете массивы объектов storeCredit и / или Giftcard с помощью вызова Magento XML-RPC для получения информации о заказе.

(код обновлен по предложению Бьёрна Тантау - для лучшей работы с объектами и коллекциями)

public function processingMethodResult($result)
{
    if (is_object($result) && is_callable(array($result, 'toArray'))) {
        $result = $result->toArray();
    }
    if (is_array($result)) {
        foreach ($result as &$row) {
            if (is_object($row) && is_callable(array($row, 'toArray'))) {
                $row = $row->toArray();
            }
            if (!is_null($row) && !is_bool($row) && !is_numeric($row)) {
                if (is_array($row)) {
                    $row = $this->processingMethodResult($row);
                } else {
                    $row = $this->processingRow($row);
                }
            }
        }
    } else {
        if (!is_null($result) && !is_bool($result) && !is_numeric($result)) {
            $result = $this->processingRow($result);
        }
    }
    return $result;
}
Кристоффер Бубах
источник