Closed valdeir2000 closed 8 years ago
ricardoon
commented
8 years ago Por favor, aceitem logo o commit do Valdeir2000. Substitui a mudança dele na mão e funcionou. Vocês toda hora davam esse fatal error. É horrível trabalhar assim. Desde 28 de agosto para aprovarem o commit do cara. Ao menos ajeitem isso, por favor.
pnegri
commented
8 years ago Desculpe Ricardo. Mas isto não é uma correção de código. Retornar True em ambos os casos só está corrigindo algo errado em outro lugar.
ricardoon
commented
8 years ago Como eu disse, seria interessante corrigir, pois esta dando fatal error direto e estou tendo que tirar voces do sistema financeiro da Xtech Commerce por conta disso, pois ele vive dando o erro do titulo no php_error.log.. ;)
Por enquanto o commit dele me salvou.
On Tuesday, 24 November 2015, Patrick Negri notifications@github.com wrote:
Desculpe Ricardo. Mas isto não é uma correção de código. Retornar True em ambos os casos só está corrigindo algo errado em outro lugar.
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159197315.
Abs. Ricardo
pnegri
commented
8 years ago Então Ricardo. Mas tem algo errado aí. Esta correção só está fazendo a função retornar True qndo não consegue cancelar tbm. Provavelmente o erro continua no código de vcs, alguma invoice que era para ser cancelada e não está sendo cancelada.
Manda um gist para mim com o trecho de código de vcs que usa a rotina de cancel
Eu uso este código em um sistema meu e funciona, além de passar em todos os testes e ter outros clientes utilizando. Todas as vezes que averiguei o erro, ele estava no código do cliente, por isto não aceitei o merge.
Abs Em 24/11/2015 12:19, "Ricardo Oliveira" notifications@github.com escreveu:
Como eu disse, seria interessante corrigir, pois esta dando fatal error direto e estou tendo que tirar voces do sistema financeiro da Xtech Commerce por conta disso, pois ele vive dando o erro do titulo no php_error.log.. ;)
Por enquanto o commit dele me salvou.
On Tuesday, 24 November 2015, Patrick Negri notifications@github.com wrote:
Desculpe Ricardo. Mas isto não é uma correção de código. Retornar True em ambos os casos só está corrigindo algo errado em outro lugar.
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159197315.
Abs. Ricardo
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159280680.
ricardoon
commented
8 years ago Show, estou na rua no momento, mas te passo sim.
Eu uso exatamente o codigo que tem na docs em php com a minha conta em modo de teste e da o fatal error para cancelar uma fatura.
On Tuesday, 24 November 2015, Patrick Negri notifications@github.com wrote:
Então Ricardo. Mas tem algo errado aí. Esta correção só está fazendo a função retornar True qndo não consegue cancelar tbm. Provavelmente o erro continua no código de vcs, alguma invoice que era para ser cancelada e não está sendo cancelada.
Manda um gist para mim com o trecho de código de vcs que usa a rotina de cancel
Eu uso este código em um sistema meu e funciona, além de passar em todos os testes e ter outros clientes utilizando. Todas as vezes que averiguei o erro, ele estava no código do cliente, por isto não aceitei o merge.
Abs Em 24/11/2015 12:19, "Ricardo Oliveira" <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> escreveu:
Como eu disse, seria interessante corrigir, pois esta dando fatal error direto e estou tendo que tirar voces do sistema financeiro da Xtech Commerce por conta disso, pois ele vive dando o erro do titulo no php_error.log.. ;)
Por enquanto o commit dele me salvou.
On Tuesday, 24 November 2015, Patrick Negri <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:
Desculpe Ricardo. Mas isto não é uma correção de código. Retornar True em ambos os casos só está corrigindo algo errado em outro lugar.
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159197315.
Abs. Ricardo
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159280680.
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159281617.
Abs. Ricardo
pnegri
commented
8 years ago Legal. Estou no aguardo. Chegando já analiso e se vermos que há algo errado por aki corrigimos hoje mesmo.
Abs Em 24/11/2015 12:26, "Ricardo Oliveira" notifications@github.com escreveu:
Show, estou na rua no momento, mas te passo sim.
Eu uso exatamente o codigo que tem na docs em php com a minha conta em modo de teste e da o fatal error para cancelar uma fatura.
On Tuesday, 24 November 2015, Patrick Negri notifications@github.com wrote:
Então Ricardo. Mas tem algo errado aí. Esta correção só está fazendo a função retornar True qndo não consegue cancelar tbm. Provavelmente o erro continua no código de vcs, alguma invoice que era para ser cancelada e não está sendo cancelada.
Manda um gist para mim com o trecho de código de vcs que usa a rotina de cancel
Eu uso este código em um sistema meu e funciona, além de passar em todos os testes e ter outros clientes utilizando. Todas as vezes que averiguei o erro, ele estava no código do cliente, por isto não aceitei o merge.
Abs Em 24/11/2015 12:19, "Ricardo Oliveira" <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> escreveu:
Como eu disse, seria interessante corrigir, pois esta dando fatal error direto e estou tendo que tirar voces do sistema financeiro da Xtech Commerce por conta disso, pois ele vive dando o erro do titulo no php_error.log.. ;)
Por enquanto o commit dele me salvou.
On Tuesday, 24 November 2015, Patrick Negri <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:
Desculpe Ricardo. Mas isto não é uma correção de código. Retornar True em ambos os casos só está corrigindo algo errado em outro lugar.
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159197315.
Abs. Ricardo
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159280680.
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159281617.
Abs. Ricardo
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159282311.
ricardoon
commented
8 years ago Fala Patrick,
Segue o código, bem simples. Sempre tenho tela em branco, fatal error, e o seguinte erro no php_error.log.
[24-Nov-2015 01:00:32 America/Sao_Paulo] PHP Fatal error: Call to undefined function IuguRequestException() in /Applications/MAMP/htdocs/xtech/loja/vendor/iugu/iugu/lib/Iugu/Invoice.php on line 26
Meu sistema usa codigniter 2.. e uso a seguinte versão via composer da biblioteca de vocês "iugu/iugu": "1.0.5"..
https://gist.github.com/ricardoon/c5e2bb79d8f7544c9db2
Abs. Ricardo
2015-11-24 12:28 GMT-02:00 Patrick Negri notifications@github.com:
Legal. Estou no aguardo. Chegando já analiso e se vermos que há algo errado por aki corrigimos hoje mesmo.
Abs Em 24/11/2015 12:26, "Ricardo Oliveira" notifications@github.com escreveu:
Show, estou na rua no momento, mas te passo sim.
Eu uso exatamente o codigo que tem na docs em php com a minha conta em modo de teste e da o fatal error para cancelar uma fatura.
On Tuesday, 24 November 2015, Patrick Negri notifications@github.com wrote:
Então Ricardo. Mas tem algo errado aí. Esta correção só está fazendo a função retornar True qndo não consegue cancelar tbm. Provavelmente o erro continua no código de vcs, alguma invoice que era para ser cancelada e não está sendo cancelada.
Manda um gist para mim com o trecho de código de vcs que usa a rotina de cancel
Eu uso este código em um sistema meu e funciona, além de passar em todos os testes e ter outros clientes utilizando. Todas as vezes que averiguei o erro, ele estava no código do cliente, por isto não aceitei o merge.
Abs Em 24/11/2015 12:19, "Ricardo Oliveira" <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> escreveu:
Como eu disse, seria interessante corrigir, pois esta dando fatal error direto e estou tendo que tirar voces do sistema financeiro da Xtech Commerce por conta disso, pois ele vive dando o erro do titulo no php_error.log.. ;)
Por enquanto o commit dele me salvou.
On Tuesday, 24 November 2015, Patrick Negri < notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:
Desculpe Ricardo. Mas isto não é uma correção de código. Retornar True em ambos os casos só está corrigindo algo errado em outro lugar.
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159197315.
Abs. Ricardo
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159280680.
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159281617.
Abs. Ricardo
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159282311.
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159282835.
ricardoon
commented
8 years ago E não entendi o retornar true em ambos os casos, acredito que não seja o caso, pode ter visto o código errado.. no código dele, ele realmente parece ter corrigido o problema de "undefined function IuguRequestException()".
pnegri
commented
8 years ago Então. Ele corrigiu pq tá retornando uma string (string == true).
Irei testar logo mais. Em 24/11/2015 18:36, "Ricardo Oliveira" notifications@github.com escreveu:
E não entendi o retornar true em ambos os casos, acredito que não seja o caso, pode ter visto o código errado.. no código dele, ele realmente parece ter corrigido o problema de "undefined function IuguRequestException()".
[image: fullscreen_11_24_15__6_35_pm] https://cloud.githubusercontent.com/assets/3991034/11379930/363d18a2-92da-11e5-8017-24721cb6b3e3.png
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159397849.
ricardoon
commented
8 years ago Saquei. Obrigado.
Abs. Ricardo
2015-11-24 18:49 GMT-02:00 Patrick Negri notifications@github.com:
Então. Ele corrigiu pq tá retornando uma string (string == true).
Irei testar logo mais. Em 24/11/2015 18:36, "Ricardo Oliveira" notifications@github.com escreveu:
E não entendi o retornar true em ambos os casos, acredito que não seja o caso, pode ter visto o código errado.. no código dele, ele realmente parece ter corrigido o problema de "undefined function IuguRequestException()".
[image: fullscreen_11_24_15__6_35_pm] < https://cloud.githubusercontent.com/assets/3991034/11379930/363d18a2-92da-11e5-8017-24721cb6b3e3.png
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159397849.
— Reply to this email directly or view it on GitHub https://github.com/iugu/iugu-php/pull/8#issuecomment-159401137.
pnegri
commented
8 years ago Ricardo. O fix do do valdeir era o "new". Estou aceitando parcialmente a modificação. Rejeitando o merge completo.
Incorporado na versão 1.0.6
Correction: Fatal error: Call to undefined function IuguRequestException