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