activemerchant 1.28.0 → 1.29.0

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

@@ -51,7 +51,7 @@ module ActiveMerchant #:nodoc:
       #
       # ==== Parameters
       # * <tt>options</tt> - A Hash of options
-      # 
+      #
       # ==== Options Hash
       # * <tt>:login</tt> - A String containing your PaymentClearing assigned API Access Username
       # * <tt>:password</tt> - A String containing your PaymentClearing assigned API Access Key
@@ -60,7 +60,6 @@ module ActiveMerchant #:nodoc:
       #
       def initialize(options = {})
         requires!(options, :login, :password, :gateway_id)
-        @options = options
         super
       end
 
@@ -196,7 +195,6 @@ module ActiveMerchant #:nodoc:
       # This will reverse a previously run transaction which *has* *not* settled.
       #
       # ==== Parameters
-      # * <tt>money</tt> - This parameter is ignored -- the PaymentClearing gateway does not allow partial voids.
       # * <tt>authorization</tt> - The authorization returned from the previous capture or purchase request
       # * <tt>options</tt> - A Hash of options, all are optional
       #
@@ -209,7 +207,7 @@ module ActiveMerchant #:nodoc:
       # * <tt>:test_mode</tt> - <tt>true</tt> or <tt>false</tt>. Runs the transaction with the 'TestMode' element set to 'TRUE' or 'FALSE'.
       #
       # ==== Examples
-      #  response = gateway.void(nil, '9999999999',
+      #  response = gateway.void('9999999999',
       #    :vendor_data => [{'repId' => '1234567'}, {'customerId' => '9886'}],
       #    :send_customer_email => true,
       #    :send_merchant_email => true,
@@ -217,7 +215,7 @@ module ActiveMerchant #:nodoc:
       #    :test_mode => true
       #  )
       #
