search

studiomado / query-bundle

Query doctrine database from query string and provide HATEOAS responses.
MIT License
13 stars 5 forks source link

Enhance QueryBuilderOptions for different purposes #163

Open sensorario opened 6 years ago

sensorario commented 6 years ago
Q A
Bug report? no
Feature request? yes
QueryBundle version 2.5

Actually QueryBuilderOptions is used for both CLI and HTTP interfaces. It is used mainly in controllers. This means that it must be built within a request. This suggest a "RequestQueryBuilderOption" component. Also, ... a "CLIQueryBuilderOption" should exists. These names are just ideas.

            └── QueryBuilderOptions
                       ├── ...
                       ├── CliOptions
                       └── HttpOptions

This could affect also BaseRepository and all its initializers. We may also have a CliBaseRepository and or HttpBaseRepository?

/cc @orociscaverai @AlessandroMinoccheri

AlessandroMinoccheri commented 6 years ago

Nice feature, at the moment cli and http are sufficient in my opinion.

👍