Correction: Fatal error: Call to undefined function IuguRequestException

[valdeir2000]

Correction: Fatal error: Call to undefined function IuguRequestException

[ricardoon]

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]

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]

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
#8 (comment).

Abs.
Ricardo

[pnegri]

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
#8 (comment).

Abs.
Ricardo

—
Reply to this email directly or view it on GitHub
#8 (comment).

[ricardoon]

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
#8 (comment).

Abs.
Ricardo

—
Reply to this email directly or view it on GitHub
#8 (comment).

—
Reply to this email directly or view it on GitHub
#8 (comment).

Abs.
Ricardo

[pnegri]

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
#8 (comment).

Abs.
Ricardo

—
Reply to this email directly or view it on GitHub
#8 (comment).

—
Reply to this email directly or view it on GitHub
#8 (comment).

Abs.
Ricardo

—
Reply to this email directly or view it on GitHub
#8 (comment).

[ricardoon]

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
#8 (comment).

Abs.
Ricardo

—
Reply to this email directly or view it on GitHub
#8 (comment).

—
Reply to this email directly or view it on GitHub
#8 (comment).

Abs.
Ricardo

—
Reply to this email directly or view it on GitHub
#8 (comment).

—
Reply to this email directly or view it on GitHub
#8 (comment).

[ricardoon]

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]

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
#8 (comment).

[ricardoon]

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
#8 (comment).

—
Reply to this email directly or view it on GitHub
#8 (comment).

[pnegri]

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