##// END OF EJS Templates
jro_apps » redmine
RSS
Refactor: Extract a new IssueMovesController from IssuesController....
Eric Davis -
r3822:ff43bb1a74a7
parent child
Show More
Side by Side Unified
Context file:
r3822:ff43bb1a74a7 Loading diff...:
Show file before | Show file after | Hide comments
@@ -0,0 +1,94
1 class IssueMovesController < ApplicationController
2 default_search_scope :issues
3 before_filter :find_issues
4 before_filter :authorize
5
6 def new
7 prepare_for_issue_move
8 render :layout => false if request.xhr?
9 end
10
11 def create
12 prepare_for_issue_move
13
14 if request.post?
15 new_tracker = params[:new_tracker_id].blank? ? nil : @target_project.trackers.find_by_id(params[:new_tracker_id])
16 unsaved_issue_ids = []
17 moved_issues = []
18 @issues.each do |issue|
19 issue.reload
20 issue.init_journal(User.current)
21 call_hook(:controller_issues_move_before_save, { :params => params, :issue => issue, :target_project => @target_project, :copy => !!@copy })
22 if r = issue.move_to_project(@target_project, new_tracker, {:copy => @copy, :attributes => extract_changed_attributes_for_move(params)})
23 moved_issues << r
24 else
25 unsaved_issue_ids << issue.id
26 end
27 end
28 set_flash_from_bulk_issue_save(@issues, unsaved_issue_ids)
29
30 if params[:follow]
31 if @issues.size == 1 && moved_issues.size == 1
32 redirect_to :controller => 'issues', :action => 'show', :id => moved_issues.first
33 else
34 redirect_to :controller => 'issues', :action => 'index', :project_id => (@target_project || @project)
35 end
36 else
37 redirect_to :controller => 'issues', :action => 'index', :project_id => @project
38 end
39 return
40 end
41 end
42
43 private
44
45 def prepare_for_issue_move
46 @issues.sort!
47 @copy = params[:copy_options] && params[:copy_options][:copy]
48 @allowed_projects = Issue.allowed_target_projects_on_move
49 @target_project = @allowed_projects.detect {|p| p.id.to_s == params[:new_project_id]} if params[:new_project_id]
50 @target_project ||= @project
51 @trackers = @target_project.trackers
52 @available_statuses = Workflow.available_statuses(@project)
53 end
54
55 # Filter for bulk operations
56 # TODO: duplicated in IssuesController
57 def find_issues
58 @issues = Issue.find_all_by_id(params[:id] || params[:ids])
59 raise ActiveRecord::RecordNotFound if @issues.empty?
60 projects = @issues.collect(&:project).compact.uniq
61 if projects.size == 1
62 @project = projects.first
63 else
64 # TODO: let users bulk edit/move/destroy issues from different projects
65 render_error 'Can not bulk edit/move/destroy issues from different projects'
66 return false
67 end
68 rescue ActiveRecord::RecordNotFound
69 render_404
70 end
71
72 # TODO: duplicated in IssuesController
73 def set_flash_from_bulk_issue_save(issues, unsaved_issue_ids)
74 if unsaved_issue_ids.empty?
75 flash[:notice] = l(:notice_successful_update) unless issues.empty?
76 else
77 flash[:error] = l(:notice_failed_to_save_issues,
78 :count => unsaved_issue_ids.size,
79 :total => issues.size,
80 :ids => '#' + unsaved_issue_ids.join(', #'))
81 end
82 end
83
84 def extract_changed_attributes_for_move(params)
85 changed_attributes = {}
86 [:assigned_to_id, :status_id, :start_date, :due_date].each do |valid_attribute|
87 unless params[valid_attribute].blank?
88 changed_attributes[valid_attribute] = (params[valid_attribute] == 'none' ? nil : params[valid_attribute])
89 end
90 end
91 changed_attributes
92 end
93
94 end
Show file before | Show file after | Hide comments
@@ -0,0 +1,2
1 module IssueMovesHelper
2 end
Show file before | Show file after | Hide comments
@@ -0,0 +1,99
1 require 'test_helper'
2
3 class IssueMovesControllerTest < ActionController::TestCase
4 fixtures :all
5
6 def setup
7 User.current = nil
8 end
9
10 def test_create_one_issue_to_another_project
11 @request.session[:user_id] = 2
12 post :create, :id => 1, :new_project_id => 2, :tracker_id => '', :assigned_to_id => '', :status_id => '', :start_date => '', :due_date => ''
13 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
14 assert_equal 2, Issue.find(1).project_id
15 end
16
17 def test_create_one_issue_to_another_project_should_follow_when_needed
18 @request.session[:user_id] = 2
19 post :create, :id => 1, :new_project_id => 2, :follow => '1'
20 assert_redirected_to '/issues/1'
21 end
22
23 def test_bulk_create_to_another_project
24 @request.session[:user_id] = 2
25 post :create, :ids => [1, 2], :new_project_id => 2
26 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
27 # Issues moved to project 2
28 assert_equal 2, Issue.find(1).project_id
29 assert_equal 2, Issue.find(2).project_id
30 # No tracker change
31 assert_equal 1, Issue.find(1).tracker_id
32 assert_equal 2, Issue.find(2).tracker_id
33 end
34
35 def test_bulk_create_to_another_tracker
36 @request.session[:user_id] = 2
37 post :create, :ids => [1, 2], :new_tracker_id => 2
38 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
39 assert_equal 2, Issue.find(1).tracker_id
40 assert_equal 2, Issue.find(2).tracker_id
41 end
42
43 def test_bulk_copy_to_another_project
44 @request.session[:user_id] = 2
45 assert_difference 'Issue.count', 2 do
46 assert_no_difference 'Project.find(1).issues.count' do
47 post :create, :ids => [1, 2], :new_project_id => 2, :copy_options => {:copy => '1'}
48 end
49 end
50 assert_redirected_to 'projects/ecookbook/issues'
51 end
52
53 context "#create via bulk copy" do
54 should "allow not changing the issue's attributes" do
55 @request.session[:user_id] = 2
56 issue_before_move = Issue.find(1)
57 assert_difference 'Issue.count', 1 do
58 assert_no_difference 'Project.find(1).issues.count' do
59 post :create, :ids => [1], :new_project_id => 2, :copy_options => {:copy => '1'}, :new_tracker_id => '', :assigned_to_id => '', :status_id => '', :start_date => '', :due_date => ''
60 end
61 end
62 issue_after_move = Issue.first(:order => 'id desc', :conditions => {:project_id => 2})
63 assert_equal issue_before_move.tracker_id, issue_after_move.tracker_id
64 assert_equal issue_before_move.status_id, issue_after_move.status_id
65 assert_equal issue_before_move.assigned_to_id, issue_after_move.assigned_to_id
66 end
67
68 should "allow changing the issue's attributes" do
69 # Fixes random test failure with Mysql
70 # where Issue.all(:limit => 2, :order => 'id desc', :conditions => {:project_id => 2}) doesn't return the expected results
71 Issue.delete_all("project_id=2")
72
73 @request.session[:user_id] = 2
74 assert_difference 'Issue.count', 2 do
75 assert_no_difference 'Project.find(1).issues.count' do
76 post :create, :ids => [1, 2], :new_project_id => 2, :copy_options => {:copy => '1'}, :new_tracker_id => '', :assigned_to_id => 4, :status_id => 3, :start_date => '2009-12-01', :due_date => '2009-12-31'
77 end
78 end
79
80 copied_issues = Issue.all(:limit => 2, :order => 'id desc', :conditions => {:project_id => 2})
81 assert_equal 2, copied_issues.size
82 copied_issues.each do |issue|
83 assert_equal 2, issue.project_id, "Project is incorrect"
84 assert_equal 4, issue.assigned_to_id, "Assigned to is incorrect"
85 assert_equal 3, issue.status_id, "Status is incorrect"
86 assert_equal '2009-12-01', issue.start_date.to_s, "Start date is incorrect"
87 assert_equal '2009-12-31', issue.due_date.to_s, "Due date is incorrect"
88 end
89 end
90 end
91
92 def test_copy_to_another_project_should_follow_when_needed
93 @request.session[:user_id] = 2
94 post :create, :ids => [1], :new_project_id => 2, :copy_options => {:copy => '1'}, :follow => '1'
95 issue = Issue.first(:order => 'id DESC')
96 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
97 end
98
99 end
Show file before | Show file after | Hide comments
@@ -0,0 +1,4
1 require 'test_helper'
2
3 class IssueMovesHelperTest < ActionView::TestCase
4 end
Show file before | Show file after | Hide comments
@@ -1,502 +1,444
1 1 # Redmine - project management software
2 2 # Copyright (C) 2006-2008 Jean-Philippe Lang
3 3 #
4 4 # This program is free software; you can redistribute it and/or
5 5 # modify it under the terms of the GNU General Public License
6 6 # as published by the Free Software Foundation; either version 2
7 7 # of the License, or (at your option) any later version.
8 8 #
9 9 # This program is distributed in the hope that it will be useful,
10 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 12 # GNU General Public License for more details.
13 13 #
14 14 # You should have received a copy of the GNU General Public License
15 15 # along with this program; if not, write to the Free Software
16 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17 17
18 18 class IssuesController < ApplicationController
19 19 menu_item :new_issue, :only => [:new, :create]
20 20 default_search_scope :issues
21 21
22 22 before_filter :find_issue, :only => [:show, :edit, :update, :reply]
23 23 before_filter :find_issues, :only => [:bulk_edit, :move, :perform_move, :destroy]
24 24 before_filter :find_project, :only => [:new, :create, :update_form, :preview, :auto_complete]
25 25 before_filter :authorize, :except => [:index, :changes, :preview, :context_menu]
26 26 before_filter :find_optional_project, :only => [:index, :changes]
27 27 before_filter :check_for_default_issue_status, :only => [:new, :create]
28 28 before_filter :build_new_issue_from_params, :only => [:new, :create]
29 29 accept_key_auth :index, :show, :changes
30 30
31 31 rescue_from Query::StatementInvalid, :with => :query_statement_invalid
32 32
33 33 helper :journals
34 34 helper :projects
35 35 include ProjectsHelper
36 36 helper :custom_fields
37 37 include CustomFieldsHelper
38 38 helper :issue_relations
39 39 include IssueRelationsHelper
40 40 helper :watchers
41 41 include WatchersHelper
42 42 helper :attachments
43 43 include AttachmentsHelper
44 44 helper :queries
45 45 include QueriesHelper
46 46 helper :sort
47 47 include SortHelper
48 48 include IssuesHelper
49 49 helper :timelog
50 50 include Redmine::Export::PDF
51 51
52 52 verify :method => [:post, :delete],
53 53 :only => :destroy,
54 54 :render => { :nothing => true, :status => :method_not_allowed }
55 55
56 56 verify :method => :post, :only => :create, :render => {:nothing => true, :status => :method_not_allowed }
57 57 verify :method => :put, :only => :update, :render => {:nothing => true, :status => :method_not_allowed }
58 58
59 59 def index
60 60 retrieve_query
61 61 sort_init(@query.sort_criteria.empty? ? [['id', 'desc']] : @query.sort_criteria)
62 62 sort_update(@query.sortable_columns)
63 63
64 64 if @query.valid?
65 65 limit = case params[:format]
66 66 when 'csv', 'pdf'
67 67 Setting.issues_export_limit.to_i
68 68 when 'atom'
69 69 Setting.feeds_limit.to_i
70 70 else
71 71 per_page_option
72 72 end
73 73
74 74 @issue_count = @query.issue_count
75 75 @issue_pages = Paginator.new self, @issue_count, limit, params['page']
76 76 @issues = @query.issues(:include => [:assigned_to, :tracker, :priority, :category, :fixed_version],
77 77 :order => sort_clause,
78 78 :offset => @issue_pages.current.offset,
79 79 :limit => limit)
80 80 @issue_count_by_group = @query.issue_count_by_group
81 81
82 82 respond_to do |format|
83 83 format.html { render :template => 'issues/index.rhtml', :layout => !request.xhr? }
84 84 format.xml { render :layout => false }
85 85 format.json { render :text => @issues.to_json, :layout => false }
86 86 format.atom { render_feed(@issues, :title => "#{@project || Setting.app_title}: #{l(:label_issue_plural)}") }
87 87 format.csv { send_data(issues_to_csv(@issues, @project), :type => 'text/csv; header=present', :filename => 'export.csv') }
88 88 format.pdf { send_data(issues_to_pdf(@issues, @project, @query), :type => 'application/pdf', :filename => 'export.pdf') }
89 89 end
90 90 else
91 91 # Send html if the query is not valid
92 92 render(:template => 'issues/index.rhtml', :layout => !request.xhr?)
93 93 end
94 94 rescue ActiveRecord::RecordNotFound
95 95 render_404
96 96 end
97 97
98 98 def changes
99 99 retrieve_query
100 100 sort_init 'id', 'desc'
101 101 sort_update(@query.sortable_columns)
102 102
103 103 if @query.valid?
104 104 @journals = @query.journals(:order => "#{Journal.table_name}.created_on DESC",
105 105 :limit => 25)
106 106 end
107 107 @title = (@project ? @project.name : Setting.app_title) + ": " + (@query.new_record? ? l(:label_changes_details) : @query.name)
108 108 render :layout => false, :content_type => 'application/atom+xml'
109 109 rescue ActiveRecord::RecordNotFound
110 110 render_404
111 111 end
112 112
113 113 def show
114 114 @journals = @issue.journals.find(:all, :include => [:user, :details], :order => "#{Journal.table_name}.created_on ASC")
115 115 @journals.each_with_index {|j,i| j.indice = i+1}
116 116 @journals.reverse! if User.current.wants_comments_in_reverse_order?
117 117 @changesets = @issue.changesets.visible.all
118 118 @changesets.reverse! if User.current.wants_comments_in_reverse_order?
119 119 @allowed_statuses = @issue.new_statuses_allowed_to(User.current)
120 120 @edit_allowed = User.current.allowed_to?(:edit_issues, @project)
121 121 @priorities = IssuePriority.all
122 122 @time_entry = TimeEntry.new
123 123 respond_to do |format|
124 124 format.html { render :template => 'issues/show.rhtml' }
125 125 format.xml { render :layout => false }
126 126 format.json { render :text => @issue.to_json, :layout => false }
127 127 format.atom { render :action => 'changes', :layout => false, :content_type => 'application/atom+xml' }
128 128 format.pdf { send_data(issue_to_pdf(@issue), :type => 'application/pdf', :filename => "#{@project.identifier}-#{@issue.id}.pdf") }
129 129 end
130 130 end
131 131
132 132 # Add a new issue
133 133 # The new issue will be created from an existing one if copy_from parameter is given
134 134 def new
135 135 render :action => 'new', :layout => !request.xhr?
136 136 end
137 137
138 138 def create
139 139 call_hook(:controller_issues_new_before_save, { :params => params, :issue => @issue })
140 140 if @issue.save
141 141 attachments = Attachment.attach_files(@issue, params[:attachments])
142 142 render_attachment_warning_if_needed(@issue)
143 143 flash[:notice] = l(:notice_successful_create)
144 144 call_hook(:controller_issues_new_after_save, { :params => params, :issue => @issue})
145 145 respond_to do |format|
146 146 format.html {
147 147 redirect_to(params[:continue] ? { :action => 'new', :issue => {:tracker_id => @issue.tracker, :parent_issue_id => @issue.parent_issue_id}.reject {|k,v| v.nil?} } :
148 148 { :action => 'show', :id => @issue })
149 149 }
150 150 format.xml { render :action => 'show', :status => :created, :location => url_for(:controller => 'issues', :action => 'show', :id => @issue) }
151 151 format.json { render :text => @issue.to_json, :status => :created, :location => url_for(:controller => 'issues', :action => 'show'), :layout => false }
152 152 end
153 153 return
154 154 else
155 155 respond_to do |format|
156 156 format.html { render :action => 'new' }
157 157 format.xml { render(:xml => @issue.errors, :status => :unprocessable_entity); return }
158 158 format.json { render :text => object_errors_to_json(@issue), :status => :unprocessable_entity, :layout => false }
159 159 end
160 160 end
161 161 end
162 162
163 163 # Attributes that can be updated on workflow transition (without :edit permission)
164 164 # TODO: make it configurable (at least per role)
165 165 UPDATABLE_ATTRS_ON_TRANSITION = %w(status_id assigned_to_id fixed_version_id done_ratio) unless const_defined?(:UPDATABLE_ATTRS_ON_TRANSITION)
166 166
167 167 def edit
168 168 update_issue_from_params
169 169
170 170 @journal = @issue.current_journal
171 171
172 172 respond_to do |format|
173 173 format.html { }
174 174 format.xml { }
175 175 end
176 176 end
177 177
178 178 def update
179 179 update_issue_from_params
180 180
181 181 if @issue.save_issue_with_child_records(params, @time_entry)
182 182 render_attachment_warning_if_needed(@issue)
183 183 flash[:notice] = l(:notice_successful_update) unless @issue.current_journal.new_record?
184 184
185 185 respond_to do |format|
186 186 format.html { redirect_back_or_default({:action => 'show', :id => @issue}) }
187 187 format.xml { head :ok }
188 188 format.json { head :ok }
189 189 end
190 190 else
191 191 render_attachment_warning_if_needed(@issue)
192 192 flash[:notice] = l(:notice_successful_update) unless @issue.current_journal.new_record?
193 193 @journal = @issue.current_journal
194 194
195 195 respond_to do |format|
196 196 format.html { render :action => 'edit' }
197 197 format.xml { render :xml => @issue.errors, :status => :unprocessable_entity }
198 198 format.json { render :text => object_errors_to_json(@issue), :status => :unprocessable_entity, :layout => false }
199 199 end
200 200 end
201 201 end
202 202
203 203 def reply
204 204 journal = Journal.find(params[:journal_id]) if params[:journal_id]
205 205 if journal
206 206 user = journal.user
207 207 text = journal.notes
208 208 else
209 209 user = @issue.author
210 210 text = @issue.description
211 211 end
212 212 # Replaces pre blocks with [...]
213 213 text = text.to_s.strip.gsub(%r{<pre>((.|\s)*?)</pre>}m, '[...]')
214 214 content = "#{ll(Setting.default_language, :text_user_wrote, user)}\n> "
215 215 content << text.gsub(/(\r?\n|\r\n?)/, "\n> ") + "\n\n"
216 216
217 217 render(:update) { |page|
218 218 page.<< "$('notes').value = \"#{escape_javascript content}\";"
219 219 page.show 'update'
220 220 page << "Form.Element.focus('notes');"
221 221 page << "Element.scrollTo('update');"
222 222 page << "$('notes').scrollTop = $('notes').scrollHeight - $('notes').clientHeight;"
223 223 }
224 224 end
225 225
226 226 # Bulk edit a set of issues
227 227 def bulk_edit
228 228 @issues.sort!
229 229 if request.post?
230 230 attributes = (params[:issue] || {}).reject {|k,v| v.blank?}
231 231 attributes.keys.each {|k| attributes[k] = '' if attributes[k] == 'none'}
232 232 attributes[:custom_field_values].reject! {|k,v| v.blank?} if attributes[:custom_field_values]
233 233
234 234 unsaved_issue_ids = []
235 235 @issues.each do |issue|
236 236 issue.reload
237 237 journal = issue.init_journal(User.current, params[:notes])
238 238 issue.safe_attributes = attributes
239 239 call_hook(:controller_issues_bulk_edit_before_save, { :params => params, :issue => issue })
240 240 unless issue.save
241 241 # Keep unsaved issue ids to display them in flash error
242 242 unsaved_issue_ids << issue.id
243 243 end
244 244 end
245 245 set_flash_from_bulk_issue_save(@issues, unsaved_issue_ids)
246 246 redirect_back_or_default({:controller => 'issues', :action => 'index', :project_id => @project})
247 247 return
248 248 end
249 249 @available_statuses = Workflow.available_statuses(@project)
250 250 @custom_fields = @project.all_issue_custom_fields
251 251 end
252 252
253 def move
254 prepare_for_issue_move
255 render :layout => false if request.xhr?
256 end
257
258 # TODO: more descriptive name? move to separate controller like IssueMovesController?
259 def perform_move
260 prepare_for_issue_move
261
262 if request.post?
263 new_tracker = params[:new_tracker_id].blank? ? nil : @target_project.trackers.find_by_id(params[:new_tracker_id])
264 unsaved_issue_ids = []
265 moved_issues = []
266 @issues.each do |issue|
267 issue.reload
268 issue.init_journal(User.current)
269 call_hook(:controller_issues_move_before_save, { :params => params, :issue => issue, :target_project => @target_project, :copy => !!@copy })
270 if r = issue.move_to_project(@target_project, new_tracker, {:copy => @copy, :attributes => extract_changed_attributes_for_move(params)})
271 moved_issues << r
272 else
273 unsaved_issue_ids << issue.id
274 end
275 end
276 set_flash_from_bulk_issue_save(@issues, unsaved_issue_ids)
277
278 if params[:follow]
279 if @issues.size == 1 && moved_issues.size == 1
280 redirect_to :controller => 'issues', :action => 'show', :id => moved_issues.first
281 else
282 redirect_to :controller => 'issues', :action => 'index', :project_id => (@target_project || @project)
283 end
284 else
285 redirect_to :controller => 'issues', :action => 'index', :project_id => @project
286 end
287 return
288 end
289 end
290
291 253 def destroy
292 254 @hours = TimeEntry.sum(:hours, :conditions => ['issue_id IN (?)', @issues]).to_f
293 255 if @hours > 0
294 256 case params[:todo]
295 257 when 'destroy'
296 258 # nothing to do
297 259 when 'nullify'
298 260 TimeEntry.update_all('issue_id = NULL', ['issue_id IN (?)', @issues])
299 261 when 'reassign'
300 262 reassign_to = @project.issues.find_by_id(params[:reassign_to_id])
301 263 if reassign_to.nil?
302 264 flash.now[:error] = l(:error_issue_not_found_in_project)
303 265 return
304 266 else
305 267 TimeEntry.update_all("issue_id = #{reassign_to.id}", ['issue_id IN (?)', @issues])
306 268 end
307 269 else
308 270 unless params[:format] == 'xml' || params[:format] == 'json'
309 271 # display the destroy form if it's a user request
310 272 return
311 273 end
312 274 end
313 275 end
314 276 @issues.each(&:destroy)
315 277 respond_to do |format|
316 278 format.html { redirect_to :action => 'index', :project_id => @project }
317 279 format.xml { head :ok }
318 280 format.json { head :ok }
319 281 end
320 282 end
321 283
322 284 def context_menu
323 285 @issues = Issue.find_all_by_id(params[:ids], :include => :project)
324 286 if (@issues.size == 1)
325 287 @issue = @issues.first
326 288 @allowed_statuses = @issue.new_statuses_allowed_to(User.current)
327 289 end
328 290 projects = @issues.collect(&:project).compact.uniq
329 291 @project = projects.first if projects.size == 1
330 292
331 293 @can = {:edit => (@project && User.current.allowed_to?(:edit_issues, @project)),
332 294 :log_time => (@project && User.current.allowed_to?(:log_time, @project)),
333 295 :update => (@project && (User.current.allowed_to?(:edit_issues, @project) || (User.current.allowed_to?(:change_status, @project) && @allowed_statuses && !@allowed_statuses.empty?))),
334 296 :move => (@project && User.current.allowed_to?(:move_issues, @project)),
335 297 :copy => (@issue && @project.trackers.include?(@issue.tracker) && User.current.allowed_to?(:add_issues, @project)),
336 298 :delete => (@project && User.current.allowed_to?(:delete_issues, @project))
337 299 }
338 300 if @project
339 301 @assignables = @project.assignable_users
340 302 @assignables << @issue.assigned_to if @issue && @issue.assigned_to && !@assignables.include?(@issue.assigned_to)
341 303 @trackers = @project.trackers
342 304 end
343 305
344 306 @priorities = IssuePriority.all.reverse
345 307 @statuses = IssueStatus.find(:all, :order => 'position')
346 308 @back = back_url
347 309
348 310 render :layout => false
349 311 end
350 312
351 313 def update_form
352 314 if params[:id].blank?
353 315 @issue = Issue.new
354 316 @issue.project = @project
355 317 else
356 318 @issue = @project.issues.visible.find(params[:id])
357 319 end
358 320 @issue.attributes = params[:issue]
359 321 @allowed_statuses = ([@issue.status] + @issue.status.find_new_statuses_allowed_to(User.current.roles_for_project(@project), @issue.tracker)).uniq
360 322 @priorities = IssuePriority.all
361 323
362 324 render :partial => 'attributes'
363 325 end
364 326
365 327 def preview
366 328 @issue = @project.issues.find_by_id(params[:id]) unless params[:id].blank?
367 329 if @issue
368 330 @attachements = @issue.attachments
369 331 @description = params[:issue] && params[:issue][:description]
370 332 if @description && @description.gsub(/(\r?\n|\n\r?)/, "\n") == @issue.description.to_s.gsub(/(\r?\n|\n\r?)/, "\n")
371 333 @description = nil
372 334 end
373 335 @notes = params[:notes]
374 336 else
375 337 @description = (params[:issue] ? params[:issue][:description] : nil)
376 338 end
377 339 render :layout => false
378 340 end
379 341
380 342 def auto_complete
381 343 @issues = []
382 344 q = params[:q].to_s
383 345 if q.match(/^\d+$/)
384 346 @issues << @project.issues.visible.find_by_id(q.to_i)
385 347 end
386 348 unless q.blank?
387 349 @issues += @project.issues.visible.find(:all, :conditions => ["LOWER(#{Issue.table_name}.subject) LIKE ?", "%#{q.downcase}%"], :limit => 10)
388 350 end
389 351 render :layout => false
390 352 end
391 353
392 354 private
393 355 def find_issue
394 356 @issue = Issue.find(params[:id], :include => [:project, :tracker, :status, :author, :priority, :category])
395 357 @project = @issue.project
396 358 rescue ActiveRecord::RecordNotFound
397 359 render_404
398 360 end
399 361
400 362 # Filter for bulk operations
401 363 def find_issues
402 364 @issues = Issue.find_all_by_id(params[:id] || params[:ids])
403 365 raise ActiveRecord::RecordNotFound if @issues.empty?
404 366 projects = @issues.collect(&:project).compact.uniq
405 367 if projects.size == 1
406 368 @project = projects.first
407 369 else
408 370 # TODO: let users bulk edit/move/destroy issues from different projects
409 371 render_error 'Can not bulk edit/move/destroy issues from different projects'
410 372 return false
411 373 end
412 374 rescue ActiveRecord::RecordNotFound
413 375 render_404
414 376 end
415 377
416 378 def find_project
417 379 project_id = (params[:issue] && params[:issue][:project_id]) || params[:project_id]
418 380 @project = Project.find(project_id)
419 381 rescue ActiveRecord::RecordNotFound
420 382 render_404
421 383 end
422 384
423 385 # Used by #edit and #update to set some common instance variables
424 386 # from the params
425 387 # TODO: Refactor, not everything in here is needed by #edit
426 388 def update_issue_from_params
427 389 @allowed_statuses = @issue.new_statuses_allowed_to(User.current)
428 390 @priorities = IssuePriority.all
429 391 @edit_allowed = User.current.allowed_to?(:edit_issues, @project)
430 392 @time_entry = TimeEntry.new
431 393
432 394 @notes = params[:notes]
433 395 @issue.init_journal(User.current, @notes)
434 396 # User can change issue attributes only if he has :edit permission or if a workflow transition is allowed
435 397 if (@edit_allowed || !@allowed_statuses.empty?) && params[:issue]
436 398 attrs = params[:issue].dup
437 399 attrs.delete_if {|k,v| !UPDATABLE_ATTRS_ON_TRANSITION.include?(k) } unless @edit_allowed
438 400 attrs.delete(:status_id) unless @allowed_statuses.detect {|s| s.id.to_s == attrs[:status_id].to_s}
439 401 @issue.safe_attributes = attrs
440 402 end
441 403
442 404 end
443 405
444 406 # TODO: Refactor, lots of extra code in here
445 407 def build_new_issue_from_params
446 408 @issue = Issue.new
447 409 @issue.copy_from(params[:copy_from]) if params[:copy_from]
448 410 @issue.project = @project
449 411 # Tracker must be set before custom field values
450 412 @issue.tracker ||= @project.trackers.find((params[:issue] && params[:issue][:tracker_id]) || params[:tracker_id] || :first)
451 413 if @issue.tracker.nil?
452 414 render_error l(:error_no_tracker_in_project)
453 415 return false
454 416 end
455 417 if params[:issue].is_a?(Hash)
456 418 @issue.safe_attributes = params[:issue]
457 419 @issue.watcher_user_ids = params[:issue]['watcher_user_ids'] if User.current.allowed_to?(:add_issue_watchers, @project)
458 420 end
459 421 @issue.author = User.current
460 422 @issue.start_date ||= Date.today
461 423 @priorities = IssuePriority.all
462 424 @allowed_statuses = @issue.new_statuses_allowed_to(User.current, true)
463 425 end
464 426
465 def prepare_for_issue_move
466 @issues.sort!
467 @copy = params[:copy_options] && params[:copy_options][:copy]
468 @allowed_projects = Issue.allowed_target_projects_on_move
469 @target_project = @allowed_projects.detect {|p| p.id.to_s == params[:new_project_id]} if params[:new_project_id]
470 @target_project ||= @project
471 @trackers = @target_project.trackers
472 @available_statuses = Workflow.available_statuses(@project)
473 end
474
475 427 def set_flash_from_bulk_issue_save(issues, unsaved_issue_ids)
476 428 if unsaved_issue_ids.empty?
477 429 flash[:notice] = l(:notice_successful_update) unless issues.empty?
478 430 else
479 431 flash[:error] = l(:notice_failed_to_save_issues,
480 432 :count => unsaved_issue_ids.size,
481 433 :total => issues.size,
482 434 :ids => '#' + unsaved_issue_ids.join(', #'))
483 435 end
484 436 end
485 437
486 438 def check_for_default_issue_status
487 439 if IssueStatus.default.nil?
488 440 render_error l(:error_no_default_issue_status)
489 441 return false
490 442 end
491 443 end
492
493 def extract_changed_attributes_for_move(params)
494 changed_attributes = {}
495 [:assigned_to_id, :status_id, :start_date, :due_date].each do |valid_attribute|
496 unless params[valid_attribute].blank?
497 changed_attributes[valid_attribute] = (params[valid_attribute] == 'none' ? nil : params[valid_attribute])
498 end
499 end
500 changed_attributes
501 end
502 444 end
Show file before | Show file after | Hide comments
@@ -1,57 +1,57
1 1 <h2><%= @copy ? l(:button_copy) : l(:button_move) %></h2>
2 2
3 3 <ul>
4 4 <% @issues.each do |issue| -%>
5 5 <li><%= link_to_issue issue %></li>
6 6 <% end -%>
7 7 </ul>
8 8
9 <% form_tag({:action => 'perform_move'}, :id => 'move_form') do %>
9 <% form_tag({:action => 'create'}, :id => 'move_form') do %>
10 10 <%= @issues.collect {|i| hidden_field_tag('ids[]', i.id)}.join %>
11 11
12 12 <div class="box tabular">
13 13 <p><label for="new_project_id"><%=l(:field_project)%>:</label>
14 14 <%= select_tag "new_project_id",
15 15 project_tree_options_for_select(@allowed_projects, :selected => @target_project),
16 :onchange => remote_function(:url => { :action => 'move' },
16 :onchange => remote_function(:url => { :action => 'new' },
17 17 :method => :get,
18 18 :update => 'content',
19 19 :with => "Form.serialize('move_form')") %></p>
20 20
21 21 <p><label for="new_tracker_id"><%=l(:field_tracker)%>:</label>
22 22 <%= select_tag "new_tracker_id", "<option value=\"\">#{l(:label_no_change_option)}</option>" + options_from_collection_for_select(@trackers, "id", "name") %></p>
23 23
24 24 <p>
25 25 <label><%= l(:field_assigned_to) %></label>
26 26 <%= select_tag('assigned_to_id', content_tag('option', l(:label_no_change_option), :value => '') +
27 27 content_tag('option', l(:label_nobody), :value => 'none') +
28 28 options_from_collection_for_select(@target_project.assignable_users, :id, :name)) %>
29 29 </p>
30 30
31 31 <p>
32 32 <label><%= l(:field_status) %></label>
33 33 <%= select_tag('status_id', "<option value=\"\">#{l(:label_no_change_option)}</option>" + options_from_collection_for_select(@available_statuses, :id, :name)) %>
34 34 </p>
35 35
36 36 <p>
37 37 <label><%= l(:field_start_date) %></label>
38 38 <%= text_field_tag 'start_date', '', :size => 10 %><%= calendar_for('start_date') %>
39 39 </p>
40 40
41 41 <p>
42 42 <label><%= l(:field_due_date) %></label>
43 43 <%= text_field_tag 'due_date', '', :size => 10 %><%= calendar_for('due_date') %>
44 44 </p>
45 45
46 46 <%= call_hook(:view_issues_move_bottom, :issues => @issues, :target_project => @target_project, :copy => !!@copy) %>
47 47 </div>
48 48
49 49 <% if @copy %>
50 50 <%= hidden_field_tag("copy_options[copy]", "1") %>
51 51 <%= submit_tag l(:button_copy) %>
52 52 <%= submit_tag l(:button_copy_and_follow), :name => 'follow' %>
53 53 <% else %>
54 54 <%= submit_tag l(:button_move) %>
55 55 <%= submit_tag l(:button_move_and_follow), :name => 'follow' %>
56 56 <% end %>
57 57 <% end %>
Show file before | Show file after | Hide comments
@@ -1,10 +1,10
1 1 <div class="contextual">
2 2 <%= link_to_if_authorized(l(:button_update), {:controller => 'issues', :action => 'edit', :id => @issue }, :onclick => 'showAndScrollTo("update", "notes"); return false;', :class => 'icon icon-edit', :accesskey => accesskey(:edit)) %>
3 3 <%= link_to_if_authorized l(:button_log_time), {:controller => 'timelog', :action => 'edit', :issue_id => @issue}, :class => 'icon icon-time-add' %>
4 4 <% replace_watcher ||= 'watcher' %>
5 5 <%= watcher_tag(@issue, User.current, {:id => replace_watcher, :replace => ['watcher','watcher2']}) %>
6 6 <%= link_to_if_authorized l(:button_duplicate), {:controller => 'issues', :action => 'new', :project_id => @project, :copy_from => @issue }, :class => 'icon icon-duplicate' %>
7 <%= link_to_if_authorized l(:button_copy), {:controller => 'issues', :action => 'move', :id => @issue, :copy_options => {:copy => 't'} }, :class => 'icon icon-copy' %>
8 <%= link_to_if_authorized l(:button_move), {:controller => 'issues', :action => 'move', :id => @issue }, :class => 'icon icon-move' %>
7 <%= link_to_if_authorized l(:button_copy), new_issue_move_path(:id => @issue, :copy_options => {:copy => 't'}), :class => 'icon icon-copy' %>
8 <%= link_to_if_authorized l(:button_move), new_issue_move_path(:id => @issue), :class => 'icon icon-move' %>
9 9 <%= link_to_if_authorized l(:button_delete), {:controller => 'issues', :action => 'destroy', :id => @issue}, :confirm => l(:text_are_you_sure), :method => :post, :class => 'icon icon-del' %>
10 10 </div>
Show file before | Show file after | Hide comments
@@ -1,113 +1,113
1 1 <ul>
2 2 <%= call_hook(:view_issues_context_menu_start, {:issues => @issues, :can => @can, :back => @back }) %>
3 3
4 4 <% if !@issue.nil? -%>
5 5 <li><%= context_menu_link l(:button_edit), {:controller => 'issues', :action => 'edit', :id => @issue},
6 6 :class => 'icon-edit', :disabled => !@can[:edit] %></li>
7 7 <li class="folder">
8 8 <a href="#" class="submenu" onclick="return false;"><%= l(:field_status) %></a>
9 9 <ul>
10 10 <% @statuses.each do |s| -%>
11 11 <li><%= context_menu_link s.name, {:controller => 'issues', :action => 'update', :id => @issue, :issue => {:status_id => s}, :back_url => @back}, :method => :put,
12 12 :selected => (s == @issue.status), :disabled => !(@can[:update] && @allowed_statuses.include?(s)) %></li>
13 13 <% end -%>
14 14 </ul>
15 15 </li>
16 16 <% else %>
17 17 <li><%= context_menu_link l(:button_edit), {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id)},
18 18 :class => 'icon-edit', :disabled => !@can[:edit] %></li>
19 19 <% end %>
20 20
21 21 <% unless @trackers.nil? %>
22 22 <li class="folder">
23 23 <a href="#" class="submenu"><%= l(:field_tracker) %></a>
24 24 <ul>
25 25 <% @trackers.each do |t| -%>
26 26 <li><%= context_menu_link t.name, {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :issue => {'tracker_id' => t}, :back_url => @back}, :method => :post,
27 27 :selected => (@issue && t == @issue.tracker), :disabled => !@can[:edit] %></li>
28 28 <% end -%>
29 29 </ul>
30 30 </li>
31 31 <% end %>
32 32 <li class="folder">
33 33 <a href="#" class="submenu"><%= l(:field_priority) %></a>
34 34 <ul>
35 35 <% @priorities.each do |p| -%>
36 36 <li><%= context_menu_link p.name, {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :issue => {'priority_id' => p}, :back_url => @back}, :method => :post,
37 37 :selected => (@issue && p == @issue.priority), :disabled => (!@can[:edit] || @issues.detect {|i| !i.leaf?}) %></li>
38 38 <% end -%>
39 39 </ul>
40 40 </li>
41 41 <% unless @project.nil? || @project.shared_versions.open.empty? -%>
42 42 <li class="folder">
43 43 <a href="#" class="submenu"><%= l(:field_fixed_version) %></a>
44 44 <ul>
45 45 <% @project.shared_versions.open.sort.each do |v| -%>
46 46 <li><%= context_menu_link format_version_name(v), {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :issue => {'fixed_version_id' => v}, :back_url => @back}, :method => :post,
47 47 :selected => (@issue && v == @issue.fixed_version), :disabled => !@can[:update] %></li>
48 48 <% end -%>
49 49 <li><%= context_menu_link l(:label_none), {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :issue => {'fixed_version_id' => 'none'}, :back_url => @back}, :method => :post,
50 50 :selected => (@issue && @issue.fixed_version.nil?), :disabled => !@can[:update] %></li>
51 51 </ul>
52 52 </li>
53 53 <% end %>
54 54 <% unless @assignables.nil? || @assignables.empty? -%>
55 55 <li class="folder">
56 56 <a href="#" class="submenu"><%= l(:field_assigned_to) %></a>
57 57 <ul>
58 58 <% @assignables.each do |u| -%>
59 59 <li><%= context_menu_link u.name, {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :issue => {'assigned_to_id' => u}, :back_url => @back}, :method => :post,
60 60 :selected => (@issue && u == @issue.assigned_to), :disabled => !@can[:update] %></li>
61 61 <% end -%>
62 62 <li><%= context_menu_link l(:label_nobody), {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :issue => {'assigned_to_id' => 'none'}, :back_url => @back}, :method => :post,
63 63 :selected => (@issue && @issue.assigned_to.nil?), :disabled => !@can[:update] %></li>
64 64 </ul>
65 65 </li>
66 66 <% end %>
67 67 <% unless @project.nil? || @project.issue_categories.empty? -%>
68 68 <li class="folder">
69 69 <a href="#" class="submenu"><%= l(:field_category) %></a>
70 70 <ul>
71 71 <% @project.issue_categories.each do |u| -%>
72 72 <li><%= context_menu_link u.name, {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :issue => {'category_id' => u}, :back_url => @back}, :method => :post,
73 73 :selected => (@issue && u == @issue.category), :disabled => !@can[:update] %></li>
74 74 <% end -%>
75 75 <li><%= context_menu_link l(:label_none), {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :issue => {'category_id' => 'none'}, :back_url => @back}, :method => :post,
76 76 :selected => (@issue && @issue.category.nil?), :disabled => !@can[:update] %></li>
77 77 </ul>
78 78 </li>
79 79 <% end -%>
80 80 <% if Issue.use_field_for_done_ratio? %>
81 81 <li class="folder">
82 82 <a href="#" class="submenu"><%= l(:field_done_ratio) %></a>
83 83 <ul>
84 84 <% (0..10).map{|x|x*10}.each do |p| -%>
85 85 <li><%= context_menu_link "#{p}%", {:controller => 'issues', :action => 'bulk_edit', :ids => @issues.collect(&:id), :issue => {'done_ratio' => p}, :back_url => @back}, :method => :post,
86 86 :selected => (@issue && p == @issue.done_ratio), :disabled => (!@can[:edit] || @issues.detect {|i| !i.leaf?}) %></li>
87 87 <% end -%>
88 88 </ul>
89 89 </li>
90 90 <% end %>
91 91 <% if !@issue.nil? %>
92 92 <% if @can[:log_time] -%>
93 93 <li><%= context_menu_link l(:button_log_time), {:controller => 'timelog', :action => 'edit', :issue_id => @issue},
94 94 :class => 'icon-time-add' %></li>
95 95 <% end %>
96 96 <% if User.current.logged? %>
97 97 <li><%= watcher_link(@issue, User.current) %></li>
98 98 <% end %>
99 99 <% end %>
100 100
101 101 <% if @issue.present? %>
102 102 <li><%= context_menu_link l(:button_duplicate), {:controller => 'issues', :action => 'new', :project_id => @project, :copy_from => @issue},
103 103 :class => 'icon-duplicate', :disabled => !@can[:copy] %></li>
104 104 <% end %>
105 <li><%= context_menu_link l(:button_copy), {:controller => 'issues', :action => 'move', :ids => @issues.collect(&:id), :copy_options => {:copy => 't'}},
105 <li><%= context_menu_link l(:button_copy), new_issue_move_path(:ids => @issues.collect(&:id), :copy_options => {:copy => 't'}),
106 106 :class => 'icon-copy', :disabled => !@can[:move] %></li>
107 <li><%= context_menu_link l(:button_move), {:controller => 'issues', :action => 'move', :ids => @issues.collect(&:id)},
107 <li><%= context_menu_link l(:button_move), new_issue_move_path(:ids => @issues.collect(&:id)),
108 108 :class => 'icon-move', :disabled => !@can[:move] %></li>
109 109 <li><%= context_menu_link l(:button_delete), {:controller => 'issues', :action => 'destroy', :ids => @issues.collect(&:id)},
110 110 :method => :post, :confirm => l(:text_issues_destroy_confirmation), :class => 'icon-del', :disabled => !@can[:delete] %></li>
111 111
112 112 <%= call_hook(:view_issues_context_menu_end, {:issues => @issues, :can => @can, :back => @back }) %>
113 113 </ul>
Show file before | Show file after | Hide comments
@@ -1,293 +1,294
1 1 ActionController::Routing::Routes.draw do |map|
2 2 # Add your own custom routes here.
3 3 # The priority is based upon order of creation: first created -> highest priority.
4 4
5 5 # Here's a sample route:
6 6 # map.connect 'products/:id', :controller => 'catalog', :action => 'view'
7 7 # Keep in mind you can assign values other than :controller and :action
8 8
9 9 map.home '', :controller => 'welcome'
10 10
11 11 map.signin 'login', :controller => 'account', :action => 'login'
12 12 map.signout 'logout', :controller => 'account', :action => 'logout'
13 13
14 14 map.connect 'roles/workflow/:id/:role_id/:tracker_id', :controller => 'roles', :action => 'workflow'
15 15 map.connect 'help/:ctrl/:page', :controller => 'help'
16 16
17 17 map.connect 'time_entries/:id/edit', :action => 'edit', :controller => 'timelog'
18 18 map.connect 'projects/:project_id/time_entries/new', :action => 'edit', :controller => 'timelog'
19 19 map.connect 'projects/:project_id/issues/:issue_id/time_entries/new', :action => 'edit', :controller => 'timelog'
20 20
21 21 map.with_options :controller => 'timelog' do |timelog|
22 22 timelog.connect 'projects/:project_id/time_entries', :action => 'details'
23 23
24 24 timelog.with_options :action => 'details', :conditions => {:method => :get} do |time_details|
25 25 time_details.connect 'time_entries'
26 26 time_details.connect 'time_entries.:format'
27 27 time_details.connect 'issues/:issue_id/time_entries'
28 28 time_details.connect 'issues/:issue_id/time_entries.:format'
29 29 time_details.connect 'projects/:project_id/time_entries.:format'
30 30 time_details.connect 'projects/:project_id/issues/:issue_id/time_entries'
31 31 time_details.connect 'projects/:project_id/issues/:issue_id/time_entries.:format'
32 32 end
33 33 timelog.connect 'projects/:project_id/time_entries/report', :action => 'report'
34 34 timelog.with_options :action => 'report',:conditions => {:method => :get} do |time_report|
35 35 time_report.connect 'time_entries/report'
36 36 time_report.connect 'time_entries/report.:format'
37 37 time_report.connect 'projects/:project_id/time_entries/report.:format'
38 38 end
39 39
40 40 timelog.with_options :action => 'edit', :conditions => {:method => :get} do |time_edit|
41 41 time_edit.connect 'issues/:issue_id/time_entries/new'
42 42 end
43 43
44 44 timelog.connect 'time_entries/:id/destroy', :action => 'destroy', :conditions => {:method => :post}
45 45 end
46 46
47 47 map.connect 'projects/:id/wiki', :controller => 'wikis', :action => 'edit', :conditions => {:method => :post}
48 48 map.connect 'projects/:id/wiki/destroy', :controller => 'wikis', :action => 'destroy', :conditions => {:method => :get}
49 49 map.connect 'projects/:id/wiki/destroy', :controller => 'wikis', :action => 'destroy', :conditions => {:method => :post}
50 50 map.with_options :controller => 'wiki' do |wiki_routes|
51 51 wiki_routes.with_options :conditions => {:method => :get} do |wiki_views|
52 52 wiki_views.connect 'projects/:id/wiki/:page', :action => 'special', :page => /page_index|date_index|export/i
53 53 wiki_views.connect 'projects/:id/wiki/:page', :action => 'index', :page => nil
54 54 wiki_views.connect 'projects/:id/wiki/:page/edit', :action => 'edit'
55 55 wiki_views.connect 'projects/:id/wiki/:page/rename', :action => 'rename'
56 56 wiki_views.connect 'projects/:id/wiki/:page/history', :action => 'history'
57 57 wiki_views.connect 'projects/:id/wiki/:page/diff/:version/vs/:version_from', :action => 'diff'
58 58 wiki_views.connect 'projects/:id/wiki/:page/annotate/:version', :action => 'annotate'
59 59 end
60 60
61 61 wiki_routes.connect 'projects/:id/wiki/:page/:action',
62 62 :action => /edit|rename|destroy|preview|protect/,
63 63 :conditions => {:method => :post}
64 64 end
65 65
66 66 map.with_options :controller => 'messages' do |messages_routes|
67 67 messages_routes.with_options :conditions => {:method => :get} do |messages_views|
68 68 messages_views.connect 'boards/:board_id/topics/new', :action => 'new'
69 69 messages_views.connect 'boards/:board_id/topics/:id', :action => 'show'
70 70 messages_views.connect 'boards/:board_id/topics/:id/edit', :action => 'edit'
71 71 end
72 72 messages_routes.with_options :conditions => {:method => :post} do |messages_actions|
73 73 messages_actions.connect 'boards/:board_id/topics/new', :action => 'new'
74 74 messages_actions.connect 'boards/:board_id/topics/:id/replies', :action => 'reply'
75 75 messages_actions.connect 'boards/:board_id/topics/:id/:action', :action => /edit|destroy/
76 76 end
77 77 end
78 78
79 79 map.with_options :controller => 'boards' do |board_routes|
80 80 board_routes.with_options :conditions => {:method => :get} do |board_views|
81 81 board_views.connect 'projects/:project_id/boards', :action => 'index'
82 82 board_views.connect 'projects/:project_id/boards/new', :action => 'new'
83 83 board_views.connect 'projects/:project_id/boards/:id', :action => 'show'
84 84 board_views.connect 'projects/:project_id/boards/:id.:format', :action => 'show'
85 85 board_views.connect 'projects/:project_id/boards/:id/edit', :action => 'edit'
86 86 end
87 87 board_routes.with_options :conditions => {:method => :post} do |board_actions|
88 88 board_actions.connect 'projects/:project_id/boards', :action => 'new'
89 89 board_actions.connect 'projects/:project_id/boards/:id/:action', :action => /edit|destroy/
90 90 end
91 91 end
92 92
93 93 map.with_options :controller => 'documents' do |document_routes|
94 94 document_routes.with_options :conditions => {:method => :get} do |document_views|
95 95 document_views.connect 'projects/:project_id/documents', :action => 'index'
96 96 document_views.connect 'projects/:project_id/documents/new', :action => 'new'
97 97 document_views.connect 'documents/:id', :action => 'show'
98 98 document_views.connect 'documents/:id/edit', :action => 'edit'
99 99 end
100 100 document_routes.with_options :conditions => {:method => :post} do |document_actions|
101 101 document_actions.connect 'projects/:project_id/documents', :action => 'new'
102 102 document_actions.connect 'documents/:id/:action', :action => /destroy|edit/
103 103 end
104 104 end
105 105
106 map.resources :issue_moves, :only => [:new, :create], :path_prefix => '/issues', :as => 'move'
107
106 108 map.with_options :controller => 'issues' do |issues_routes|
107 109 issues_routes.with_options :conditions => {:method => :get} do |issues_views|
108 110 issues_views.connect 'issues', :action => 'index'
109 111 issues_views.connect 'issues.:format', :action => 'index'
110 112 issues_views.connect 'projects/:project_id/issues', :action => 'index'
111 113 issues_views.connect 'projects/:project_id/issues.:format', :action => 'index'
112 114 issues_views.connect 'projects/:project_id/issues/new', :action => 'new'
113 115 issues_views.connect 'projects/:project_id/issues/gantt', :controller => 'gantts', :action => 'show'
114 116 issues_views.connect 'projects/:project_id/issues/calendar', :controller => 'calendars', :action => 'show'
115 117 issues_views.connect 'projects/:project_id/issues/:copy_from/copy', :action => 'new'
116 118 issues_views.connect 'issues/:id', :action => 'show', :id => /\d+/
117 119 issues_views.connect 'issues/:id.:format', :action => 'show', :id => /\d+/
118 120 issues_views.connect 'issues/:id/edit', :action => 'edit', :id => /\d+/
119 issues_views.connect 'issues/:id/move', :action => 'move', :id => /\d+/
120 121 end
121 122 issues_routes.with_options :conditions => {:method => :post} do |issues_actions|
122 123 issues_actions.connect 'issues', :action => 'index'
123 124 issues_actions.connect 'projects/:project_id/issues', :action => 'create'
124 125 issues_actions.connect 'projects/:project_id/issues/gantt', :controller => 'gantts', :action => 'show'
125 126 issues_actions.connect 'projects/:project_id/issues/calendar', :controller => 'calendars', :action => 'show'
126 127 issues_actions.connect 'issues/:id/quoted', :action => 'reply', :id => /\d+/
127 issues_actions.connect 'issues/:id/:action', :action => /edit|perform_move|destroy/, :id => /\d+/
128 issues_actions.connect 'issues/:id/:action', :action => /edit|destroy/, :id => /\d+/
128 129 issues_actions.connect 'issues.:format', :action => 'create', :format => /xml/
129 130 end
130 131 issues_routes.with_options :conditions => {:method => :put} do |issues_actions|
131 132 issues_actions.connect 'issues/:id/edit', :action => 'update', :id => /\d+/
132 133 issues_actions.connect 'issues/:id.:format', :action => 'update', :id => /\d+/, :format => /xml/
133 134 end
134 135 issues_routes.with_options :conditions => {:method => :delete} do |issues_actions|
135 136 issues_actions.connect 'issues/:id.:format', :action => 'destroy', :id => /\d+/, :format => /xml/
136 137 end
137 138 issues_routes.connect 'issues/gantt', :controller => 'gantts', :action => 'show'
138 139 issues_routes.connect 'issues/calendar', :controller => 'calendars', :action => 'show'
139 140 issues_routes.connect 'issues/:action'
140 141 end
141 142
142 143 map.with_options :controller => 'issue_relations', :conditions => {:method => :post} do |relations|
143 144 relations.connect 'issues/:issue_id/relations/:id', :action => 'new'
144 145 relations.connect 'issues/:issue_id/relations/:id/destroy', :action => 'destroy'
145 146 end
146 147
147 148 map.with_options :controller => 'reports', :conditions => {:method => :get} do |reports|
148 149 reports.connect 'projects/:id/issues/report', :action => 'issue_report'
149 150 reports.connect 'projects/:id/issues/report/:detail', :action => 'issue_report_details'
150 151 end
151 152
152 153 map.with_options :controller => 'news' do |news_routes|
153 154 news_routes.with_options :conditions => {:method => :get} do |news_views|
154 155 news_views.connect 'news', :action => 'index'
155 156 news_views.connect 'projects/:project_id/news', :action => 'index'
156 157 news_views.connect 'projects/:project_id/news.:format', :action => 'index'
157 158 news_views.connect 'news.:format', :action => 'index'
158 159 news_views.connect 'projects/:project_id/news/new', :action => 'new'
159 160 news_views.connect 'news/:id', :action => 'show'
160 161 news_views.connect 'news/:id/edit', :action => 'edit'
161 162 end
162 163 news_routes.with_options do |news_actions|
163 164 news_actions.connect 'projects/:project_id/news', :action => 'new'
164 165 news_actions.connect 'news/:id/edit', :action => 'edit'
165 166 news_actions.connect 'news/:id/destroy', :action => 'destroy'
166 167 end
167 168 end
168 169
169 170 map.connect 'projects/:id/members/new', :controller => 'members', :action => 'new'
170 171
171 172 map.with_options :controller => 'users' do |users|
172 173 users.with_options :conditions => {:method => :get} do |user_views|
173 174 user_views.connect 'users', :action => 'index'
174 175 user_views.connect 'users/:id', :action => 'show', :id => /\d+/
175 176 user_views.connect 'users/new', :action => 'add'
176 177 user_views.connect 'users/:id/edit/:tab', :action => 'edit', :tab => nil
177 178 end
178 179 users.with_options :conditions => {:method => :post} do |user_actions|
179 180 user_actions.connect 'users', :action => 'add'
180 181 user_actions.connect 'users/new', :action => 'add'
181 182 user_actions.connect 'users/:id/edit', :action => 'edit'
182 183 user_actions.connect 'users/:id/memberships', :action => 'edit_membership'
183 184 user_actions.connect 'users/:id/memberships/:membership_id', :action => 'edit_membership'
184 185 user_actions.connect 'users/:id/memberships/:membership_id/destroy', :action => 'destroy_membership'
185 186 end
186 187 end
187 188
188 189 map.with_options :controller => 'projects' do |projects|
189 190 projects.with_options :conditions => {:method => :get} do |project_views|
190 191 project_views.connect 'projects', :action => 'index'
191 192 project_views.connect 'projects.:format', :action => 'index'
192 193 project_views.connect 'projects/new', :action => 'add'
193 194 project_views.connect 'projects/:id', :action => 'show'
194 195 project_views.connect 'projects/:id.:format', :action => 'show'
195 196 project_views.connect 'projects/:id/:action', :action => /roadmap|destroy|settings/
196 197 project_views.connect 'projects/:id/files', :action => 'list_files'
197 198 project_views.connect 'projects/:id/files/new', :action => 'add_file'
198 199 project_views.connect 'projects/:id/settings/:tab', :action => 'settings'
199 200 end
200 201
201 202 projects.with_options :action => 'activity', :conditions => {:method => :get} do |activity|
202 203 activity.connect 'projects/:id/activity'
203 204 activity.connect 'projects/:id/activity.:format'
204 205 activity.connect 'activity', :id => nil
205 206 activity.connect 'activity.:format', :id => nil
206 207 end
207 208
208 209 projects.with_options :conditions => {:method => :post} do |project_actions|
209 210 project_actions.connect 'projects/new', :action => 'add'
210 211 project_actions.connect 'projects', :action => 'add'
211 212 project_actions.connect 'projects.:format', :action => 'add', :format => /xml/
212 213 project_actions.connect 'projects/:id/:action', :action => /edit|destroy|archive|unarchive/
213 214 project_actions.connect 'projects/:id/files/new', :action => 'add_file'
214 215 project_actions.connect 'projects/:id/activities/save', :action => 'save_activities'
215 216 end
216 217
217 218 projects.with_options :conditions => {:method => :put} do |project_actions|
218 219 project_actions.conditions 'projects/:id.:format', :action => 'edit', :format => /xml/
219 220 end
220 221
221 222 projects.with_options :conditions => {:method => :delete} do |project_actions|
222 223 project_actions.conditions 'projects/:id.:format', :action => 'destroy', :format => /xml/
223 224 project_actions.conditions 'projects/:id/reset_activities', :action => 'reset_activities'
224 225 end
225 226 end
226 227
227 228 map.with_options :controller => 'versions' do |versions|
228 229 versions.connect 'projects/:project_id/versions/new', :action => 'new'
229 230 versions.with_options :conditions => {:method => :post} do |version_actions|
230 231 version_actions.connect 'projects/:project_id/versions/close_completed', :action => 'close_completed'
231 232 end
232 233 end
233 234
234 235 map.with_options :controller => 'issue_categories' do |categories|
235 236 categories.connect 'projects/:project_id/issue_categories/new', :action => 'new'
236 237 end
237 238
238 239 map.with_options :controller => 'repositories' do |repositories|
239 240 repositories.with_options :conditions => {:method => :get} do |repository_views|
240 241 repository_views.connect 'projects/:id/repository', :action => 'show'
241 242 repository_views.connect 'projects/:id/repository/edit', :action => 'edit'
242 243 repository_views.connect 'projects/:id/repository/statistics', :action => 'stats'
243 244 repository_views.connect 'projects/:id/repository/revisions', :action => 'revisions'
244 245 repository_views.connect 'projects/:id/repository/revisions.:format', :action => 'revisions'
245 246 repository_views.connect 'projects/:id/repository/revisions/:rev', :action => 'revision'
246 247 repository_views.connect 'projects/:id/repository/revisions/:rev/diff', :action => 'diff'
247 248 repository_views.connect 'projects/:id/repository/revisions/:rev/diff.:format', :action => 'diff'
248 249 repository_views.connect 'projects/:id/repository/revisions/:rev/raw/*path', :action => 'entry', :format => 'raw', :requirements => { :rev => /[a-z0-9\.\-_]+/ }
249 250 repository_views.connect 'projects/:id/repository/revisions/:rev/:action/*path', :requirements => { :rev => /[a-z0-9\.\-_]+/ }
250 251 repository_views.connect 'projects/:id/repository/raw/*path', :action => 'entry', :format => 'raw'
251 252 # TODO: why the following route is required?
252 253 repository_views.connect 'projects/:id/repository/entry/*path', :action => 'entry'
253 254 repository_views.connect 'projects/:id/repository/:action/*path'
254 255 end
255 256
256 257 repositories.connect 'projects/:id/repository/:action', :conditions => {:method => :post}
257 258 end
258 259
259 260 map.connect 'attachments/:id', :controller => 'attachments', :action => 'show', :id => /\d+/
260 261 map.connect 'attachments/:id/:filename', :controller => 'attachments', :action => 'show', :id => /\d+/, :filename => /.*/
261 262 map.connect 'attachments/download/:id/:filename', :controller => 'attachments', :action => 'download', :id => /\d+/, :filename => /.*/
262 263
263 264 map.resources :groups
264 265
265 266 #left old routes at the bottom for backwards compat
266 267 map.connect 'projects/:project_id/issues/:action', :controller => 'issues'
267 268 map.connect 'projects/:project_id/documents/:action', :controller => 'documents'
268 269 map.connect 'projects/:project_id/boards/:action/:id', :controller => 'boards'
269 270 map.connect 'boards/:board_id/topics/:action/:id', :controller => 'messages'
270 271 map.connect 'wiki/:id/:page/:action', :page => nil, :controller => 'wiki'
271 272 map.connect 'issues/:issue_id/relations/:action/:id', :controller => 'issue_relations'
272 273 map.connect 'projects/:project_id/news/:action', :controller => 'news'
273 274 map.connect 'projects/:project_id/timelog/:action/:id', :controller => 'timelog', :project_id => /.+/
274 275 map.with_options :controller => 'repositories' do |omap|
275 276 omap.repositories_show 'repositories/browse/:id/*path', :action => 'browse'
276 277 omap.repositories_changes 'repositories/changes/:id/*path', :action => 'changes'
277 278 omap.repositories_diff 'repositories/diff/:id/*path', :action => 'diff'
278 279 omap.repositories_entry 'repositories/entry/:id/*path', :action => 'entry'
279 280 omap.repositories_entry 'repositories/annotate/:id/*path', :action => 'annotate'
280 281 omap.connect 'repositories/revision/:id/:rev', :action => 'revision'
281 282 end
282 283
283 284 map.with_options :controller => 'sys' do |sys|
284 285 sys.connect 'sys/projects.:format', :action => 'projects', :conditions => {:method => :get}
285 286 sys.connect 'sys/projects/:id/repository.:format', :action => 'create_project_repository', :conditions => {:method => :post}
286 287 end
287 288
288 289 # Install the default route as the lowest priority.
289 290 map.connect ':controller/:action/:id'
290 291 map.connect 'robots.txt', :controller => 'welcome', :action => 'robots'
291 292 # Used for OpenID
292 293 map.root :controller => 'account', :action => 'login'
293 294 end
Show file before | Show file after | Hide comments
@@ -1,221 +1,221
1 1 require 'redmine/access_control'
2 2 require 'redmine/menu_manager'
3 3 require 'redmine/activity'
4 4 require 'redmine/search'
5 5 require 'redmine/custom_field_format'
6 6 require 'redmine/mime_type'
7 7 require 'redmine/core_ext'
8 8 require 'redmine/themes'
9 9 require 'redmine/hook'
10 10 require 'redmine/plugin'
11 11 require 'redmine/wiki_formatting'
12 12 require 'redmine/scm/base'
13 13
14 14 begin
15 15 require_library_or_gem 'RMagick' unless Object.const_defined?(:Magick)
16 16 rescue LoadError
17 17 # RMagick is not available
18 18 end
19 19
20 20 if RUBY_VERSION < '1.9'
21 21 require 'faster_csv'
22 22 else
23 23 require 'csv'
24 24 FCSV = CSV
25 25 end
26 26
27 27 Redmine::Scm::Base.add "Subversion"
28 28 Redmine::Scm::Base.add "Darcs"
29 29 Redmine::Scm::Base.add "Mercurial"
30 30 Redmine::Scm::Base.add "Cvs"
31 31 Redmine::Scm::Base.add "Bazaar"
32 32 Redmine::Scm::Base.add "Git"
33 33 Redmine::Scm::Base.add "Filesystem"
34 34
35 35 Redmine::CustomFieldFormat.map do |fields|
36 36 fields.register Redmine::CustomFieldFormat.new('string', :label => :label_string, :order => 1)
37 37 fields.register Redmine::CustomFieldFormat.new('text', :label => :label_text, :order => 2)
38 38 fields.register Redmine::CustomFieldFormat.new('int', :label => :label_integer, :order => 3)
39 39 fields.register Redmine::CustomFieldFormat.new('float', :label => :label_float, :order => 4)
40 40 fields.register Redmine::CustomFieldFormat.new('list', :label => :label_list, :order => 5)
41 41 fields.register Redmine::CustomFieldFormat.new('date', :label => :label_date, :order => 6)
42 42 fields.register Redmine::CustomFieldFormat.new('bool', :label => :label_boolean, :order => 7)
43 43 end
44 44
45 45 # Permissions
46 46 Redmine::AccessControl.map do |map|
47 47 map.permission :view_project, {:projects => [:show, :activity]}, :public => true
48 48 map.permission :search_project, {:search => :index}, :public => true
49 49 map.permission :add_project, {:projects => :add}, :require => :loggedin
50 50 map.permission :edit_project, {:projects => [:settings, :edit]}, :require => :member
51 51 map.permission :select_project_modules, {:projects => :modules}, :require => :member
52 52 map.permission :manage_members, {:projects => :settings, :members => [:new, :edit, :destroy, :autocomplete_for_member]}, :require => :member
53 53 map.permission :manage_versions, {:projects => :settings, :versions => [:new, :edit, :close_completed, :destroy]}, :require => :member
54 54 map.permission :add_subprojects, {:projects => :add}, :require => :member
55 55
56 56 map.project_module :issue_tracking do |map|
57 57 # Issue categories
58 58 map.permission :manage_categories, {:projects => :settings, :issue_categories => [:new, :edit, :destroy]}, :require => :member
59 59 # Issues
60 60 map.permission :view_issues, {:projects => :roadmap,
61 61 :issues => [:index, :changes, :show, :context_menu, :auto_complete],
62 62 :versions => [:show, :status_by],
63 63 :queries => :index,
64 64 :reports => [:issue_report, :issue_report_details]}
65 65 map.permission :add_issues, {:issues => [:new, :create, :update_form]}
66 66 map.permission :edit_issues, {:issues => [:edit, :update, :reply, :bulk_edit, :update_form]}
67 67 map.permission :manage_issue_relations, {:issue_relations => [:new, :destroy]}
68 68 map.permission :manage_subtasks, {}
69 69 map.permission :add_issue_notes, {:issues => [:edit, :update, :reply]}
70 70 map.permission :edit_issue_notes, {:journals => :edit}, :require => :loggedin
71 71 map.permission :edit_own_issue_notes, {:journals => :edit}, :require => :loggedin
72 map.permission :move_issues, {:issues => [:move, :perform_move]}, :require => :loggedin
72 map.permission :move_issues, {:issue_moves => [:new, :create]}, :require => :loggedin
73 73 map.permission :delete_issues, {:issues => :destroy}, :require => :member
74 74 # Queries
75 75 map.permission :manage_public_queries, {:queries => [:new, :edit, :destroy]}, :require => :member
76 76 map.permission :save_queries, {:queries => [:new, :edit, :destroy]}, :require => :loggedin
77 77 # Gantt & calendar
78 78 map.permission :view_gantt, :gantts => :show
79 79 map.permission :view_calendar, :calendars => :show
80 80 # Watchers
81 81 map.permission :view_issue_watchers, {}
82 82 map.permission :add_issue_watchers, {:watchers => :new}
83 83 map.permission :delete_issue_watchers, {:watchers => :destroy}
84 84 end
85 85
86 86 map.project_module :time_tracking do |map|
87 87 map.permission :log_time, {:timelog => :edit}, :require => :loggedin
88 88 map.permission :view_time_entries, :timelog => [:details, :report]
89 89 map.permission :edit_time_entries, {:timelog => [:edit, :destroy]}, :require => :member
90 90 map.permission :edit_own_time_entries, {:timelog => [:edit, :destroy]}, :require => :loggedin
91 91 map.permission :manage_project_activities, {:projects => [:save_activities, :reset_activities]}, :require => :member
92 92 end
93 93
94 94 map.project_module :news do |map|
95 95 map.permission :manage_news, {:news => [:new, :edit, :destroy, :destroy_comment]}, :require => :member
96 96 map.permission :view_news, {:news => [:index, :show]}, :public => true
97 97 map.permission :comment_news, {:news => :add_comment}
98 98 end
99 99
100 100 map.project_module :documents do |map|
101 101 map.permission :manage_documents, {:documents => [:new, :edit, :destroy, :add_attachment]}, :require => :loggedin
102 102 map.permission :view_documents, :documents => [:index, :show, :download]
103 103 end
104 104
105 105 map.project_module :files do |map|
106 106 map.permission :manage_files, {:projects => :add_file}, :require => :loggedin
107 107 map.permission :view_files, :projects => :list_files, :versions => :download
108 108 end
109 109
110 110 map.project_module :wiki do |map|
111 111 map.permission :manage_wiki, {:wikis => [:edit, :destroy]}, :require => :member
112 112 map.permission :rename_wiki_pages, {:wiki => :rename}, :require => :member
113 113 map.permission :delete_wiki_pages, {:wiki => :destroy}, :require => :member
114 114 map.permission :view_wiki_pages, :wiki => [:index, :special]
115 115 map.permission :export_wiki_pages, {}
116 116 map.permission :view_wiki_edits, :wiki => [:history, :diff, :annotate]
117 117 map.permission :edit_wiki_pages, :wiki => [:edit, :preview, :add_attachment]
118 118 map.permission :delete_wiki_pages_attachments, {}
119 119 map.permission :protect_wiki_pages, {:wiki => :protect}, :require => :member
120 120 end
121 121
122 122 map.project_module :repository do |map|
123 123 map.permission :manage_repository, {:repositories => [:edit, :committers, :destroy]}, :require => :member
124 124 map.permission :browse_repository, :repositories => [:show, :browse, :entry, :annotate, :changes, :diff, :stats, :graph]
125 125 map.permission :view_changesets, :repositories => [:show, :revisions, :revision]
126 126 map.permission :commit_access, {}
127 127 end
128 128
129 129 map.project_module :boards do |map|
130 130 map.permission :manage_boards, {:boards => [:new, :edit, :destroy]}, :require => :member
131 131 map.permission :view_messages, {:boards => [:index, :show], :messages => [:show]}, :public => true
132 132 map.permission :add_messages, {:messages => [:new, :reply, :quote]}
133 133 map.permission :edit_messages, {:messages => :edit}, :require => :member
134 134 map.permission :edit_own_messages, {:messages => :edit}, :require => :loggedin
135 135 map.permission :delete_messages, {:messages => :destroy}, :require => :member
136 136 map.permission :delete_own_messages, {:messages => :destroy}, :require => :loggedin
137 137 end
138 138 end
139 139
140 140 Redmine::MenuManager.map :top_menu do |menu|
141 141 menu.push :home, :home_path
142 142 menu.push :my_page, { :controller => 'my', :action => 'page' }, :if => Proc.new { User.current.logged? }
143 143 menu.push :projects, { :controller => 'projects', :action => 'index' }, :caption => :label_project_plural
144 144 menu.push :administration, { :controller => 'admin', :action => 'index' }, :if => Proc.new { User.current.admin? }, :last => true
145 145 menu.push :help, Redmine::Info.help_url, :last => true
146 146 end
147 147
148 148 Redmine::MenuManager.map :account_menu do |menu|
149 149 menu.push :login, :signin_path, :if => Proc.new { !User.current.logged? }
150 150 menu.push :register, { :controller => 'account', :action => 'register' }, :if => Proc.new { !User.current.logged? && Setting.self_registration? }
151 151 menu.push :my_account, { :controller => 'my', :action => 'account' }, :if => Proc.new { User.current.logged? }
152 152 menu.push :logout, :signout_path, :if => Proc.new { User.current.logged? }
153 153 end
154 154
155 155 Redmine::MenuManager.map :application_menu do |menu|
156 156 # Empty
157 157 end
158 158
159 159 Redmine::MenuManager.map :admin_menu do |menu|
160 160 menu.push :projects, {:controller => 'admin', :action => 'projects'}, :caption => :label_project_plural
161 161 menu.push :users, {:controller => 'users'}, :caption => :label_user_plural
162 162 menu.push :groups, {:controller => 'groups'}, :caption => :label_group_plural
163 163 menu.push :roles, {:controller => 'roles'}, :caption => :label_role_and_permissions
164 164 menu.push :trackers, {:controller => 'trackers'}, :caption => :label_tracker_plural
165 165 menu.push :issue_statuses, {:controller => 'issue_statuses'}, :caption => :label_issue_status_plural,
166 166 :html => {:class => 'issue_statuses'}
167 167 menu.push :workflows, {:controller => 'workflows', :action => 'edit'}, :caption => :label_workflow
168 168 menu.push :custom_fields, {:controller => 'custom_fields'}, :caption => :label_custom_field_plural,
169 169 :html => {:class => 'custom_fields'}
170 170 menu.push :enumerations, {:controller => 'enumerations'}
171 171 menu.push :settings, {:controller => 'settings'}
172 172 menu.push :ldap_authentication, {:controller => 'ldap_auth_sources', :action => 'index'},
173 173 :html => {:class => 'server_authentication'}
174 174 menu.push :plugins, {:controller => 'admin', :action => 'plugins'}, :last => true
175 175 menu.push :info, {:controller => 'admin', :action => 'info'}, :caption => :label_information_plural, :last => true
176 176 end
177 177
178 178 Redmine::MenuManager.map :project_menu do |menu|
179 179 menu.push :overview, { :controller => 'projects', :action => 'show' }
180 180 menu.push :activity, { :controller => 'projects', :action => 'activity' }
181 181 menu.push :roadmap, { :controller => 'projects', :action => 'roadmap' },
182 182 :if => Proc.new { |p| p.shared_versions.any? }
183 183 menu.push :issues, { :controller => 'issues', :action => 'index' }, :param => :project_id, :caption => :label_issue_plural
184 184 menu.push :new_issue, { :controller => 'issues', :action => 'new' }, :param => :project_id, :caption => :label_issue_new,
185 185 :html => { :accesskey => Redmine::AccessKeys.key_for(:new_issue) }
186 186 menu.push :news, { :controller => 'news', :action => 'index' }, :param => :project_id, :caption => :label_news_plural
187 187 menu.push :documents, { :controller => 'documents', :action => 'index' }, :param => :project_id, :caption => :label_document_plural
188 188 menu.push :wiki, { :controller => 'wiki', :action => 'index', :page => nil },
189 189 :if => Proc.new { |p| p.wiki && !p.wiki.new_record? }
190 190 menu.push :boards, { :controller => 'boards', :action => 'index', :id => nil }, :param => :project_id,
191 191 :if => Proc.new { |p| p.boards.any? }, :caption => :label_board_plural
192 192 menu.push :files, { :controller => 'projects', :action => 'list_files' }, :caption => :label_file_plural
193 193 menu.push :repository, { :controller => 'repositories', :action => 'show' },
194 194 :if => Proc.new { |p| p.repository && !p.repository.new_record? }
195 195 menu.push :settings, { :controller => 'projects', :action => 'settings' }, :last => true
196 196 end
197 197
198 198 Redmine::Activity.map do |activity|
199 199 activity.register :issues, :class_name => %w(Issue Journal)
200 200 activity.register :changesets
201 201 activity.register :news
202 202 activity.register :documents, :class_name => %w(Document Attachment)
203 203 activity.register :files, :class_name => 'Attachment'
204 204 activity.register :wiki_edits, :class_name => 'WikiContent::Version', :default => false
205 205 activity.register :messages, :default => false
206 206 activity.register :time_entries, :default => false
207 207 end
208 208
209 209 Redmine::Search.map do |search|
210 210 search.register :issues
211 211 search.register :news
212 212 search.register :documents
213 213 search.register :changesets
214 214 search.register :wiki_pages
215 215 search.register :messages
216 216 search.register :projects
217 217 end
218 218
219 219 Redmine::WikiFormatting.map do |format|
220 220 format.register :textile, Redmine::WikiFormatting::Textile::Formatter, Redmine::WikiFormatting::Textile::Helper
221 221 end
Show file before | Show file after | Hide comments
@@ -1,1292 +1,1203
1 1 # Redmine - project management software
2 2 # Copyright (C) 2006-2008 Jean-Philippe Lang
3 3 #
4 4 # This program is free software; you can redistribute it and/or
5 5 # modify it under the terms of the GNU General Public License
6 6 # as published by the Free Software Foundation; either version 2
7 7 # of the License, or (at your option) any later version.
8 8 #
9 9 # This program is distributed in the hope that it will be useful,
10 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 12 # GNU General Public License for more details.
13 13 #
14 14 # You should have received a copy of the GNU General Public License
15 15 # along with this program; if not, write to the Free Software
16 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17 17
18 18 require File.dirname(__FILE__) + '/../test_helper'
19 19 require 'issues_controller'
20 20
21 21 # Re-raise errors caught by the controller.
22 22 class IssuesController; def rescue_action(e) raise e end; end
23 23
24 24 class IssuesControllerTest < ActionController::TestCase
25 25 fixtures :projects,
26 26 :users,
27 27 :roles,
28 28 :members,
29 29 :member_roles,
30 30 :issues,
31 31 :issue_statuses,
32 32 :versions,
33 33 :trackers,
34 34 :projects_trackers,
35 35 :issue_categories,
36 36 :enabled_modules,
37 37 :enumerations,
38 38 :attachments,
39 39 :workflows,
40 40 :custom_fields,
41 41 :custom_values,
42 42 :custom_fields_projects,
43 43 :custom_fields_trackers,
44 44 :time_entries,
45 45 :journals,
46 46 :journal_details,
47 47 :queries
48 48
49 49 def setup
50 50 @controller = IssuesController.new
51 51 @request = ActionController::TestRequest.new
52 52 @response = ActionController::TestResponse.new
53 53 User.current = nil
54 54 end
55 55
56 56 def test_index
57 57 Setting.default_language = 'en'
58 58
59 59 get :index
60 60 assert_response :success
61 61 assert_template 'index.rhtml'
62 62 assert_not_nil assigns(:issues)
63 63 assert_nil assigns(:project)
64 64 assert_tag :tag => 'a', :content => /Can't print recipes/
65 65 assert_tag :tag => 'a', :content => /Subproject issue/
66 66 # private projects hidden
67 67 assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
68 68 assert_no_tag :tag => 'a', :content => /Issue on project 2/
69 69 # project column
70 70 assert_tag :tag => 'th', :content => /Project/
71 71 end
72 72
73 73 def test_index_should_not_list_issues_when_module_disabled
74 74 EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
75 75 get :index
76 76 assert_response :success
77 77 assert_template 'index.rhtml'
78 78 assert_not_nil assigns(:issues)
79 79 assert_nil assigns(:project)
80 80 assert_no_tag :tag => 'a', :content => /Can't print recipes/
81 81 assert_tag :tag => 'a', :content => /Subproject issue/
82 82 end
83 83
84 84 def test_index_should_not_list_issues_when_module_disabled
85 85 EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
86 86 get :index
87 87 assert_response :success
88 88 assert_template 'index.rhtml'
89 89 assert_not_nil assigns(:issues)
90 90 assert_nil assigns(:project)
91 91 assert_no_tag :tag => 'a', :content => /Can't print recipes/
92 92 assert_tag :tag => 'a', :content => /Subproject issue/
93 93 end
94 94
95 95 def test_index_with_project
96 96 Setting.display_subprojects_issues = 0
97 97 get :index, :project_id => 1
98 98 assert_response :success
99 99 assert_template 'index.rhtml'
100 100 assert_not_nil assigns(:issues)
101 101 assert_tag :tag => 'a', :content => /Can't print recipes/
102 102 assert_no_tag :tag => 'a', :content => /Subproject issue/
103 103 end
104 104
105 105 def test_index_with_project_and_subprojects
106 106 Setting.display_subprojects_issues = 1
107 107 get :index, :project_id => 1
108 108 assert_response :success
109 109 assert_template 'index.rhtml'
110 110 assert_not_nil assigns(:issues)
111 111 assert_tag :tag => 'a', :content => /Can't print recipes/
112 112 assert_tag :tag => 'a', :content => /Subproject issue/
113 113 assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
114 114 end
115 115
116 116 def test_index_with_project_and_subprojects_should_show_private_subprojects
117 117 @request.session[:user_id] = 2
118 118 Setting.display_subprojects_issues = 1
119 119 get :index, :project_id => 1
120 120 assert_response :success
121 121 assert_template 'index.rhtml'
122 122 assert_not_nil assigns(:issues)
123 123 assert_tag :tag => 'a', :content => /Can't print recipes/
124 124 assert_tag :tag => 'a', :content => /Subproject issue/
125 125 assert_tag :tag => 'a', :content => /Issue of a private subproject/
126 126 end
127 127
128 128 def test_index_with_project_and_filter
129 129 get :index, :project_id => 1, :set_filter => 1
130 130 assert_response :success
131 131 assert_template 'index.rhtml'
132 132 assert_not_nil assigns(:issues)
133 133 end
134 134
135 135 def test_index_with_query
136 136 get :index, :project_id => 1, :query_id => 5
137 137 assert_response :success
138 138 assert_template 'index.rhtml'
139 139 assert_not_nil assigns(:issues)
140 140 assert_nil assigns(:issue_count_by_group)
141 141 end
142 142
143 143 def test_index_with_query_grouped_by_tracker
144 144 get :index, :project_id => 1, :query_id => 6
145 145 assert_response :success
146 146 assert_template 'index.rhtml'
147 147 assert_not_nil assigns(:issues)
148 148 assert_not_nil assigns(:issue_count_by_group)
149 149 end
150 150
151 151 def test_index_with_query_grouped_by_list_custom_field
152 152 get :index, :project_id => 1, :query_id => 9
153 153 assert_response :success
154 154 assert_template 'index.rhtml'
155 155 assert_not_nil assigns(:issues)
156 156 assert_not_nil assigns(:issue_count_by_group)
157 157 end
158 158
159 159 def test_index_sort_by_field_not_included_in_columns
160 160 Setting.issue_list_default_columns = %w(subject author)
161 161 get :index, :sort => 'tracker'
162 162 end
163 163
164 164 def test_index_csv_with_project
165 165 Setting.default_language = 'en'
166 166
167 167 get :index, :format => 'csv'
168 168 assert_response :success
169 169 assert_not_nil assigns(:issues)
170 170 assert_equal 'text/csv', @response.content_type
171 171 assert @response.body.starts_with?("#,")
172 172
173 173 get :index, :project_id => 1, :format => 'csv'
174 174 assert_response :success
175 175 assert_not_nil assigns(:issues)
176 176 assert_equal 'text/csv', @response.content_type
177 177 end
178 178
179 179 def test_index_pdf
180 180 get :index, :format => 'pdf'
181 181 assert_response :success
182 182 assert_not_nil assigns(:issues)
183 183 assert_equal 'application/pdf', @response.content_type
184 184
185 185 get :index, :project_id => 1, :format => 'pdf'
186 186 assert_response :success
187 187 assert_not_nil assigns(:issues)
188 188 assert_equal 'application/pdf', @response.content_type
189 189
190 190 get :index, :project_id => 1, :query_id => 6, :format => 'pdf'
191 191 assert_response :success
192 192 assert_not_nil assigns(:issues)
193 193 assert_equal 'application/pdf', @response.content_type
194 194 end
195 195
196 196 def test_index_pdf_with_query_grouped_by_list_custom_field
197 197 get :index, :project_id => 1, :query_id => 9, :format => 'pdf'
198 198 assert_response :success
199 199 assert_not_nil assigns(:issues)
200 200 assert_not_nil assigns(:issue_count_by_group)
201 201 assert_equal 'application/pdf', @response.content_type
202 202 end
203 203
204 204 def test_index_sort
205 205 get :index, :sort => 'tracker,id:desc'
206 206 assert_response :success
207 207
208 208 sort_params = @request.session['issues_index_sort']
209 209 assert sort_params.is_a?(String)
210 210 assert_equal 'tracker,id:desc', sort_params
211 211
212 212 issues = assigns(:issues)
213 213 assert_not_nil issues
214 214 assert !issues.empty?
215 215 assert_equal issues.sort {|a,b| a.tracker == b.tracker ? b.id <=> a.id : a.tracker <=> b.tracker }.collect(&:id), issues.collect(&:id)
216 216 end
217 217
218 218 def test_index_with_columns
219 219 columns = ['tracker', 'subject', 'assigned_to']
220 220 get :index, :set_filter => 1, :query => { 'column_names' => columns}
221 221 assert_response :success
222 222
223 223 # query should use specified columns
224 224 query = assigns(:query)
225 225 assert_kind_of Query, query
226 226 assert_equal columns, query.column_names.map(&:to_s)
227 227
228 228 # columns should be stored in session
229 229 assert_kind_of Hash, session[:query]
230 230 assert_kind_of Array, session[:query][:column_names]
231 231 assert_equal columns, session[:query][:column_names].map(&:to_s)
232 232 end
233 233
234 234 def test_changes
235 235 get :changes, :project_id => 1
236 236 assert_response :success
237 237 assert_not_nil assigns(:journals)
238 238 assert_equal 'application/atom+xml', @response.content_type
239 239 end
240 240
241 241 def test_show_by_anonymous
242 242 get :show, :id => 1
243 243 assert_response :success
244 244 assert_template 'show.rhtml'
245 245 assert_not_nil assigns(:issue)
246 246 assert_equal Issue.find(1), assigns(:issue)
247 247
248 248 # anonymous role is allowed to add a note
249 249 assert_tag :tag => 'form',
250 250 :descendant => { :tag => 'fieldset',
251 251 :child => { :tag => 'legend',
252 252 :content => /Notes/ } }
253 253 end
254 254
255 255 def test_show_by_manager
256 256 @request.session[:user_id] = 2
257 257 get :show, :id => 1
258 258 assert_response :success
259 259
260 260 assert_tag :tag => 'form',
261 261 :descendant => { :tag => 'fieldset',
262 262 :child => { :tag => 'legend',
263 263 :content => /Change properties/ } },
264 264 :descendant => { :tag => 'fieldset',
265 265 :child => { :tag => 'legend',
266 266 :content => /Log time/ } },
267 267 :descendant => { :tag => 'fieldset',
268 268 :child => { :tag => 'legend',
269 269 :content => /Notes/ } }
270 270 end
271 271
272 272 def test_show_should_deny_anonymous_access_without_permission
273 273 Role.anonymous.remove_permission!(:view_issues)
274 274 get :show, :id => 1
275 275 assert_response :redirect
276 276 end
277 277
278 278 def test_show_should_deny_non_member_access_without_permission
279 279 Role.non_member.remove_permission!(:view_issues)
280 280 @request.session[:user_id] = 9
281 281 get :show, :id => 1
282 282 assert_response 403
283 283 end
284 284
285 285 def test_show_should_deny_member_access_without_permission
286 286 Role.find(1).remove_permission!(:view_issues)
287 287 @request.session[:user_id] = 2
288 288 get :show, :id => 1
289 289 assert_response 403
290 290 end
291 291
292 292 def test_show_should_not_disclose_relations_to_invisible_issues
293 293 Setting.cross_project_issue_relations = '1'
294 294 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => 'relates')
295 295 # Relation to a private project issue
296 296 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(4), :relation_type => 'relates')
297 297
298 298 get :show, :id => 1
299 299 assert_response :success
300 300
301 301 assert_tag :div, :attributes => { :id => 'relations' },
302 302 :descendant => { :tag => 'a', :content => /#2$/ }
303 303 assert_no_tag :div, :attributes => { :id => 'relations' },
304 304 :descendant => { :tag => 'a', :content => /#4$/ }
305 305 end
306 306
307 307 def test_show_atom
308 308 get :show, :id => 2, :format => 'atom'
309 309 assert_response :success
310 310 assert_template 'changes.rxml'
311 311 # Inline image
312 312 assert_select 'content', :text => Regexp.new(Regexp.quote('http://test.host/attachments/download/10'))
313 313 end
314 314
315 315 def test_show_export_to_pdf
316 316 get :show, :id => 3, :format => 'pdf'
317 317 assert_response :success
318 318 assert_equal 'application/pdf', @response.content_type
319 319 assert @response.body.starts_with?('%PDF')
320 320 assert_not_nil assigns(:issue)
321 321 end
322 322
323 323 def test_get_new
324 324 @request.session[:user_id] = 2
325 325 get :new, :project_id => 1, :tracker_id => 1
326 326 assert_response :success
327 327 assert_template 'new'
328 328
329 329 assert_tag :tag => 'input', :attributes => { :name => 'issue[custom_field_values][2]',
330 330 :value => 'Default string' }
331 331 end
332 332
333 333 def test_get_new_without_tracker_id
334 334 @request.session[:user_id] = 2
335 335 get :new, :project_id => 1
336 336 assert_response :success
337 337 assert_template 'new'
338 338
339 339 issue = assigns(:issue)
340 340 assert_not_nil issue
341 341 assert_equal Project.find(1).trackers.first, issue.tracker
342 342 end
343 343
344 344 def test_get_new_with_no_default_status_should_display_an_error
345 345 @request.session[:user_id] = 2
346 346 IssueStatus.delete_all
347 347
348 348 get :new, :project_id => 1
349 349 assert_response 500
350 350 assert_not_nil flash[:error]
351 351 assert_tag :tag => 'div', :attributes => { :class => /error/ },
352 352 :content => /No default issue/
353 353 end
354 354
355 355 def test_get_new_with_no_tracker_should_display_an_error
356 356 @request.session[:user_id] = 2
357 357 Tracker.delete_all
358 358
359 359 get :new, :project_id => 1
360 360 assert_response 500
361 361 assert_not_nil flash[:error]
362 362 assert_tag :tag => 'div', :attributes => { :class => /error/ },
363 363 :content => /No tracker/
364 364 end
365 365
366 366 def test_update_new_form
367 367 @request.session[:user_id] = 2
368 368 xhr :post, :update_form, :project_id => 1,
369 369 :issue => {:tracker_id => 2,
370 370 :subject => 'This is the test_new issue',
371 371 :description => 'This is the description',
372 372 :priority_id => 5}
373 373 assert_response :success
374 374 assert_template 'attributes'
375 375
376 376 issue = assigns(:issue)
377 377 assert_kind_of Issue, issue
378 378 assert_equal 1, issue.project_id
379 379 assert_equal 2, issue.tracker_id
380 380 assert_equal 'This is the test_new issue', issue.subject
381 381 end
382 382
383 383 def test_post_create
384 384 @request.session[:user_id] = 2
385 385 assert_difference 'Issue.count' do
386 386 post :create, :project_id => 1,
387 387 :issue => {:tracker_id => 3,
388 388 :status_id => 2,
389 389 :subject => 'This is the test_new issue',
390 390 :description => 'This is the description',
391 391 :priority_id => 5,
392 392 :estimated_hours => '',
393 393 :custom_field_values => {'2' => 'Value for field 2'}}
394 394 end
395 395 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
396 396
397 397 issue = Issue.find_by_subject('This is the test_new issue')
398 398 assert_not_nil issue
399 399 assert_equal 2, issue.author_id
400 400 assert_equal 3, issue.tracker_id
401 401 assert_equal 2, issue.status_id
402 402 assert_nil issue.estimated_hours
403 403 v = issue.custom_values.find(:first, :conditions => {:custom_field_id => 2})
404 404 assert_not_nil v
405 405 assert_equal 'Value for field 2', v.value
406 406 end
407 407
408 408 def test_post_create_and_continue
409 409 @request.session[:user_id] = 2
410 410 post :create, :project_id => 1,
411 411 :issue => {:tracker_id => 3,
412 412 :subject => 'This is first issue',
413 413 :priority_id => 5},
414 414 :continue => ''
415 415 assert_redirected_to :controller => 'issues', :action => 'new', :issue => {:tracker_id => 3}
416 416 end
417 417
418 418 def test_post_create_without_custom_fields_param
419 419 @request.session[:user_id] = 2
420 420 assert_difference 'Issue.count' do
421 421 post :create, :project_id => 1,
422 422 :issue => {:tracker_id => 1,
423 423 :subject => 'This is the test_new issue',
424 424 :description => 'This is the description',
425 425 :priority_id => 5}
426 426 end
427 427 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
428 428 end
429 429
430 430 def test_post_create_with_required_custom_field_and_without_custom_fields_param
431 431 field = IssueCustomField.find_by_name('Database')
432 432 field.update_attribute(:is_required, true)
433 433
434 434 @request.session[:user_id] = 2
435 435 post :create, :project_id => 1,
436 436 :issue => {:tracker_id => 1,
437 437 :subject => 'This is the test_new issue',
438 438 :description => 'This is the description',
439 439 :priority_id => 5}
440 440 assert_response :success
441 441 assert_template 'new'
442 442 issue = assigns(:issue)
443 443 assert_not_nil issue
444 444 assert_equal I18n.translate('activerecord.errors.messages.invalid'), issue.errors.on(:custom_values)
445 445 end
446 446
447 447 def test_post_create_with_watchers
448 448 @request.session[:user_id] = 2
449 449 ActionMailer::Base.deliveries.clear
450 450
451 451 assert_difference 'Watcher.count', 2 do
452 452 post :create, :project_id => 1,
453 453 :issue => {:tracker_id => 1,
454 454 :subject => 'This is a new issue with watchers',
455 455 :description => 'This is the description',
456 456 :priority_id => 5,
457 457 :watcher_user_ids => ['2', '3']}
458 458 end
459 459 issue = Issue.find_by_subject('This is a new issue with watchers')
460 460 assert_not_nil issue
461 461 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
462 462
463 463 # Watchers added
464 464 assert_equal [2, 3], issue.watcher_user_ids.sort
465 465 assert issue.watched_by?(User.find(3))
466 466 # Watchers notified
467 467 mail = ActionMailer::Base.deliveries.last
468 468 assert_kind_of TMail::Mail, mail
469 469 assert [mail.bcc, mail.cc].flatten.include?(User.find(3).mail)
470 470 end
471 471
472 472 def test_post_create_subissue
473 473 @request.session[:user_id] = 2
474 474
475 475 assert_difference 'Issue.count' do
476 476 post :create, :project_id => 1,
477 477 :issue => {:tracker_id => 1,
478 478 :subject => 'This is a child issue',
479 479 :parent_issue_id => 2}
480 480 end
481 481 issue = Issue.find_by_subject('This is a child issue')
482 482 assert_not_nil issue
483 483 assert_equal Issue.find(2), issue.parent
484 484 end
485 485
486 486 def test_post_create_should_send_a_notification
487 487 ActionMailer::Base.deliveries.clear
488 488 @request.session[:user_id] = 2
489 489 assert_difference 'Issue.count' do
490 490 post :create, :project_id => 1,
491 491 :issue => {:tracker_id => 3,
492 492 :subject => 'This is the test_new issue',
493 493 :description => 'This is the description',
494 494 :priority_id => 5,
495 495 :estimated_hours => '',
496 496 :custom_field_values => {'2' => 'Value for field 2'}}
497 497 end
498 498 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
499 499
500 500 assert_equal 1, ActionMailer::Base.deliveries.size
501 501 end
502 502
503 503 def test_post_create_should_preserve_fields_values_on_validation_failure
504 504 @request.session[:user_id] = 2
505 505 post :create, :project_id => 1,
506 506 :issue => {:tracker_id => 1,
507 507 # empty subject
508 508 :subject => '',
509 509 :description => 'This is a description',
510 510 :priority_id => 6,
511 511 :custom_field_values => {'1' => 'Oracle', '2' => 'Value for field 2'}}
512 512 assert_response :success
513 513 assert_template 'new'
514 514
515 515 assert_tag :textarea, :attributes => { :name => 'issue[description]' },
516 516 :content => 'This is a description'
517 517 assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
518 518 :child => { :tag => 'option', :attributes => { :selected => 'selected',
519 519 :value => '6' },
520 520 :content => 'High' }
521 521 # Custom fields
522 522 assert_tag :select, :attributes => { :name => 'issue[custom_field_values][1]' },
523 523 :child => { :tag => 'option', :attributes => { :selected => 'selected',
524 524 :value => 'Oracle' },
525 525 :content => 'Oracle' }
526 526 assert_tag :input, :attributes => { :name => 'issue[custom_field_values][2]',
527 527 :value => 'Value for field 2'}
528 528 end
529 529
530 530 def test_post_create_should_ignore_non_safe_attributes
531 531 @request.session[:user_id] = 2
532 532 assert_nothing_raised do
533 533 post :create, :project_id => 1, :issue => { :tracker => "A param can not be a Tracker" }
534 534 end
535 535 end
536 536
537 537 context "without workflow privilege" do
538 538 setup do
539 539 Workflow.delete_all(["role_id = ?", Role.anonymous.id])
540 540 Role.anonymous.add_permission! :add_issues
541 541 end
542 542
543 543 context "#new" do
544 544 should "propose default status only" do
545 545 get :new, :project_id => 1
546 546 assert_response :success
547 547 assert_template 'new'
548 548 assert_tag :tag => 'select',
549 549 :attributes => {:name => 'issue[status_id]'},
550 550 :children => {:count => 1},
551 551 :child => {:tag => 'option', :attributes => {:value => IssueStatus.default.id.to_s}}
552 552 end
553 553
554 554 should "accept default status" do
555 555 assert_difference 'Issue.count' do
556 556 post :create, :project_id => 1,
557 557 :issue => {:tracker_id => 1,
558 558 :subject => 'This is an issue',
559 559 :status_id => 1}
560 560 end
561 561 issue = Issue.last(:order => 'id')
562 562 assert_equal IssueStatus.default, issue.status
563 563 end
564 564
565 565 should "ignore unauthorized status" do
566 566 assert_difference 'Issue.count' do
567 567 post :create, :project_id => 1,
568 568 :issue => {:tracker_id => 1,
569 569 :subject => 'This is an issue',
570 570 :status_id => 3}
571 571 end
572 572 issue = Issue.last(:order => 'id')
573 573 assert_equal IssueStatus.default, issue.status
574 574 end
575 575 end
576 576 end
577 577
578 578 def test_copy_issue
579 579 @request.session[:user_id] = 2
580 580 get :new, :project_id => 1, :copy_from => 1
581 581 assert_template 'new'
582 582 assert_not_nil assigns(:issue)
583 583 orig = Issue.find(1)
584 584 assert_equal orig.subject, assigns(:issue).subject
585 585 end
586 586
587 587 def test_get_edit
588 588 @request.session[:user_id] = 2
589 589 get :edit, :id => 1
590 590 assert_response :success
591 591 assert_template 'edit'
592 592 assert_not_nil assigns(:issue)
593 593 assert_equal Issue.find(1), assigns(:issue)
594 594 end
595 595
596 596 def test_get_edit_with_params
597 597 @request.session[:user_id] = 2
598 598 get :edit, :id => 1, :issue => { :status_id => 5, :priority_id => 7 }
599 599 assert_response :success
600 600 assert_template 'edit'
601 601
602 602 issue = assigns(:issue)
603 603 assert_not_nil issue
604 604
605 605 assert_equal 5, issue.status_id
606 606 assert_tag :select, :attributes => { :name => 'issue[status_id]' },
607 607 :child => { :tag => 'option',
608 608 :content => 'Closed',
609 609 :attributes => { :selected => 'selected' } }
610 610
611 611 assert_equal 7, issue.priority_id
612 612 assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
613 613 :child => { :tag => 'option',
614 614 :content => 'Urgent',
615 615 :attributes => { :selected => 'selected' } }
616 616 end
617 617
618 618 def test_update_edit_form
619 619 @request.session[:user_id] = 2
620 620 xhr :post, :update_form, :project_id => 1,
621 621 :id => 1,
622 622 :issue => {:tracker_id => 2,
623 623 :subject => 'This is the test_new issue',
624 624 :description => 'This is the description',
625 625 :priority_id => 5}
626 626 assert_response :success
627 627 assert_template 'attributes'
628 628
629 629 issue = assigns(:issue)
630 630 assert_kind_of Issue, issue
631 631 assert_equal 1, issue.id
632 632 assert_equal 1, issue.project_id
633 633 assert_equal 2, issue.tracker_id
634 634 assert_equal 'This is the test_new issue', issue.subject
635 635 end
636 636
637 637 def test_reply_to_issue
638 638 @request.session[:user_id] = 2
639 639 get :reply, :id => 1
640 640 assert_response :success
641 641 assert_select_rjs :show, "update"
642 642 end
643 643
644 644 def test_reply_to_note
645 645 @request.session[:user_id] = 2
646 646 get :reply, :id => 1, :journal_id => 2
647 647 assert_response :success
648 648 assert_select_rjs :show, "update"
649 649 end
650 650
651 651 def test_update_using_invalid_http_verbs
652 652 @request.session[:user_id] = 2
653 653 subject = 'Updated by an invalid http verb'
654 654
655 655 get :update, :id => 1, :issue => {:subject => subject}
656 656 assert_not_equal subject, Issue.find(1).subject
657 657
658 658 post :update, :id => 1, :issue => {:subject => subject}
659 659 assert_not_equal subject, Issue.find(1).subject
660 660
661 661 delete :update, :id => 1, :issue => {:subject => subject}
662 662 assert_not_equal subject, Issue.find(1).subject
663 663 end
664 664
665 665 def test_put_update_without_custom_fields_param
666 666 @request.session[:user_id] = 2
667 667 ActionMailer::Base.deliveries.clear
668 668
669 669 issue = Issue.find(1)
670 670 assert_equal '125', issue.custom_value_for(2).value
671 671 old_subject = issue.subject
672 672 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
673 673
674 674 assert_difference('Journal.count') do
675 675 assert_difference('JournalDetail.count', 2) do
676 676 put :update, :id => 1, :issue => {:subject => new_subject,
677 677 :priority_id => '6',
678 678 :category_id => '1' # no change
679 679 }
680 680 end
681 681 end
682 682 assert_redirected_to :action => 'show', :id => '1'
683 683 issue.reload
684 684 assert_equal new_subject, issue.subject
685 685 # Make sure custom fields were not cleared
686 686 assert_equal '125', issue.custom_value_for(2).value
687 687
688 688 mail = ActionMailer::Base.deliveries.last
689 689 assert_kind_of TMail::Mail, mail
690 690 assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
691 691 assert mail.body.include?("Subject changed from #{old_subject} to #{new_subject}")
692 692 end
693 693
694 694 def test_put_update_with_custom_field_change
695 695 @request.session[:user_id] = 2
696 696 issue = Issue.find(1)
697 697 assert_equal '125', issue.custom_value_for(2).value
698 698
699 699 assert_difference('Journal.count') do
700 700 assert_difference('JournalDetail.count', 3) do
701 701 put :update, :id => 1, :issue => {:subject => 'Custom field change',
702 702 :priority_id => '6',
703 703 :category_id => '1', # no change
704 704 :custom_field_values => { '2' => 'New custom value' }
705 705 }
706 706 end
707 707 end
708 708 assert_redirected_to :action => 'show', :id => '1'
709 709 issue.reload
710 710 assert_equal 'New custom value', issue.custom_value_for(2).value
711 711
712 712 mail = ActionMailer::Base.deliveries.last
713 713 assert_kind_of TMail::Mail, mail
714 714 assert mail.body.include?("Searchable field changed from 125 to New custom value")
715 715 end
716 716
717 717 def test_put_update_with_status_and_assignee_change
718 718 issue = Issue.find(1)
719 719 assert_equal 1, issue.status_id
720 720 @request.session[:user_id] = 2
721 721 assert_difference('TimeEntry.count', 0) do
722 722 put :update,
723 723 :id => 1,
724 724 :issue => { :status_id => 2, :assigned_to_id => 3 },
725 725 :notes => 'Assigned to dlopper',
726 726 :time_entry => { :hours => '', :comments => '', :activity_id => TimeEntryActivity.first }
727 727 end
728 728 assert_redirected_to :action => 'show', :id => '1'
729 729 issue.reload
730 730 assert_equal 2, issue.status_id
731 731 j = Journal.find(:first, :order => 'id DESC')
732 732 assert_equal 'Assigned to dlopper', j.notes
733 733 assert_equal 2, j.details.size
734 734
735 735 mail = ActionMailer::Base.deliveries.last
736 736 assert mail.body.include?("Status changed from New to Assigned")
737 737 # subject should contain the new status
738 738 assert mail.subject.include?("(#{ IssueStatus.find(2).name })")
739 739 end
740 740
741 741 def test_put_update_with_note_only
742 742 notes = 'Note added by IssuesControllerTest#test_update_with_note_only'
743 743 # anonymous user
744 744 put :update,
745 745 :id => 1,
746 746 :notes => notes
747 747 assert_redirected_to :action => 'show', :id => '1'
748 748 j = Journal.find(:first, :order => 'id DESC')
749 749 assert_equal notes, j.notes
750 750 assert_equal 0, j.details.size
751 751 assert_equal User.anonymous, j.user
752 752
753 753 mail = ActionMailer::Base.deliveries.last
754 754 assert mail.body.include?(notes)
755 755 end
756 756
757 757 def test_put_update_with_note_and_spent_time
758 758 @request.session[:user_id] = 2
759 759 spent_hours_before = Issue.find(1).spent_hours
760 760 assert_difference('TimeEntry.count') do
761 761 put :update,
762 762 :id => 1,
763 763 :notes => '2.5 hours added',
764 764 :time_entry => { :hours => '2.5', :comments => 'test_put_update_with_note_and_spent_time', :activity_id => TimeEntryActivity.first.id }
765 765 end
766 766 assert_redirected_to :action => 'show', :id => '1'
767 767
768 768 issue = Issue.find(1)
769 769
770 770 j = Journal.find(:first, :order => 'id DESC')
771 771 assert_equal '2.5 hours added', j.notes
772 772 assert_equal 0, j.details.size
773 773
774 774 t = issue.time_entries.find_by_comments('test_put_update_with_note_and_spent_time')
775 775 assert_not_nil t
776 776 assert_equal 2.5, t.hours
777 777 assert_equal spent_hours_before + 2.5, issue.spent_hours
778 778 end
779 779
780 780 def test_put_update_with_attachment_only
781 781 set_tmp_attachments_directory
782 782
783 783 # Delete all fixtured journals, a race condition can occur causing the wrong
784 784 # journal to get fetched in the next find.
785 785 Journal.delete_all
786 786
787 787 # anonymous user
788 788 put :update,
789 789 :id => 1,
790 790 :notes => '',
791 791 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
792 792 assert_redirected_to :action => 'show', :id => '1'
793 793 j = Issue.find(1).journals.find(:first, :order => 'id DESC')
794 794 assert j.notes.blank?
795 795 assert_equal 1, j.details.size
796 796 assert_equal 'testfile.txt', j.details.first.value
797 797 assert_equal User.anonymous, j.user
798 798
799 799 mail = ActionMailer::Base.deliveries.last
800 800 assert mail.body.include?('testfile.txt')
801 801 end
802 802
803 803 def test_put_update_with_attachment_that_fails_to_save
804 804 set_tmp_attachments_directory
805 805
806 806 # Delete all fixtured journals, a race condition can occur causing the wrong
807 807 # journal to get fetched in the next find.
808 808 Journal.delete_all
809 809
810 810 # Mock out the unsaved attachment
811 811 Attachment.any_instance.stubs(:create).returns(Attachment.new)
812 812
813 813 # anonymous user
814 814 put :update,
815 815 :id => 1,
816 816 :notes => '',
817 817 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
818 818 assert_redirected_to :action => 'show', :id => '1'
819 819 assert_equal '1 file(s) could not be saved.', flash[:warning]
820 820
821 821 end if Object.const_defined?(:Mocha)
822 822
823 823 def test_put_update_with_no_change
824 824 issue = Issue.find(1)
825 825 issue.journals.clear
826 826 ActionMailer::Base.deliveries.clear
827 827
828 828 put :update,
829 829 :id => 1,
830 830 :notes => ''
831 831 assert_redirected_to :action => 'show', :id => '1'
832 832
833 833 issue.reload
834 834 assert issue.journals.empty?
835 835 # No email should be sent
836 836 assert ActionMailer::Base.deliveries.empty?
837 837 end
838 838
839 839 def test_put_update_should_send_a_notification
840 840 @request.session[:user_id] = 2
841 841 ActionMailer::Base.deliveries.clear
842 842 issue = Issue.find(1)
843 843 old_subject = issue.subject
844 844 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
845 845
846 846 put :update, :id => 1, :issue => {:subject => new_subject,
847 847 :priority_id => '6',
848 848 :category_id => '1' # no change
849 849 }
850 850 assert_equal 1, ActionMailer::Base.deliveries.size
851 851 end
852 852
853 853 def test_put_update_with_invalid_spent_time
854 854 @request.session[:user_id] = 2
855 855 notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
856 856
857 857 assert_no_difference('Journal.count') do
858 858 put :update,
859 859 :id => 1,
860 860 :notes => notes,
861 861 :time_entry => {"comments"=>"", "activity_id"=>"", "hours"=>"2z"}
862 862 end
863 863 assert_response :success
864 864 assert_template 'edit'
865 865
866 866 assert_tag :textarea, :attributes => { :name => 'notes' },
867 867 :content => notes
868 868 assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => "2z" }
869 869 end
870 870
871 871 def test_put_update_should_allow_fixed_version_to_be_set_to_a_subproject
872 872 issue = Issue.find(2)
873 873 @request.session[:user_id] = 2
874 874
875 875 put :update,
876 876 :id => issue.id,
877 877 :issue => {
878 878 :fixed_version_id => 4
879 879 }
880 880
881 881 assert_response :redirect
882 882 issue.reload
883 883 assert_equal 4, issue.fixed_version_id
884 884 assert_not_equal issue.project_id, issue.fixed_version.project_id
885 885 end
886 886
887 887 def test_put_update_should_redirect_back_using_the_back_url_parameter
888 888 issue = Issue.find(2)
889 889 @request.session[:user_id] = 2
890 890
891 891 put :update,
892 892 :id => issue.id,
893 893 :issue => {
894 894 :fixed_version_id => 4
895 895 },
896 896 :back_url => '/issues'
897 897
898 898 assert_response :redirect
899 899 assert_redirected_to '/issues'
900 900 end
901 901
902 902 def test_put_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
903 903 issue = Issue.find(2)
904 904 @request.session[:user_id] = 2
905 905
906 906 put :update,
907 907 :id => issue.id,
908 908 :issue => {
909 909 :fixed_version_id => 4
910 910 },
911 911 :back_url => 'http://google.com'
912 912
913 913 assert_response :redirect
914 914 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue.id
915 915 end
916 916
917 917 def test_get_bulk_edit
918 918 @request.session[:user_id] = 2
919 919 get :bulk_edit, :ids => [1, 2]
920 920 assert_response :success
921 921 assert_template 'bulk_edit'
922 922
923 923 # Project specific custom field, date type
924 924 field = CustomField.find(9)
925 925 assert !field.is_for_all?
926 926 assert_equal 'date', field.field_format
927 927 assert_tag :input, :attributes => {:name => 'issue[custom_field_values][9]'}
928 928
929 929 # System wide custom field
930 930 assert CustomField.find(1).is_for_all?
931 931 assert_tag :select, :attributes => {:name => 'issue[custom_field_values][1]'}
932 932 end
933 933
934 934 def test_bulk_edit
935 935 @request.session[:user_id] = 2
936 936 # update issues priority
937 937 post :bulk_edit, :ids => [1, 2], :notes => 'Bulk editing',
938 938 :issue => {:priority_id => 7,
939 939 :assigned_to_id => '',
940 940 :custom_field_values => {'2' => ''}}
941 941
942 942 assert_response 302
943 943 # check that the issues were updated
944 944 assert_equal [7, 7], Issue.find_all_by_id([1, 2]).collect {|i| i.priority.id}
945 945
946 946 issue = Issue.find(1)
947 947 journal = issue.journals.find(:first, :order => 'created_on DESC')
948 948 assert_equal '125', issue.custom_value_for(2).value
949 949 assert_equal 'Bulk editing', journal.notes
950 950 assert_equal 1, journal.details.size
951 951 end
952 952
953 953 def test_bullk_edit_should_send_a_notification
954 954 @request.session[:user_id] = 2
955 955 ActionMailer::Base.deliveries.clear
956 956 post(:bulk_edit,
957 957 {
958 958 :ids => [1, 2],
959 959 :notes => 'Bulk editing',
960 960 :issue => {
961 961 :priority_id => 7,
962 962 :assigned_to_id => '',
963 963 :custom_field_values => {'2' => ''}
964 964 }
965 965 })
966 966
967 967 assert_response 302
968 968 assert_equal 2, ActionMailer::Base.deliveries.size
969 969 end
970 970
971 971 def test_bulk_edit_status
972 972 @request.session[:user_id] = 2
973 973 # update issues priority
974 974 post :bulk_edit, :ids => [1, 2], :notes => 'Bulk editing status',
975 975 :issue => {:priority_id => '',
976 976 :assigned_to_id => '',
977 977 :status_id => '5'}
978 978
979 979 assert_response 302
980 980 issue = Issue.find(1)
981 981 assert issue.closed?
982 982 end
983 983
984 984 def test_bulk_edit_custom_field
985 985 @request.session[:user_id] = 2
986 986 # update issues priority
987 987 post :bulk_edit, :ids => [1, 2], :notes => 'Bulk editing custom field',
988 988 :issue => {:priority_id => '',
989 989 :assigned_to_id => '',
990 990 :custom_field_values => {'2' => '777'}}
991 991
992 992 assert_response 302
993 993
994 994 issue = Issue.find(1)
995 995 journal = issue.journals.find(:first, :order => 'created_on DESC')
996 996 assert_equal '777', issue.custom_value_for(2).value
997 997 assert_equal 1, journal.details.size
998 998 assert_equal '125', journal.details.first.old_value
999 999 assert_equal '777', journal.details.first.value
1000 1000 end
1001 1001
1002 1002 def test_bulk_unassign
1003 1003 assert_not_nil Issue.find(2).assigned_to
1004 1004 @request.session[:user_id] = 2
1005 1005 # unassign issues
1006 1006 post :bulk_edit, :ids => [1, 2], :notes => 'Bulk unassigning', :issue => {:assigned_to_id => 'none'}
1007 1007 assert_response 302
1008 1008 # check that the issues were updated
1009 1009 assert_nil Issue.find(2).assigned_to
1010 1010 end
1011 1011
1012 1012 def test_post_bulk_edit_should_allow_fixed_version_to_be_set_to_a_subproject
1013 1013 @request.session[:user_id] = 2
1014 1014
1015 1015 post :bulk_edit, :ids => [1,2], :issue => {:fixed_version_id => 4}
1016 1016
1017 1017 assert_response :redirect
1018 1018 issues = Issue.find([1,2])
1019 1019 issues.each do |issue|
1020 1020 assert_equal 4, issue.fixed_version_id
1021 1021 assert_not_equal issue.project_id, issue.fixed_version.project_id
1022 1022 end
1023 1023 end
1024 1024
1025 1025 def test_post_bulk_edit_should_redirect_back_using_the_back_url_parameter
1026 1026 @request.session[:user_id] = 2
1027 1027 post :bulk_edit, :ids => [1,2], :back_url => '/issues'
1028 1028
1029 1029 assert_response :redirect
1030 1030 assert_redirected_to '/issues'
1031 1031 end
1032 1032
1033 1033 def test_post_bulk_edit_should_not_redirect_back_using_the_back_url_parameter_off_the_host
1034 1034 @request.session[:user_id] = 2
1035 1035 post :bulk_edit, :ids => [1,2], :back_url => 'http://google.com'
1036 1036
1037 1037 assert_response :redirect
1038 1038 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => Project.find(1).identifier
1039 1039 end
1040 1040
1041 def test_perform_move_one_issue_to_another_project
1042 @request.session[:user_id] = 2
1043 post :perform_move, :id => 1, :new_project_id => 2, :tracker_id => '', :assigned_to_id => '', :status_id => '', :start_date => '', :due_date => ''
1044 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1045 assert_equal 2, Issue.find(1).project_id
1046 end
1047
1048 def test_perform_move_one_issue_to_another_project_should_follow_when_needed
1049 @request.session[:user_id] = 2
1050 post :perform_move, :id => 1, :new_project_id => 2, :follow => '1'
1051 assert_redirected_to '/issues/1'
1052 end
1053
1054 def test_bulk_perform_move_to_another_project
1055 @request.session[:user_id] = 2
1056 post :perform_move, :ids => [1, 2], :new_project_id => 2
1057 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1058 # Issues moved to project 2
1059 assert_equal 2, Issue.find(1).project_id
1060 assert_equal 2, Issue.find(2).project_id
1061 # No tracker change
1062 assert_equal 1, Issue.find(1).tracker_id
1063 assert_equal 2, Issue.find(2).tracker_id
1064 end
1065
1066 def test_bulk_perform_move_to_another_tracker
1067 @request.session[:user_id] = 2
1068 post :perform_move, :ids => [1, 2], :new_tracker_id => 2
1069 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1070 assert_equal 2, Issue.find(1).tracker_id
1071 assert_equal 2, Issue.find(2).tracker_id
1072 end
1073
1074 def test_bulk_copy_to_another_project
1075 @request.session[:user_id] = 2
1076 assert_difference 'Issue.count', 2 do
1077 assert_no_difference 'Project.find(1).issues.count' do
1078 post :perform_move, :ids => [1, 2], :new_project_id => 2, :copy_options => {:copy => '1'}
1079 end
1080 end
1081 assert_redirected_to 'projects/ecookbook/issues'
1082 end
1083
1084 context "#perform_move via bulk copy" do
1085 should "allow not changing the issue's attributes" do
1086 @request.session[:user_id] = 2
1087 issue_before_move = Issue.find(1)
1088 assert_difference 'Issue.count', 1 do
1089 assert_no_difference 'Project.find(1).issues.count' do
1090 post :perform_move, :ids => [1], :new_project_id => 2, :copy_options => {:copy => '1'}, :new_tracker_id => '', :assigned_to_id => '', :status_id => '', :start_date => '', :due_date => ''
1091 end
1092 end
1093 issue_after_move = Issue.first(:order => 'id desc', :conditions => {:project_id => 2})
1094 assert_equal issue_before_move.tracker_id, issue_after_move.tracker_id
1095 assert_equal issue_before_move.status_id, issue_after_move.status_id
1096 assert_equal issue_before_move.assigned_to_id, issue_after_move.assigned_to_id
1097 end
1098
1099 should "allow changing the issue's attributes" do
1100 # Fixes random test failure with Mysql
1101 # where Issue.all(:limit => 2, :order => 'id desc', :conditions => {:project_id => 2}) doesn't return the expected results
1102 Issue.delete_all("project_id=2")
1103
1104 @request.session[:user_id] = 2
1105 assert_difference 'Issue.count', 2 do
1106 assert_no_difference 'Project.find(1).issues.count' do
1107 post :perform_move, :ids => [1, 2], :new_project_id => 2, :copy_options => {:copy => '1'}, :new_tracker_id => '', :assigned_to_id => 4, :status_id => 3, :start_date => '2009-12-01', :due_date => '2009-12-31'
1108 end
1109 end
1110
1111 copied_issues = Issue.all(:limit => 2, :order => 'id desc', :conditions => {:project_id => 2})
1112 assert_equal 2, copied_issues.size
1113 copied_issues.each do |issue|
1114 assert_equal 2, issue.project_id, "Project is incorrect"
1115 assert_equal 4, issue.assigned_to_id, "Assigned to is incorrect"
1116 assert_equal 3, issue.status_id, "Status is incorrect"
1117 assert_equal '2009-12-01', issue.start_date.to_s, "Start date is incorrect"
1118 assert_equal '2009-12-31', issue.due_date.to_s, "Due date is incorrect"
1119 end
1120 end
1121 end
1122
1123 def test_copy_to_another_project_should_follow_when_needed
1124 @request.session[:user_id] = 2
1125 post :perform_move, :ids => [1], :new_project_id => 2, :copy_options => {:copy => '1'}, :follow => '1'
1126 issue = Issue.first(:order => 'id DESC')
1127 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
1128 end
1129
1130 1041 def test_context_menu_one_issue
1131 1042 @request.session[:user_id] = 2
1132 1043 get :context_menu, :ids => [1]
1133 1044 assert_response :success
1134 1045 assert_template 'context_menu'
1135 1046 assert_tag :tag => 'a', :content => 'Edit',
1136 1047 :attributes => { :href => '/issues/1/edit',
1137 1048 :class => 'icon-edit' }
1138 1049 assert_tag :tag => 'a', :content => 'Closed',
1139 1050 :attributes => { :href => '/issues/1/edit?issue%5Bstatus_id%5D=5',
1140 1051 :class => '' }
1141 1052 assert_tag :tag => 'a', :content => 'Immediate',
1142 1053 :attributes => { :href => '/issues/bulk_edit?ids%5B%5D=1&amp;issue%5Bpriority_id%5D=8',
1143 1054 :class => '' }
1144 1055 # Versions
1145 1056 assert_tag :tag => 'a', :content => '2.0',
1146 1057 :attributes => { :href => '/issues/bulk_edit?ids%5B%5D=1&amp;issue%5Bfixed_version_id%5D=3',
1147 1058 :class => '' }
1148 1059 assert_tag :tag => 'a', :content => 'eCookbook Subproject 1 - 2.0',
1149 1060 :attributes => { :href => '/issues/bulk_edit?ids%5B%5D=1&amp;issue%5Bfixed_version_id%5D=4',
1150 1061 :class => '' }
1151 1062
1152 1063 assert_tag :tag => 'a', :content => 'Dave Lopper',
1153 1064 :attributes => { :href => '/issues/bulk_edit?ids%5B%5D=1&amp;issue%5Bassigned_to_id%5D=3',
1154 1065 :class => '' }
1155 1066 assert_tag :tag => 'a', :content => 'Duplicate',
1156 1067 :attributes => { :href => '/projects/ecookbook/issues/1/copy',
1157 1068 :class => 'icon-duplicate' }
1158 1069 assert_tag :tag => 'a', :content => 'Copy',
1159 :attributes => { :href => '/issues/move?copy_options%5Bcopy%5D=t&amp;ids%5B%5D=1',
1070 :attributes => { :href => '/issues/move/new?copy_options%5Bcopy%5D=t&amp;ids%5B%5D=1',
1160 1071 :class => 'icon-copy' }
1161 1072 assert_tag :tag => 'a', :content => 'Move',
1162 :attributes => { :href => '/issues/move?ids%5B%5D=1',
1073 :attributes => { :href => '/issues/move/new?ids%5B%5D=1',
1163 1074 :class => 'icon-move' }
1164 1075 assert_tag :tag => 'a', :content => 'Delete',
1165 1076 :attributes => { :href => '/issues/destroy?ids%5B%5D=1',
1166 1077 :class => 'icon-del' }
1167 1078 end
1168 1079
1169 1080 def test_context_menu_one_issue_by_anonymous
1170 1081 get :context_menu, :ids => [1]
1171 1082 assert_response :success
1172 1083 assert_template 'context_menu'
1173 1084 assert_tag :tag => 'a', :content => 'Delete',
1174 1085 :attributes => { :href => '#',
1175 1086 :class => 'icon-del disabled' }
1176 1087 end
1177 1088
1178 1089 def test_context_menu_multiple_issues_of_same_project
1179 1090 @request.session[:user_id] = 2
1180 1091 get :context_menu, :ids => [1, 2]
1181 1092 assert_response :success
1182 1093 assert_template 'context_menu'
1183 1094 assert_tag :tag => 'a', :content => 'Edit',
1184 1095 :attributes => { :href => '/issues/bulk_edit?ids%5B%5D=1&amp;ids%5B%5D=2',
1185 1096 :class => 'icon-edit' }
1186 1097 assert_tag :tag => 'a', :content => 'Immediate',
1187 1098 :attributes => { :href => '/issues/bulk_edit?ids%5B%5D=1&amp;ids%5B%5D=2&amp;issue%5Bpriority_id%5D=8',
1188 1099 :class => '' }
1189 1100 assert_tag :tag => 'a', :content => 'Dave Lopper',
1190 1101 :attributes => { :href => '/issues/bulk_edit?ids%5B%5D=1&amp;ids%5B%5D=2&amp;issue%5Bassigned_to_id%5D=3',
1191 1102 :class => '' }
1192 1103 assert_tag :tag => 'a', :content => 'Copy',
1193 :attributes => { :href => '/issues/move?copy_options%5Bcopy%5D=t&amp;ids%5B%5D=1&amp;ids%5B%5D=2',
1104 :attributes => { :href => '/issues/move/new?copy_options%5Bcopy%5D=t&amp;ids%5B%5D=1&amp;ids%5B%5D=2',
1194 1105 :class => 'icon-copy' }
1195 1106 assert_tag :tag => 'a', :content => 'Move',
1196 :attributes => { :href => '/issues/move?ids%5B%5D=1&amp;ids%5B%5D=2',
1107 :attributes => { :href => '/issues/move/new?ids%5B%5D=1&amp;ids%5B%5D=2',
1197 1108 :class => 'icon-move' }
1198 1109 assert_tag :tag => 'a', :content => 'Delete',
1199 1110 :attributes => { :href => '/issues/destroy?ids%5B%5D=1&amp;ids%5B%5D=2',
1200 1111 :class => 'icon-del' }
1201 1112 end
1202 1113
1203 1114 def test_context_menu_multiple_issues_of_different_project
1204 1115 @request.session[:user_id] = 2
1205 1116 get :context_menu, :ids => [1, 2, 4]
1206 1117 assert_response :success
1207 1118 assert_template 'context_menu'
1208 1119 assert_tag :tag => 'a', :content => 'Delete',
1209 1120 :attributes => { :href => '#',
1210 1121 :class => 'icon-del disabled' }
1211 1122 end
1212 1123
1213 1124 def test_preview_new_issue
1214 1125 @request.session[:user_id] = 2
1215 1126 post :preview, :project_id => '1', :issue => {:description => 'Foo'}
1216 1127 assert_response :success
1217 1128 assert_template 'preview'
1218 1129 assert_not_nil assigns(:description)
1219 1130 end
1220 1131
1221 1132 def test_preview_notes
1222 1133 @request.session[:user_id] = 2
1223 1134 post :preview, :project_id => '1', :id => 1, :issue => {:description => Issue.find(1).description}, :notes => 'Foo'
1224 1135 assert_response :success
1225 1136 assert_template 'preview'
1226 1137 assert_not_nil assigns(:notes)
1227 1138 end
1228 1139
1229 1140 def test_auto_complete_should_not_be_case_sensitive
1230 1141 get :auto_complete, :project_id => 'ecookbook', :q => 'ReCiPe'
1231 1142 assert_response :success
1232 1143 assert_not_nil assigns(:issues)
1233 1144 assert assigns(:issues).detect {|issue| issue.subject.match /recipe/}
1234 1145 end
1235 1146
1236 1147 def test_auto_complete_should_return_issue_with_given_id
1237 1148 get :auto_complete, :project_id => 'subproject1', :q => '13'
1238 1149 assert_response :success
1239 1150 assert_not_nil assigns(:issues)
1240 1151 assert assigns(:issues).include?(Issue.find(13))
1241 1152 end
1242 1153
1243 1154 def test_destroy_issue_with_no_time_entries
1244 1155 assert_nil TimeEntry.find_by_issue_id(2)
1245 1156 @request.session[:user_id] = 2
1246 1157 post :destroy, :id => 2
1247 1158 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1248 1159 assert_nil Issue.find_by_id(2)
1249 1160 end
1250 1161
1251 1162 def test_destroy_issues_with_time_entries
1252 1163 @request.session[:user_id] = 2
1253 1164 post :destroy, :ids => [1, 3]
1254 1165 assert_response :success
1255 1166 assert_template 'destroy'
1256 1167 assert_not_nil assigns(:hours)
1257 1168 assert Issue.find_by_id(1) && Issue.find_by_id(3)
1258 1169 end
1259 1170
1260 1171 def test_destroy_issues_and_destroy_time_entries
1261 1172 @request.session[:user_id] = 2
1262 1173 post :destroy, :ids => [1, 3], :todo => 'destroy'
1263 1174 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1264 1175 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
1265 1176 assert_nil TimeEntry.find_by_id([1, 2])
1266 1177 end
1267 1178
1268 1179 def test_destroy_issues_and_assign_time_entries_to_project
1269 1180 @request.session[:user_id] = 2
1270 1181 post :destroy, :ids => [1, 3], :todo => 'nullify'
1271 1182 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1272 1183 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
1273 1184 assert_nil TimeEntry.find(1).issue_id
1274 1185 assert_nil TimeEntry.find(2).issue_id
1275 1186 end
1276 1187
1277 1188 def test_destroy_issues_and_reassign_time_entries_to_another_issue
1278 1189 @request.session[:user_id] = 2
1279 1190 post :destroy, :ids => [1, 3], :todo => 'reassign', :reassign_to_id => 2
1280 1191 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1281 1192 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
1282 1193 assert_equal 2, TimeEntry.find(1).issue_id
1283 1194 assert_equal 2, TimeEntry.find(2).issue_id
1284 1195 end
1285 1196
1286 1197 def test_default_search_scope
1287 1198 get :index
1288 1199 assert_tag :div, :attributes => {:id => 'quick-search'},
1289 1200 :child => {:tag => 'form',
1290 1201 :child => {:tag => 'input', :attributes => {:name => 'issues', :type => 'hidden', :value => '1'}}}
1291 1202 end
1292 1203 end
Show file before | Show file after | Hide comments
@@ -1,281 +1,281
1 1 # redMine - project management software
2 2 # Copyright (C) 2006-2010 Jean-Philippe Lang
3 3 #
4 4 # This program is free software; you can redistribute it and/or
5 5 # modify it under the terms of the GNU General Public License
6 6 # as published by the Free Software Foundation; either version 2
7 7 # of the License, or (at your option) any later version.
8 8 #
9 9 # This program is distributed in the hope that it will be useful,
10 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 12 # GNU General Public License for more details.
13 13 #
14 14 # You should have received a copy of the GNU General Public License
15 15 # along with this program; if not, write to the Free Software
16 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17 17
18 18 require "#{File.dirname(__FILE__)}/../test_helper"
19 19
20 20 class RoutingTest < ActionController::IntegrationTest
21 21 context "activities" do
22 22 should_route :get, "/activity", :controller => 'projects', :action => 'activity', :id => nil
23 23 should_route :get, "/activity.atom", :controller => 'projects', :action => 'activity', :id => nil, :format => 'atom'
24 24 end
25 25
26 26 context "attachments" do
27 27 should_route :get, "/attachments/1", :controller => 'attachments', :action => 'show', :id => '1'
28 28 should_route :get, "/attachments/1/filename.ext", :controller => 'attachments', :action => 'show', :id => '1', :filename => 'filename.ext'
29 29 should_route :get, "/attachments/download/1", :controller => 'attachments', :action => 'download', :id => '1'
30 30 should_route :get, "/attachments/download/1/filename.ext", :controller => 'attachments', :action => 'download', :id => '1', :filename => 'filename.ext'
31 31 end
32 32
33 33 context "boards" do
34 34 should_route :get, "/projects/world_domination/boards", :controller => 'boards', :action => 'index', :project_id => 'world_domination'
35 35 should_route :get, "/projects/world_domination/boards/new", :controller => 'boards', :action => 'new', :project_id => 'world_domination'
36 36 should_route :get, "/projects/world_domination/boards/44", :controller => 'boards', :action => 'show', :project_id => 'world_domination', :id => '44'
37 37 should_route :get, "/projects/world_domination/boards/44.atom", :controller => 'boards', :action => 'show', :project_id => 'world_domination', :id => '44', :format => 'atom'
38 38 should_route :get, "/projects/world_domination/boards/44/edit", :controller => 'boards', :action => 'edit', :project_id => 'world_domination', :id => '44'
39 39
40 40 should_route :post, "/projects/world_domination/boards/new", :controller => 'boards', :action => 'new', :project_id => 'world_domination'
41 41 should_route :post, "/projects/world_domination/boards/44/edit", :controller => 'boards', :action => 'edit', :project_id => 'world_domination', :id => '44'
42 42 should_route :post, "/projects/world_domination/boards/44/destroy", :controller => 'boards', :action => 'destroy', :project_id => 'world_domination', :id => '44'
43 43
44 44 end
45 45
46 46 context "documents" do
47 47 should_route :get, "/projects/567/documents", :controller => 'documents', :action => 'index', :project_id => '567'
48 48 should_route :get, "/projects/567/documents/new", :controller => 'documents', :action => 'new', :project_id => '567'
49 49 should_route :get, "/documents/22", :controller => 'documents', :action => 'show', :id => '22'
50 50 should_route :get, "/documents/22/edit", :controller => 'documents', :action => 'edit', :id => '22'
51 51
52 52 should_route :post, "/projects/567/documents/new", :controller => 'documents', :action => 'new', :project_id => '567'
53 53 should_route :post, "/documents/567/edit", :controller => 'documents', :action => 'edit', :id => '567'
54 54 should_route :post, "/documents/567/destroy", :controller => 'documents', :action => 'destroy', :id => '567'
55 55 end
56 56
57 57 context "issues" do
58 58 # REST actions
59 59 should_route :get, "/issues", :controller => 'issues', :action => 'index'
60 60 should_route :get, "/issues.pdf", :controller => 'issues', :action => 'index', :format => 'pdf'
61 61 should_route :get, "/issues.atom", :controller => 'issues', :action => 'index', :format => 'atom'
62 62 should_route :get, "/issues.xml", :controller => 'issues', :action => 'index', :format => 'xml'
63 63 should_route :get, "/projects/23/issues", :controller => 'issues', :action => 'index', :project_id => '23'
64 64 should_route :get, "/projects/23/issues.pdf", :controller => 'issues', :action => 'index', :project_id => '23', :format => 'pdf'
65 65 should_route :get, "/projects/23/issues.atom", :controller => 'issues', :action => 'index', :project_id => '23', :format => 'atom'
66 66 should_route :get, "/projects/23/issues.xml", :controller => 'issues', :action => 'index', :project_id => '23', :format => 'xml'
67 67 should_route :get, "/issues/64", :controller => 'issues', :action => 'show', :id => '64'
68 68 should_route :get, "/issues/64.pdf", :controller => 'issues', :action => 'show', :id => '64', :format => 'pdf'
69 69 should_route :get, "/issues/64.atom", :controller => 'issues', :action => 'show', :id => '64', :format => 'atom'
70 70 should_route :get, "/issues/64.xml", :controller => 'issues', :action => 'show', :id => '64', :format => 'xml'
71 71
72 72 should_route :get, "/projects/23/issues/new", :controller => 'issues', :action => 'new', :project_id => '23'
73 73 should_route :post, "/projects/23/issues", :controller => 'issues', :action => 'create', :project_id => '23'
74 74 should_route :post, "/issues.xml", :controller => 'issues', :action => 'create', :format => 'xml'
75 75
76 76 should_route :get, "/issues/64/edit", :controller => 'issues', :action => 'edit', :id => '64'
77 77 # TODO: Should use PUT
78 78 should_route :post, "/issues/64/edit", :controller => 'issues', :action => 'edit', :id => '64'
79 79 should_route :put, "/issues/1.xml", :controller => 'issues', :action => 'update', :id => '1', :format => 'xml'
80 80
81 81 # TODO: Should use DELETE
82 82 should_route :post, "/issues/64/destroy", :controller => 'issues', :action => 'destroy', :id => '64'
83 83 should_route :delete, "/issues/1.xml", :controller => 'issues', :action => 'destroy', :id => '1', :format => 'xml'
84 84
85 85 # Extra actions
86 86 should_route :get, "/projects/23/issues/64/copy", :controller => 'issues', :action => 'new', :project_id => '23', :copy_from => '64'
87 87
88 should_route :get, "/issues/1/move", :controller => 'issues', :action => 'move', :id => '1'
89 should_route :post, "/issues/1/perform_move", :controller => 'issues', :action => 'perform_move', :id => '1'
88 should_route :get, "/issues/move/new", :controller => 'issue_moves', :action => 'new'
89 should_route :post, "/issues/move", :controller => 'issue_moves', :action => 'create'
90 90
91 91 should_route :post, "/issues/1/quoted", :controller => 'issues', :action => 'reply', :id => '1'
92 92
93 93 should_route :get, "/issues/calendar", :controller => 'calendars', :action => 'show'
94 94 should_route :post, "/issues/calendar", :controller => 'calendars', :action => 'show'
95 95 should_route :get, "/projects/project-name/issues/calendar", :controller => 'calendars', :action => 'show', :project_id => 'project-name'
96 96 should_route :post, "/projects/project-name/issues/calendar", :controller => 'calendars', :action => 'show', :project_id => 'project-name'
97 97
98 98 should_route :get, "/issues/gantt", :controller => 'gantts', :action => 'show'
99 99 should_route :post, "/issues/gantt", :controller => 'gantts', :action => 'show'
100 100 should_route :get, "/projects/project-name/issues/gantt", :controller => 'gantts', :action => 'show', :project_id => 'project-name'
101 101 should_route :post, "/projects/project-name/issues/gantt", :controller => 'gantts', :action => 'show', :project_id => 'project-name'
102 102
103 103 should_route :get, "/issues/auto_complete", :controller => 'issues', :action => 'auto_complete'
104 104 end
105 105
106 106 context "issue categories" do
107 107 should_route :get, "/projects/test/issue_categories/new", :controller => 'issue_categories', :action => 'new', :project_id => 'test'
108 108
109 109 should_route :post, "/projects/test/issue_categories/new", :controller => 'issue_categories', :action => 'new', :project_id => 'test'
110 110 end
111 111
112 112 context "issue relations" do
113 113 should_route :post, "/issues/1/relations", :controller => 'issue_relations', :action => 'new', :issue_id => '1'
114 114 should_route :post, "/issues/1/relations/23/destroy", :controller => 'issue_relations', :action => 'destroy', :issue_id => '1', :id => '23'
115 115 end
116 116
117 117 context "issue reports" do
118 118 should_route :get, "/projects/567/issues/report", :controller => 'reports', :action => 'issue_report', :id => '567'
119 119 should_route :get, "/projects/567/issues/report/assigned_to", :controller => 'reports', :action => 'issue_report_details', :id => '567', :detail => 'assigned_to'
120 120 end
121 121
122 122 context "members" do
123 123 should_route :post, "/projects/5234/members/new", :controller => 'members', :action => 'new', :id => '5234'
124 124 end
125 125
126 126 context "messages" do
127 127 should_route :get, "/boards/22/topics/2", :controller => 'messages', :action => 'show', :id => '2', :board_id => '22'
128 128 should_route :get, "/boards/lala/topics/new", :controller => 'messages', :action => 'new', :board_id => 'lala'
129 129 should_route :get, "/boards/lala/topics/22/edit", :controller => 'messages', :action => 'edit', :id => '22', :board_id => 'lala'
130 130
131 131 should_route :post, "/boards/lala/topics/new", :controller => 'messages', :action => 'new', :board_id => 'lala'
132 132 should_route :post, "/boards/lala/topics/22/edit", :controller => 'messages', :action => 'edit', :id => '22', :board_id => 'lala'
133 133 should_route :post, "/boards/22/topics/555/replies", :controller => 'messages', :action => 'reply', :id => '555', :board_id => '22'
134 134 should_route :post, "/boards/22/topics/555/destroy", :controller => 'messages', :action => 'destroy', :id => '555', :board_id => '22'
135 135 end
136 136
137 137 context "news" do
138 138 should_route :get, "/news", :controller => 'news', :action => 'index'
139 139 should_route :get, "/news.atom", :controller => 'news', :action => 'index', :format => 'atom'
140 140 should_route :get, "/news.xml", :controller => 'news', :action => 'index', :format => 'xml'
141 141 should_route :get, "/news.json", :controller => 'news', :action => 'index', :format => 'json'
142 142 should_route :get, "/projects/567/news", :controller => 'news', :action => 'index', :project_id => '567'
143 143 should_route :get, "/projects/567/news.atom", :controller => 'news', :action => 'index', :format => 'atom', :project_id => '567'
144 144 should_route :get, "/projects/567/news.xml", :controller => 'news', :action => 'index', :format => 'xml', :project_id => '567'
145 145 should_route :get, "/projects/567/news.json", :controller => 'news', :action => 'index', :format => 'json', :project_id => '567'
146 146 should_route :get, "/news/2", :controller => 'news', :action => 'show', :id => '2'
147 147 should_route :get, "/projects/567/news/new", :controller => 'news', :action => 'new', :project_id => '567'
148 148 should_route :get, "/news/234", :controller => 'news', :action => 'show', :id => '234'
149 149
150 150 should_route :post, "/projects/567/news/new", :controller => 'news', :action => 'new', :project_id => '567'
151 151 should_route :post, "/news/567/edit", :controller => 'news', :action => 'edit', :id => '567'
152 152 should_route :post, "/news/567/destroy", :controller => 'news', :action => 'destroy', :id => '567'
153 153 end
154 154
155 155 context "projects" do
156 156 should_route :get, "/projects", :controller => 'projects', :action => 'index'
157 157 should_route :get, "/projects.atom", :controller => 'projects', :action => 'index', :format => 'atom'
158 158 should_route :get, "/projects.xml", :controller => 'projects', :action => 'index', :format => 'xml'
159 159 should_route :get, "/projects/new", :controller => 'projects', :action => 'add'
160 160 should_route :get, "/projects/test", :controller => 'projects', :action => 'show', :id => 'test'
161 161 should_route :get, "/projects/1.xml", :controller => 'projects', :action => 'show', :id => '1', :format => 'xml'
162 162 should_route :get, "/projects/4223/settings", :controller => 'projects', :action => 'settings', :id => '4223'
163 163 should_route :get, "/projects/4223/settings/members", :controller => 'projects', :action => 'settings', :id => '4223', :tab => 'members'
164 164 should_route :get, "/projects/567/destroy", :controller => 'projects', :action => 'destroy', :id => '567'
165 165 should_route :get, "/projects/33/files", :controller => 'projects', :action => 'list_files', :id => '33'
166 166 should_route :get, "/projects/33/files/new", :controller => 'projects', :action => 'add_file', :id => '33'
167 167 should_route :get, "/projects/33/roadmap", :controller => 'projects', :action => 'roadmap', :id => '33'
168 168 should_route :get, "/projects/33/activity", :controller => 'projects', :action => 'activity', :id => '33'
169 169 should_route :get, "/projects/33/activity.atom", :controller => 'projects', :action => 'activity', :id => '33', :format => 'atom'
170 170
171 171 should_route :post, "/projects/new", :controller => 'projects', :action => 'add'
172 172 should_route :post, "/projects.xml", :controller => 'projects', :action => 'add', :format => 'xml'
173 173 should_route :post, "/projects/4223/edit", :controller => 'projects', :action => 'edit', :id => '4223'
174 174 should_route :post, "/projects/64/destroy", :controller => 'projects', :action => 'destroy', :id => '64'
175 175 should_route :post, "/projects/33/files/new", :controller => 'projects', :action => 'add_file', :id => '33'
176 176 should_route :post, "/projects/64/archive", :controller => 'projects', :action => 'archive', :id => '64'
177 177 should_route :post, "/projects/64/unarchive", :controller => 'projects', :action => 'unarchive', :id => '64'
178 178 should_route :post, "/projects/64/activities/save", :controller => 'projects', :action => 'save_activities', :id => '64'
179 179
180 180 should_route :put, "/projects/1.xml", :controller => 'projects', :action => 'edit', :id => '1', :format => 'xml'
181 181
182 182 should_route :delete, "/projects/1.xml", :controller => 'projects', :action => 'destroy', :id => '1', :format => 'xml'
183 183 should_route :delete, "/projects/64/reset_activities", :controller => 'projects', :action => 'reset_activities', :id => '64'
184 184 end
185 185
186 186 context "repositories" do
187 187 should_route :get, "/projects/redmine/repository", :controller => 'repositories', :action => 'show', :id => 'redmine'
188 188 should_route :get, "/projects/redmine/repository/edit", :controller => 'repositories', :action => 'edit', :id => 'redmine'
189 189 should_route :get, "/projects/redmine/repository/revisions", :controller => 'repositories', :action => 'revisions', :id => 'redmine'
190 190 should_route :get, "/projects/redmine/repository/revisions.atom", :controller => 'repositories', :action => 'revisions', :id => 'redmine', :format => 'atom'
191 191 should_route :get, "/projects/redmine/repository/revisions/2457", :controller => 'repositories', :action => 'revision', :id => 'redmine', :rev => '2457'
192 192 should_route :get, "/projects/redmine/repository/revisions/2457/diff", :controller => 'repositories', :action => 'diff', :id => 'redmine', :rev => '2457'
193 193 should_route :get, "/projects/redmine/repository/revisions/2457/diff.diff", :controller => 'repositories', :action => 'diff', :id => 'redmine', :rev => '2457', :format => 'diff'
194 194 should_route :get, "/projects/redmine/repository/diff/path/to/file.c", :controller => 'repositories', :action => 'diff', :id => 'redmine', :path => %w[path to file.c]
195 195 should_route :get, "/projects/redmine/repository/revisions/2/diff/path/to/file.c", :controller => 'repositories', :action => 'diff', :id => 'redmine', :path => %w[path to file.c], :rev => '2'
196 196 should_route :get, "/projects/redmine/repository/browse/path/to/file.c", :controller => 'repositories', :action => 'browse', :id => 'redmine', :path => %w[path to file.c]
197 197 should_route :get, "/projects/redmine/repository/entry/path/to/file.c", :controller => 'repositories', :action => 'entry', :id => 'redmine', :path => %w[path to file.c]
198 198 should_route :get, "/projects/redmine/repository/revisions/2/entry/path/to/file.c", :controller => 'repositories', :action => 'entry', :id => 'redmine', :path => %w[path to file.c], :rev => '2'
199 199 should_route :get, "/projects/redmine/repository/raw/path/to/file.c", :controller => 'repositories', :action => 'entry', :id => 'redmine', :path => %w[path to file.c], :format => 'raw'
200 200 should_route :get, "/projects/redmine/repository/revisions/2/raw/path/to/file.c", :controller => 'repositories', :action => 'entry', :id => 'redmine', :path => %w[path to file.c], :rev => '2', :format => 'raw'
201 201 should_route :get, "/projects/redmine/repository/annotate/path/to/file.c", :controller => 'repositories', :action => 'annotate', :id => 'redmine', :path => %w[path to file.c]
202 202 should_route :get, "/projects/redmine/repository/changes/path/to/file.c", :controller => 'repositories', :action => 'changes', :id => 'redmine', :path => %w[path to file.c]
203 203 should_route :get, "/projects/redmine/repository/statistics", :controller => 'repositories', :action => 'stats', :id => 'redmine'
204 204
205 205
206 206 should_route :post, "/projects/redmine/repository/edit", :controller => 'repositories', :action => 'edit', :id => 'redmine'
207 207 end
208 208
209 209 context "timelogs" do
210 210 should_route :get, "/issues/567/time_entries/new", :controller => 'timelog', :action => 'edit', :issue_id => '567'
211 211 should_route :get, "/projects/ecookbook/time_entries/new", :controller => 'timelog', :action => 'edit', :project_id => 'ecookbook'
212 212 should_route :get, "/projects/ecookbook/issues/567/time_entries/new", :controller => 'timelog', :action => 'edit', :project_id => 'ecookbook', :issue_id => '567'
213 213 should_route :get, "/time_entries/22/edit", :controller => 'timelog', :action => 'edit', :id => '22'
214 214 should_route :get, "/time_entries/report", :controller => 'timelog', :action => 'report'
215 215 should_route :get, "/projects/567/time_entries/report", :controller => 'timelog', :action => 'report', :project_id => '567'
216 216 should_route :get, "/projects/567/time_entries/report.csv", :controller => 'timelog', :action => 'report', :project_id => '567', :format => 'csv'
217 217 should_route :get, "/time_entries", :controller => 'timelog', :action => 'details'
218 218 should_route :get, "/time_entries.csv", :controller => 'timelog', :action => 'details', :format => 'csv'
219 219 should_route :get, "/time_entries.atom", :controller => 'timelog', :action => 'details', :format => 'atom'
220 220 should_route :get, "/projects/567/time_entries", :controller => 'timelog', :action => 'details', :project_id => '567'
221 221 should_route :get, "/projects/567/time_entries.csv", :controller => 'timelog', :action => 'details', :project_id => '567', :format => 'csv'
222 222 should_route :get, "/projects/567/time_entries.atom", :controller => 'timelog', :action => 'details', :project_id => '567', :format => 'atom'
223 223 should_route :get, "/issues/234/time_entries", :controller => 'timelog', :action => 'details', :issue_id => '234'
224 224 should_route :get, "/issues/234/time_entries.csv", :controller => 'timelog', :action => 'details', :issue_id => '234', :format => 'csv'
225 225 should_route :get, "/issues/234/time_entries.atom", :controller => 'timelog', :action => 'details', :issue_id => '234', :format => 'atom'
226 226 should_route :get, "/projects/ecookbook/issues/123/time_entries", :controller => 'timelog', :action => 'details', :project_id => 'ecookbook', :issue_id => '123'
227 227
228 228 should_route :post, "/time_entries/55/destroy", :controller => 'timelog', :action => 'destroy', :id => '55'
229 229 end
230 230
231 231 context "users" do
232 232 should_route :get, "/users", :controller => 'users', :action => 'index'
233 233 should_route :get, "/users/44", :controller => 'users', :action => 'show', :id => '44'
234 234 should_route :get, "/users/new", :controller => 'users', :action => 'add'
235 235 should_route :get, "/users/444/edit", :controller => 'users', :action => 'edit', :id => '444'
236 236 should_route :get, "/users/222/edit/membership", :controller => 'users', :action => 'edit', :id => '222', :tab => 'membership'
237 237
238 238 should_route :post, "/users/new", :controller => 'users', :action => 'add'
239 239 should_route :post, "/users/444/edit", :controller => 'users', :action => 'edit', :id => '444'
240 240 should_route :post, "/users/123/memberships", :controller => 'users', :action => 'edit_membership', :id => '123'
241 241 should_route :post, "/users/123/memberships/55", :controller => 'users', :action => 'edit_membership', :id => '123', :membership_id => '55'
242 242 should_route :post, "/users/567/memberships/12/destroy", :controller => 'users', :action => 'destroy_membership', :id => '567', :membership_id => '12'
243 243 end
244 244
245 245 context "versions" do
246 246 should_route :get, "/projects/foo/versions/new", :controller => 'versions', :action => 'new', :project_id => 'foo'
247 247
248 248 should_route :post, "/projects/foo/versions/new", :controller => 'versions', :action => 'new', :project_id => 'foo'
249 249 end
250 250
251 251 context "wiki (singular, project's pages)" do
252 252 should_route :get, "/projects/567/wiki", :controller => 'wiki', :action => 'index', :id => '567'
253 253 should_route :get, "/projects/567/wiki/lalala", :controller => 'wiki', :action => 'index', :id => '567', :page => 'lalala'
254 254 should_route :get, "/projects/567/wiki/my_page/edit", :controller => 'wiki', :action => 'edit', :id => '567', :page => 'my_page'
255 255 should_route :get, "/projects/1/wiki/CookBook_documentation/history", :controller => 'wiki', :action => 'history', :id => '1', :page => 'CookBook_documentation'
256 256 should_route :get, "/projects/1/wiki/CookBook_documentation/diff/2/vs/1", :controller => 'wiki', :action => 'diff', :id => '1', :page => 'CookBook_documentation', :version => '2', :version_from => '1'
257 257 should_route :get, "/projects/1/wiki/CookBook_documentation/annotate/2", :controller => 'wiki', :action => 'annotate', :id => '1', :page => 'CookBook_documentation', :version => '2'
258 258 should_route :get, "/projects/22/wiki/ladida/rename", :controller => 'wiki', :action => 'rename', :id => '22', :page => 'ladida'
259 259 should_route :get, "/projects/567/wiki/page_index", :controller => 'wiki', :action => 'special', :id => '567', :page => 'page_index'
260 260 should_route :get, "/projects/567/wiki/Page_Index", :controller => 'wiki', :action => 'special', :id => '567', :page => 'Page_Index'
261 261 should_route :get, "/projects/567/wiki/date_index", :controller => 'wiki', :action => 'special', :id => '567', :page => 'date_index'
262 262 should_route :get, "/projects/567/wiki/export", :controller => 'wiki', :action => 'special', :id => '567', :page => 'export'
263 263
264 264 should_route :post, "/projects/567/wiki/my_page/edit", :controller => 'wiki', :action => 'edit', :id => '567', :page => 'my_page'
265 265 should_route :post, "/projects/567/wiki/CookBook_documentation/preview", :controller => 'wiki', :action => 'preview', :id => '567', :page => 'CookBook_documentation'
266 266 should_route :post, "/projects/22/wiki/ladida/rename", :controller => 'wiki', :action => 'rename', :id => '22', :page => 'ladida'
267 267 should_route :post, "/projects/22/wiki/ladida/destroy", :controller => 'wiki', :action => 'destroy', :id => '22', :page => 'ladida'
268 268 should_route :post, "/projects/22/wiki/ladida/protect", :controller => 'wiki', :action => 'protect', :id => '22', :page => 'ladida'
269 269 end
270 270
271 271 context "wikis (plural, admin setup)" do
272 272 should_route :get, "/projects/ladida/wiki/destroy", :controller => 'wikis', :action => 'destroy', :id => 'ladida'
273 273
274 274 should_route :post, "/projects/ladida/wiki", :controller => 'wikis', :action => 'edit', :id => 'ladida'
275 275 should_route :post, "/projects/ladida/wiki/destroy", :controller => 'wikis', :action => 'destroy', :id => 'ladida'
276 276 end
277 277
278 278 context "administration panel" do
279 279 should_route :get, "/admin/projects", :controller => 'admin', :action => 'projects'
280 280 end
281 281 end
General Comments 0
You need to be logged in to leave comments. Login now