github_api 0.6.0 → 0.6.1

data/lib/github_api/issues.rb

@@ -84,8 +84,8 @@ module Github
     #
     def list(*args)
       params = args.extract_options!
-      _normalize_params_keys(params)
-      _filter_params_keys(VALID_ISSUE_PARAM_NAMES, params)
+      normalize! params
+      filter! VALID_ISSUE_PARAM_NAMES, params
       # _merge_mime_type(:issue, params)
       assert_valid_values(VALID_ISSUE_PARAM_VALUES, params)
 
@@ -129,8 +129,8 @@ module Github
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
 
-      _normalize_params_keys(params)
-      _filter_params_keys(VALID_ISSUE_PARAM_NAMES, params)
+      normalize! params
+      filter! VALID_ISSUE_PARAM_NAMES, params
       # _merge_mime_type(:issue, params)
       assert_valid_values(VALID_ISSUE_PARAM_VALUES, params)
 
@@ -151,7 +151,7 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of issue_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:issue, params)
 
       get_request("/repos/#{user}/#{repo}/issues/#{issue_id}", params)
@@ -182,9 +182,9 @@ module Github
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:issue, params)
-      _filter_params_keys(VALID_ISSUE_PARAM_NAMES, params)
+      filter! VALID_ISSUE_PARAM_NAMES, params
       assert_required_keys(%w[ title ], params)
 
       post_request("/repos/#{user}/#{repo}/issues", params)
@@ -217,9 +217,9 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of issue_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:issue, params)
-      _filter_params_keys(VALID_ISSUE_PARAM_NAMES, params)
+      filter! VALID_ISSUE_PARAM_NAMES, params
 
       patch_request("/repos/#{user}/#{repo}/issues/#{issue_id}", params)
     end

data/lib/github_api/normalizer.rb

@@ -0,0 +1,27 @@
+# -*- encoding: utf-8 -*-
+
+module Github
+  # Deals with normalazing client supplied parameter keys.
+  module Normalizer
+
+    # Turns any keys from nested hashes including nested arrays into strings
+    #
+    def normalize!(params)
+      case params
+      when Hash
+        params.keys.each do |k|
+          params[k.to_s] = params.delete(k)
+          normalize!(params[k.to_s])
+        end
+      when Array
+        params.map! do |el|
+          normalize!(el)
+        end
+      else
+        params.to_s
+      end
+      return params
+    end
+
+  end # Normalizer
+end # Github

data/lib/github_api/orgs/members.rb

@@ -18,7 +18,7 @@ module Github
     #
     def list(org_name, params={})
       _validate_presence_of org_name
-      _normalize_params_keys(params)
+      normalize! params
       response = get_request("/orgs/#{org_name}/members", params)
       return response unless block_given?
       response.each { |el| yield el }
@@ -33,7 +33,7 @@ module Github
     #
     def member?(org_name, member_name, params={})
       _validate_presence_of org_name, member_name
-      _normalize_params_keys(params)
+      normalize! params
       get_request("/orgs/#{org_name}/members/#{member_name}", params)
       true
     rescue Github::Error::NotFound
@@ -50,7 +50,7 @@ module Github
     #
     def delete(org_name, member_name, params={})
       _validate_presence_of org_name, member_name
-      _normalize_params_keys(params)
+      normalize! params
       delete_request("/orgs/#{org_name}/members/#{member_name}", params)
     end
     alias :remove :delete
@@ -64,7 +64,7 @@ module Github
     #
     def list_public(org_name, params={})
       _validate_presence_of org_name
-      _normalize_params_keys(params)
+      normalize! params
       response = get_request("/orgs/#{org_name}/public_members", params)
       return response unless block_given?
       response.each { |el| yield el }
@@ -79,7 +79,7 @@ module Github
     #
     def public_member?(org_name, member_name, params={})
       _validate_presence_of org_name, member_name
-      _normalize_params_keys(params)
+      normalize! params
       get_request("/orgs/#{org_name}/public_members/#{member_name}", params)
       true
     rescue Github::Error::NotFound
@@ -94,7 +94,7 @@ module Github
     #
     def publicize(org_name, member_name, params={})
       _validate_presence_of org_name, member_name
-      _normalize_params_keys(params)
+      normalize! params
       put_request("/orgs/#{org_name}/public_members/#{member_name}", params)
     end
     alias :make_public :publicize
