activemerchant 1.45.0 → 1.126.0

data/lib/active_merchant/billing/gateways/pay_gate_xml.rb

@@ -52,7 +52,7 @@ module ActiveMerchant #:nodoc:
     #
     # PayGateXML Field        ActiveMerchant Use
     #
-    # pgid                    use :login value to gateway instantation
+    # pgid                    use :login value to gateway instantiation
     # pwd                     use :password value to gateway instantiation
     #
     # cname                   credit_card.name
@@ -76,10 +76,10 @@ module ActiveMerchant #:nodoc:
       self.live_url = 'https://www.paygate.co.za/payxml/process.trans'
 
       # The countries the gateway supports merchants from as 2 digit ISO country codes
-      self.supported_countries = ['US', 'ZA']
+      self.supported_countries = %w[US ZA]
 
       # The card types supported by the payment gateway
-      self.supported_cardtypes = [:visa, :master, :american_express, :diners_club]
+      self.supported_cardtypes = %i[visa master american_express diners_club]
 
       # The homepage URL of the gateway
       self.homepage_url = 'http://paygate.co.za/'
@@ -102,48 +102,48 @@ module ActiveMerchant #:nodoc:
 
       DECLINE_CODES = {
         # Credit Card Errors - These RESULT_CODEs are returned if the transaction cannot be authorized due to a problem with the card.  The TRANSACTION_STATUS will be 2
-        900001  => "Call for Approval",
-        900002  => "Card Expired",
-        900003  => "Insufficient Funds",
-        900004  => "Invalid Card Number",
-        900005  => "Bank Interface Timeout",  # indicates a communications failure between the banks systems
-        900006  => "Invalid Card",
-        900007  => "Declined",
-        900009  => "Lost Card",
-        900010  => "Invalid Card Length",
-        900011  => "Suspected Fraud",
-        900012  => "Card Reported As Stolen",
-        900013  => "Restricted Card",
-        900014  => "Excessive Card Usage",
-        900015  => "Card Blacklisted",
-
-        900207  => "Declined; authentication failed",  # indicates the cardholder did not enter their MasterCard SecureCode / Verified by Visa password correctly
-
-        990020  => "Auth Declined",
-
-        991001  => "Invalid expiry date",
-        991002  => "Invalid amount",
+        900001  => 'Call for Approval',
+        900002  => 'Card Expired',
+        900003  => 'Insufficient Funds',
+        900004  => 'Invalid Card Number',
+        900005  => 'Bank Interface Timeout', # indicates a communications failure between the banks systems
+        900006  => 'Invalid Card',
+        900007  => 'Declined',
+        900009  => 'Lost Card',
+        900010  => 'Invalid Card Length',
+        900011  => 'Suspected Fraud',
+        900012  => 'Card Reported As Stolen',
+        900013  => 'Restricted Card',
+        900014  => 'Excessive Card Usage',
+        900015  => 'Card Blacklisted',
+
+        900207  => 'Declined; authentication failed', # indicates the cardholder did not enter their MasterCard SecureCode / Verified by Visa password correctly
+
+        990020  => 'Auth Declined',
+
+        991001  => 'Invalid expiry date',
+        991002  => 'Invalid amount',
 
         # Communication Errors - These RESULT_CODEs are returned if the transaction cannot be completed due to an unexpected error.  TRANSACTION_STATUS will be 0.
-        900205  => "Unexpected authentication result (phase 1)",
-        900206  => "Unexpected authentication result (phase 1)",
+        900205  => 'Unexpected authentication result (phase 1)',
+        900206  => 'Unexpected authentication result (phase 1)',
 
-        990001  => "Could not insert into Database",
+        990001  => 'Could not insert into Database',
 
-        990022  => "Bank not available",
+        990022  => 'Bank not available',
 
-        990053  => "Error processing transaction",
+        990053  => 'Error processing transaction',
 
         # Miscellaneous - Unless otherwise noted, the TRANSACTION_STATUS will be 0.
-        900209  => "Transaction verification failed (phase 2)",  # Indicates the verification data returned from MasterCard SecureCode / Verified by Visa has been altered
-        900210  => "Authentication complete; transaction must be restarted",  # Indicates that the MasterCard SecuerCode / Verified by Visa transaction has already been completed.  Most likely caused by the customer clicking the refresh button
+        900209  => 'Transaction verification failed (phase 2)', # Indicates the verification data returned from MasterCard SecureCode / Verified by Visa has been altered
+        900210  => 'Authentication complete; transaction must be restarted', # Indicates that the MasterCard SecuerCode / Verified by Visa transaction has already been completed.  Most likely caused by the customer clicking the refresh button
 
-        990024  => "Duplicate Transaction Detected.  Please check before submitting",
+        990024  => 'Duplicate Transaction Detected.  Please check before submitting',
 
-        990028  => "Transaction cancelled"  # Customer clicks the 'Cancel' button on the payment page
+        990028  => 'Transaction cancelled' # Customer clicks the 'Cancel' button on the payment page
       }
 
