Refactor because of codeclimate feedback

Author: Agora@Ubuntu-dev

lib/oauth/tokens/request_token.rb

@@ -8,14 +8,14 @@ def authorize_url(params = nil)
       return nil if self.token.nil?
 
       params = (params || {}).merge(:oauth_token => self.token)
-      build_authorize_url(consumer.authorize_url, params)
+      build_url(consumer.authorize_url, params)
     end
 
     def authenticate_url(params = nil)
       return nil if self.token.nil?
 
       params = (params || {}).merge(:oauth_token => self.token)
-      build_authenticate_url(consumer.authenticate_url, params)
+      build_url(consumer.authenticate_url, params)
     end
 
     def callback_confirmed?
@@ -30,19 +30,8 @@ def get_access_token(options = {}, *arguments)
 
   protected
 
-    # construct an authorization url
-    def build_authorize_url(base_url, params)
-      uri = URI.parse(base_url.to_s)
-      queries = {}
-      queries = Hash[URI.decode_www_form(uri.query)] if uri.query
-      # TODO doesn't handle array values correctly
-      queries.merge!(params) if params
-      uri.query = URI.encode_www_form(queries) if !queries.empty?
-      uri.to_s
-    end
-
-    # construct an authenticate url
-    def build_authenticate_url(base_url, params)
+    # construct an authorization or authentication url
+    def build_url(base_url, params)
       uri = URI.parse(base_url.to_s)
       queries = {}
       queries = Hash[URI.decode_www_form(uri.query)] if uri.query