amosassis
commented
7 years ago Considerando que a função getCodigo() retorna um inteiro, não há necessidade de se manter '01', bastando apenas representar como inteiro sem o uso do zero (Ex. 01 => 1).
Closed felipph closed 7 years ago
amosassis
commented
7 years ago Considerando que a função getCodigo() retorna um inteiro, não há necessidade de se manter '01', bastando apenas representar como inteiro sem o uso do zero (Ex. 01 => 1).
felipph
commented
7 years ago Boa tarde,
No PHP7 números iniciados com "0" são entendidos como octal, por isso é necessário expressar literalmente como string e aspas simples.
Att,
Felipph
Em 17 de janeiro de 2017 12:04, Amós Assis notifications@github.com escreveu:
Considerando que a função getCodigo() retorna um inteiro, não há necessidade de se manter '01', bastando apenas representar como inteiro sem o uso do zero (Ex. 01 => 1).
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/andersondanilo/CnabPHP/pull/81#issuecomment-273193496, or mute the thread https://github.com/notifications/unsubscribe-auth/AE25FNItJ_EmfMqX-FubMaz9e1OmQBKBks5rTNiIgaJpZM4LZC9E .
amosassis
commented
7 years ago Boa tarde,
Concordo contigo, sua afirmação está correta. Minha observação é apenas quanto a necessidade de se manter o zero, ainda que entre aspas, já que a função getCodigo() utilizada para a obtenção do campo codigo_de_ocorrencia ja faz a conversão do parâmetro para inteiro:
return (int) $this->codigo_de_ocorrencia;
Sendo assim o problema pode ser corrigido apenas removendo o zero, da mesma forma dos demais bancos.
Att,
Amós
No PHP7, números octais são iniciados com "0". no processamento do detalhe, para o banco CEF, os códigos de retorno estão sendo comparados com "inteiros" iniciados com 0. Ajustada a comparação.
PHP 7.0.8-0ubuntu0.16.04.3 (cli) ( NTS )