jro_apps/redmine Commit - r10754:8ab9215ea8df

Use named routes in controllers....

Jean-Philippe Lang -

r10754:8ab9215ea8df

parent child

Context file:

r10754:8ab9215ea8df

Expand all files

app/controllers/application_controller.rb +10 0

@@ -334,6 +334,16 class ApplicationController < ActionController::Base
334	url	334	url
335	end	335	end
336		336
		337	# Returns the path to project issues or to the cross-project
		338	# issue list if project is nil
		339	def _issues_path(project, *args)
		340	if project
		341	project_issues_path(project, *args)
		342	else
		343	issues_path(*args)
		344	end
		345	end
		346
337	def redirect_back_or_default(default)	347	def redirect_back_or_default(default)
338	back_url = params[:back_url].to_s	348	back_url = params[:back_url].to_s
339	if back_url.present?	349	if back_url.present?

app/controllers/issue_statuses_controller.rb +5 -5

                 @issue_status = IssueStatus.new(params[:issue_status])
                 if request.post? && @issue_status.save
                   flash[:notice] = l(:notice_successful_create)
-                  redirect_to :action => 'index'
+                  redirect_to issue_statuses_path
                 else
                   render :action => 'new'
                 end
                 @issue_status = IssueStatus.find(params[:id])
                 if request.put? && @issue_status.update_attributes(params[:issue_status])
                   flash[:notice] = l(:notice_successful_update)
-                  redirect_to :action => 'index'
+                  redirect_to issue_statuses_path
                 else
                   render :action => 'edit'
                 end
               def destroy
                 IssueStatus.find(params[:id]).destroy
-                redirect_to :action => 'index'
+                redirect_to issue_statuses_path
               rescue
                 flash[:error] = l(:error_unable_delete_issue_status)
-                redirect_to :action => 'index'
+                redirect_to issue_statuses_path
               end
               def update_issue_done_ratio
                 else
                   flash[:error] =  l(:error_issue_done_ratios_not_updated)
                 end
-                redirect_to :action => 'index'
+                redirect_to issue_statuses_path
               end
             end

app/controllers/issues_controller.rb +11 -7

                     format.html {
                       render_attachment_warning_if_needed(@issue)
                       flash[:notice] = l(:notice_issue_successful_create, :id => view_context.link_to("##{@issue.id}", issue_path(@issue), :title => @issue.subject))
-                      redirect_to(params[:continue] ?  { :action => 'new', :project_id => @issue.project, :issue => {:tracker_id => @issue.tracker, :parent_issue_id => @issue.parent_issue_id}.reject {|k,v| v.nil?} } :
+                      if params[:continue]
-                                  { :action => 'show', :id => @issue })
+                        attrs = {:tracker_id => @issue.tracker, :parent_issue_id => @issue.parent_issue_id}.reject {|k,v| v.nil?}
+                        redirect_to new_project_issue_path(@issue.project, :issue => attrs)
+                      else
+                        redirect_to issue_path(@issue)
+                      end
                     }
                     format.api  { render :action => 'show', :status => :created, :location => issue_url(@issue) }
                   end
                   flash[:notice] = l(:notice_successful_update) unless @issue.current_journal.new_record?
                   respond_to do |format|
-                    format.html { redirect_back_or_default({:action => 'show', :id => @issue}) }
+                    format.html { redirect_back_or_default issue_path(@issue) }
                     format.api  { render_api_ok }
                   end
                 else
                 if params[:follow]
                   if @issues.size == 1 && moved_issues.size == 1
-                    redirect_to :controller => 'issues', :action => 'show', :id => moved_issues.first
+                    redirect_to issue_path(moved_issues.first)
                   elsif moved_issues.map(&:project).uniq.size == 1
-                    redirect_to :controller => 'issues', :action => 'index', :project_id => moved_issues.map(&:project).first
+                    redirect_to project_issues_path(moved_issues.map(&:project).first)
                   end
                 else
-                  redirect_back_or_default({:controller => 'issues', :action => 'index', :project_id => @project})
+                  redirect_back_or_default _issues_path(@project)
                 end
               end
                   end
                 end
                 respond_to do |format|
-                  format.html { redirect_back_or_default(:action => 'index', :project_id => @project) }
+                  format.html { redirect_back_or_default _issues_path(@project) }
                   format.api  { render_api_ok }
                 end
               end

