Apply *_params_ce
pattern to ProjectsController
This commit is contained in:
parent
87afcdff9c
commit
d59956afc0
1 changed files with 35 additions and 17 deletions
|
@ -318,26 +318,44 @@ class ProjectsController < Projects::ApplicationController
|
|||
end
|
||||
|
||||
def project_params
|
||||
project_feature_attributes =
|
||||
{
|
||||
project_feature_attributes:
|
||||
[
|
||||
:issues_access_level, :builds_access_level,
|
||||
:wiki_access_level, :merge_requests_access_level,
|
||||
:snippets_access_level, :repository_access_level
|
||||
]
|
||||
}
|
||||
params.require(:project)
|
||||
.permit(project_params_ce)
|
||||
end
|
||||
|
||||
params.require(:project).permit(
|
||||
:name, :path, :description, :issues_tracker, :tag_list, :runners_token,
|
||||
def project_params_ce
|
||||
[
|
||||
:avatar,
|
||||
:build_allow_git_fetch,
|
||||
:build_coverage_regex,
|
||||
:build_timeout_in_minutes,
|
||||
:container_registry_enabled,
|
||||
:issues_tracker_id, :default_branch,
|
||||
:visibility_level, :import_url, :last_activity_at, :namespace_id, :avatar,
|
||||
:build_allow_git_fetch, :build_timeout_in_minutes, :build_coverage_regex,
|
||||
:public_builds, :only_allow_merge_if_build_succeeds, :request_access_enabled,
|
||||
:default_branch,
|
||||
:description,
|
||||
:import_url,
|
||||
:issues_tracker,
|
||||
:issues_tracker_id,
|
||||
:last_activity_at,
|
||||
:lfs_enabled,
|
||||
:name,
|
||||
:namespace_id,
|
||||
:only_allow_merge_if_all_discussions_are_resolved,
|
||||
:lfs_enabled, project_feature_attributes
|
||||
)
|
||||
:only_allow_merge_if_build_succeeds,
|
||||
:path,
|
||||
:public_builds,
|
||||
:request_access_enabled,
|
||||
:runners_token,
|
||||
:tag_list,
|
||||
:visibility_level,
|
||||
|
||||
project_feature_attributes: %i[
|
||||
builds_access_level
|
||||
issues_access_level
|
||||
merge_requests_access_level
|
||||
repository_access_level
|
||||
snippets_access_level
|
||||
wiki_access_level
|
||||
]
|
||||
]
|
||||
end
|
||||
|
||||
def repo_exists?
|
||||
|
|
Loading…
Reference in a new issue