-      SUCCESS_CODES = %w( 990004 990005 990017 990012 990018 990031 )
+      SUCCESS_CODES = %w(990004 990005 990017 990012 990018 990031)
 
       TRANSACTION_CODES = {
         0 => 'Not Done',
@@ -162,22 +162,32 @@ module ActiveMerchant #:nodoc:
 
       def purchase(money, creditcard, options = {})
         MultiResponse.run do |r|
-          r.process{authorize(money, creditcard, options)}
-          r.process{capture(money, r.authorization, options)}
+          r.process { authorize(money, creditcard, options) }
+          r.process { capture(money, r.authorization, options) }
         end
       end
 
       def authorize(money, creditcard, options = {})
         action = 'authtx'
 
-        options.merge!(:money => money, :creditcard => creditcard)
+        options[:money] = money
+        options[:creditcard] = creditcard
         commit(action, build_request(action, options))
       end
 
       def capture(money, authorization, options = {})
         action = 'settletx'
 
-        options.merge!(:money => money, :authorization => authorization)
+        options[:money] = money
+        options[:authorization] = authorization
+        commit(action, build_request(action, options), authorization)
+      end
+
+      def refund(money, authorization, options = {})
+        action = 'refundtx'
+
+        options[:money] = money
+        options[:authorization] = authorization
         commit(action, build_request(action, options))
       end
 
@@ -187,44 +197,54 @@ module ActiveMerchant #:nodoc:
         SUCCESS_CODES.include?(response[:res])
       end
 
-      def build_request(action, options={})
+      def build_request(action, options = {})
         xml = Builder::XmlMarkup.new
         xml.instruct!
 
-        xml.tag! 'protocol', :ver => API_VERSION, :pgid => (test? ? TEST_ID : @options[:login]), :pwd => @options[:password] do |protocol|
+        xml.tag! 'protocol', ver: API_VERSION, pgid: (test? ? TEST_ID : @options[:login]), pwd: @options[:password] do |protocol|
+          money         = options.delete(:money)
+          authorization = options.delete(:authorization)
+          creditcard    = options.delete(:creditcard)
           case action
-            when 'authtx'
-              money       = options.delete(:money)
-              creditcard  = options.delete(:creditcard)
-              build_authorization(protocol, money, creditcard, options)
-            when 'settletx'
-              money           = options.delete(:money)
-              authorization   = options.delete(:authorization)
-              build_capture(protocol, money, authorization, options)
+          when 'authtx'
+            build_authorization(protocol, money, creditcard, options)
+          when 'settletx'
+            build_capture(protocol, money, authorization, options)
+          when 'refundtx'
+            build_refund(protocol, money, authorization, options)
           else
-            raise "no action specified for build_request"
+            raise 'no action specified for build_request'
           end
         end
 
         xml.target!
       end
 
-      def build_authorization(xml, money, creditcard, options={})
+      def build_authorization(xml, money, creditcard, options = {})
         xml.tag! 'authtx', {
-          :cref  => options[:order_id],
-          :cname => creditcard.name,
-          :cc    => creditcard.number,
-          :exp   => "#{format(creditcard.month, :two_digits)}#{format(creditcard.year, :four_digits)}",
-          :budp  => 0,
-          :amt   => amount(money),
-          :cur   => (options[:currency] || currency(money)),
-          :cvv   => creditcard.verification_value
+          cref: options[:order_id],
+          cname: creditcard.name,
+          cc: creditcard.number,
+          exp: "#{format(creditcard.month, :two_digits)}#{format(creditcard.year, :four_digits)}",
+          budp: 0,
+          amt: amount(money),
+          cur: (options[:currency] || currency(money)),
+          cvv: creditcard.verification_value,
+          email: options[:email],
+          ip: options[:ip]
         }
       end
 
-      def build_capture(xml, money, authorization, options={})
+      def build_capture(xml, money, authorization, options = {})
         xml.tag! 'settletx', {
-          :tid => authorization
+          tid: authorization
+        }
+      end
+
+      def build_refund(xml, money, authorization, options = {})
+        xml.tag! 'refundtx', {
+          tid: authorization,
+          amt: amount(money)
         }
       end
 
@@ -235,21 +255,18 @@ module ActiveMerchant #:nodoc:
         response_action = action.gsub(/tx/, 'rx')
         root  = REXML::XPath.first(xml.root, response_action)
         # we might have gotten an error
-        if root.nil?
-          root  = REXML::XPath.first(xml.root, 'errorrx')
-        end
+        root  = REXML::XPath.first(xml.root, 'errorrx') if root.nil?
         root.attributes.each do |name, value|
           hash[name.to_sym] = value
         end
         hash
       end
 
-      def commit(action, request)
+      def commit(action, request, authorization = nil)
         response = parse(action, ssl_post(self.live_url, request))
         Response.new(successful?(response), message_from(response), response,
-          :test           => test?,
-          :authorization  => response[:tid]
-        )
+          test: test?,
+          authorization: authorization || response[:tid])
       end
 
       def message_from(response)
@@ -258,4 +275,3 @@ module ActiveMerchant #:nodoc:
     end
   end
 end
-

data/lib/active_merchant/billing/gateways/pay_hub.rb

@@ -0,0 +1,213 @@
+module ActiveMerchant #:nodoc:
+  module Billing #:nodoc:
+    class PayHubGateway < Gateway
+      self.live_url = 'https://checkout.payhub.com/transaction/api'
+
+      self.supported_countries = ['US']
+      self.default_currency = 'USD'
+      self.supported_cardtypes = %i[visa master american_express discover]
+
+      self.homepage_url = 'http://www.payhub.com/'
+      self.display_name = 'PayHub'
+
+      CVV_CODE_TRANSLATOR = {
+        'M' => 'CVV matches',
+        'N' => 'CVV does not match',
+        'P' => 'CVV not processed',
+        'S' => 'CVV should have been present',
+        'U' => 'CVV request unable to be processed by issuer'
+      }
+
+      AVS_CODE_TRANSLATOR = {
+        '0' =>  'Approved, Address verification was not requested.',
+        'A' =>  'Approved, Address matches only.',
+        'B' =>  'Address Match. Street Address math for international transaction Postal Code not verified because of incompatible formats (Acquirer sent both street address and Postal Code)',
+        'C' =>  'Serv Unavailable. Street address and Postal Code not verified for international transaction because of incompatible formats (Acquirer sent both street and Postal Code).',
+        'D' =>  'Exact Match, Street Address and Postal Code match for international transaction.',
+        'F' =>  'Exact Match, Street Address and Postal Code match. Applies to UK only.',
+        'G' =>  'Ver Unavailable, Non-U.S. Issuer does not participate.',
+        'I' =>  'Ver Unavailable, Address information not verified for international transaction',
+        'M' =>  'Exact Match, Street Address and Postal Code match for international transaction',
+        'N' =>  'No - Address and ZIP Code does not match',
+        'P' =>  'Zip Match, Postal Codes match for international transaction Street address not verified because of incompatible formats (Acquirer sent both street address and Postal Code).',
+        'R' =>  'Retry - Issuer system unavailable',
+        'S' =>  'Serv Unavailable, Service not supported',
+        'U' =>  'Ver Unavailable, Address unavailable.',
+        'W' =>  'ZIP match - Nine character numeric ZIP match only.',
+        'X' =>  'Exact match, Address and nine-character ZIP match.',
+        'Y' =>  'Exact Match, Address and five character ZIP match.',
+        'Z' =>  'Zip Match, Five character numeric ZIP match only.',
+        '1' =>  'Cardholder name and ZIP match AMEX only.',
+        '2' =>  'Cardholder name, address, and ZIP match AMEX only.',
+        '3' =>  'Cardholder name and address match AMEX only.',
+        '4' =>  'Cardholder name match AMEX only.',
+        '5' =>  'Cardholder name incorrect, ZIP match AMEX only.',
+        '6' =>  'Cardholder name incorrect, address and ZIP match AMEX only.',
+        '7' =>  'Cardholder name incorrect, address match AMEX only.',
+        '8' =>  'Cardholder, all do not match AMEX only.'
+      }
+
+      STANDARD_ERROR_CODE_MAPPING = {
+        '14' => STANDARD_ERROR_CODE[:invalid_number],
+        '80' => STANDARD_ERROR_CODE[:invalid_expiry_date],
+        '82' => STANDARD_ERROR_CODE[:invalid_cvc],
+        '54' => STANDARD_ERROR_CODE[:expired_card],
+        '51' => STANDARD_ERROR_CODE[:card_declined],
+        '05' => STANDARD_ERROR_CODE[:card_declined],
+        '61' => STANDARD_ERROR_CODE[:card_declined],
+        '62' => STANDARD_ERROR_CODE[:card_declined],
+        '65' => STANDARD_ERROR_CODE[:card_declined],
+        '93' => STANDARD_ERROR_CODE[:card_declined],
+        '01' => STANDARD_ERROR_CODE[:call_issuer],
+        '02' => STANDARD_ERROR_CODE[:call_issuer],
+        '04' => STANDARD_ERROR_CODE[:pickup_card],
+        '07' => STANDARD_ERROR_CODE[:pickup_card],
+        '41' => STANDARD_ERROR_CODE[:pickup_card],
+        '43' => STANDARD_ERROR_CODE[:pickup_card]
+      }
+
+      def initialize(options = {})
+        requires!(options, :orgid, :username, :password, :tid)
+
+        super
+      end
+
+      def authorize(amount, creditcard, options = {})
+        post = setup_post('auth')
+        add_creditcard(post, creditcard)
+        add_amount(post, amount)
+        add_address(post, (options[:address] || options[:billing_address]))
+        add_customer_data(post, options)
+
+        commit(post)
+      end
+
+      def purchase(amount, creditcard, options = {})
+        post = setup_post('sale')
+        add_creditcard(post, creditcard)
+        add_amount(post, amount)
+        add_address(post, (options[:address] || options[:billing_address]))
+        add_customer_data(post, options)
+
+        commit(post)
+      end
+
+      def refund(amount, trans_id, options = {})
+        # Attempt a void in case the transaction is unsettled
+        post = setup_post('void')
+        add_reference(post, trans_id)
+        response = commit(post)
+        return response if response.success?
+
+        post = setup_post('refund')
+        add_reference(post, trans_id)
+        commit(post)
+      end
+
+      def capture(amount, trans_id, options = {})
+        post = setup_post('capture')
+
+        add_reference(post, trans_id)
+        add_amount(post, amount)
+
+        commit(post)
+      end
+
+      # No void, as PayHub's void does not work on authorizations
+
+      def verify(creditcard, options = {})
+        authorize(100, creditcard, options)
+      end
+
+      private
+
+      def setup_post(action)
+        post = {}
+        post[:orgid] = @options[:orgid]
+        post[:tid] = @options[:tid]
+        post[:username] = @options[:username]
+        post[:password] = @options[:password]
+        post[:mode] = (test? ? 'demo' : 'live')
+        post[:trans_type] = action
+        post
+      end
+
+      def add_reference(post, trans_id)
+        post[:trans_id] = trans_id
+      end
+
+      def add_customer_data(post, options = {})
+        post[:first_name] = options[:first_name]
+        post[:last_name] = options[:last_name]
+        post[:phone] = options[:phone]
+        post[:email] = options[:email]
+      end
+
+      def add_address(post, address)
+        return unless address
+
+        post[:address1] = address[:address1]
+        post[:address2] = address[:address2]
+        post[:zip] = address[:zip]
+        post[:state] = address[:state]
+        post[:city] = address[:city]
+      end
+
+      def add_amount(post, amount)
+        post[:amount] = amount(amount)
+      end
+
+      def add_creditcard(post, creditcard)
+        post[:cc] = creditcard.number
+        post[:month] = creditcard.month.to_s
+        post[:year] = creditcard.year.to_s
+        post[:cvv] = creditcard.verification_value
+      end
+
+      def parse(body)
+        JSON.parse(body)
+      end
+
+      def commit(post)
+        success = false
+
+        begin
+          raw_response = ssl_post(live_url, post.to_json, { 'Content-Type' => 'application/json' })
+          response = parse(raw_response)
+          success = (response['RESPONSE_CODE'] == '00')
+        rescue ResponseError => e
+          raw_response = e.response.body
+          response = response_error(raw_response)
+        rescue JSON::ParserError
+          response = json_error(raw_response)
+        end
+
+        Response.new(success,
+          response_message(response),
+          response,
+          test: test?,
+          avs_result: { code: response['AVS_RESULT_CODE'] },
+          cvv_result: response['VERIFICATION_RESULT_CODE'],
+          error_code: (success ? nil : STANDARD_ERROR_CODE_MAPPING[response['RESPONSE_CODE']]),
+          authorization: response['TRANSACTION_ID'])
+      end
+
+      def response_error(raw_response)
+        parse(raw_response)
+      rescue JSON::ParserError
+        json_error(raw_response)
+      end
+
+      def json_error(raw_response)
+        {
+          error_message: 'Invalid response received from the Payhub API.  Please contact wecare@payhub.com if you continue to receive this message.' \
+            "  (The raw response returned by the API was #{raw_response.inspect})"
+        }
+      end
+
+      def response_message(response)
+        (response['RESPONSE_TEXT'] || response['RESPONSE_CODE'] || response[:error_message])
+      end
+    end
+  end
+end

