Actions
Patch #37135
closedReduce extra queries in ProjectQuery.default
Start date:
Due date:
% Done:
0%
Estimated time:
Description
Same as #36503 but for ProjectQuery.
Currently redmine tries to select default_project_query
even if settings are empty.
diff --git a/app/models/project_query.rb b/app/models/project_query.rb index 002437aeb..8c6643c7b 100644 --- a/app/models/project_query.rb +++ b/app/models/project_query.rb @@ -39,10 +39,13 @@ class ProjectQuery < Query def self.default(project: nil, user: User.current) query = nil - if user&.logged? - query = find_by_id user.pref.default_project_query + if user&.logged? && (query_id = user.pref.default_project_query).present? + query = find_by(id: query_id) end - query || find_by_id(Setting.default_project_query) + if query.nil? && (query_id = Setting.default_project_query).present? + query = find_by(id: query_id) + end + query end def initialize(attributes=nil, *args)
Related issues
Updated by Dmitry Makurin over 2 years ago
- Copied from Patch #36503: Reduce extra queries in IssueQuery.default added
Updated by Go MAEDA over 2 years ago
- Assignee deleted (
Go MAEDA) - Target version set to 5.0.2
Setting the target version to 5.0.2.
Updated by Go MAEDA over 2 years ago
- Related to Feature #35795: Settings for global and user default custom ProjectQuery added
Updated by Marius BĂLTEANU over 2 years ago
- Status changed from New to Resolved
- Assignee set to Marius BĂLTEANU
Patch committed, thanks!
Updated by Marius BĂLTEANU over 2 years ago
- Status changed from Resolved to Closed
Actions