https://www.redmine.org/
https://www.redmine.org/favicon.ico?1679302129
2012-05-19T11:56:57Z
Redmine
Redmine - Patch #10970: Sort issues in roadmap by IssueStatus.position
https://www.redmine.org/issues/10970?journal_id=38295
2012-05-19T11:56:57Z
Александр Гурьянов
<ul></ul><p>Revision: trunk(9701)</p>
Redmine - Patch #10970: Sort issues in roadmap by IssueStatus.position
https://www.redmine.org/issues/10970?journal_id=90613
2019-03-06T06:36:30Z
Sunding Wei
<ul></ul><p>Thanks for the patch, I fixed for Redmine v3.4.4</p>
<p>Hope this feature can be merged into redmine master.</p>
<pre><code class="diff syntaxhl"><span class="gd">--- versions_controller.rb.old 2019-03-06 13:43:41.068549228 +0800
</span><span class="gi">+++ versions_controller.rb 2019-03-06 14:32:04.942060762 +0800
</span><span class="p">@@ -49,10 +49,10 @@</span>
@issues_by_version = {}
if @selected_tracker_ids.any? && @versions.any?
issues = Issue.visible.
<span class="gd">- includes(:project, :tracker).
</span><span class="gi">+ includes(:project, :tracker, :status).
</span> preload(:status, :priority, :fixed_version).
where(:tracker_id => @selected_tracker_ids, :project_id => project_ids, :fixed_version_id => @versions.map(&:id)).
<span class="gd">- order("#{Project.table_name}.lft, #{Tracker.table_name}.position, #{Issue.table_name}.id")
</span><span class="gi">+ order("#{Project.table_name}.lft, #{Tracker.table_name}.position, #{IssueStatus.table_name}.position, #{Issue.table_name}.id")
</span> @issues_by_version = issues.group_by(&:fixed_version)
end
@versions.reject! {|version| !project_ids.include?(version.project_id) && @issues_by_version[version].blank?}
<span class="p">@@ -69,7 +69,7 @@</span>
@issues = @version.fixed_issues.visible.
includes(:status, :tracker, :priority).
preload(:project).
<span class="gd">- reorder("#{Tracker.table_name}.position, #{Issue.table_name}.id").
</span><span class="gi">+ reorder("#{Tracker.table_name}.position, #{IssueStatus.table_name}.position, #{Issue.table_name}.id").
</span> to_a
}
format.api
</code></pre>