##// END OF EJS Templates
Only show allowed statuses in context menu (#10181)....
Jean-Philippe Lang -
r8705:07a93c2731c5
parent child
Show More
@@ -1,73 +1,72
1 class ContextMenusController < ApplicationController
1 class ContextMenusController < ApplicationController
2 helper :watchers
2 helper :watchers
3 helper :issues
3 helper :issues
4
4
5 def issues
5 def issues
6 @issues = Issue.visible.all(:conditions => {:id => params[:ids]}, :include => :project)
6 @issues = Issue.visible.all(:conditions => {:id => params[:ids]}, :include => :project)
7
7
8 if (@issues.size == 1)
8 if (@issues.size == 1)
9 @issue = @issues.first
9 @issue = @issues.first
10 @allowed_statuses = @issue.new_statuses_allowed_to(User.current)
10 @allowed_statuses = @issue.new_statuses_allowed_to(User.current)
11 else
11 else
12 @allowed_statuses = @issues.map do |i|
12 @allowed_statuses = @issues.map do |i|
13 i.new_statuses_allowed_to(User.current)
13 i.new_statuses_allowed_to(User.current)
14 end.inject do |memo,s|
14 end.inject do |memo,s|
15 memo & s
15 memo & s
16 end
16 end
17 end
17 end
18 @projects = @issues.collect(&:project).compact.uniq
18 @projects = @issues.collect(&:project).compact.uniq
19 @project = @projects.first if @projects.size == 1
19 @project = @projects.first if @projects.size == 1
20
20
21 @can = {:edit => User.current.allowed_to?(:edit_issues, @projects),
21 @can = {:edit => User.current.allowed_to?(:edit_issues, @projects),
22 :log_time => (@project && User.current.allowed_to?(:log_time, @project)),
22 :log_time => (@project && User.current.allowed_to?(:log_time, @project)),
23 :update => (User.current.allowed_to?(:edit_issues, @projects) || (User.current.allowed_to?(:change_status, @projects) && !@allowed_statuses.blank?)),
23 :update => (User.current.allowed_to?(:edit_issues, @projects) || (User.current.allowed_to?(:change_status, @projects) && !@allowed_statuses.blank?)),
24 :move => (@project && User.current.allowed_to?(:move_issues, @project)),
24 :move => (@project && User.current.allowed_to?(:move_issues, @project)),
25 :copy => (@issue && @project.trackers.include?(@issue.tracker) && User.current.allowed_to?(:add_issues, @project)),
25 :copy => (@issue && @project.trackers.include?(@issue.tracker) && User.current.allowed_to?(:add_issues, @project)),
26 :delete => User.current.allowed_to?(:delete_issues, @projects)
26 :delete => User.current.allowed_to?(:delete_issues, @projects)
27 }
27 }
28 if @project
28 if @project
29 if @issue
29 if @issue
30 @assignables = @issue.assignable_users
30 @assignables = @issue.assignable_users
31 else
31 else
32 @assignables = @project.assignable_users
32 @assignables = @project.assignable_users
33 end
33 end
34 @trackers = @project.trackers
34 @trackers = @project.trackers
35 else
35 else
36 #when multiple projects, we only keep the intersection of each set
36 #when multiple projects, we only keep the intersection of each set
37 @assignables = @projects.map(&:assignable_users).inject{|memo,a| memo & a}
37 @assignables = @projects.map(&:assignable_users).inject{|memo,a| memo & a}
38 @trackers = @projects.map(&:trackers).inject{|memo,t| memo & t}
38 @trackers = @projects.map(&:trackers).inject{|memo,t| memo & t}
39 end
39 end
40
40
41 @priorities = IssuePriority.active.reverse
41 @priorities = IssuePriority.active.reverse
42 @statuses = IssueStatus.find(:all, :order => 'position')
43 @back = back_url
42 @back = back_url
44
43
45 @options_by_custom_field = {}
44 @options_by_custom_field = {}
46 if @can[:edit]
45 if @can[:edit]
47 custom_fields = @issues.map(&:available_custom_fields).inject {|memo, f| memo & f}.select do |f|
46 custom_fields = @issues.map(&:available_custom_fields).inject {|memo, f| memo & f}.select do |f|
48 %w(bool list user version).include?(f.field_format) && !f.multiple?
47 %w(bool list user version).include?(f.field_format) && !f.multiple?
49 end
48 end
50 custom_fields.each do |field|
49 custom_fields.each do |field|
51 values = field.possible_values_options(@projects)
50 values = field.possible_values_options(@projects)
52 if values.any?
51 if values.any?
53 @options_by_custom_field[field] = values
52 @options_by_custom_field[field] = values
54 end
53 end
55 end
54 end
56 end
55 end
57
56
58 render :layout => false
57 render :layout => false
59 end
58 end
60
59
61 def time_entries
60 def time_entries
62 @time_entries = TimeEntry.all(
61 @time_entries = TimeEntry.all(
63 :conditions => {:id => params[:ids]}, :include => :project)
62 :conditions => {:id => params[:ids]}, :include => :project)
64 @projects = @time_entries.collect(&:project).compact.uniq
63 @projects = @time_entries.collect(&:project).compact.uniq
65 @project = @projects.first if @projects.size == 1
64 @project = @projects.first if @projects.size == 1
66 @activities = TimeEntryActivity.shared.active
65 @activities = TimeEntryActivity.shared.active
67 @can = {:edit => User.current.allowed_to?(:edit_time_entries, @projects),
66 @can = {:edit => User.current.allowed_to?(:edit_time_entries, @projects),
68 :delete => User.current.allowed_to?(:edit_time_entries, @projects)
67 :delete => User.current.allowed_to?(:edit_time_entries, @projects)
69 }
68 }
70 @back = back_url
69 @back = back_url
71 render :layout => false
70 render :layout => false
72 end
71 end
73 end
72 end
@@ -1,138 +1,138
1 <ul>
1 <ul>
2 <%= call_hook(:view_issues_context_menu_start, {:issues => @issues, :can => @can, :back => @back }) %>
2 <%= call_hook(:view_issues_context_menu_start, {:issues => @issues, :can => @can, :back => @back }) %>
3
3
4 <% if !@issue.nil? -%>
4 <% if !@issue.nil? -%>
5 <li><%= context_menu_link l(:button_edit), {:controller => 'issues', :action => 'edit', :id => @issue},
5 <li><%= context_menu_link l(:button_edit), {:controller => 'issues', :action => 'edit', :id => @issue},
6 :class => 'icon-edit', :disabled => !@can[:edit] %></li>
6 :class => 'icon-edit', :disabled => !@can[:edit] %></li>
7 <% else %>
7 <% else %>
8 <li><%= context_menu_link l(:button_edit), {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id)},
8 <li><%= context_menu_link l(:button_edit), {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id)},
9 :class => 'icon-edit', :disabled => !@can[:edit] %></li>
9 :class => 'icon-edit', :disabled => !@can[:edit] %></li>
10 <% end %>
10 <% end %>
11
11
12 <% if @allowed_statuses.present? %>
12 <% if @allowed_statuses.present? %>
13 <li class="folder">
13 <li class="folder">
14 <a href="#" class="submenu" onclick="return false;"><%= l(:field_status) %></a>
14 <a href="#" class="submenu" onclick="return false;"><%= l(:field_status) %></a>
15 <ul>
15 <ul>
16 <% @statuses.each do |s| -%>
16 <% @allowed_statuses.each do |s| -%>
17 <li><%= context_menu_link h(s.name), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {:status_id => s}, :back_url => @back}, :method => :post,
17 <li><%= context_menu_link h(s.name), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {:status_id => s}, :back_url => @back}, :method => :post,
18 :selected => (@issue && s == @issue.status), :disabled => !(@can[:update] && @allowed_statuses.include?(s)) %></li>
18 :selected => (@issue && s == @issue.status), :disabled => !@can[:update] %></li>
19 <% end -%>
19 <% end -%>
20 </ul>
20 </ul>
21 </li>
21 </li>
22 <% end %>
22 <% end %>
23
23
24 <% unless @trackers.nil? %>
24 <% unless @trackers.nil? %>
25 <li class="folder">
25 <li class="folder">
26 <a href="#" class="submenu"><%= l(:field_tracker) %></a>
26 <a href="#" class="submenu"><%= l(:field_tracker) %></a>
27 <ul>
27 <ul>
28 <% @trackers.each do |t| -%>
28 <% @trackers.each do |t| -%>
29 <li><%= context_menu_link h(t.name), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'tracker_id' => t}, :back_url => @back}, :method => :post,
29 <li><%= context_menu_link h(t.name), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'tracker_id' => t}, :back_url => @back}, :method => :post,
30 :selected => (@issue && t == @issue.tracker), :disabled => !@can[:edit] %></li>
30 :selected => (@issue && t == @issue.tracker), :disabled => !@can[:edit] %></li>
31 <% end -%>
31 <% end -%>
32 </ul>
32 </ul>
33 </li>
33 </li>
34 <% end %>
34 <% end %>
35
35
36 <li class="folder">
36 <li class="folder">
37 <a href="#" class="submenu"><%= l(:field_priority) %></a>
37 <a href="#" class="submenu"><%= l(:field_priority) %></a>
38 <ul>
38 <ul>
39 <% @priorities.each do |p| -%>
39 <% @priorities.each do |p| -%>
40 <li><%= context_menu_link h(p.name), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'priority_id' => p}, :back_url => @back}, :method => :post,
40 <li><%= context_menu_link h(p.name), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'priority_id' => p}, :back_url => @back}, :method => :post,
41 :selected => (@issue && p == @issue.priority), :disabled => (!@can[:edit] || @issues.detect {|i| !i.leaf?}) %></li>
41 :selected => (@issue && p == @issue.priority), :disabled => (!@can[:edit] || @issues.detect {|i| !i.leaf?}) %></li>
42 <% end -%>
42 <% end -%>
43 </ul>
43 </ul>
44 </li>
44 </li>
45
45
46 <% #TODO: allow editing versions when multiple projects %>
46 <% #TODO: allow editing versions when multiple projects %>
47 <% unless @project.nil? || @project.shared_versions.open.empty? -%>
47 <% unless @project.nil? || @project.shared_versions.open.empty? -%>
48 <li class="folder">
48 <li class="folder">
49 <a href="#" class="submenu"><%= l(:field_fixed_version) %></a>
49 <a href="#" class="submenu"><%= l(:field_fixed_version) %></a>
50 <ul>
50 <ul>
51 <% @project.shared_versions.open.sort.each do |v| -%>
51 <% @project.shared_versions.open.sort.each do |v| -%>
52 <li><%= context_menu_link format_version_name(v), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'fixed_version_id' => v}, :back_url => @back}, :method => :post,
52 <li><%= context_menu_link format_version_name(v), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'fixed_version_id' => v}, :back_url => @back}, :method => :post,
53 :selected => (@issue && v == @issue.fixed_version), :disabled => !@can[:update] %></li>
53 :selected => (@issue && v == @issue.fixed_version), :disabled => !@can[:update] %></li>
54 <% end -%>
54 <% end -%>
55 <li><%= context_menu_link l(:label_none), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'fixed_version_id' => 'none'}, :back_url => @back}, :method => :post,
55 <li><%= context_menu_link l(:label_none), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'fixed_version_id' => 'none'}, :back_url => @back}, :method => :post,
56 :selected => (@issue && @issue.fixed_version.nil?), :disabled => !@can[:update] %></li>
56 :selected => (@issue && @issue.fixed_version.nil?), :disabled => !@can[:update] %></li>
57 </ul>
57 </ul>
58 </li>
58 </li>
59 <% end %>
59 <% end %>
60 <% if @assignables.present? -%>
60 <% if @assignables.present? -%>
61 <li class="folder">
61 <li class="folder">
62 <a href="#" class="submenu"><%= l(:field_assigned_to) %></a>
62 <a href="#" class="submenu"><%= l(:field_assigned_to) %></a>
63 <ul>
63 <ul>
64 <% if @assignables.include?(User.current) %>
64 <% if @assignables.include?(User.current) %>
65 <li><%= context_menu_link "<< #{l(:label_me)} >>", {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'assigned_to_id' => User.current}, :back_url => @back}, :method => :post,
65 <li><%= context_menu_link "<< #{l(:label_me)} >>", {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'assigned_to_id' => User.current}, :back_url => @back}, :method => :post,
66 :disabled => !@can[:update] %></li>
66 :disabled => !@can[:update] %></li>
67 <% end %>
67 <% end %>
68 <% @assignables.each do |u| -%>
68 <% @assignables.each do |u| -%>
69 <li><%= context_menu_link h(u.name), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'assigned_to_id' => u}, :back_url => @back}, :method => :post,
69 <li><%= context_menu_link h(u.name), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'assigned_to_id' => u}, :back_url => @back}, :method => :post,
70 :selected => (@issue && u == @issue.assigned_to), :disabled => !@can[:update] %></li>
70 :selected => (@issue && u == @issue.assigned_to), :disabled => !@can[:update] %></li>
71 <% end -%>
71 <% end -%>
72 <li><%= context_menu_link l(:label_nobody), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'assigned_to_id' => 'none'}, :back_url => @back}, :method => :post,
72 <li><%= context_menu_link l(:label_nobody), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'assigned_to_id' => 'none'}, :back_url => @back}, :method => :post,
73 :selected => (@issue && @issue.assigned_to.nil?), :disabled => !@can[:update] %></li>
73 :selected => (@issue && @issue.assigned_to.nil?), :disabled => !@can[:update] %></li>
74 </ul>
74 </ul>
75 </li>
75 </li>
76 <% end %>
76 <% end %>
77 <% unless @project.nil? || @project.issue_categories.empty? -%>
77 <% unless @project.nil? || @project.issue_categories.empty? -%>
78 <li class="folder">
78 <li class="folder">
79 <a href="#" class="submenu"><%= l(:field_category) %></a>
79 <a href="#" class="submenu"><%= l(:field_category) %></a>
80 <ul>
80 <ul>
81 <% @project.issue_categories.each do |u| -%>
81 <% @project.issue_categories.each do |u| -%>
82 <li><%= context_menu_link h(u.name), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'category_id' => u}, :back_url => @back}, :method => :post,
82 <li><%= context_menu_link h(u.name), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'category_id' => u}, :back_url => @back}, :method => :post,
83 :selected => (@issue && u == @issue.category), :disabled => !@can[:update] %></li>
83 :selected => (@issue && u == @issue.category), :disabled => !@can[:update] %></li>
84 <% end -%>
84 <% end -%>
85 <li><%= context_menu_link l(:label_none), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'category_id' => 'none'}, :back_url => @back}, :method => :post,
85 <li><%= context_menu_link l(:label_none), {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'category_id' => 'none'}, :back_url => @back}, :method => :post,
86 :selected => (@issue && @issue.category.nil?), :disabled => !@can[:update] %></li>
86 :selected => (@issue && @issue.category.nil?), :disabled => !@can[:update] %></li>
87 </ul>
87 </ul>
88 </li>
88 </li>
89 <% end -%>
89 <% end -%>
90
90
91 <% if Issue.use_field_for_done_ratio? %>
91 <% if Issue.use_field_for_done_ratio? %>
92 <li class="folder">
92 <li class="folder">
93 <a href="#" class="submenu"><%= l(:field_done_ratio) %></a>
93 <a href="#" class="submenu"><%= l(:field_done_ratio) %></a>
94 <ul>
94 <ul>
95 <% (0..10).map{|x|x*10}.each do |p| -%>
95 <% (0..10).map{|x|x*10}.each do |p| -%>
96 <li><%= context_menu_link "#{p}%", {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'done_ratio' => p}, :back_url => @back}, :method => :post,
96 <li><%= context_menu_link "#{p}%", {:controller => 'issues', :action => 'bulk_update', :ids => @issues.collect(&:id), :issue => {'done_ratio' => p}, :back_url => @back}, :method => :post,
97 :selected => (@issue && p == @issue.done_ratio), :disabled => (!@can[:edit] || @issues.detect {|i| !i.leaf?}) %></li>
97 :selected => (@issue && p == @issue.done_ratio), :disabled => (!@can[:edit] || @issues.detect {|i| !i.leaf?}) %></li>
98 <% end -%>
98 <% end -%>
99 </ul>
99 </ul>
100 </li>
100 </li>
101 <% end %>
101 <% end %>
102
102
103 <% @options_by_custom_field.each do |field, options| %>
103 <% @options_by_custom_field.each do |field, options| %>
104 <li class="folder">
104 <li class="folder">
105 <a href="#" class="submenu"><%= h(field.name) %></a>
105 <a href="#" class="submenu"><%= h(field.name) %></a>
106 <ul>
106 <ul>
107 <% options.each do |text, value| %>
107 <% options.each do |text, value| %>
108 <li><%= bulk_update_custom_field_context_menu_link(field, text, value || text) %></li>
108 <li><%= bulk_update_custom_field_context_menu_link(field, text, value || text) %></li>
109 <% end %>
109 <% end %>
110 <% unless field.is_required? %>
110 <% unless field.is_required? %>
111 <li><%= bulk_update_custom_field_context_menu_link(field, l(:label_none), '') %></li>
111 <li><%= bulk_update_custom_field_context_menu_link(field, l(:label_none), '') %></li>
112 <% end %>
112 <% end %>
113 </ul>
113 </ul>
114 </li>
114 </li>
115 <% end %>
115 <% end %>
116
116
117 <% if !@issue.nil? %>
117 <% if !@issue.nil? %>
118 <% if @can[:log_time] -%>
118 <% if @can[:log_time] -%>
119 <li><%= context_menu_link l(:button_log_time), {:controller => 'timelog', :action => 'new', :issue_id => @issue},
119 <li><%= context_menu_link l(:button_log_time), {:controller => 'timelog', :action => 'new', :issue_id => @issue},
120 :class => 'icon-time-add' %></li>
120 :class => 'icon-time-add' %></li>
121 <% end %>
121 <% end %>
122 <% if User.current.logged? %>
122 <% if User.current.logged? %>
123 <li><%= watcher_link(@issue, User.current) %></li>
123 <li><%= watcher_link(@issue, User.current) %></li>
124 <% end %>
124 <% end %>
125 <% end %>
125 <% end %>
126
126
127 <% if @issue.present? %>
127 <% if @issue.present? %>
128 <li><%= context_menu_link l(:button_copy), {:controller => 'issues', :action => 'new', :project_id => @project, :copy_from => @issue},
128 <li><%= context_menu_link l(:button_copy), {:controller => 'issues', :action => 'new', :project_id => @project, :copy_from => @issue},
129 :class => 'icon-copy', :disabled => !@can[:copy] %></li>
129 :class => 'icon-copy', :disabled => !@can[:copy] %></li>
130 <% else %>
130 <% else %>
131 <li><%= context_menu_link l(:button_copy), {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :copy => '1'},
131 <li><%= context_menu_link l(:button_copy), {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :copy => '1'},
132 :class => 'icon-copy', :disabled => !@can[:move] %></li>
132 :class => 'icon-copy', :disabled => !@can[:move] %></li>
133 <% end %>
133 <% end %>
134 <li><%= context_menu_link l(:button_delete), issues_path(:ids => @issues.collect(&:id), :back_url => @back),
134 <li><%= context_menu_link l(:button_delete), issues_path(:ids => @issues.collect(&:id), :back_url => @back),
135 :method => :delete, :confirm => issues_destroy_confirmation_message(@issues), :class => 'icon-del', :disabled => !@can[:delete] %></li>
135 :method => :delete, :confirm => issues_destroy_confirmation_message(@issues), :class => 'icon-del', :disabled => !@can[:delete] %></li>
136
136
137 <%= call_hook(:view_issues_context_menu_end, {:issues => @issues, :can => @can, :back => @back }) %>
137 <%= call_hook(:view_issues_context_menu_end, {:issues => @issues, :can => @can, :back => @back }) %>
138 </ul>
138 </ul>
General Comments 0
You need to be logged in to leave comments. Login now