Skip to content
This repository has been archived by the owner on Jan 21, 2020. It is now read-only.

This repository needs a rewrite #51

Open
TomHAnderson opened this issue Jan 25, 2019 · 1 comment
Open

This repository needs a rewrite #51

TomHAnderson opened this issue Jan 25, 2019 · 1 comment

Comments

@TomHAnderson
Copy link

This repo was written in the context of Apigility. It is very useful in Apigility but the name of this repository and what it does best is to populate a QueryBuilder based on an array. For instance this is used in api-skeletons/zf-doctrine-graphql where Apigility shouldn't be a requirement.

Specific problems

  • Hydrator\Strategy\CollectionLink is specific to Apigility. This strategy is duplicated in api-skeleletons/zf-doctrine-hydrator (which also has strategies in the same tool type as this strategy)
  • Query Providers are specific to Apigility. I think these classes would do well in their own repository.

And with these out of the way this repo can do the job of array > Query Builder cleanly.

I'm willing to do this for a cleaner repo. What do you say?

@weierophinney
Copy link
Member

This repository has been closed and moved to laminas-api-tools/api-tools-doctrine-querybuilder; a new issue has been opened at laminas-api-tools/api-tools-doctrine-querybuilder#2.

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

No branches or pull requests

2 participants