data/lib/active_merchant/billing/gateways/pay_junction.rb

@@ -97,23 +97,23 @@ module ActiveMerchant #:nodoc:
     # See example use above for address AVS fields
     # See #recurring for periodic transaction fields
     class PayJunctionGateway < Gateway
-      API_VERSION   = '1.2'
+      API_VERSION = '1.2'
 
       class_attribute :test_url, :live_url
 
-      self.test_url = "https://www.payjunctionlabs.com/quick_link"
-      self.live_url = "https://payjunction.com/quick_link"
+      self.test_url = 'https://www.payjunctionlabs.com/quick_link'
+      self.live_url = 'https://payjunction.com/quick_link'
 
       TEST_LOGIN = 'pj-ql-01'
       TEST_PASSWORD = 'pj-ql-01p'
 
-      SUCCESS_CODES = ["00", "85"]
+      SUCCESS_CODES = %w[00 85]
       SUCCESS_MESSAGE = 'The transaction was approved.'
 
       FAILURE_MESSAGE = 'The transaction was declined.'
 
       DECLINE_CODES = {
-        "AE"  => 'Address verification failed because address did not match.',
+        'AE'  => 'Address verification failed because address did not match.',
         'ZE'  => 'Address verification failed because zip did not match.',
         'XE'  => 'Address verification failed because zip and address did not match.',
         'YE'  => 'Address verification failed because zip and address did not match.',
@@ -144,13 +144,13 @@ module ActiveMerchant #:nodoc:
         '96'  => 'Declined because of a system error.',
         'N7'  => 'Declined because of a CVV2/CVC2 mismatch.',
         'M4'  => 'Declined.',
-        "FE"  => "There was a format error with your Trinity Gateway Service (API) request.",
-        "LE"  => "Could not log you in (problem with dc_logon and/or dc_password).",
+        'FE'  => 'There was a format error with your Trinity Gateway Service (API) request.',
+        'LE'  => 'Could not log you in (problem with dc_logon and/or dc_password).',
         'NL'  => 'Aborted because of a system error, please try again later. ',
         'AB'  => 'Aborted because of an upstream system error, please try again later.'
       }
 
-      self.supported_cardtypes = [:visa, :master, :american_express, :discover]
+      self.supported_cardtypes = %i[visa master american_express discover]
       self.supported_countries = ['US']
       self.homepage_url = 'http://www.payjunction.com/'
       self.display_name = 'PayJunction'
@@ -165,7 +165,7 @@ module ActiveMerchant #:nodoc:
       # transaction_id that can be used later to postauthorize (capture) the funds.
       def authorize(money, payment_source, options = {})
         parameters = {
-          :transaction_amount => amount(money),
+          transaction_amount: amount(money)
         }
 
         add_payment_source(parameters, payment_source)
@@ -178,7 +178,7 @@ module ActiveMerchant #:nodoc:
       # Execute authorization and capture in a single step.
       def purchase(money, payment_source, options = {})
         parameters = {
-          :transaction_amount => amount(money),
+          transaction_amount: amount(money)
         }
 
         add_payment_source(parameters, payment_source)
@@ -191,8 +191,8 @@ module ActiveMerchant #:nodoc:
       # Retrieve funds that have been previously authorized with _authorization_
       def capture(money, authorization, options = {})
         parameters = {
-          :transaction_id => authorization,
-          :posture => 'capture'
+          transaction_id: authorization,
+          posture: 'capture'
         }
 
         add_optional_fields(parameters, options)
