diff options
| author | Stan Hu <stanhu@gmail.com> | 2019-04-15 20:55:45 +0300 |
|---|---|---|
| committer | Stan Hu <stanhu@gmail.com> | 2019-04-15 21:27:33 +0300 |
| commit | d3fa9c9539aac42844a297a99ef56254ce1c06a3 (patch) | |
| tree | ba94fe9d979a003585ede0c8f499030c731fa3f1 /changelogs | |
| parent | 1a50801cd0801d3134b41e96ff2a6b27a96a1047 (diff) | |
Fix remove_source_branch merge request API handling
Users attempting to set merge requests to `remove_source_branch` to
`false` would encounter an Error 500 because the UpdateService and API
checked `present?`, which would always return `false`. We now just
use `has_key?` to decide whether the parameter is present.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/60530
Diffstat (limited to 'changelogs')
| -rw-r--r-- | changelogs/unreleased/sh-fix-merge-requests-api-remove-branch-param.yml | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/changelogs/unreleased/sh-fix-merge-requests-api-remove-branch-param.yml b/changelogs/unreleased/sh-fix-merge-requests-api-remove-branch-param.yml new file mode 100644 index 00000000000..d13c972ccc9 --- /dev/null +++ b/changelogs/unreleased/sh-fix-merge-requests-api-remove-branch-param.yml @@ -0,0 +1,5 @@ +--- +title: Fix remove_source_branch merge request API handling +merge_request: 27392 +author: +type: fixed |