diff options
| author | Stan Hu <stanhu@gmail.com> | 2019-08-27 20:31:59 +0300 |
|---|---|---|
| committer | Stan Hu <stanhu@gmail.com> | 2019-08-29 08:49:58 +0300 |
| commit | 680f437715dcf7a8871aa997559cf57362b43217 (patch) | |
| tree | 83ee049647b181ed9e74ad3d221f5d78106c5c8b /app/services/groups/create_service.rb | |
| parent | 549e95b8f921dfb30bc7982e9957ce9ccdfd916e (diff) | |
Fix snippets API not working with visibility level
When a restricted visibility level of `private` is set in the instance,
creating a snippet with the `visibility` level would always fail.
This happened because:
1. `params[:visibility]` was a string (e.g. "public")
2. `CreateSnippetService` and `UpdateSnippetService` only looked
at `params[:visibility_level]`, which was `nil`.
To fix this, we:
1. Make `CreateSnippetService` look at the newly-built
`snippet.visibility_level`, since the right value is assigned by the
`VisibilityLevel#visibility=` method.
2. Modify `UpdateSnippetService` to handle both `visibility_level` and
`visibility` parameters.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/66050
Diffstat (limited to 'app/services/groups/create_service.rb')
| -rw-r--r-- | app/services/groups/create_service.rb | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/app/services/groups/create_service.rb b/app/services/groups/create_service.rb index e78e5d5fc2c..1dd22d7a3ae 100644 --- a/app/services/groups/create_service.rb +++ b/app/services/groups/create_service.rb @@ -68,9 +68,5 @@ module Groups true end - - def visibility_level - params[:visibility].present? ? Gitlab::VisibilityLevel.level_value(params[:visibility]) : params[:visibility_level] - end end end |