Chave do array faltando na verificação do retorno do erro

[rodrigopeixotobr]

Olá! A função process_error_body() da classe Entity ao chamar o RecuperableError tenta pegar o elemento error da $message, mas ele nem sempre existe.

function process_error_body($message){
        $recuperable_error = new RecuperableError(
            $message['message'],
            $message['error'],
            $message['status']
        );
...

No retorno na falha da criação de uma assinatura por exemplo, não possui o elemento error.
Ao enviar dados incorretos de um cartão, o retorno é este:

(
[message] => CC_VAL_433 Credit card validation has failed
[code] => cc_rejected_high_risk
[status] => 401
)

Ou esse retorno aqui também sem o error que é quando é enviado um cardToken inválido.

(
[message] => Card token service not found
[status] => 404
)

Então ocorre o erro Undefined index: error e acaba não retornando o real motivo da transação não ter sido aprovada.
É necessário algo como uma verificação se possui o [error] no $message e se não, se tem o [code], e se não tem nenhum dos dois.

[lucmantovani]

Holá, @rodrigopeixotobr!
Obrigado pela observação!

O PR #329 provavelmente irá solucionar o problema relatado, poderia verificar por favor?
Caso tenha sugestões diferentes da proposta pelo PR, sua contribuição será bem-vinda 🙂

[rodrigopeixotobr]

Olá @lucmantovani

O PR #329 se mostrou resolver o poblema, foram realizados os mesmos testes para receber retornos com erros e a process_error_body funcionou, retornando o atributo error com a instância da classe MercadoPago\RecuperableError.

Porém, outro problema foi observado, a função process_error_body está esperando que a $message tenha a chave ['cause'] para processar as causas do erro, mas as causas do erro vieram na chave ['cause_error'], então ele não processou as causas.

Retorno recebido ao enviar dados errados de um cartão:

(
    [message] => User cards bad request
    [code] => bad_request
    [status] => 400
    [cause_error] => Array
        (
            [0] => Array
                (
                    [description] => Invalid parameter 'number' in site_id: MLB
                    [code] => 120
                )
        )
)

Então se fez necessário acrescentar uma verificação pela ['cause_error'] para processar as causas do erro, além da ['cause'] já existente, exemplo abaixo:

if(isset($message['cause_error']))
    $recuperable_error->proccess_causes($message['cause_error']);

[lucmantovani]

Olá @rodrigopeixotobr!
Obrigado pela validação!

Como essa discussion e o PR #329 tratam do mesmo assunto, poderia adicionar essa sugestão lá no PR para centralizarmos a solução?

[rodrigopeixotobr]

Ok, claro. Adicionado lá.