app/controllers/journals_controller.rb +1 -1

                   @journal.destroy if @journal.details.empty? && @journal.notes.blank?
                   call_hook(:controller_journals_edit_post, { :journal => @journal, :params => params})
                   respond_to do |format|
-                    format.html { redirect_to :controller => 'issues', :action => 'show', :id => @journal.journalized_id }
+                    format.html { redirect_to issue_path(@journal.journalized) }
                     format.js { render :action => 'update' }
                   end
                 else

app/controllers/members_controller.rb +9 -3

                 end
                 respond_to do |format|
-                  format.html { redirect_to :controller => 'projects', :action => 'settings', :tab => 'members', :id => @project }
+                  format.html { redirect_to_settings_in_projects }
                   format.js { @members = members }
                   format.api {
                     @member = members.first
                 end
                 saved = @member.save
                 respond_to do |format|
-                  format.html { redirect_to :controller => 'projects', :action => 'settings', :tab => 'members', :id => @project }
+                  format.html { redirect_to_settings_in_projects }
                   format.js
                   format.api {
                     if saved
                   @member.destroy
                 end
                 respond_to do |format|
-                  format.html { redirect_to :controller => 'projects', :action => 'settings', :tab => 'members', :id => @project }
+                  format.html { redirect_to_settings_in_projects }
                   format.js
                   format.api {
                     if @member.destroyed?
                 @principals = Principal.active.not_member_of(@project).like(params[:q]).all(:limit => 100)
                 render :layout => false
               end
+              private
+              def redirect_to_settings_in_projects
+                redirect_to settings_project_path(@project, :tab => 'members')
+              end
             end

app/controllers/my_controller.rb +8 -8

                     @user.notified_project_ids = (@user.mail_notification == 'selected' ? params[:notified_project_ids] : [])
                     set_language_if_valid @user.language
                     flash[:notice] = l(:notice_account_updated)
-                    redirect_to :action => 'account'
+                    redirect_to my_account_path
                     return
                   end
                 end
               def destroy
                 @user = User.current
                 unless @user.own_account_deletable?
-                  redirect_to :action => 'account'
+                  redirect_to my_account_path
                   return
                 end
                 @user = User.current
                 unless @user.change_password_allowed?
                   flash[:error] = l(:notice_can_t_change_password)
-                  redirect_to :action => 'account'
+                  redirect_to my_account_path
                   return
                 end
                 if request.post?
                     @user.password, @user.password_confirmation = params[:new_password], params[:new_password_confirmation]
                     if @user.save
                       flash[:notice] = l(:notice_account_password_updated)
-                      redirect_to :action => 'account'
+                      redirect_to my_account_path
                     end
                   else
                     flash[:error] = l(:notice_account_wrong_password)
                   User.current.rss_key
                   flash[:notice] = l(:notice_feeds_access_key_reseted)
                 end
-                redirect_to :action => 'account'
+                redirect_to my_account_path
               end
               # Create a new API key
                   User.current.api_key
                   flash[:notice] = l(:notice_api_access_key_reseted)
                 end
-                redirect_to :action => 'account'
+                redirect_to my_account_path
               end
               # User's page layout configuration
                 layout['top'].unshift block
                 @user.pref[:my_page_layout] = layout
                 @user.pref.save
-                redirect_to :action => 'page_layout'
+                redirect_to my_page_layout_path
               end
               # Remove a block to user's page
                 %w(top left right).each {|f| (layout[f] ||= []).delete block }
                 @user.pref[:my_page_layout] = layout
                 @user.pref.save
-                redirect_to :action => 'page_layout'
+                redirect_to my_page_layout_path
               end
               # Change blocks order on user's page

app/controllers/news_controller.rb +3 -3

                 if @news.save
                   render_attachment_warning_if_needed(@news)
                   flash[:notice] = l(:notice_successful_create)
-                  redirect_to :controller => 'news', :action => 'index', :project_id => @project
+                  redirect_to project_news_index_path(@project)
                 else
                   render :action => 'new'
                 end
                 if @news.save
                   render_attachment_warning_if_needed(@news)
                   flash[:notice] = l(:notice_successful_update)
-                  redirect_to :action => 'show', :id => @news
+                  redirect_to news_path(@news)
                 else
                   render :action => 'edit'
                 end
               def destroy
                 @news.destroy
-                redirect_to :action => 'index', :project_id => @project
+                redirect_to project_news_index_path(@project)
               end
               private

app/controllers/project_enumerations_controller.rb +2 -3

                   flash[:notice] = l(:notice_successful_update)
                 end
-                redirect_to :controller => 'projects', :action => 'settings', :tab => 'activities', :id => @project
+                redirect_to settings_project_path(@project, :tab => 'activities')
               end
               def destroy
                   time_entry_activity.destroy(time_entry_activity.parent)
                 end
                 flash[:notice] = l(:notice_successful_update)
-                redirect_to :controller => 'projects', :action => 'settings', :tab => 'activities', :id => @project
+                redirect_to settings_project_path(@project, :tab => 'activities')
               end
             end

app/controllers/projects_controller.rb +13 -12

                   respond_to do |format|
                     format.html {
                       flash[:notice] = l(:notice_successful_create)
-                      redirect_to(params[:continue] ?
+                      if params[:continue]
-                        {:controller => 'projects', :action => 'new', :project => {:parent_id => @project.parent_id}.reject {|k,v| v.nil?}} :
+                        attrs = {:parent_id => @project.parent_id}.reject {|k,v| v.nil?}
-                        {:controller => 'projects', :action => 'settings', :id => @project}
+                        redirect_to new_project_path(attrs)
+                      else
+                        redirect_to settings_project_path(@project)
+                      end
                     }
                     format.api  { render :action => 'show', :status => :created, :location => url_for(:controller => 'projects', :action => 'show', :id => @project.id) }
                   end
                     format.api  { render_validation_errors(@project) }
                   end
                 end
               end
               def copy
                     if validate_parent_id && @project.copy(@source_project, :only => params[:only])
                       @project.set_allowed_parent!(params[:project]['parent_id']) if params[:project].has_key?('parent_id')
                       flash[:notice] = l(:notice_successful_create)
-                      redirect_to :controller => 'projects', :action => 'settings', :id => @project
+                      redirect_to settings_project_path(@project)
                     elsif !@project.new_record?
                       # Project was created
                       # But some objects were not copied due to validation failures
                       # (eg. issues from disabled trackers)
                       # TODO: inform about that
-                      redirect_to :controller => 'projects', :action => 'settings', :id => @project
+                      redirect_to settings_project_path(@project)
                     end
                   end
                 end
                   respond_to do |format|
                     format.html {
                       flash[:notice] = l(:notice_successful_update)
-                      redirect_to :action => 'settings', :id => @project
+                      redirect_to settings_project_path(@project)
                     }
                     format.api  { render_api_ok }
                   end
               def modules
                 @project.enabled_module_names = params[:enabled_module_names]
                 flash[:notice] = l(:notice_successful_update)
-                redirect_to :action => 'settings', :id => @project, :tab => 'modules'
+                redirect_to settings_project_path(@project, :tab => 'modules')
               end
               def archive
                     flash[:error] = l(:error_can_not_archive_project)
                   end
                 end
-                redirect_to(url_for(:controller => 'admin', :action => 'projects', :status => params[:status]))
+                redirect_to admin_projects_path(:status => params[:status])
               end
               def unarchive
                 @project.unarchive if request.post? && !@project.active?
-                redirect_to(url_for(:controller => 'admin', :action => 'projects', :status => params[:status]))
+                redirect_to admin_projects_path(:status => params[:status])
               end
               def close
                 if api_request? || params[:confirm]
                   @project_to_destroy.destroy
                   respond_to do |format|
-                    format.html { redirect_to :controller => 'admin', :action => 'projects' }
+                    format.html { redirect_to admin_projects_path }
                     format.api  { render_api_ok }
                   end
                 end

app/controllers/queries_controller.rb +3 -3

                 if @query.save
                   flash[:notice] = l(:notice_successful_create)
-                  redirect_to :controller => 'issues', :action => 'index', :project_id => @project, :query_id => @query
+                  redirect_to _issues_path(@project, :query_id => @query)
                 else
                   render :action => 'new', :layout => !request.xhr?
                 end
                 if @query.save
                   flash[:notice] = l(:notice_successful_update)
-                  redirect_to :controller => 'issues', :action => 'index', :project_id => @project, :query_id => @query
+                  redirect_to _issues_path(@project, :query_id => @query)
                 else
                   render :action => 'edit'
                 end
               def destroy
                 @query.destroy
-                redirect_to :controller => 'issues', :action => 'index', :project_id => @project, :set_filter => 1
+                redirect_to _issues_path(@project, :set_filter => 1)
               end
             private

app/controllers/roles_controller.rb +5 -5

                     @role.workflow_rules.copy(copy_from)
                   end
                   flash[:notice] = l(:notice_successful_create)
-                  redirect_to :action => 'index'
+                  redirect_to roles_path
                 else
                   @roles = Role.sorted.all
                   render :action => 'new'
               def update
                 if request.put? and @role.update_attributes(params[:role])
                   flash[:notice] = l(:notice_successful_update)
-                  redirect_to :action => 'index'
+                  redirect_to roles_path
                 else
                   render :action => 'edit'
                 end
               def destroy
                 @role.destroy
-                redirect_to :action => 'index'
+                redirect_to roles_path
               rescue
                 flash[:error] =  l(:error_can_not_remove_role)
-                redirect_to :action => 'index'
+                redirect_to roles_path
               end
               def permissions
                     role.save
                   end
                   flash[:notice] = l(:notice_successful_update)
-                  redirect_to :action => 'index'
+                  redirect_to roles_path
                 end
               end

app/controllers/settings_controller.rb +2 -2

                     Setting[name] = value
                   end
                   flash[:notice] = l(:notice_successful_update)
-                  redirect_to :action => 'edit', :tab => params[:tab]
+                  redirect_to settings_path(:tab => params[:tab])
                 else
                   @options = {}
                   user_format = User::USER_FORMATS.collect{|key, value| [key, value[:setting_order]]}.sort{|a, b| a[1] <=> b[1]}
                 if request.post?
                   Setting.send "plugin_#{@plugin.id}=", params[:settings]
                   flash[:notice] = l(:notice_successful_update)
-                  redirect_to :action => 'plugin', :id => @plugin.id
+                  redirect_to plugin_settings_path(@plugin.id)
                 else
                   @partial = @plugin.settings[:partial]
                   @settings = Setting.send "plugin_#{@plugin.id}"

app/controllers/timelog_controller.rb +14 -6

                       flash[:notice] = l(:notice_successful_create)
                       if params[:continue]
                         if params[:project_id]
-                          redirect_to :action => 'new', :project_id => @time_entry.project, :issue_id => @time_entry.issue,
+                          options = {
                             :time_entry => {:issue_id => @time_entry.issue_id, :activity_id => @time_entry.activity_id},
                             :back_url => params[:back_url]
+                          }
+                          if @time_entry.issue
+                            redirect_to new_project_issue_time_entry_path(@time_entry.project, @time_entry.issue, options)
+                          else
+                            redirect_to new_project_time_entry_path(@time_entry.project, options)
+                          end
                         else
-                          redirect_to :action => 'new',
+                          options = {
                             :time_entry => {:project_id => @time_entry.project_id, :issue_id => @time_entry.issue_id, :activity_id => @time_entry.activity_id},
                             :back_url => params[:back_url]
+                          }
+                          redirect_to new_time_entry_path(options)
                         end
                       else
-                        redirect_back_or_default :action => 'index', :project_id => @time_entry.project
+                        redirect_back_or_default project_time_entries_path(@time_entry.project)
                       end
                     }
                     format.api  { render :action => 'show', :status => :created, :location => time_entry_url(@time_entry) }
                   respond_to do |format|
                     format.html {
                       flash[:notice] = l(:notice_successful_update)
-                      redirect_back_or_default :action => 'index', :project_id => @time_entry.project
+                      redirect_back_or_default project_time_entries_path(@time_entry.project)
                     }
                     format.api  { render_api_ok }
                   end
                   end
                 end
                 set_flash_from_bulk_time_entry_save(@time_entries, unsaved_time_entry_ids)
-                redirect_back_or_default({:controller => 'timelog', :action => 'index', :project_id => @projects.first})
+                redirect_back_or_default project_time_entries_path(@projects.first)
               end
               def destroy
                     else
                       flash[:error] = l(:notice_unable_delete_time_entry)
                     end
-                    redirect_back_or_default(:action => 'index', :project_id => @projects.first)
+                    redirect_back_or_default project_time_entries_path(@projects.first)
                   }
                   format.api  {
                     if destroyed

app/controllers/trackers_controller.rb +4 -4

                     @tracker.workflow_rules.copy(copy_from)
                   end
                   flash[:notice] = l(:notice_successful_create)
-                  redirect_to :action => 'index'
+                  redirect_to trackers_path
                   return
                 end
                 new
                 @tracker = Tracker.find(params[:id])
                 if @tracker.update_attributes(params[:tracker])
                   flash[:notice] = l(:notice_successful_update)
-                  redirect_to :action => 'index'
+                  redirect_to trackers_path
                   return
                 end
                 edit
                 else
                   @tracker.destroy
                 end
-                redirect_to :action => 'index'
+                redirect_to trackers_path
               end
               def fields
                     end
                   end
                   flash[:notice] = l(:notice_successful_update)
-                  redirect_to :action => 'fields'
+                  redirect_to fields_trackers_path
                   return
                 end
                 @trackers = Tracker.sorted.all

app/controllers/users_controller.rb +8 -7

                   respond_to do |format|
                     format.html {
                       flash[:notice] = l(:notice_user_successful_create, :id => view_context.link_to(@user.login, user_path(@user)))
-                      redirect_to(params[:continue] ?
+                      if params[:continue]
-                        {:controller => 'users', :action => 'new'} :
+                        redirect_to new_user_path
-                        {:controller => 'users', :action => 'edit', :id => @user}
+                      else
+                        redirect_to edit_user_path(@user)
+                      end
                     }
                     format.api  { render :action => 'show', :status => :created, :location => user_url(@user) }
                   end
               def destroy
                 @user.destroy
                 respond_to do |format|
-                  format.html { redirect_back_or_default(users_url) }
+                  format.html { redirect_back_or_default(users_path) }
                   format.api  { render_api_ok }
                 end
               end
                 @membership = Member.edit_membership(params[:membership_id], params[:membership], @user)
                 @membership.save
                 respond_to do |format|
-                  format.html { redirect_to :controller => 'users', :action => 'edit', :id => @user, :tab => 'memberships' }
+                  format.html { redirect_to edit_user_path(@user, :tab => 'memberships') }
                   format.js
                 end
               end
                   @membership.destroy
                 end
                 respond_to do |format|
-                  format.html { redirect_to :controller => 'users', :action => 'edit', :id => @user, :tab => 'memberships' }
+                  format.html { redirect_to edit_user_path(@user, :tab => 'memberships') }
                   format.js
                 end
               end

app/controllers/versions_controller.rb +5 -5

                     respond_to do |format|
                       format.html do
                         flash[:notice] = l(:notice_successful_create)
-                        redirect_back_or_default :controller => 'projects', :action => 'settings', :tab => 'versions', :id => @project
+                        redirect_back_or_default settings_project_path(@project, :tab => 'versions')
                       end
                       format.js
                       format.api do
                     respond_to do |format|
                       format.html {
                         flash[:notice] = l(:notice_successful_update)
-                        redirect_back_or_default :controller => 'projects', :action => 'settings', :tab => 'versions', :id => @project
+                        redirect_back_or_default settings_project_path(@project, :tab => 'versions')
                       }
                       format.api  { render_api_ok }
                     end
                 if request.put?
                   @project.close_completed_versions
                 end
-                redirect_to :controller => 'projects', :action => 'settings', :tab => 'versions', :id => @project
+                redirect_to settings_project_path(@project, :tab => 'versions')
               end
               def destroy
                 if @version.fixed_issues.empty?
                   @version.destroy
                   respond_to do |format|
-                    format.html { redirect_back_or_default :controller => 'projects', :action => 'settings', :tab => 'versions', :id => @project }
+                    format.html { redirect_back_or_default settings_project_path(@project, :tab => 'versions') }
                     format.api  { render_api_ok }
                   end
                 else
                   respond_to do |format|
                     format.html {
                       flash[:error] = l(:notice_unable_delete_version)
-                      redirect_to :controller => 'projects', :action => 'settings', :tab => 'versions', :id => @project
+                      redirect_to settings_project_path(@project, :tab => 'versions')
                     }
                     format.api  { head :unprocessable_entity }
                   end

app/controllers/wiki_controller.rb +6 -6

                   call_hook(:controller_wiki_edit_after_save, { :params => params, :page => @page})
                   respond_to do |format|
-                    format.html { redirect_to :action => 'show', :project_id => @project, :id => @page.title }
+                    format.html { redirect_to project_wiki_page_path(@project, @page.title) }
                     format.api {
                       if was_new_page
-                        render :action => 'show', :status => :created, :location => url_for(:controller => 'wiki', :action => 'show', :project_id => @project, :id => @page.title)
+                        render :action => 'show', :status => :created, :location => project_wiki_page_path(@project, @page.title)
                       else
                         render_api_ok
                       end
                 @original_title = @page.pretty_title
                 if request.post? && @page.update_attributes(params[:wiki_page])
                   flash[:notice] = l(:notice_successful_update)
-                  redirect_to :action => 'show', :project_id => @project, :id => @page.title
+                  redirect_to project_wiki_page_path(@project, @page.title)
                 end
               end
               def protect
                 @page.update_attribute :protected, params[:protected]
-                redirect_to :action => 'show', :project_id => @project, :id => @page.title
+                redirect_to project_wiki_page_path(@project, @page.title)
               end
               # show page history
                 end
                 @page.destroy
                 respond_to do |format|
-                  format.html { redirect_to :action => 'index', :project_id => @project }
+                  format.html { redirect_to project_wiki_index_path(@project) }
                   format.api { render_api_ok }
                 end
               end
                 @content = @page.content_for_version(params[:version])
                 @content.destroy
-                redirect_to_referer_or :action => 'history', :id => @page.title, :project_id => @project
+                redirect_to_referer_or history_project_wiki_page_path(@project, @page.title)
               end
               # Export wiki to a single pdf or html file

app/controllers/wikis_controller.rb +1 -1

               def destroy
                 if request.post? && params[:confirm] && @project.wiki
                   @project.wiki.destroy
-                  redirect_to :controller => 'projects', :action => 'settings', :id => @project, :tab => 'wiki'
+                  redirect_to settings_project_path(@project, :tab => 'wiki')
                 end
               end
             end

app/controllers/workflows_controller.rb +3 -3

                     }
                   }
                   if @role.save
-                    redirect_to :action => 'edit', :role_id => @role, :tracker_id => @tracker, :used_statuses_only => params[:used_statuses_only]
+                    redirect_to workflows_edit_path(:role_id => @role, :tracker_id => @tracker, :used_statuses_only => params[:used_statuses_only])
                     return
                   end
                 end
                 if request.post? && @role && @tracker
                   WorkflowPermission.replace_permissions(@tracker, @role, params[:permissions] || {})
-                  redirect_to :action => 'permissions', :role_id => @role, :tracker_id => @tracker, :used_statuses_only => params[:used_statuses_only]
+                  redirect_to workflows_permissions_path(:role_id => @role, :tracker_id => @tracker, :used_statuses_only => params[:used_statuses_only])
                   return
                 end
                   else
                     WorkflowRule.copy(@source_tracker, @source_role, @target_trackers, @target_roles)
                     flash[:notice] = l(:notice_successful_update)
-                    redirect_to :action => 'copy', :source_tracker_id => @source_tracker, :source_role_id => @source_role
+                    redirect_to workflows_copy_path(:source_tracker_id => @source_tracker, :source_role_id => @source_role)
                   end
                 end
               end

config/routes.rb +2 -2

                 end
                 match 'wiki/index', :controller => 'wiki', :action => 'index', :via => :get
-                resources :wiki, :except => [:index, :new, :create] do
+                resources :wiki, :except => [:index, :new, :create], :as => 'wiki_page' do
                   member do
                     get 'rename'
                     post 'rename'
               match 'workflows/copy', :controller => 'workflows', :action => 'copy', :via => [:get, :post]
               match 'settings', :controller => 'settings', :action => 'index', :via => :get
               match 'settings/edit', :controller => 'settings', :action => 'edit', :via => [:get, :post]
-              match 'settings/plugin/:id', :controller => 'settings', :action => 'plugin', :via => [:get, :post]
+              match 'settings/plugin/:id', :controller => 'settings', :action => 'plugin', :via => [:get, :post], :as => 'plugin_settings'
               match 'sys/projects', :to => 'sys#projects', :via => :get
               match 'sys/projects/:id/repository', :to => 'sys#create_project_repository', :via => :post

test/functional/projects_controller_test.rb +1 -1

                     assert_difference 'Project.count' do
                       post :create, :project => {:name => "blog", :identifier => "blog"}, :continue => 'Create and continue'
                     end
-                    assert_redirected_to '/projects/new?'
+                    assert_redirected_to '/projects/new'
                   end
                 end

test/functional/settings_controller_test.rb +1 -1

                                           :notified_events => %w(issue_added issue_updated news_added),
                                           :emails_footer => 'Test footer'
                                           }
-                assert_redirected_to '/settings/edit'
+                assert_redirected_to '/settings'
                 assert_equal 'functional@test.foo', Setting.mail_from
                 assert !Setting.bcc_recipients?
                 assert_equal %w(issue_added issue_updated news_added), Setting.notified_events

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages