Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove useless foreach #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fonini
Copy link

@fonini fonini commented Oct 1, 2018

No description provided.

@marcosbrasil
Copy link

Esse foreach não é somente "sem uso", mas está removendo os IDs dos itens (produtos). É muito estranho ninguém ter percebido isso já que esses IDs são necessários caso seja deseje gerar uma segunda via da fatura.

O que está acontecendo é que o foreach está sobrepondo a chave items que vem da response e contém um array com os itens pelo array que é enviado para a API.

Iugu_Invoice antes e depois do foreach

Antes:
https://gist.github.com/marcosbrasil/8b0b7deb369b0f77c2beb8fdb55d0520

Depois:
https://gist.github.com/marcosbrasil/bae88386a9de33a0bf36e2bbbc7990f6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants