##// END OF EJS Templates
Removed obsolete routes....
Jean-Philippe Lang -
r7897:526736dedced
parent child
Show More
@@ -1,245 +1,241
1 1 ActionController::Routing::Routes.draw do |map|
2 2 # Add your own custom routes here.
3 3 # The priority is based upon order of creation: first created -> highest priority.
4 4
5 5 # Here's a sample route:
6 6 # map.connect 'products/:id', :controller => 'catalog', :action => 'view'
7 7 # Keep in mind you can assign values other than :controller and :action
8 8
9 9 map.home '', :controller => 'welcome'
10 10
11 11 map.signin 'login', :controller => 'account', :action => 'login'
12 12 map.signout 'logout', :controller => 'account', :action => 'logout'
13 13
14 14 map.connect 'roles/workflow/:id/:role_id/:tracker_id', :controller => 'roles', :action => 'workflow'
15 15 map.connect 'help/:ctrl/:page', :controller => 'help'
16 16
17 17 map.with_options :controller => 'time_entry_reports', :action => 'report',:conditions => {:method => :get} do |time_report|
18 18 time_report.connect 'projects/:project_id/issues/:issue_id/time_entries/report'
19 19 time_report.connect 'projects/:project_id/issues/:issue_id/time_entries/report.:format'
20 20 time_report.connect 'projects/:project_id/time_entries/report'
21 21 time_report.connect 'projects/:project_id/time_entries/report.:format'
22 22 time_report.connect 'time_entries/report'
23 23 time_report.connect 'time_entries/report.:format'
24 24 end
25 25
26 26 map.bulk_edit_time_entry 'time_entries/bulk_edit',
27 27 :controller => 'timelog', :action => 'bulk_edit', :conditions => { :method => :get }
28 28 map.bulk_update_time_entry 'time_entries/bulk_edit',
29 29 :controller => 'timelog', :action => 'bulk_update', :conditions => { :method => :post }
30 30 map.time_entries_context_menu '/time_entries/context_menu',
31 31 :controller => 'context_menus', :action => 'time_entries'
32 32 # TODO: wasteful since this is also nested under issues, projects, and projects/issues
33 33 map.resources :time_entries, :controller => 'timelog'
34 34
35 35 map.connect 'projects/:id/wiki', :controller => 'wikis', :action => 'edit', :conditions => {:method => :post}
36 36 map.connect 'projects/:id/wiki/destroy', :controller => 'wikis', :action => 'destroy', :conditions => {:method => :get}
37 37 map.connect 'projects/:id/wiki/destroy', :controller => 'wikis', :action => 'destroy', :conditions => {:method => :post}
38 38
39 39 map.with_options :controller => 'messages' do |messages_routes|
40 40 messages_routes.with_options :conditions => {:method => :get} do |messages_views|
41 41 messages_views.connect 'boards/:board_id/topics/new', :action => 'new'
42 42 messages_views.connect 'boards/:board_id/topics/:id', :action => 'show'
43 43 messages_views.connect 'boards/:board_id/topics/:id/edit', :action => 'edit'
44 44 end
45 45 messages_routes.with_options :conditions => {:method => :post} do |messages_actions|
46 46 messages_actions.connect 'boards/:board_id/topics/new', :action => 'new'
47 47 messages_actions.connect 'boards/:board_id/topics/:id/replies', :action => 'reply'
48 48 messages_actions.connect 'boards/:board_id/topics/:id/:action', :action => /edit|destroy/
49 49 end
50 50 end
51 51
52 52 map.with_options :controller => 'boards' do |board_routes|
53 53 board_routes.with_options :conditions => {:method => :get} do |board_views|
54 54 board_views.connect 'projects/:project_id/boards', :action => 'index'
55 55 board_views.connect 'projects/:project_id/boards/new', :action => 'new'
56 56 board_views.connect 'projects/:project_id/boards/:id', :action => 'show'
57 57 board_views.connect 'projects/:project_id/boards/:id.:format', :action => 'show'
58 58 board_views.connect 'projects/:project_id/boards/:id/edit', :action => 'edit'
59 59 end
60 60 board_routes.with_options :conditions => {:method => :post} do |board_actions|
61 61 board_actions.connect 'projects/:project_id/boards', :action => 'new'
62 62 board_actions.connect 'projects/:project_id/boards/:id/:action', :action => /edit|destroy/
63 63 end
64 64 end
65 65
66 66 map.resources :issue_moves, :only => [:new, :create], :path_prefix => '/issues', :as => 'move'
67 67 map.resources :queries, :except => [:show]
68 68
69 69 # Misc issue routes. TODO: move into resources
70 70 map.auto_complete_issues '/issues/auto_complete', :controller => 'auto_completes', :action => 'issues'
71 71 map.preview_issue '/issues/preview/:id', :controller => 'previews', :action => 'issue' # TODO: would look nicer as /issues/:id/preview
72 72 map.issues_context_menu '/issues/context_menu', :controller => 'context_menus', :action => 'issues'
73 73 map.issue_changes '/issues/changes', :controller => 'journals', :action => 'index'
74 74 map.bulk_edit_issue 'issues/bulk_edit', :controller => 'issues', :action => 'bulk_edit', :conditions => { :method => :get }
75 75 map.bulk_update_issue 'issues/bulk_edit', :controller => 'issues', :action => 'bulk_update', :conditions => { :method => :post }
76 76 map.quoted_issue '/issues/:id/quoted', :controller => 'journals', :action => 'new', :id => /\d+/, :conditions => { :method => :post }
77 77 map.connect '/issues/:id/destroy', :controller => 'issues', :action => 'destroy', :conditions => { :method => :post } # legacy
78 78
79 79 map.with_options :controller => 'gantts', :action => 'show' do |gantts_routes|
80 80 gantts_routes.connect '/projects/:project_id/issues/gantt'
81 81 gantts_routes.connect '/projects/:project_id/issues/gantt.:format'
82 82 gantts_routes.connect '/issues/gantt.:format'
83 83 end
84 84
85 85 map.with_options :controller => 'calendars', :action => 'show' do |calendars_routes|
86 86 calendars_routes.connect '/projects/:project_id/issues/calendar'
87 87 calendars_routes.connect '/issues/calendar'
88 88 end
89 89
90 90 map.with_options :controller => 'reports', :conditions => {:method => :get} do |reports|
91 91 reports.connect 'projects/:id/issues/report', :action => 'issue_report'
92 92 reports.connect 'projects/:id/issues/report/:detail', :action => 'issue_report_details'
93 93 end
94 94
95 # Following two routes conflict with the resources because #index allows POST
96 map.connect '/issues', :controller => 'issues', :action => 'index', :conditions => { :method => :post }
97 map.connect '/issues/create', :controller => 'issues', :action => 'index', :conditions => { :method => :post }
98
99 95 map.resources :issues, :member => { :edit => :post }, :collection => {} do |issues|
100 96 issues.resources :time_entries, :controller => 'timelog'
101 97 issues.resources :relations, :shallow => true, :controller => 'issue_relations', :only => [:index, :show, :create, :destroy]
102 98 end
103 99
104 100 map.resources :issues, :path_prefix => '/projects/:project_id', :collection => { :create => :post } do |issues|
105 101 issues.resources :time_entries, :controller => 'timelog'
106 102 end
107 103
108 104 map.connect 'projects/:id/members/new', :controller => 'members', :action => 'new'
109 105
110 106 map.with_options :controller => 'users' do |users|
111 107 users.connect 'users/:id/edit/:tab', :action => 'edit', :tab => nil, :conditions => {:method => :get}
112 108
113 109 users.with_options :conditions => {:method => :post} do |user_actions|
114 110 user_actions.connect 'users/:id/memberships', :action => 'edit_membership'
115 111 user_actions.connect 'users/:id/memberships/:membership_id', :action => 'edit_membership'
116 112 user_actions.connect 'users/:id/memberships/:membership_id/destroy', :action => 'destroy_membership'
117 113 end
118 114 end
119 115
120 116 map.resources :users, :member => {
121 117 :edit_membership => :post,
122 118 :destroy_membership => :post
123 119 }
124 120
125 121 # For nice "roadmap" in the url for the index action
126 122 map.connect 'projects/:project_id/roadmap', :controller => 'versions', :action => 'index'
127 123
128 124 map.all_news 'news', :controller => 'news', :action => 'index'
129 125 map.formatted_all_news 'news.:format', :controller => 'news', :action => 'index'
130 126 map.preview_news '/news/preview', :controller => 'previews', :action => 'news'
131 127 map.connect 'news/:id/comments', :controller => 'comments', :action => 'create', :conditions => {:method => :post}
132 128 map.connect 'news/:id/comments/:comment_id', :controller => 'comments', :action => 'destroy', :conditions => {:method => :delete}
133 129
134 130 map.resources :projects, :member => {
135 131 :copy => [:get, :post],
136 132 :settings => :get,
137 133 :modules => :post,
138 134 :archive => :post,
139 135 :unarchive => :post
140 136 } do |project|
141 137 project.resource :project_enumerations, :as => 'enumerations', :only => [:update, :destroy]
142 138 project.resources :files, :only => [:index, :new, :create]
143 139 project.resources :versions, :shallow => true, :collection => {:close_completed => :put}, :member => {:status_by => :post}
144 140 project.resources :news, :shallow => true
145 141 project.resources :time_entries, :controller => 'timelog', :path_prefix => 'projects/:project_id'
146 142 project.resources :queries, :only => [:new, :create]
147 143 project.resources :issue_categories, :shallow => true
148 144 project.resources :documents, :shallow => true, :member => {:add_attachment => :post}
149 145
150 146 project.wiki_start_page 'wiki', :controller => 'wiki', :action => 'show', :conditions => {:method => :get}
151 147 project.wiki_index 'wiki/index', :controller => 'wiki', :action => 'index', :conditions => {:method => :get}
152 148 project.wiki_diff 'wiki/:id/diff/:version', :controller => 'wiki', :action => 'diff', :version => nil
153 149 project.wiki_diff 'wiki/:id/diff/:version/vs/:version_from', :controller => 'wiki', :action => 'diff'
154 150 project.wiki_annotate 'wiki/:id/annotate/:version', :controller => 'wiki', :action => 'annotate'
155 151 project.resources :wiki, :except => [:new, :create], :member => {
156 152 :rename => [:get, :post],
157 153 :history => :get,
158 154 :preview => :any,
159 155 :protect => :post,
160 156 :add_attachment => :post
161 157 }, :collection => {
162 158 :export => :get,
163 159 :date_index => :get
164 160 }
165 161
166 162 end
167 163
168 164 # Destroy uses a get request to prompt the user before the actual DELETE request
169 165 map.project_destroy_confirm 'projects/:id/destroy', :controller => 'projects', :action => 'destroy', :conditions => {:method => :get}
170 166
171 167 # TODO: port to be part of the resources route(s)
172 168 map.with_options :controller => 'projects' do |project_mapper|
173 169 project_mapper.with_options :conditions => {:method => :get} do |project_views|
174 170 project_views.connect 'projects/:id/settings/:tab', :controller => 'projects', :action => 'settings'
175 171 project_views.connect 'projects/:project_id/issues/:copy_from/copy', :controller => 'issues', :action => 'new'
176 172 end
177 173 end
178 174
179 175 map.with_options :controller => 'activities', :action => 'index', :conditions => {:method => :get} do |activity|
180 176 activity.connect 'projects/:id/activity'
181 177 activity.connect 'projects/:id/activity.:format'
182 178 activity.connect 'activity', :id => nil
183 179 activity.connect 'activity.:format', :id => nil
184 180 end
185 181
186 182 map.with_options :controller => 'repositories' do |repositories|
187 183 repositories.with_options :conditions => {:method => :get} do |repository_views|
188 184 repository_views.connect 'projects/:id/repository', :action => 'show'
189 185 repository_views.connect 'projects/:id/repository/edit', :action => 'edit'
190 186 repository_views.connect 'projects/:id/repository/statistics', :action => 'stats'
191 187 repository_views.connect 'projects/:id/repository/revisions', :action => 'revisions'
192 188 repository_views.connect 'projects/:id/repository/revisions.:format', :action => 'revisions'
193 189 repository_views.connect 'projects/:id/repository/revisions/:rev', :action => 'revision'
194 190 repository_views.connect 'projects/:id/repository/revisions/:rev/diff', :action => 'diff'
195 191 repository_views.connect 'projects/:id/repository/revisions/:rev/diff.:format', :action => 'diff'
196 192 repository_views.connect 'projects/:id/repository/revisions/:rev/raw/*path', :action => 'entry', :format => 'raw', :requirements => { :rev => /[a-z0-9\.\-_]+/ }
197 193 repository_views.connect 'projects/:id/repository/revisions/:rev/:action/*path', :requirements => { :rev => /[a-z0-9\.\-_]+/ }
198 194 repository_views.connect 'projects/:id/repository/raw/*path', :action => 'entry', :format => 'raw'
199 195 # TODO: why the following route is required?
200 196 repository_views.connect 'projects/:id/repository/entry/*path', :action => 'entry'
201 197 repository_views.connect 'projects/:id/repository/:action/*path'
202 198 end
203 199
204 200 repositories.connect 'projects/:id/repository/:action', :conditions => {:method => :post}
205 201 end
206 202
207 203 map.resources :attachments, :only => [:show, :destroy]
208 204 # additional routes for having the file name at the end of url
209 205 map.connect 'attachments/:id/:filename', :controller => 'attachments', :action => 'show', :id => /\d+/, :filename => /.*/
210 206 map.connect 'attachments/download/:id/:filename', :controller => 'attachments', :action => 'download', :id => /\d+/, :filename => /.*/
211 207
212 208 map.resources :groups, :member => {:autocomplete_for_user => :get}
213 209 map.group_users 'groups/:id/users', :controller => 'groups', :action => 'add_users', :id => /\d+/, :conditions => {:method => :post}
214 210 map.group_user 'groups/:id/users/:user_id', :controller => 'groups', :action => 'remove_user', :id => /\d+/, :conditions => {:method => :delete}
215 211
216 212 map.resources :trackers, :except => :show
217 213 map.resources :issue_statuses, :except => :show, :collection => {:update_issue_done_ratio => :post}
218 214
219 215 #left old routes at the bottom for backwards compat
220 216 map.connect 'projects/:project_id/issues/:action', :controller => 'issues'
221 217 map.connect 'projects/:project_id/boards/:action/:id', :controller => 'boards'
222 218 map.connect 'boards/:board_id/topics/:action/:id', :controller => 'messages'
223 219 map.connect 'wiki/:id/:page/:action', :page => nil, :controller => 'wiki'
224 220 map.connect 'projects/:project_id/news/:action', :controller => 'news'
225 221 map.connect 'projects/:project_id/timelog/:action/:id', :controller => 'timelog', :project_id => /.+/
226 222 map.with_options :controller => 'repositories' do |omap|
227 223 omap.repositories_show 'repositories/browse/:id/*path', :action => 'browse'
228 224 omap.repositories_changes 'repositories/changes/:id/*path', :action => 'changes'
229 225 omap.repositories_diff 'repositories/diff/:id/*path', :action => 'diff'
230 226 omap.repositories_entry 'repositories/entry/:id/*path', :action => 'entry'
231 227 omap.repositories_entry 'repositories/annotate/:id/*path', :action => 'annotate'
232 228 omap.connect 'repositories/revision/:id/:rev', :action => 'revision'
233 229 end
234 230
235 231 map.with_options :controller => 'sys' do |sys|
236 232 sys.connect 'sys/projects.:format', :action => 'projects', :conditions => {:method => :get}
237 233 sys.connect 'sys/projects/:id/repository.:format', :action => 'create_project_repository', :conditions => {:method => :post}
238 234 end
239 235
240 236 # Install the default route as the lowest priority.
241 237 map.connect ':controller/:action/:id'
242 238 map.connect 'robots.txt', :controller => 'welcome', :action => 'robots'
243 239 # Used for OpenID
244 240 map.root :controller => 'account', :action => 'login'
245 241 end
@@ -1,200 +1,200
1 1 # Redmine - project management software
2 2 # Copyright (C) 2006-2011 Jean-Philippe Lang
3 3 #
4 4 # This program is free software; you can redistribute it and/or
5 5 # modify it under the terms of the GNU General Public License
6 6 # as published by the Free Software Foundation; either version 2
7 7 # of the License, or (at your option) any later version.
8 8 #
9 9 # This program is distributed in the hope that it will be useful,
10 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 12 # GNU General Public License for more details.
13 13 #
14 14 # You should have received a copy of the GNU General Public License
15 15 # along with this program; if not, write to the Free Software
16 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17 17
18 18 require File.expand_path('../../test_helper', __FILE__)
19 19
20 20 class IssuesTest < ActionController::IntegrationTest
21 21 fixtures :projects,
22 22 :users,
23 23 :roles,
24 24 :members,
25 25 :trackers,
26 26 :projects_trackers,
27 27 :enabled_modules,
28 28 :issue_statuses,
29 29 :issues,
30 30 :enumerations,
31 31 :custom_fields,
32 32 :custom_values,
33 33 :custom_fields_trackers
34 34
35 35 # create an issue
36 36 def test_add_issue
37 37 log_user('jsmith', 'jsmith')
38 38 get 'projects/1/issues/new', :tracker_id => '1'
39 39 assert_response :success
40 40 assert_template 'issues/new'
41 41
42 42 post 'projects/1/issues', :tracker_id => "1",
43 43 :issue => { :start_date => "2006-12-26",
44 44 :priority_id => "4",
45 45 :subject => "new test issue",
46 46 :category_id => "",
47 47 :description => "new issue",
48 48 :done_ratio => "0",
49 49 :due_date => "",
50 50 :assigned_to_id => "" },
51 51 :custom_fields => {'2' => 'Value for field 2'}
52 52 # find created issue
53 53 issue = Issue.find_by_subject("new test issue")
54 54 assert_kind_of Issue, issue
55 55
56 56 # check redirection
57 57 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
58 58 follow_redirect!
59 59 assert_equal issue, assigns(:issue)
60 60
61 61 # check issue attributes
62 62 assert_equal 'jsmith', issue.author.login
63 63 assert_equal 1, issue.project.id
64 64 assert_equal 1, issue.status.id
65 65 end
66 66
67 67 # add then remove 2 attachments to an issue
68 68 def test_issue_attachments
69 69 log_user('jsmith', 'jsmith')
70 70 set_tmp_attachments_directory
71 71
72 72 put 'issues/1',
73 73 :notes => 'Some notes',
74 74 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'This is an attachment'}}
75 75 assert_redirected_to "/issues/1"
76 76
77 77 # make sure attachment was saved
78 78 attachment = Issue.find(1).attachments.find_by_filename("testfile.txt")
79 79 assert_kind_of Attachment, attachment
80 80 assert_equal Issue.find(1), attachment.container
81 81 assert_equal 'This is an attachment', attachment.description
82 82 # verify the size of the attachment stored in db
83 83 #assert_equal file_data_1.length, attachment.filesize
84 84 # verify that the attachment was written to disk
85 85 assert File.exist?(attachment.diskfile)
86 86
87 87 # remove the attachments
88 88 Issue.find(1).attachments.each(&:destroy)
89 89 assert_equal 0, Issue.find(1).attachments.length
90 90 end
91 91
92 def test_other_formats_links_on_get_index
92 def test_other_formats_links_on_index
93 93 get '/projects/ecookbook/issues'
94 94
95 95 %w(Atom PDF CSV).each do |format|
96 96 assert_tag :a, :content => format,
97 97 :attributes => { :href => "/projects/ecookbook/issues.#{format.downcase}",
98 98 :rel => 'nofollow' }
99 99 end
100 100 end
101 101
102 def test_other_formats_links_on_post_index_without_project_id_in_url
103 post '/issues', :project_id => 'ecookbook'
102 def test_other_formats_links_on_index_without_project_id_in_url
103 get '/issues', :project_id => 'ecookbook'
104 104
105 105 %w(Atom PDF CSV).each do |format|
106 106 assert_tag :a, :content => format,
107 107 :attributes => { :href => "/projects/ecookbook/issues.#{format.downcase}",
108 108 :rel => 'nofollow' }
109 109 end
110 110 end
111 111
112 def test_pagination_links_on_get_index
112 def test_pagination_links_on_index
113 113 Setting.per_page_options = '2'
114 114 get '/projects/ecookbook/issues'
115 115
116 116 assert_tag :a, :content => '2',
117 117 :attributes => { :href => '/projects/ecookbook/issues?page=2' }
118 118
119 119 end
120 120
121 def test_pagination_links_on_post_index_without_project_id_in_url
121 def test_pagination_links_on_index_without_project_id_in_url
122 122 Setting.per_page_options = '2'
123 post '/issues', :project_id => 'ecookbook'
123 get '/issues', :project_id => 'ecookbook'
124 124
125 125 assert_tag :a, :content => '2',
126 126 :attributes => { :href => '/projects/ecookbook/issues?page=2' }
127 127
128 128 end
129 129
130 130 def test_issue_with_user_custom_field
131 131 @field = IssueCustomField.create!(:name => 'Tester', :field_format => 'user', :is_for_all => true, :trackers => Tracker.all)
132 132 Role.anonymous.add_permission! :add_issues, :edit_issues
133 133 users = Project.find(1).users
134 134 tester = users.first
135 135
136 136 # Issue form
137 137 get '/projects/ecookbook/issues/new'
138 138 assert_response :success
139 139 assert_tag :select,
140 140 :attributes => {:name => "issue[custom_field_values][#{@field.id}]"},
141 141 :children => {:count => (users.size + 1)}, # +1 for blank value
142 142 :child => {
143 143 :tag => 'option',
144 144 :attributes => {:value => tester.id.to_s},
145 145 :content => tester.name
146 146 }
147 147
148 148 # Create issue
149 149 assert_difference 'Issue.count' do
150 150 post '/projects/ecookbook/issues',
151 151 :issue => {
152 152 :tracker_id => '1',
153 153 :priority_id => '4',
154 154 :subject => 'Issue with user custom field',
155 155 :custom_field_values => {@field.id.to_s => users.first.id.to_s}
156 156 }
157 157 end
158 158 issue = Issue.first(:order => 'id DESC')
159 159 assert_response 302
160 160
161 161 # Issue view
162 162 follow_redirect!
163 163 assert_tag :th,
164 164 :content => /Tester/,
165 165 :sibling => {
166 166 :tag => 'td',
167 167 :content => tester.name
168 168 }
169 169 assert_tag :select,
170 170 :attributes => {:name => "issue[custom_field_values][#{@field.id}]"},
171 171 :children => {:count => (users.size + 1)}, # +1 for blank value
172 172 :child => {
173 173 :tag => 'option',
174 174 :attributes => {:value => tester.id.to_s, :selected => 'selected'},
175 175 :content => tester.name
176 176 }
177 177
178 178 # Update issue
179 179 new_tester = users[1]
180 180 assert_difference 'Journal.count' do
181 181 put "/issues/#{issue.id}",
182 182 :notes => 'Updating custom field',
183 183 :issue => {
184 184 :custom_field_values => {@field.id.to_s => new_tester.id.to_s}
185 185 }
186 186 end
187 187 assert_response 302
188 188
189 189 # Issue view
190 190 follow_redirect!
191 191 assert_tag :content => 'Tester',
192 192 :ancestor => {:tag => 'ul', :attributes => {:class => /details/}},
193 193 :sibling => {
194 194 :content => tester.name,
195 195 :sibling => {
196 196 :content => new_tester.name
197 197 }
198 198 }
199 199 end
200 200 end
General Comments 0
You need to be logged in to leave comments. Login now