Patch #25265
closed
QueriesController can not handle subclass of IssueQuery
Added by Haihan Ji over 7 years ago.
Updated over 4 years ago.
Description
I have a PivotQuery, it's a subclass of IssueQuery.
But Query don't think it is subclass of himself.
So, r15635 let my plugin down.
I changed QueriesController to fix it.
OLD CODE:
def query_class
Query.get_subclass(params[:type] || 'IssueQuery')
end
New Code
def query_class
get_query_class(params[:type] || 'IssueQuery')
end
def get_query_class(class_name)
klass = nil
begin
klass = class_name.to_s.classify.constantize
rescue
# invalid class name
end
unless klass && klass.new.is_a?(Query)
klass = nil
end
klass
end
Maybe klass < Query
is better than klass && klass.new.is_a?(Query)
.
We should be able to fix get_subclass
instead:
Index: lib/redmine/subclass_factory.rb
===================================================================
--- lib/redmine/subclass_factory.rb (revision 16379)
+++ lib/redmine/subclass_factory.rb (working copy)
@@ -29,7 +29,7 @@
rescue
# invalid class name
end
- unless subclasses.include? klass
+ unless descendants.include? klass
klass = nil
end
klass
Would it work for you?
Hi Jean-Philippe,
your suggested change would be fix problems for some plugins, which I maintain. It would be nice to get this change in a upcoming release.
- Assignee set to Jean-Philippe Lang
- Target version set to 4.1.0
- Status changed from New to Closed
This change is only available in trunk, but not in Branch 4.1-stable - as this issue shows. Can we get this to 4.1-stable branch?
You are right! Sorry for bothering you about this.
Also available in: Atom
PDF