cd88fa8f80
also, I refactored the MergeRequest#fetch_ref method to express the side-effect that this method has. MergeRequest#fetch_ref -> MergeRequest#fetch_ref! Repository#fetch_source_branch -> Repository#fetch_source_branch!
41 lines
1 KiB
Ruby
41 lines
1 KiB
Ruby
class Projects::MergeRequests::ApplicationController < Projects::ApplicationController
|
|
before_action :check_merge_requests_available!
|
|
before_action :merge_request
|
|
before_action :authorize_read_merge_request!
|
|
|
|
private
|
|
|
|
def merge_request
|
|
@issuable = @merge_request ||= @project.merge_requests.find_by!(iid: params[:id])
|
|
end
|
|
|
|
def merge_request_params
|
|
params.require(:merge_request).permit(merge_request_params_attributes)
|
|
end
|
|
|
|
def merge_request_params_attributes
|
|
[
|
|
:assignee_id,
|
|
:description,
|
|
:force_remove_source_branch,
|
|
:lock_version,
|
|
:milestone_id,
|
|
:source_branch,
|
|
:source_project_id,
|
|
:state_event,
|
|
:target_branch,
|
|
:target_project_id,
|
|
:task_num,
|
|
:title,
|
|
:discussion_locked,
|
|
|
|
label_ids: []
|
|
]
|
|
end
|
|
|
|
def set_pipeline_variables
|
|
@pipelines = @merge_request.all_pipelines
|
|
@pipeline = @merge_request.head_pipeline
|
|
@statuses_count = @pipeline.present? ? @pipeline.statuses.relevant.count : 0
|
|
end
|
|
end
|