diff options
author | Stan Hu <stanhu@gmail.com> | 2016-10-24 02:46:21 +0300 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2016-10-25 08:40:08 +0300 |
commit | ddafea060d4b607cd3f5c29e947cdbf6483dcd5d (patch) | |
tree | 18228dce12f544994f1c77d448a76a18fd8eb7d2 /app/finders | |
parent | add3a2c4431f29ca84c956f5604769505f0f0904 (diff) |
Fix bug where labels would be assigned to issues that were moved
If you attempt to move an issue from one project to another and leave
labels blank, LabelsFinder would assign all labels in the new project
to that issue. The issue is that :title is passed along to the Finder,
but since it appears empty no filtering is done. As a result, all
labels in the group are returned. This fix handles that case.
Closes #23668
Diffstat (limited to 'app/finders')
-rw-r--r-- | app/finders/labels_finder.rb | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/app/finders/labels_finder.rb b/app/finders/labels_finder.rb index 6ace14a4bb5..2291c64b84d 100644 --- a/app/finders/labels_finder.rb +++ b/app/finders/labels_finder.rb @@ -35,6 +35,10 @@ class LabelsFinder < UnionFinder end def with_title(items) + # Match no labels if an empty title is supplied to avoid matching all + # labels (e.g. when an issue is moved) + return Label.none if params[:title] && params[:title].empty? + items = items.where(title: title) if title items end |