diff options
| author | dineshpanda <dineshpanda92@gmail.com> | 2019-09-01 00:51:48 +0530 |
|---|---|---|
| committer | dineshpanda <dineshpanda92@gmail.com> | 2019-09-04 09:19:07 +0530 |
| commit | a3de0cd60369f6ee70e98a47b8db5a309a655354 (patch) | |
| tree | c6934056ec7ba789a0438f4fa568230d330ea600 | |
| parent | df6f1dd93f0f427a2c66c07ea31bdb17a7efaca6 (diff) | |
| download | gitlab-ce-a3de0cd60369f6ee70e98a47b8db5a309a655354.tar.gz | |
Avoid calling freeze on already frozen strings in app/finders
| -rw-r--r-- | app/finders/issuable_finder.rb | 6 | ||||
| -rw-r--r-- | app/finders/todos_finder.rb | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/app/finders/issuable_finder.rb b/app/finders/issuable_finder.rb index 1773ac2d508..b735f9ff3b8 100644 --- a/app/finders/issuable_finder.rb +++ b/app/finders/issuable_finder.rb @@ -40,11 +40,11 @@ class IssuableFinder requires_cross_project_access unless: -> { project? } # This is used as a common filter for None / Any - FILTER_NONE = 'none'.freeze - FILTER_ANY = 'any'.freeze + FILTER_NONE = 'none' + FILTER_ANY = 'any' # This is used in unassigning users - NONE = '0'.freeze + NONE = '0' attr_accessor :current_user, :params diff --git a/app/finders/todos_finder.rb b/app/finders/todos_finder.rb index d001e18fea9..ed6d20b9585 100644 --- a/app/finders/todos_finder.rb +++ b/app/finders/todos_finder.rb @@ -21,7 +21,7 @@ class TodosFinder requires_cross_project_access unless: -> { project? } - NONE = '0'.freeze + NONE = '0' TODO_TYPES = Set.new(%w(Issue MergeRequest Epic)).freeze |