@@ -108,7 +108,7 @@ module Github
     #
     def conceal(org_name, member_name, params={})
       _validate_presence_of org_name, member_name
-      _normalize_params_keys(params)
+      normalize! params
       delete_request("/orgs/#{org_name}/public_members/#{member_name}", params)
     end
     alias :conceal_membership :conceal

data/lib/github_api/orgs/teams.rb

@@ -19,7 +19,7 @@ module Github
     #
     def list(org_name, params={})
       _validate_presence_of org_name
-      _normalize_params_keys(params)
+      normalize! params
 
       response = get_request("/orgs/#{org_name}/teams", params)
       return response unless block_given?
@@ -35,7 +35,7 @@ module Github
     #
     def get(team_name, params={})
       _validate_presence_of team_name
-      _normalize_params_keys(params)
+      normalize! params
 
       get_request("/teams/#{team_name}", params)
     end
@@ -63,8 +63,8 @@ module Github
     #
     def create(org_name, params={})
       _validate_presence_of org_name
-      _normalize_params_keys(params)
-      _filter_params_keys(VALID_TEAM_PARAM_NAMES, params)
+      normalize! params
+      filter! VALID_TEAM_PARAM_NAMES, params
       assert_valid_values(VALID_TEAM_PARAM_VALUES, params)
       assert_required_keys(%w[ name ], params)
 
@@ -89,9 +89,8 @@ module Github
     #
     def edit(team_name, params={})
       _validate_presence_of team_name
-      _normalize_params_keys(params)
-
-      _filter_params_keys(VALID_TEAM_PARAM_NAMES, params)
+      normalize! params
+      filter! VALID_TEAM_PARAM_NAMES, params
       assert_valid_values(VALID_TEAM_PARAM_VALUES, params)
       assert_required_keys(%w[ name ], params)
 
@@ -107,7 +106,7 @@ module Github
     #
     def delete(team_name, params={})
       _validate_presence_of team_name
-      _normalize_params_keys(params)
+      normalize! params
       delete_request("/teams/#{team_name}", params)
     end
     alias :remove :delete
@@ -122,7 +121,7 @@ module Github
     #
     def list_members(team_name, params={})
       _validate_presence_of team_name
-      _normalize_params_keys(params)
+      normalize! params
 
       response = get_request("/teams/#{team_name}/members", params)
       return response unless block_given?
@@ -138,7 +137,7 @@ module Github
     #
     def team_member?(team_name, member_name, params={})
       _validate_presence_of team_name, member_name
-      _normalize_params_keys(params)
+      normalize! params
       get_request("/teams/#{team_name}/members/#{member_name}", params)
       true
     rescue Github::Error::NotFound
@@ -154,7 +153,7 @@ module Github
     #
     def add_member(team_name, member_name, params={})
       _validate_presence_of team_name, member_name
-      _normalize_params_keys(params)
+      normalize! params
       put_request("/teams/#{team_name}/members/#{member_name}", params)
     end
     alias :add_team_member :add_member
@@ -171,7 +170,7 @@ module Github
     #
     def remove_member(team_name, member_name, params={})
       _validate_presence_of team_name, member_name
-      _normalize_params_keys(params)
+      normalize! params
       delete_request("/teams/#{team_name}/members/#{member_name}", params)
     end
     alias :remove_team_member :remove_member
@@ -184,7 +183,7 @@ module Github
     #
     def list_repos(team_name, params={})
       _validate_presence_of team_name
-      _normalize_params_keys(params)
+      normalize! params
 
       response = get_request("/teams/#{team_name}/repos", params)
       return response unless block_given?
@@ -200,7 +199,7 @@ module Github
     #
     def team_repo?(team_name, user_name, repo_name, params={})
       _validate_presence_of team_name, user_name, repo_name
-      _normalize_params_keys(params)
+      normalize! params
       get_request("/teams/#{team_name}/repos/#{user_name}/#{repo_name}", params)
       true
     rescue Github::Error::NotFound
@@ -221,7 +220,7 @@ module Github
     #
     def add_repo(team_name, user_name, repo_name, params={})
       _validate_presence_of team_name, user_name, repo_name
-      _normalize_params_keys(params)
+      normalize! params
       put_request("/teams/#{team_name}/repos/#{user_name}/#{repo_name}", params)
     end
     alias :add_repository :add_repo
