From 6b35c4ad65205391a054725e2bf0b86985895b86 Mon Sep 17 00:00:00 2001 From: Douwe Maan Date: Thu, 17 Nov 2016 09:39:23 +0000 Subject: [PATCH] Revert "Merge branch '22680-unlabel-limit-autocomplete-to-selected-items' into 'master'" This reverts merge request !6796 --- app/assets/javascripts/gfm_auto_complete.js.es6 | 6 ------ app/controllers/projects_controller.rb | 4 +--- app/services/projects/autocomplete_service.rb | 9 +-------- ...2680-unlabel-limit-autocomplete-to-selected-items.yml | 4 ---- 4 files changed, 2 insertions(+), 21 deletions(-) delete mode 100644 changelogs/unreleased/22680-unlabel-limit-autocomplete-to-selected-items.yml diff --git a/app/assets/javascripts/gfm_auto_complete.js.es6 b/app/assets/javascripts/gfm_auto_complete.js.es6 index 19bfdf1de8c..58c1179d250 100644 --- a/app/assets/javascripts/gfm_auto_complete.js.es6 +++ b/app/assets/javascripts/gfm_auto_complete.js.es6 @@ -51,11 +51,6 @@ if (!GitLab.GfmAutoComplete.dataLoaded) { return this.at; } else { - if (value.indexOf("unlabel") !== -1) { - GitLab.GfmAutoComplete.input.atwho('load', '~', GitLab.GfmAutoComplete.cachedData.unlabels); - } else { - GitLab.GfmAutoComplete.input.atwho('load', '~', GitLab.GfmAutoComplete.cachedData.labels); - } return value; } } @@ -363,4 +358,3 @@ }; }).call(this); - diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb index 7376c2bfeb7..a8a18b4fa16 100644 --- a/app/controllers/projects_controller.rb +++ b/app/controllers/projects_controller.rb @@ -144,15 +144,13 @@ class ProjectsController < Projects::ApplicationController autocomplete = ::Projects::AutocompleteService.new(@project, current_user) participants = ::Projects::ParticipantsService.new(@project, current_user).execute(noteable) - unlabels = autocomplete.unlabels(noteable) @suggestions = { emojis: Gitlab::AwardEmoji.urls, issues: autocomplete.issues, milestones: autocomplete.milestones, mergerequests: autocomplete.merge_requests, - labels: autocomplete.labels - unlabels, - unlabels: unlabels, + labels: autocomplete.labels, members: participants, commands: autocomplete.commands(noteable, params[:type]) } diff --git a/app/services/projects/autocomplete_service.rb b/app/services/projects/autocomplete_service.rb index 223461e88b6..015f2828921 100644 --- a/app/services/projects/autocomplete_service.rb +++ b/app/services/projects/autocomplete_service.rb @@ -13,14 +13,7 @@ module Projects end def labels - LabelsFinder.new(current_user, project_id: project.id).execute. - pluck(:title, :color).map { |l| { title: l.first, color: l.second } } - end - - def unlabels(noteable) - return [] unless noteable && noteable.respond_to?(:labels) - - noteable.labels.pluck(:title, :color).map { |l| { title: l.first, color: l.second } } + LabelsFinder.new(current_user, project_id: project.id).execute.select([:title, :color]) end def commands(noteable, type) diff --git a/changelogs/unreleased/22680-unlabel-limit-autocomplete-to-selected-items.yml b/changelogs/unreleased/22680-unlabel-limit-autocomplete-to-selected-items.yml deleted file mode 100644 index 95fd07c12e1..00000000000 --- a/changelogs/unreleased/22680-unlabel-limit-autocomplete-to-selected-items.yml +++ /dev/null @@ -1,4 +0,0 @@ ---- -title: Limit autocomplete to currently selected items for unlabel slash command -merge_request: 22680 -author: Akram Fares -- GitLab