Employ declared_params in finder methods for MR and Issue API lists

- Ensure that unwanted params are no passed to actual finder classes

Employ declared_params in finder methods for MR and Issue API lists
c9732b3c · Mark Fletcher · 5b880f0d · c9732b3c · c9732b3c
Commit c9732b3c authored Dec 19, 2017 by Mark Fletcher
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

issues.rb lib/api/issues.rb +1 -1

merge_requests.rb lib/api/merge_requests.rb +1 -1

No files found.
--- a/lib/api/issues.rb
+++ b/lib/api/issues.rb
@@ -8,7 +8,7 @@ module API

    helpers do
      def find_issues(args = {})
-        args = params.merge(args)
+        args = declared_params.merge(args)

        args.delete(:id)
        args[:milestone_title] = args.delete(:milestone)

--- a/lib/api/merge_requests.rb
+++ b/lib/api/merge_requests.rb
@@ -8,7 +8,7 @@ module API

    helpers do
      def find_merge_requests(args = {})
-        args = params.merge(args)
+        args = declared_params.merge(args)

        args[:milestone_title] = args.delete(:milestone)
        args[:label_name] = args.delete(:labels)