@@ -238,7 +237,7 @@ module Github
     #
     def remove_repo(team_name, user_name, repo_name, params={})
       _validate_presence_of team_name, user_name, repo_name
-      _normalize_params_keys(params)
+      normalize! params
       delete_request("/teams/#{team_name}/repos/#{user_name}/#{repo_name}", params)
     end
     alias :remove_repository :remove_repo

data/lib/github_api/orgs.rb

@@ -44,7 +44,7 @@ module Github
     #
     def list(*args)
       params = args.extract_options!
-      _normalize_params_keys(params)
+      normalize! params
 
       response = if (user_name = params.delete("user"))
         get_request("/users/#{user_name}/orgs", params)
@@ -65,6 +65,7 @@ module Github
     #
     def get(org_name, params={})
       _validate_presence_of org_name
+      normalize! params
       get_request("/orgs/#{org_name}", params)
     end
     alias :find :get
@@ -90,8 +91,8 @@ module Github
     #
     def edit(org_name, params={})
       _validate_presence_of org_name
-      _normalize_params_keys(params)
-      _filter_params_keys(VALID_ORG_PARAM_NAMES, params)
+      normalize! params
+      filter! VALID_ORG_PARAM_NAMES, params
 
       patch_request("/orgs/#{org_name}", params)
     end

data/lib/github_api/parameter_filter.rb

@@ -0,0 +1,32 @@
+# -*- encoding: utf-8 -*-
+
+module Github
+  # Allows you to specify parameters keys which will be preserved
+  # in parameters hash and its subhashes. Any keys from the nested
+  # hash that do not match will be removed.
+  module ParameterFilter
+
+    # Removes any keys from nested hashes that don't match predefiend keys
+    #
+    def filter!(keys, params, options={:recursive => true})  # :nodoc:
+      case params
+      when Hash
+        params.keys.each do |k, v|
+          unless (keys.include?(k) or Github::Validations::VALID_API_KEYS.include?(k))
+            params.delete(k)
+          else
+            filter!(keys, params[k]) if options[:recursive]
+          end
+        end
+      when Array
+        params.map! do |el|
+          filter!(keys, el) if options[:recursive]
+        end
+      else
+        params
+      end
+      return params
+    end
+
+  end # Filter
+end # Github

data/lib/github_api/pull_requests/comments.rb

@@ -24,7 +24,7 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of request_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_comment, params)
 
       response = get_request("/repos/#{user}/#{repo}/pulls/#{request_id}/comments", params)
@@ -43,7 +43,7 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of comment_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_comment, params)
 
       get_request("/repos/#{user}/#{repo}/pulls/comments/#{comment_id}", params)
@@ -83,9 +83,9 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of request_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_comment, params)
-      _filter_params_keys(VALID_REQUEST_COM_PARAM_NAMES, params)
+      filter! VALID_REQUEST_COM_PARAM_NAMES, params
       # _validate_reply_to(params)
 
       post_request("/repos/#{user}/#{repo}/pulls/#{request_id}/comments", params)
@@ -106,9 +106,9 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of comment_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_comment, params)
-      _filter_params_keys(VALID_REQUEST_COM_PARAM_NAMES, params)
+      filter! VALID_REQUEST_COM_PARAM_NAMES, params
 
       patch_request("/repos/#{user}/#{repo}/pulls/comments/#{comment_id}", params)
     end
@@ -124,7 +124,7 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of comment_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_comment, params)
 
       delete_request("/repos/#{user}/#{repo}/pulls/comments/#{comment_id}", params)

data/lib/github_api/pull_requests.rb

@@ -47,8 +47,8 @@ module Github
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless (user? && repo?)
 
-      _normalize_params_keys(params)
-      _filter_params_keys(VALID_REQUEST_PARAM_NAMES, params)
+      normalize! params
+      filter! VALID_REQUEST_PARAM_NAMES, params
       # _merge_mime_type(:pull_request, params)
       assert_valid_values(VALID_REQUEST_PARAM_VALUES, params)
 
@@ -72,7 +72,7 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of request_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_request, params)
 
       get_request("/repos/#{user}/#{repo}/pulls/#{request_id}", params)
@@ -112,9 +112,9 @@ module Github
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_request, params)
-      _filter_params_keys(VALID_REQUEST_PARAM_NAMES, params)
+      filter! VALID_REQUEST_PARAM_NAMES, params
 
       post_request("/repos/#{user}/#{repo}/pulls", params)
     end
