diff options
author | Heinrich Lee Yu <heinrich@gitlab.com> | 2018-12-14 07:13:51 +0300 |
---|---|---|
committer | Heinrich Lee Yu <heinrich@gitlab.com> | 2018-12-14 07:58:40 +0300 |
commit | 7d1a7261f07886fceeb932cfa8ba4aa5001f8b83 (patch) | |
tree | f9840115f8e51191dd6ba9e217486345219b1fa8 /app/helpers | |
parent | 0087396036b265879771380e60787e2072318fd1 (diff) |
Change `page_filter_path` default behavior
`label_name` is now not excluded by default
Also removed `label: true` from calls to the helper
Diffstat (limited to 'app/helpers')
-rw-r--r-- | app/helpers/application_helper.rb | 7 | ||||
-rw-r--r-- | app/helpers/sorting_helper.rb | 2 |
2 files changed, 2 insertions, 7 deletions
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb index 74042f0bae8..82bb2d1a805 100644 --- a/app/helpers/application_helper.rb +++ b/app/helpers/application_helper.rb @@ -171,7 +171,6 @@ module ApplicationHelper def page_filter_path(options = {}) without = options.delete(:without) - add_label = options.delete(:label) options = request.query_parameters.merge(options) @@ -181,11 +180,7 @@ module ApplicationHelper end end - params = options.compact - - params.delete(:label_name) unless add_label - - "#{request.path}?#{params.to_param}" + "#{request.path}?#{options.compact.to_param}" end def outdated_browser? diff --git a/app/helpers/sorting_helper.rb b/app/helpers/sorting_helper.rb index 67c808b167a..f51b96ba8ce 100644 --- a/app/helpers/sorting_helper.rb +++ b/app/helpers/sorting_helper.rb @@ -164,7 +164,7 @@ module SortingHelper reverse_sort = issuable_reverse_sort_order_hash[sort_value] if reverse_sort - reverse_url = page_filter_path(sort: reverse_sort, label: true) + reverse_url = page_filter_path(sort: reverse_sort) else reverse_url = '#' link_class += ' disabled' |