@@ -203,8 +203,8 @@ module ActiveMerchant #:nodoc:
       # _authorization_ should be the transaction id of the transaction we are returning.
       def refund(money, authorization, options = {})
         parameters = {
-          :transaction_amount => amount(money),
-          :transaction_id => authorization
+          transaction_amount: amount(money),
+          transaction_id: authorization
         }
 
         commit('CREDIT', parameters)
@@ -219,8 +219,8 @@ module ActiveMerchant #:nodoc:
       # through the batch process.
       def void(authorization, options = {})
         parameters = {
-          :transaction_id => authorization,
-          :posture => 'void'
+          transaction_id: authorization,
+          posture: 'void'
         }
 
         add_optional_fields(parameters, options)
@@ -241,16 +241,17 @@ module ActiveMerchant #:nodoc:
       def recurring(money, payment_source, options = {})
         ActiveMerchant.deprecated RECURRING_DEPRECATION_MESSAGE
 
-        requires!(options, [:periodicity, :monthly, :weekly, :daily], :payments)
+        requires!(options, %i[periodicity monthly weekly daily], :payments)
 
-        periodic_type = case options[:periodicity]
-        when :monthly
-          'month'
-        when :weekly
-          'week'
-        when :daily
-          'day'
-        end
+        periodic_type =
+          case options[:periodicity]
+          when :monthly
+            'month'
+          when :weekly
+            'week'
+          when :daily
+            'day'
+          end
 
         if options[:starting_at].nil?
           start_date = Time.now.strftime('%Y-%m-%d')
@@ -262,12 +263,12 @@ module ActiveMerchant #:nodoc:
         end
 
         parameters = {
-          :transaction_amount => amount(money),
-          :schedule_periodic_type => periodic_type,
-          :schedule_create => 'true',
-          :schedule_limit => options[:payments].to_i > 1 ? options[:payments] : 1,
-          :schedule_periodic_number => 1,
-          :schedule_start => start_date
+          transaction_amount: amount(money),
+          schedule_periodic_type: periodic_type,
+          schedule_create: 'true',
+          schedule_limit: options[:payments].to_i > 1 ? options[:payments] : 1,
+          schedule_periodic_number: 1,
+          schedule_start: start_date
         }
 
         add_payment_source(parameters, payment_source)
@@ -318,7 +319,7 @@ module ActiveMerchant #:nodoc:
         address = options[:billing_address] || options[:address]
 
         if address
-          params[:address]  = address[:address1] unless address[:address1].blank?
+          params[:address]   = address[:address1] unless address[:address1].blank?
           params[:city]      = address[:city]     unless address[:city].blank?
           params[:state]     = address[:state]    unless address[:state].blank?
           params[:zipcode]   = address[:zip]      unless address[:zip].blank?
@@ -334,12 +335,11 @@ module ActiveMerchant #:nodoc:
       def commit(action, parameters)
         url = test? ? self.test_url : self.live_url
 
-        response = parse( ssl_post(url, post_data(action, parameters)) )
+        response = parse(ssl_post(url, post_data(action, parameters)))
 
         Response.new(successful?(response), message_from(response), response,
-          :test => test?,
-          :authorization => response[:transaction_id] || parameters[:transaction_id]
-        )
+          test: test?,
+          authorization: response[:transaction_id] || parameters[:transaction_id])
       end
 
       def successful?(response)
@@ -366,7 +366,7 @@ module ActiveMerchant #:nodoc:
         params[:version] = API_VERSION
         params[:transaction_type] = action
 
-        params.reject{|k,v| v.blank?}.collect{ |k, v| "dc_#{k.to_s}=#{CGI.escape(v.to_s)}" }.join("&")
+        params.reject { |_k, v| v.blank? }.collect { |k, v| "dc_#{k}=#{CGI.escape(v.to_s)}" }.join('&')
       end
 
       def parse(body)