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

                  url
                end
+               # Returns the path to project issues or to the cross-project
+               # issue list if project is nil
+               def _issues_path(project, *args)
+                 if project
+                   project_issues_path(project, *args)
+                 else
+                   issues_path(*args)
+                 end
+               end
                def redirect_back_or_default(default)
                  back_url = params[:back_url].to_s
                  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?} } :
-                                   { :action => 'show', :id => @issue })
+                       if params[:continue]
+                         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] ?
-                         {:controller => 'projects', :action => 'new', :project => {:parent_id => @project.parent_id}.reject {|k,v| v.nil?}} :
-                         {:controller => 'projects', :action => 'settings', :id => @project}
+                       )
+                       if params[:continue]
+                         attrs = {:parent_id => @project.parent_id}.reject {|k,v| v.nil?}
+                         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] ?
-                         {:controller => 'users', :action => 'new'} :
-                         {:controller => 'users', :action => 'edit', :id => @user}
+                       )
+                       if params[:continue]
+                         redirect_to new_user_path
+                       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