-      def void(money, authorization, options = {})
+      def void(authorization, options = {})
         payload = Nokogiri::XML::Builder.new do |xml|
           xml.VoidTransaction {
             xml.OperationXID(authorization)
@@ -232,7 +230,7 @@ module ActiveMerchant #:nodoc:
       # This will reverse a previously run transaction which *has* settled.
       #
       # ==== Parameters
-      # * <tt>money</tt> - The amount to be credited. Should be <tt>nil</tt> or an Integer amount in cents
+      # * <tt>money</tt> - The amount to be credited. Should be an Integer amount in cents
       # * <tt>authorization</tt> - The authorization returned from the previous capture or purchase request
       # * <tt>options</tt> - A Hash of options, all are optional
       #
@@ -245,7 +243,7 @@ module ActiveMerchant #:nodoc:
       # * <tt>:test_mode</tt> - <tt>true</tt> or <tt>false</tt>. Runs the transaction with the 'TestMode' element set to 'TRUE' or 'FALSE'.
       #
       # ==== Examples
-      #  response = gateway.credit(nil, '9999999999',
+      #  response = gateway.refund(555, '9999999999',
       #    :vendor_data => [{'repId' => '1234567'}, {'customerId' => '9886'}],
       #    :send_customer_email => true,
       #    :send_merchant_email => true,
@@ -253,9 +251,9 @@ module ActiveMerchant #:nodoc:
       #    :test_mode => true
       #  )
       #
-      def credit(money, authorization, options = {})
+      def refund(money, authorization, options = {})
         payload = Nokogiri::XML::Builder.new do |xml|
-          xml.TranCreditTransaction {
+          xml.TranCredTransaction {
             xml.OperationXID(authorization)
             add_invoice(xml, money, options)
             add_transaction_control(xml, options)

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

@@ -3,22 +3,22 @@ module ActiveMerchant #:nodoc:
     class JetpayGateway < Gateway
       self.test_url = 'https://test1.jetpay.com/jetpay'
       self.live_url = 'https://gateway17.jetpay.com/jetpay'
-      
+
       # The countries the gateway supports merchants from as 2 digit ISO country codes
       self.supported_countries = ['US']
-      
+
       # The card types supported by the payment gateway
       self.supported_cardtypes = [:visa, :master, :american_express, :discover]
-      
+
       # The homepage URL of the gateway
       self.homepage_url = 'http://www.jetpay.com/'
-      
+
       # The name of the gateway
       self.display_name = 'JetPay'
-      
+
       # all transactions are in cents
       self.money_format = :cents
-      
+
       ACTION_CODE_MESSAGES = {
         "001" =>  "Refer to card issuer.",
         "002" =>  "Refer to card issuer, special condition.",
@@ -61,25 +61,24 @@ module ActiveMerchant #:nodoc:
         "996" =>  "Terminal ID Not Found.",
         nil   =>  "No response returned (missing credentials?)."
       }
-      
+
       def initialize(options = {})
         requires!(options, :login)
-        @options = options
         super
-      end  
-      
+      end
+
       def purchase(money, credit_card, options = {})
         commit(money, build_sale_request(money, credit_card, options))
       end
-      
+
       def authorize(money, credit_card, options = {})
         commit(money, build_authonly_request(money, credit_card, options))
       end
-      
+
       def capture(money, reference, options = {})
         commit(money, build_capture_request('CAPT', reference.split(";").first))
       end
-      
+
       def void(reference, options = {})
         transaction_id, approval, amount = reference.split(";")
         commit(amount.to_i, build_void_request(amount.to_i, transaction_id, approval))
@@ -100,9 +99,9 @@ module ActiveMerchant #:nodoc:
         commit(money, build_credit_request('CREDIT', money, transaction_id, credit_card))
       end
 
-      
+
       private
-      
+
       def build_xml_request(transaction_type, transaction_id = nil, &block)
         xml = Builder::XmlMarkup.new
         xml.tag! 'JetPay' do
@@ -110,15 +109,15 @@ module ActiveMerchant #:nodoc:
           xml.tag! 'TerminalID', @options[:login]
           xml.tag! 'TransactionType', transaction_type
           xml.tag! 'TransactionID', transaction_id.nil? ? generate_unique_id.slice(0, 18) : transaction_id
-          
+
           if block_given?
             yield xml
-          else 
+          else
             xml.target!
           end
         end
       end
-      
+
       def build_sale_request(money, credit_card, options)
         build_xml_request('SALE') do |xml|
           add_credit_card(xml, credit_card)
@@ -126,11 +125,11 @@ module ActiveMerchant #:nodoc:
           add_customer_data(xml, options)
           add_invoice_data(xml, options)
           xml.tag! 'TotalAmount', amount(money)
-          
+
           xml.target!
         end
       end
-      
+
       def build_authonly_request(money, credit_card, options)
         build_xml_request('AUTHONLY') do |xml|
           add_credit_card(xml, credit_card)
@@ -138,22 +137,22 @@ module ActiveMerchant #:nodoc:
           add_customer_data(xml, options)
           add_invoice_data(xml, options)
           xml.tag! 'TotalAmount', amount(money)
-          
+
           xml.target!
         end
       end
-      
+
       def build_capture_request(transaction_type, transaction_id)
         build_xml_request(transaction_type, transaction_id)
       end
-      
+
       def build_void_request(money, transaction_id, approval)
         build_xml_request('VOID', transaction_id) do |xml|
           xml.tag! 'Approval', approval
           xml.tag! 'TotalAmount', amount(money)
-          
+
           xml.target!
-        end        
+        end
       end
 
       # `transaction_id` may be nil for unlinked credit transactions.
@@ -161,37 +160,37 @@ module ActiveMerchant #:nodoc:
         build_xml_request(transaction_type, transaction_id) do |xml|
           add_credit_card(xml, card) if card
           xml.tag! 'TotalAmount', amount(money)
-          
+
           xml.target!
         end
       end
-      
+
       def commit(money, request)
         response = parse(ssl_post(test? ? self.test_url : self.live_url, request))
-        
+
         success = success?(response)
-        Response.new(success, 
-          success ? 'APPROVED' : message_from(response), 
-          response, 
-          :test => test?, 
+        Response.new(success,
+          success ? 'APPROVED' : message_from(response),
+          response,
+          :test => test?,
           :authorization => authorization_from(response, money),
           :avs_result => { :code => response[:avs] },
           :cvv_result => response[:cvv2]
         )
       end
-      
+
       def parse(body)
         return {} if body.blank?
 
         xml = REXML::Document.new(body)
-        
+
         response = {}
         xml.root.elements.to_a.each do |node|
           parse_element(response, node)
         end
         response
       end
-      
+
       def parse_element(response, node)
         if node.has_elements?
           node.elements.each{|element| parse_element(response, element) }
@@ -199,24 +198,24 @@ module ActiveMerchant #:nodoc:
           response[node.name.underscore.to_sym] = node.text
         end
       end
-      
+
       def format_exp(value)
         format(value, :two_digits)
       end
-      
+
       def success?(response)
         response[:action_code] == "000"
       end
-      
+
       def message_from(response)
         ACTION_CODE_MESSAGES[response[:action_code]]
       end
-      
+
       def authorization_from(response, money)
         original_amount = amount(money) if money
         [ response[:transaction_id], response[:approval], original_amount ].join(";")
       end
-      
+
       def add_credit_card(xml, credit_card)
         xml.tag! 'CardNum', credit_card.number
         xml.tag! 'CardExpMonth', format_exp(credit_card.month)
@@ -230,7 +229,7 @@ module ActiveMerchant #:nodoc:
           xml.tag! 'CVV2', credit_card.verification_value
         end
       end
-      
+
       def add_addresses(xml, options)
         if billing_address = options[:billing_address] || options[:address]
           xml.tag! 'BillingAddress', [billing_address[:address1], billing_address[:address2]].compact.join(" ")
@@ -240,11 +239,11 @@ module ActiveMerchant #:nodoc:
           xml.tag! 'BillingCountry', lookup_country_code(billing_address[:country])
           xml.tag! 'BillingPhone', billing_address[:phone]
         end
-        
+
         if shipping_address = options[:shipping_address]
           xml.tag! 'ShippingInfo' do
             xml.tag! 'ShippingName', shipping_address[:name]
-            
+
             xml.tag! 'ShippingAddr' do
               xml.tag! 'Address', [shipping_address[:address1], shipping_address[:address2]].compact.join(" ")
               xml.tag! 'City', shipping_address[:city]
@@ -260,14 +259,14 @@ module ActiveMerchant #:nodoc:
         xml.tag! 'Email', options[:email] if options[:email]
         xml.tag! 'UserIPAddress', options[:ip] if options[:ip]
       end
-      
+
       def add_invoice_data(xml, options)
         xml.tag! 'OrderNumber', options[:order_id] if options[:order_id]
         xml.tag! 'TaxAmount', amount(options[:tax]) if options[:tax]
       end
-      
+
       def lookup_country_code(code)
-        country = Country.find(code) rescue nil        
+        country = Country.find(code) rescue nil
         country && country.code(:alpha3)
       end
     end

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

@@ -2,36 +2,36 @@ require 'rexml/document'
 
 module ActiveMerchant #:nodoc:
   module Billing #:nodoc:
-    
+
     # Initialization Options
     # :login                Your store number
     # :pem                  The text of your linkpoint PEM file. Note
     #                       this is not the path to file, but its
     #                       contents. If you are only using one PEM
-    #                       file on your site you can declare it 
+    #                       file on your site you can declare it
     #                       globally and then you won't need to
     #                       include this option
     #
     #
-    # A valid store number is required. Unfortunately, with LinkPoint 
-    # YOU CAN'T JUST USE ANY OLD STORE NUMBER. Also, you can't just 
-    # generate your own PEM file. You'll need to use a special PEM file 
-    # provided by LinkPoint. 
+    # A valid store number is required. Unfortunately, with LinkPoint
+    # YOU CAN'T JUST USE ANY OLD STORE NUMBER. Also, you can't just
+    # generate your own PEM file. You'll need to use a special PEM file
+    # provided by LinkPoint.
     #
-    # Go to http://www.linkpoint.com/support/sup_teststore.asp to set up 
+    # Go to http://www.linkpoint.com/support/sup_teststore.asp to set up
     # a test account and obtain your PEM file.
     #
     # Declaring PEM file Globally
     # ActiveMerchant::Billing::LinkpointGateway.pem_file = File.read( File.dirname(__FILE__) + '/../mycert.pem' )
-    # 
-    # 
+    #
+    #
     # Valid Order Options
-    # :result => 
+    # :result =>
     #   LIVE                  Production mode
     #   GOOD                  Approved response in test mode
     #   DECLINE               Declined response in test mode
     #   DUPLICATE             Duplicate response in test mode
-    #                     
+    #
     # :ponumber               Order number
     #
     # :transactionorigin =>   Source of the transaction
@@ -41,35 +41,35 @@ module ActiveMerchant #:nodoc:
     #    TELEPHONE            Telephone
     #    RETAIL               Face-to-face
     #
-    # :ordertype =>       
+    # :ordertype =>
     #    SALE                 Real live sale
     #    PREAUTH              Authorize only
     #    POSTAUTH             Forced Ticket or Ticket Only transaction
-    #    VOID             
-    #    CREDIT           
+    #    VOID
+    #    CREDIT
     #    CALCSHIPPING         For shipping charges calculations
     #    CALCTAX              For sales tax calculations
-    #                     
-    # Recurring Options   
-    # :action =>          
-    #    SUBMIT           
-    #    MODIFY           
-    #    CANCEL           
-    #                     
+    #
+    # Recurring Options
+    # :action =>
+    #    SUBMIT
+    #    MODIFY
+    #    CANCEL
+    #
     # :installments           Identifies how many recurring payments to charge the customer
     # :startdate              Date to begin charging the recurring payments. Format: YYYYMMDD or "immediate"
-    # :periodicity  =>    
-    #     MONTHLY         
-    #     BIMONTHLY       
-    #     WEEKLY          
-    #     BIWEEKLY        
-    #     YEARLY          
-    #     DAILY           
+    # :periodicity  =>
+    #     MONTHLY
+    #     BIMONTHLY
+    #     WEEKLY
+    #     BIWEEKLY
+    #     YEARLY
+    #     DAILY
     # :threshold              Tells how many times to retry the transaction (if it fails) before contacting the merchant.
     # :comments               Uh... comments
     #
     #
-    # For reference: 
+    # For reference:
     #
     # https://www.linkpointcentral.com/lpc/docs/Help/APIHelp/lpintguide.htm
     #
@@ -85,7 +85,7 @@ module ActiveMerchant #:nodoc:
     #    :items => [:item => [:price, :quantity, :description, :id, :options => [:option => [:name, :value]]]]
     #  }
     #
-    # 
+    #
     # LinkPoint's Items entity is an optional entity that can be attached to orders.
     # It is entered as :line_items to be consistent with the CyberSource implementation
     #
@@ -117,57 +117,57 @@ module ActiveMerchant #:nodoc:
     #         ]
     # This functionality is only supported by this particular gateway may
     # be changed at any time
-    # 
-    class LinkpointGateway < Gateway     
+    #
+    class LinkpointGateway < Gateway
       # Your global PEM file. This will be assigned to you by linkpoint
-      # 
-      # Example: 
-      # 
+      #
+      # Example:
+      #
       # ActiveMerchant::Billing::LinkpointGateway.pem_file = File.read( File.dirname(__FILE__) + '/../mycert.pem' )
-      # 
+      #
       cattr_accessor :pem_file
-      
+
       self.test_url  = 'https://staging.linkpt.net:1129/'
       self.live_url  = 'https://secure.linkpt.net:1129/'
-      
+
       self.supported_countries = ['US']
       self.supported_cardtypes = [:visa, :master, :american_express, :discover, :jcb, :diners_club]
       self.homepage_url = 'http://www.linkpoint.com/'
       self.display_name = 'LinkPoint'
-           
+
       def initialize(options = {})
         requires!(options, :login)
-        
+
         @options = {
           :result => 'LIVE',
           :pem => LinkpointGateway.pem_file
         }.update(options)
-        
+
         raise ArgumentError, "You need to pass in your pem file using the :pem parameter or set it globally using ActiveMerchant::Billing::LinkpointGateway.pem_file = File.read( File.dirname(__FILE__) + '/../mycert.pem' ) or similar" if @options[:pem].blank?
       end
-      
+
       # Send a purchase request with periodic options
-      # Recurring Options   
-      # :action =>          
-      #    SUBMIT           
-      #    MODIFY           
-      #    CANCEL           
-      #                     
+      # Recurring Options
+      # :action =>
+      #    SUBMIT
+      #    MODIFY
+      #    CANCEL
+      #
       # :installments           Identifies how many recurring payments to charge the customer
       # :startdate              Date to begin charging the recurring payments. Format: YYYYMMDD or "immediate"
-      # :periodicity  =>    
-      #     :monthly         
-      #     :bimonthly       
-      #     :weekly          
-      #     :biweekly        
-      #     :yearly          
-      #     :daily           
+      # :periodicity  =>
+      #     :monthly
+      #     :bimonthly
+      #     :weekly
+      #     :biweekly
+      #     :yearly
+      #     :daily
       # :threshold              Tells how many times to retry the transaction (if it fails) before contacting the merchant.
       # :comments               Uh... comments
       #
       def recurring(money, creditcard, options={})
         requires!(options, [:periodicity, :bimonthly, :monthly, :biweekly, :weekly, :yearly, :daily], :installments, :order_id )
-        
+
         options.update(
           :ordertype => "SALE",
           :action => options[:action] || "SUBMIT",
@@ -179,7 +179,7 @@ module ActiveMerchant #:nodoc:
         )
         commit(money, creditcard, options)
       end
-      
+
       # Buy the thing
       def purchase(money, creditcard, options={})
         requires!(options, :order_id)
@@ -188,10 +188,10 @@ module ActiveMerchant #:nodoc:
         )
         commit(money, creditcard, options)
       end
-      
+
       #
       # Authorize the transaction
-      # 
+      #
       # Reserves the funds on the customer's credit card, but does not charge the card.
       #
       def authorize(money, creditcard, options = {})
@@ -201,21 +201,21 @@ module ActiveMerchant #:nodoc:
         )
         commit(money, creditcard, options)
       end
-      
+
       #
-      # Post an authorization. 
+      # Post an authorization.
       #
-      # Captures the funds from an authorized transaction. 
+      # Captures the funds from an authorized transaction.
       # Order_id must be a valid order id from a prior authorized transaction.
-      # 
+      #
       def capture(money, authorization, options = {})
         options.update(
           :order_id => authorization,
           :ordertype => "POSTAUTH"
         )
-        commit(money, nil, options)  
+        commit(money, nil, options)
       end
-      
+
       # Void a previous transaction
       def void(identification, options = {})
         options.update(
@@ -224,10 +224,10 @@ module ActiveMerchant #:nodoc:
         )
         commit(nil, nil, options)
       end
-      
-      # 
+
+      #
       # Refund an order
-      # 
+      #
       # identification must be a valid order id previously submitted by SALE
       #
       def refund(money, identification, options = {})
@@ -242,39 +242,35 @@ module ActiveMerchant #:nodoc:
         deprecated CREDIT_DEPRECATION_MESSAGE
         refund(money, identification, options)
       end
-    
-      def test?
-        @options[:test] || super
-      end
-      
+
       private
       # Commit the transaction by posting the XML file to the LinkPoint server
       def commit(money, creditcard, options = {})
         response = parse(ssl_post(test? ? self.test_url : self.live_url, post_data(money, creditcard, options)))
-        
-        Response.new(successful?(response), response[:message], response, 
+
+        Response.new(successful?(response), response[:message], response,
           :test => test?,
           :authorization => response[:ordernum],
           :avs_result => { :code => response[:avs].to_s[2,1] },
           :cvv_result => response[:avs].to_s[3,1]
         )
       end
-      
+
       def successful?(response)
         response[:approved] == "APPROVED"
       end
-      
+
       # Build the XML file
       def post_data(money, creditcard, options)
         params = parameters(money, creditcard, options)
-        
+
         xml = REXML::Document.new
         order = xml.add_element("order")
-        
+
         # Merchant Info
         merchantinfo = order.add_element("merchantinfo")
         merchantinfo.add_element("configfile").text = @options[:login]
-        
+
         # Loop over the params hash to construct the XML string
         for key, value in params
           elem = order.add_element(key.to_s)
@@ -285,7 +281,7 @@ module ActiveMerchant #:nodoc:
               elem.add_element(k.to_s).text = params[key][k].to_s if params[key][k]
             end
           end
-          # Linkpoint doesn't understand empty elements: 
+          # Linkpoint doesn't understand empty elements:
           order.delete(elem) if elem.size == 0
         end
         return xml.to_s
@@ -311,9 +307,9 @@ module ActiveMerchant #:nodoc:
       end
 
       # Set up the parameters hash just once so we don't have to do it
-      # for every action. 
+      # for every action.
       def parameters(money, creditcard, options = {})
-        
+
         params = {
           :payment => {
             :subtotal => amount(options[:subtotal]),
@@ -330,7 +326,7 @@ module ActiveMerchant #:nodoc:
             :terminaltype => options[:terminaltype],
             :ip => options[:ip],
             :reference_number => options[:reference_number],
-            :recurring => options[:recurring] || "NO",  #DO NOT USE if you are using the periodic billing option. 
+            :recurring => options[:recurring] || "NO",  #DO NOT USE if you are using the periodic billing option.
             :tdate => options[:tdate]
           },
           :orderoptions => {
@@ -339,10 +335,10 @@ module ActiveMerchant #:nodoc:
           },
           :periodic => {
             :action => options[:action],
-            :installments => options[:installments], 
-            :threshold => options[:threshold], 
-            :startdate => options[:startdate], 
-            :periodicity => options[:periodicity], 
+            :installments => options[:installments],
+            :threshold => options[:threshold],
+            :startdate => options[:startdate],
+            :periodicity => options[:periodicity],
             :comments => options[:comments]
           },
           :telecheck => {
@@ -357,7 +353,7 @@ module ActiveMerchant #:nodoc:
             :ssn => options[:telecheck_ssn],
           }
         }
-      
+
         if creditcard
           params[:creditcard] = {
             :cardnumber => creditcard.number,
@@ -365,18 +361,18 @@ module ActiveMerchant #:nodoc:
             :cardexpyear => format_creditcard_expiry_year(creditcard.year),
             :track => nil
           }
-          
+
           if creditcard.verification_value?
             params[:creditcard][:cvmvalue] = creditcard.verification_value
             params[:creditcard][:cvmindicator] = 'provided'
           else
             params[:creditcard][:cvmindicator] = 'not_provided'
-          end          
+          end
         end
-        
-        if billing_address = options[:billing_address] || options[:address]          
-          
-          params[:billing] = {}        
+
+        if billing_address = options[:billing_address] || options[:address]
+
+          params[:billing] = {}
           params[:billing][:name]      = billing_address[:name] || (creditcard ? creditcard.name : nil)
           params[:billing][:address1]  = billing_address[:address1] unless billing_address[:address1].blank?
           params[:billing][:address2]  = billing_address[:address2] unless billing_address[:address2].blank?
@@ -387,27 +383,27 @@ module ActiveMerchant #:nodoc:
           params[:billing][:company]   = billing_address[:company]  unless billing_address[:company].blank?
           params[:billing][:phone]     = billing_address[:phone]  unless billing_address[:phone].blank?
           params[:billing][:email]     = options[:email] unless options[:email].blank?
-        end                
+        end
 
-        if shipping_address = options[:shipping_address] 
+        if shipping_address = options[:shipping_address]
 
           params[:shipping] = {}
-          params[:shipping][:name]      = shipping_address[:name] || creditcard ? creditcard.name : nil
+          params[:shipping][:name]      = shipping_address[:name] || (creditcard ? creditcard.name : nil)
           params[:shipping][:address1]  = shipping_address[:address1] unless shipping_address[:address1].blank?
           params[:shipping][:address2]  = shipping_address[:address2] unless shipping_address[:address2].blank?
           params[:shipping][:city]      = shipping_address[:city]     unless shipping_address[:city].blank?
           params[:shipping][:state]     = shipping_address[:state]    unless shipping_address[:state].blank?
           params[:shipping][:zip]       = shipping_address[:zip]      unless shipping_address[:zip].blank?
           params[:shipping][:country]   = shipping_address[:country]  unless shipping_address[:country].blank?
-        end        
+        end
 
         params[:items] = options[:line_items] if options[:line_items]
-        
+
         return params
       end
-        
+
       def parse(xml)
-        
+
         # For reference, a typical response...
         # <r_csp></r_csp>
         # <r_time></r_time>
@@ -421,17 +417,17 @@ module ActiveMerchant #:nodoc:
         # <r_authresponse></r_authresponse>
         # <r_approved>APPROVED</r_approved>
         # <r_avs></r_avs>
-        
+
         response = {:message => "Global Error Receipt", :complete => false}
-        
+
         xml = REXML::Document.new("<response>#{xml}</response>")
         xml.root.elements.each do |node|
           response[node.name.downcase.sub(/^r_/, '').to_sym] = normalize(node.text)
         end unless xml.root.nil?
-        
+
         response
       end
-    
+
       # Make a ruby type out of the response string
       def normalize(field)
         case field
@@ -440,12 +436,12 @@ module ActiveMerchant #:nodoc:
         when ""       then nil
         when "null"   then nil
         else field
-        end        
+        end
       end
 
       def format_creditcard_expiry_year(year)
         sprintf("%.4i", year)[-2..-1]
-      end      
+      end
     end
   end
 end