As discussed in T10521, it is useful to refactor out the generic query building logic in FileSearchQuery and create the base class SearchQuery. This would allow us to create other domain specific queries without having to reimplement all the building logic, as will be the case with TeamdriveSearchQuery.
- Maniphest Tasks
- T10521: [Drive] Implement Team Drives portion of the API
- R477:4f226717ee1f: Refactor FileSearchQuery and create base class SearchQuery
All tests defined in FileSearchQueryTest should remain valid and, as of this patch, passing.
Again, C++11 allows us to force the compiler to generate code for us, so let's make use of that.
If you just chain up to the parent implementation here, there's no need to have FileSearchQuery::addQuery() at all.