Defect #27533

Cannot change the priority of the parent issue in issue query context menu when parent priority is independent of children

Added by Vito Marolda 8 months ago. Updated 7 months ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Go MAEDA% Done:

0%

Category:Issues list
Target version:3.3.6
Resolution:Fixed Affected version:3.3.1

Description

Also if Tracking settings are set to have parent priority independent of children, the context menu in issue query has priorities grayed out.
In fact, when normally editing, the priorities of parent issues are perfetcly editable.
This bug affects the context menu only.

Redmine Version.PNG - Screenshot of Version information (13 KB) Vito Marolda, 2017-11-14 19:58

Tracking settings.PNG - Screenshot of tracking settings (6.06 KB) Vito Marolda, 2017-11-14 19:58

Priority grayed out.PNG - Screenshot of issue query context menu (15.4 KB) Vito Marolda, 2017-11-14 19:58

0001-Fixato-priority-disabilitato-in-query-list.patch Magnifier - resolving patch (1.09 KB) Vito Marolda, 2017-11-15 12:15


Related issues

Related to Redmine - Defect #20992: Parent priority "Independent of subtasks" setting doesn't... Closed
Related to Redmine - Patch #28168: Allow context-menu edit of % done and priority of parent ... Closed

Associated revisions

Revision 17129
Added by Go MAEDA 7 months ago

Fix: Cannot change the priority of the parent issue in issues context menu (#27533).

Patch by Vito Marolda.

Revision 17130
Added by Go MAEDA 7 months ago

Merged r17129 to 3.4-stable (#27533).

Revision 17131
Added by Go MAEDA 7 months ago

Merged r17129 to 3.3-stable (#27533).

History

#1 Updated by Vito Marolda 8 months ago

I have fixed my installation. Attached the patch. Fortunately it was very simple. I am not a Ruby programmer.

#2 Updated by Toshi MARUYAMA 7 months ago

  • Target version set to 3.3.6

LGTM.
This patch is same with #20992.

#3 Updated by Marius BALTEANU 7 months ago

  • Related to Defect #20992: Parent priority "Independent of subtasks" setting doesn't work added

#4 Updated by Go MAEDA 7 months ago

Toshi MARUYAMA wrote:

LGTM.
This patch is same with #20992.

Absolutely. The following code is the change made by #20992 (r14671).

Index: app/views/issues/_attributes.html.erb
===================================================================
--- app/views/issues/_attributes.html.erb    (revision 14670)
+++ app/views/issues/_attributes.html.erb    (revision 14671)
@@ -11,7 +11,7 @@
 <% end %>

 <% if @issue.safe_attribute? 'priority_id' %>
-<p><%= f.select :priority_id, (@priorities.collect {|p| [p.name, p.id]}), {:required => true}, :disabled => !@issue.leaf? %></p>
+<p><%= f.select :priority_id, (@priorities.collect {|p| [p.name, p.id]}), {:required => true} %></p>
 <% end %>

 <% if @issue.safe_attribute? 'assigned_to_id' %>

#5 Updated by Go MAEDA 7 months ago

  • Status changed from New to Resolved
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed to the trunk. Thank you for detecting and fixing this issue.

#6 Updated by Go MAEDA 7 months ago

  • Status changed from Resolved to Closed

Merged from trunk to 3.4-stable and 3.3-stable.

#7 Updated by Go MAEDA 5 months ago

  • Related to Patch #28168: Allow context-menu edit of % done and priority of parent issues if the fields are not derived added

Also available in: Atom PDF