@@ -138,8 +138,8 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of request_id
 
-      _normalize_params_keys(params)
-      _filter_params_keys(VALID_REQUEST_PARAM_NAMES, params)
+      normalize! params
+      filter! VALID_REQUEST_PARAM_NAMES, params
       # _merge_mime_type(:pull_request, params)
       assert_valid_values(VALID_REQUEST_PARAM_VALUES, params)
 
@@ -157,7 +157,7 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of request_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_request, params)
 
       response = get_request("/repos/#{user}/#{repo}/pulls/#{request_id}/commits", params)
@@ -176,7 +176,7 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of request_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_request, params)
 
       response = get_request("/repos/#{user}/#{repo}/pulls/#{request_id}/files", params)
@@ -195,7 +195,7 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of request_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_request, params)
 
       get_request("/repos/#{user}/#{repo}/pulls/#{request_id}/merge", params)
@@ -218,9 +218,9 @@ module Github
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of request_id
 
-      _normalize_params_keys(params)
+      normalize! params
       # _merge_mime_type(:pull_request, params)
-      _filter_params_keys(VALID_REQUEST_PARAM_NAMES, params)
+      filter! VALID_REQUEST_PARAM_NAMES, params
 
       put_request("/repos/#{user}/#{repo}/pulls/#{request_id}/merge", params)
     end

data/lib/github_api/repos/collaborators.rb

@@ -16,7 +16,7 @@ module Github
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of collaborator
-      _normalize_params_keys(params)
+      normalize! params
 
       put_request("/repos/#{user}/#{repo}/collaborators/#{collaborator}", params)
     end
@@ -32,7 +32,7 @@ module Github
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of collaborator
-      _normalize_params_keys(params)
+      normalize! params
 
       get_request("/repos/#{user}/#{repo}/collaborators/#{collaborator}", params)
       true
@@ -50,7 +50,7 @@ module Github
     def list(user_name, repo_name, params={})
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless (user? && repo?)
-      _normalize_params_keys(params)
+      normalize! params
 
       response = get_request("/repos/#{user}/#{repo}/collaborators", params)
       return response unless block_given?
@@ -68,7 +68,7 @@ module Github
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of collaborator
-      _normalize_params_keys(params)
+      normalize! params
 
       delete_request("/repos/#{user}/#{repo}/collaborators/#{collaborator}", params)
     end

data/lib/github_api/repos/commits.rb

@@ -23,7 +23,7 @@ module Github
       #
       def compare(user_name, repo_name, base, head, params={})
         _validate_presence_of base, head
-        _normalize_params_keys(params)
+        normalize! params
 
         get_request("/repos/#{user_name}/#{repo_name}/compare/#{base}...#{head}", params)
       end
@@ -51,8 +51,8 @@ module Github
         _validate_user_repo_params(user, repo) unless user? && repo?
         _validate_presence_of sha
 
-        _normalize_params_keys(params)
-        _filter_params_keys(REQUIRED_COMMENT_PARAMS, params)
+        normalize! params
+        filter! REQUIRED_COMMENT_PARAMS, params
 
         assert_required_keys(REQUIRED_COMMENT_PARAMS, params)
 
@@ -69,7 +69,7 @@ module Github
         _update_user_repo_params(user_name, repo_name)
         _validate_user_repo_params(user, repo) unless user? && repo?
         _validate_presence_of comment_id
-        _normalize_params_keys(params)
+        normalize! params
 
         delete_request("/repos/#{user}/#{repo}/comments/#{comment_id}", params)
       end
@@ -88,8 +88,8 @@ module Github
       def list(user_name, repo_name, params={})
         _update_user_repo_params(user_name, repo_name)
         _validate_user_repo_params(user, repo) unless user? && repo?
-        _normalize_params_keys(params)
-        _filter_params_keys(%w[ sha path], params)
+        normalize! params
+        filter! %w[ sha path], params
 
         response = get_request("/repos/#{user}/#{repo}/commits", params)
         return response unless block_given?
@@ -107,7 +107,7 @@ module Github
       def repo_comments(user_name, repo_name, params={})
         _update_user_repo_params(user_name, repo_name)
         _validate_user_repo_params(user, repo) unless user? && repo?
-        _normalize_params_keys(params)
+        normalize! params
 
         response = get_request("/repos/#{user}/#{repo}/comments", params)
         return response unless block_given?
