Skip to content

Return project when project was successfully updated

Issue: when changed project settings and submit, the notice("Project '#{@project.name}' was successfully updated.") was not dispalyed.

https://gitlab.com/gitlab-org/gitlab-ce/blob/master/app/controllers/projects_controller.rb#L46

def update
    status = ::Projects::UpdateService.new(@project, current_user, project_params).execute

    respond_to do |format|
      if status
        flash[:notice] = "Project '#{@project.name}' was successfully updated."
        format.html do
          redirect_to(
            edit_project_path(@project),
            notice: "Project '#{@project.name}' was successfully updated."
          )
        end
        format.js
      else
        format.html { render 'edit' }
        format.js
      end
    end
  end

Version: 8.5(latest release)

So I add a return value for the project update service to fix the issue.

Merge request reports