Problemas de conexão SOAP após a atualização 1.9.3.0

12

Atualizei minha Magento Store de 1.9.2.4 para 1.9.3.0

Utilizamos software de expedição (Shipworks) que se conecta por meio de um usuário SOAP / XML-RPC.

Após a atualização, o registro nas obras mostra esta resposta nos logs:

<?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>

Então, liguei o log de exceção no Magento e recebi o seguinte erro:

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}

Então, eu fiz um diff entre o Abstract.php de 1.9.2.4 e 1.9.3.0 e obtive o seguinte:

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;

Qualquer ajuda seria apreciada.

LandonL
fonte

Respostas:

14

Mesmo erro com outra extensão aqui. system.log diz

O argumento 1 passado para Mage_Api_Model_Server_Handler_Abstract :: processingMethodResult () deve ser do tipo array, sequência fornecida, chamada em app / code / core / Mage / Api / Model / Server / Handler / Abstract.php ...

Eu acho que o problema é o novo método

Mage_Api_Model_Server_Handler_Abstract::processingMethodResult(array $result)

que aceita apenas matrizes. Portanto, toda função da API que retorna um valor escalar gera esse erro. Para obter este curso novamente i copiado app/code/core/Mage/Api/Model/Server/Handler/Abstract.phppara app/code/local/Mage/Api/Model/Server/Handler/Abstract.phpe remendado 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;
}
Belgor
fonte
qual versão do magento você está executando?
LandonL 14/10
Isso resolve o problema. Por que o arquivo principal do magento seria alterado para array vs string de qualquer maneira aqui?
LandonL 14/10
Excelente correção, obrigado. Alguém levantou isso como um bug no Magento? Muitas pessoas vão ser mordidas por isso.
BlueC 14/10
A mudança é mais em torno do PHP7, veja minha resposta abaixo. Além disso, stackoverflow.com/a/4103536/158325 não é realmente um bug, mas torna a API PHP7 compatível.
B00MER
1
@LandonL: Atualizei de 1.9.2.4 para 1.9.3.0.
Belgor 15/10/16
1

Muito provavelmente o módulo ShipStation e / ou a versão PHP que você está executando não são compatíveis:

Adivinhando a mensagem de erro PHP truncada retornada:

Argument 1 pass...' é mais provável Argument 1 passed to methodhere() must be an instance of string, string given

Qual versão do PHP você está executando e consultou o ShipStation para verificar se há alguma versão mais nova e / ou problemas de compatibilidade com as versões / patches mais recentes do Magento.

Além disso, você pode adicionar um pouco mais de log aqui: https://github.com/OpenMage/magento-mirror/blob/magento-1.9/lib/Zend/Soap/Server.php#L889 para capturar mais erros de PHP truncados que está sendo retornado, para obter a validação de que é o erro correto que está sendo retornado.

Espero que isto ajude.

B00MER
fonte
Como obteria mais erros truncados? Não sei como aumentar o comprimento da linha.
LandonL
Atualmente, o sistema está executando o php 5.6.25, liguei para obras navais hoje, mas desde que a atualização magento 1.9.3.0 foi lançada ontem, não acredito que eles tenham tido a chance de examinar o problema ainda.
LandonL
1

A resposta dos belgas realmente me ajudou, mas acabei modificando um pouco o patch para permitir a inclusão de objetos extras no reposnse da API.

Por exemplo, agora você obtém as matrizes de objeto de armazenamento em cache e / ou cartão-presente do aheadWorks com a chamada Magento XML-RPC para obter informações sobre pedidos.

(código atualizado a partir da sugestão de Björn Tantau - para melhor trabalhar com objetos e coleções)

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;
}
Christoffer Bubach
fonte