diff options
author | Fatih Acet <acetfatih@gmail.com> | 2016-07-28 13:40:29 +0300 |
---|---|---|
committer | Fatih Acet <acetfatih@gmail.com> | 2016-08-11 01:15:27 +0300 |
commit | 5dffd21cd526fadc3daa041fe88b60388dc786ed (patch) | |
tree | dbc6c9d649cf292ec1059fa2e883f00fd9d7dab4 | |
parent | 279c61f3cbe79f37578ff4366adfc55a63cc5b5b (diff) | |
download | gitlab-ce-5dffd21cd526fadc3daa041fe88b60388dc786ed.tar.gz |
Fix error message for existing labels.label-flash
-rw-r--r-- | CHANGELOG | 3 | ||||
-rw-r--r-- | app/assets/javascripts/labels_select.js | 12 |
2 files changed, 10 insertions, 5 deletions
diff --git a/CHANGELOG b/CHANGELOG index 7126f390417..721d05a3bda 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -112,6 +112,9 @@ v 8.10.3 - Fix importer for GitHub Pull Requests when a branch was removed. !5573 - Ignore invalid IPs in X-Forwarded-For when trusted proxies are configured. !5584 - Trim extra displayed carriage returns in diffs and files with CRLFs. !5588 + - Fix label already exist error message in the right sidebar. + +v 8.10.3 (unreleased) v 8.10.2 - User can now search branches by name. !5144 diff --git a/app/assets/javascripts/labels_select.js b/app/assets/javascripts/labels_select.js index 675dd5b7cea..1bb0b67d0e8 100644 --- a/app/assets/javascripts/labels_select.js +++ b/app/assets/javascripts/labels_select.js @@ -70,13 +70,15 @@ name: newLabelField.val(), color: newColorField.val() }, function(label) { - var errors; $newLabelCreateButton.enable(); if (label.message != null) { - errors = _.map(label.message, function(value, key) { - return key + " " + value[0]; - }); - return $newLabelError.html(errors.join("<br/>")).show(); + var errorText = label.message; + if (_.isObject(label.message)) { + errorText = _.map(label.message, function(value, key) { + return key + " " + value[0]; + }).join('<br/>'); + } + return $newLabelError.html(errorText).show(); } else { return $('.dropdown-menu-back', $dropdown.parent()).trigger('click'); } |