@@ -126,7 +126,7 @@ module Github
         _update_user_repo_params(user_name, repo_name)
         _validate_user_repo_params(user, repo) unless user? && repo?
         _validate_presence_of sha
-        _normalize_params_keys(params)
+        normalize! params
 
         response = get_request("/repos/#{user}/#{repo}/commits/#{sha}/comments", params)
         return response unless block_given?
@@ -144,7 +144,7 @@ module Github
         _update_user_repo_params(user_name, repo_name)
         _validate_user_repo_params(user, repo) unless user? && repo?
         _validate_presence_of sha
-        _normalize_params_keys(params)
+        normalize! params
 
         get_request("/repos/#{user}/#{repo}/commits/#{sha}", params)
       end
@@ -160,7 +160,7 @@ module Github
         _update_user_repo_params(user_name, repo_name)
         _validate_user_repo_params(user, repo) unless user? && repo?
         _validate_presence_of comment_id
-        _normalize_params_keys(params)
+        normalize! params
 
         get_request("/repos/#{user}/#{repo}/comments/#{comment_id}", params)
       end
@@ -181,7 +181,7 @@ module Github
         _validate_user_repo_params(user, repo) unless user? && repo?
         _validate_presence_of comment_id
 
-        _normalize_params_keys(params)
+        normalize! params
         assert_required_keys(%w[ body ], params)
 
         patch_request("/repos/#{user}/#{repo}/comments/#{comment_id}", params)

data/lib/github_api/repos/contents.rb

@@ -13,7 +13,7 @@ module Github
     #  github.repos.contents.readme 'user-name', 'repo-name'
     #
     def readme(user_name, repo_name, params={})
-      _normalize_params_keys(params)
+      normalize! params
 
       get_request("/repos/#{user_name}/#{repo_name}/readme", params)
     end
@@ -27,7 +27,7 @@ module Github
     #  github.repos.contents.get 'user-name', 'repo-name', 'path'
     #
     def get(user_name, repo_name, path, params={})
-      _normalize_params_keys(params)
+      normalize! params
 
       get_request("/repos/#{user_name}/#{repo_name}/contents/#{path}", params)
     end
@@ -53,7 +53,7 @@ module Github
     #    "ref" => "master"
     #
     def archive(user_name, repo_name, params={})
-      _normalize_params_keys(params)
+      normalize! params
       archive_format = params.delete('archive_format') || 'zipball'
       ref = params.delete('ref') || 'master'
 

data/lib/github_api/repos/downloads.rb

@@ -24,7 +24,7 @@ module Github
     def list(user_name, repo_name, params={})
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
-      _normalize_params_keys(params)
+      normalize! params
 
       response = get_request("/repos/#{user}/#{repo}/downloads", params)
       return response unless block_given?
@@ -42,7 +42,7 @@ module Github
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of download_id
-      _normalize_params_keys(params)
+      normalize! params
 
       get_request("/repos/#{user}/#{repo}/downloads/#{download_id}", params)
     end
@@ -58,7 +58,7 @@ module Github
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
       _validate_presence_of download_id
-      _normalize_params_keys(params)
+      normalize! params
 
       delete_request("/repos/#{user}/#{repo}/downloads/#{download_id}", params)
     end
@@ -85,8 +85,8 @@ module Github
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
 
-      _normalize_params_keys(params)
-      _filter_params_keys(VALID_DOWNLOAD_PARAM_NAMES, params)
+      normalize! params
+      filter! VALID_DOWNLOAD_PARAM_NAMES, params
       assert_required_keys(REQUIRED_PARAMS, params)
 
       post_request("/repos/#{user}/#{repo}/downloads", params)

data/lib/github_api/repos/forks.rb

@@ -16,7 +16,7 @@ module Github
     def list(user_name, repo_name, params = {})
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
-      _normalize_params_keys(params)
+      normalize! params
 
       response = get_request("/repos/#{user}/#{repo}/forks", params)
       return response unless block_given?
@@ -37,8 +37,8 @@ module Github
     def create(user_name, repo_name, params={})
       _update_user_repo_params(user_name, repo_name)
       _validate_user_repo_params(user, repo) unless user? && repo?
-      _normalize_params_keys(params)
-      _filter_params_keys(%w[ org ], params)
+      normalize! params
+      filter! %w[ org ], params
 
       post_request("/repos/#{user}/#{repo}/forks", params)
     end