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

associação hasMany poderia gerar um Collection fluente com where #29

Open
rodolforg opened this issue May 27, 2019 · 9 comments
Open

Comments

@rodolforg
Copy link
Contributor

Por várias vezes, encontro erro no meu código por tentar algo como:

$biblioteca->livros->where(['formato'=>'brochura'])->next();

sendo $biblioteca um objeto com hasMany para classe Livro...

O que se obtém, contudo, é o primeiro elemento de $biblioteca->livros, ignorando completamente o where.

Seria possível implementar isso?

@taq
Copy link
Owner

taq commented May 28, 2019

Mas o next retorna o próximo.

@taq
Copy link
Owner

taq commented May 28, 2019

Ah, espera, foi mal, ignora as condições do where?

@rodolforg
Copy link
Contributor Author

Sim, ignora. Pelo menos nas minhas três classes que tentei usar e descobri que ignora o where completamente

@taq
Copy link
Owner

taq commented May 29, 2019

É, pelo que vi aqui, parece que eu não implementei mesmo. Como faz tempo que não atualizo aqui - e nem uso o TORM mais, não tenho mais projetos em PHP - até a suíte de testes caducou com alterações no PHPUnit.

@rodolforg
Copy link
Contributor Author

rodolforg commented May 29, 2019 via email

@taq
Copy link
Owner

taq commented May 29, 2019

Obrigado! O duro que eu não sei se vou continuar mantendo ele, o tempo aqui está curto, eu fiz para o uso dele em um projeto mesmo que o pessoal não queria usar framework e faz muito tempo que eu não tenho contato com esse projeto, que é de um cliente. Então ficou parado. :-(
Eu teria que arrumar tempo para dar uma atualizada e colocar mais recursos, mas agora tá bem complicado. :-(

@taq
Copy link
Owner

taq commented May 30, 2019

Daria para você dar uma atualizada na suíte de testes? Ai eu poderia dar uma olhada nessa questão.

@rodolforg
Copy link
Contributor Author

Feito: #30

@taq
Copy link
Owner

taq commented Jun 1, 2019

Valeu! Estou numa correria dando treinamento aqui, no Domingo eu dou uma olhada!

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

No branches or pull requests

2 participants