Skip to content

providing ability to manage flat_query on a filter class basis#215

Merged
ssteffen merged 4 commits intomainfrom
filter_smart_defaults_for_flat_query
May 6, 2025
Merged

providing ability to manage flat_query on a filter class basis#215
ssteffen merged 4 commits intomainfrom
filter_smart_defaults_for_flat_query

Conversation

@ssteffen
Copy link
Copy Markdown
Collaborator

@ssteffen ssteffen commented May 5, 2025

This makes flat_query more easily accessible by doing the following:

  • Moves the FlatQueryTools module out of the Filter class. Instead, the implementing filter class should include it - and then flat_query will be enabled by default
  • Creates a smart default method on filter called can_use_flat_query?. Can be extended to handle custom use cases
  • Changes the standard get_query method to check can_use_flat_query? method and whether the FlatQueryTools module is included. If so, it uses flat_query. Otherwise it calls get_complex_query which is simply the sub-select path which is the standard default

@ssteffen ssteffen self-assigned this May 5, 2025
@etisonMachineUser etisonMachineUser added the enhancement New feature or request label May 5, 2025
@ssteffen ssteffen merged commit 52a4c39 into main May 6, 2025
1 check passed
@ssteffen ssteffen deleted the filter_smart_defaults_for_flat_query branch May 6, 2025 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants