activemerchant 1.28.0 → 1.29.0

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

@@ -8,22 +8,17 @@ module ActiveMerchant #:nodoc:
       self.supported_cardtypes = [:visa, :master, :american_express, :discover]
       self.homepage_url = 'http://www.concordefsnet.com/'
       self.display_name = 'Efsnet'
-     
+
       self.test_url = 'https://testefsnet.concordebiz.com/efsnet.dll'
       self.live_url = 'https://efsnet.concordebiz.com/efsnet.dll'
-      
+
       # login is your Store ID
       # password is your Store Key
       def initialize(options = {})
         requires!(options, :login, :password)
-        @options = options
-        super      
-      end
-      
-      def test?
-        @options[:test] || super
+        super
       end
-      
+
       def authorize(money, creditcard, options = {})
         request = build_credit_card_request(money, creditcard, options)
         commit(:credit_card_authorize, request)
@@ -33,7 +28,7 @@ module ActiveMerchant #:nodoc:
         request = build_credit_card_request(money, creditcard, options)
         commit(:credit_card_charge, request)
       end
-     
+
       def capture(money, identification, options = {})
         request = build_refund_or_settle_request(money, identification, options)
         commit(:credit_card_settle, request)
@@ -62,28 +57,28 @@ module ActiveMerchant #:nodoc:
         original_transaction_id, original_transaction_amount = identification.split(";")
         commit(:void_transaction, {:reference_number => format_reference_number(options[:order_id]), :transaction_id => original_transaction_id})
       end
-      
+
       def voice_authorize(money, authorization_code, creditcard, options = {})
         options[:authorization_number] = authorization_code
         request = build_credit_card_request(money, creditcard, options)
         commit(:credit_card_voice_authorize, request)
       end
-      
+
       def force(money, authorization_code, creditcard, options = {})
         options[:authorization_number] = authorization_code
         request = build_credit_card_request(money, creditcard, options)
         commit(:credit_card_capture, request)
       end
-      
+
       def system_check
-        commit(:system_check, {})      
+        commit(:system_check, {})
       end
 
       private
 
       def build_refund_or_settle_request(money, identification, options = {})
         original_transaction_id, original_transaction_amount = identification.split(";")
-        
+
         requires!(options, :order_id)
 
         post = {
@@ -94,7 +89,7 @@ module ActiveMerchant #:nodoc:
           :client_ip_address => options[:ip]
         }
       end
-    
+
       def build_credit_card_request(money, creditcard, options = {})
         requires!(options, :order_id)
 
@@ -103,17 +98,17 @@ module ActiveMerchant #:nodoc:
           :authorization_number => options[:authorization_number],
           :transaction_amount => amount(money),
           :client_ip_address => options[:ip]
-          
+
         }
         add_creditcard(post,creditcard)
         add_address(post,options)
         post
       end
-      
+
       def format_reference_number(number)
         number.to_s.slice(0,12)
       end
-    
+
       def add_address(post,options)
         if address = options[:billing_address] || options[:address]
           if address[:address2]
@@ -123,7 +118,7 @@ module ActiveMerchant #:nodoc:
           end
           post[:billing_city]         = address[:city].to_s
           post[:billing_state]        = address[:state].blank?  ? 'n/a' : address[:state]
-          post[:billing_postal_code]  = address[:zip].to_s       
+          post[:billing_postal_code]  = address[:zip].to_s
           post[:billing_country]      = address[:country].to_s
         end
 
@@ -135,12 +130,12 @@ module ActiveMerchant #:nodoc:
           end
           post[:shipping_city]        = address[:city].to_s
           post[:shipping_state]       = address[:state].blank?  ? 'n/a' : address[:state]
-          post[:shipping_postal_code] = address[:zip].to_s       
+          post[:shipping_postal_code] = address[:zip].to_s
           post[:shipping_country]     = address[:country].to_s
         end
       end
 
-      def add_creditcard(post, creditcard)      
+      def add_creditcard(post, creditcard)
         post[:billing_name]  = creditcard.name if creditcard.name
         post[:account_number]  = creditcard.number
         post[:card_verification_value] = creditcard.verification_value if creditcard.verification_value?
@@ -148,8 +143,8 @@ module ActiveMerchant #:nodoc:
         post[:expiration_year]  = sprintf("%.4i", creditcard.year)[-2..-1]
       end
 
-  
-      def commit(action, parameters)  
+
+      def commit(action, parameters)
         response = parse(ssl_post(test? ? self.test_url : self.live_url, post_data(action, parameters), 'Content-Type' => 'text/xml'))
 
         Response.new(success?(response), message_from(response[:result_message]), response,
@@ -159,19 +154,19 @@ module ActiveMerchant #:nodoc:
           :cvv_result => response[:cvv_response_code]
         )
       end
-      
+
       def success?(response)
         response[:response_code] == '0'
       end
-      
+
       def authorization_from(response, params)
         [ response[:transaction_id], params[:transaction_amount] ].compact.join(';')
       end
-                                               
+
       def parse(xml)
         response = {}
 
-        xml = REXML::Document.new(xml)          
+        xml = REXML::Document.new(xml)
 
         xml.elements.each('//Reply//TransactionReply/*') do |node|
 
@@ -180,7 +175,7 @@ module ActiveMerchant #:nodoc:
         end unless xml.root.nil?
 
         response
-      end     
+      end
 
       def post_data(action, parameters = {})
         xml   = REXML::Document.new("<?xml version='1.0' encoding='UTF-8'?>")
@@ -196,7 +191,7 @@ module ActiveMerchant #:nodoc:
 
         xml.to_s
       end
-    
+
       def message_from(message)
         return 'Unspecified error' if message.blank?
         message.gsub(/[^\w]/, ' ').split.join(" ").capitalize
@@ -210,9 +205,9 @@ module ActiveMerchant #:nodoc:
         when ""       then nil
         when "null"   then nil
         else field
-        end        
-      end         
-    
+        end
+      end
+
       def actions
         ACTIONS
       end
@@ -229,7 +224,7 @@ module ActiveMerchant #:nodoc:
            :void_transaction			=> %w(ReferenceNumber TransactionID),
            :credit_card_settle			=> %w(ReferenceNumber TransactionAmount OriginalTransactionAmount OriginalTransactionID ClientIpAddress),
            :system_check			=> %w(SystemCheck),
-      }    
+      }
     end
   end
 end

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

@@ -63,13 +63,13 @@ module ActiveMerchant #:nodoc:
       # * <tt>:test => +true+ or +false+</tt> -- Force test transactions
       def initialize(options = {})
         requires!(options, :login, :password)
-        @options = options
         super
       end
 
       # Make a purchase
       def purchase(money, creditcard, options = {})
         form = {}
+        add_salestax(form, options)
         add_invoice(form, options)
         add_creditcard(form, creditcard)
         add_address(form, options)
@@ -87,6 +87,7 @@ module ActiveMerchant #:nodoc:
       #   * <tt>:billing_address</tt> - The billing address for the cardholder.
       def authorize(money, creditcard, options = {})
         form = {}
+        add_salestax(form, options)
         add_invoice(form, options)
         add_creditcard(form, creditcard)
         add_address(form, options)
@@ -106,6 +107,7 @@ module ActiveMerchant #:nodoc:
         requires!(options, :credit_card)
 
         form = {}
+        add_salestax(form, options)
         add_approval_code(form, authorization)
         add_invoice(form, options)
         add_creditcard(form, options[:credit_card])
@@ -205,6 +207,10 @@ module ActiveMerchant #:nodoc:
         form[:customer_code] = options[:customer].to_s.slice(0, 10) unless options[:customer].blank?
       end
 
+      def add_salestax(form, options)
+        form[:salestax] = options[:tax] if options[:tax].present?
+      end
+
       def expdate(creditcard)
         year  = sprintf("%.4i", creditcard.year)
         month = sprintf("%.2i", creditcard.month)

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

@@ -54,7 +54,6 @@ module ActiveMerchant #:nodoc:
       # password: referrer url (for authorize authentication)
       def initialize(options = {})
         requires!(options, :login)
-        @options = options
         super
       end
 

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

@@ -3,7 +3,7 @@ require 'rexml/document'
 module ActiveMerchant #:nodoc:
   module Billing #:nodoc:
     # First, make sure you have everything setup correctly and all of your dependencies in place with:
-    # 
+    #
     #   require 'rubygems'
     #   require 'active_merchant'
     #
@@ -17,10 +17,10 @@ module ActiveMerchant #:nodoc:
     # Next, create a credit card object using a eWay approved test card number (4444333322221111).
     #
     #   creditcard = ActiveMerchant::Billing::CreditCard.new(
-    #	    :number => '4444333322221111',
-    #	    :month => 8,
-    #	    :year => 2006,
-    #	    :first_name => 'Longbob',
+    #     :number => '4444333322221111',
+    #     :month => 8,
+    #     :year => 2006,
+    #     :first_name => 'Longbob',
     #     :last_name => 'Longsen',
     #     :verification_value => '123'
     #   )
@@ -28,7 +28,7 @@ module ActiveMerchant #:nodoc:
     #     :order_id => '1230123',
     #     :email => 'bob@testbob.com',
     #     :address => { :address1 => '47 Bobway',
-    #                   :city => 'Bobville', 
+    #                   :city => 'Bobville',
     #                   :state => 'WA',
     #                   :country => 'Australia',
     #                   :zip => '2000'
@@ -63,88 +63,87 @@ module ActiveMerchant #:nodoc:
     # This should be enough to get you started with eWay and active_merchant. For further information, review the methods
     # below and the rest of active_merchant's documentation.
 
-    class EwayGateway < Gateway 
+    class EwayGateway < Gateway
       self.test_url = 'https://www.eway.com.au/gateway/xmltest/testpage.asp'
       self.live_url = 'https://www.eway.com.au/gateway/xmlpayment.asp'
 
       class_attribute :test_cvn_url, :live_cvn_url
       self.test_cvn_url = 'https://www.eway.com.au/gateway_cvn/xmltest/testpage.asp'
       self.live_cvn_url = 'https://www.eway.com.au/gateway_cvn/xmlpayment.asp'
-      
+
       MESSAGES = {
         "00" => "Transaction Approved",
         "01" => "Refer to Issuer",
-        "02" => "Refer to Issuer, special",	
+        "02" => "Refer to Issuer, special",
         "03" => "No Merchant",
-        "04" => "Pick Up Card",	
-        "05" => "Do Not Honour",	
+        "04" => "Pick Up Card",
+        "05" => "Do Not Honour",
         "06" => "Error",
-        "07" => "Pick Up Card, Special",	
-        "08" => "Honour With Identification",	
+        "07" => "Pick Up Card, Special",
+        "08" => "Honour With Identification",
         "09" => "Request In Progress",
-        "10" => "Approved For Partial Amount",	
-        "11" => "Approved, VIP",	
-        "12" => "Invalid Transaction",	
+        "10" => "Approved For Partial Amount",
+        "11" => "Approved, VIP",
+        "12" => "Invalid Transaction",
         "13" => "Invalid Amount",
-        "14" => "Invalid Card Number",	
-        "15" => "No Issuer",	
-        "16" => "Approved, Update Track 3",	
-        "19" => "Re-enter Last Transaction",	
-        "21" => "No Action Taken",	
-        "22" => "Suspected Malfunction",	
-        "23" => "Unacceptable Transaction Fee",	
-        "25" => "Unable to Locate Record On File",	
-        "30" => "Format Error",	
-        "31" => "Bank Not Supported By Switch",	
-        "33" => "Expired Card, Capture",	
-        "34" => "Suspected Fraud, Retain Card",	
-        "35" => "Card Acceptor, Contact Acquirer, Retain Card",	
-        "36" => "Restricted Card, Retain Card",	
-        "37" => "Contact Acquirer Security Department, Retain Card",	
-        "38" => "PIN Tries Exceeded, Capture",	
-        "39" => "No Credit Account",	
-        "40" => "Function Not Supported",	
-        "41" => "Lost Card",	
-        "42" => "No Universal Account",	
-        "43" => "Stolen Card",	
-        "44" => "No Investment Account",	
-        "51" => "Insufficient Funds",	
-        "52" => "No Cheque Account",	
-        "53" => "No Savings Account",	
-        "54" => "Expired Card",	
-        "55" => "Incorrect PIN",	
-        "56" => "No Card Record",	
-        "57" => "Function Not Permitted to Cardholder",	
-        "58" => "Function Not Permitted to Terminal",	
-        "59" => "Suspected Fraud",	
-        "60" => "Acceptor Contact Acquirer",	
-        "61" => "Exceeds Withdrawal Limit",	
-        "62" => "Restricted Card",	
-        "63" => "Security Violation",	
-        "64" => "Original Amount Incorrect",	
-        "66" => "Acceptor Contact Acquirer, Security",	
-        "67" => "Capture Card",	
-        "75" => "PIN Tries Exceeded",	
-        "82" => "CVV Validation Error",	
-        "90" => "Cutoff In Progress",	
-        "91" => "Card Issuer Unavailable",	
-        "92" => "Unable To Route Transaction",	
-        "93" => "Cannot Complete, Violation Of The Law",	
-        "94" => "Duplicate Transaction",	
+        "14" => "Invalid Card Number",
+        "15" => "No Issuer",
+        "16" => "Approved, Update Track 3",
+        "19" => "Re-enter Last Transaction",
+        "21" => "No Action Taken",
+        "22" => "Suspected Malfunction",
+        "23" => "Unacceptable Transaction Fee",
+        "25" => "Unable to Locate Record On File",
+        "30" => "Format Error",
+        "31" => "Bank Not Supported By Switch",
+        "33" => "Expired Card, Capture",
+        "34" => "Suspected Fraud, Retain Card",
+        "35" => "Card Acceptor, Contact Acquirer, Retain Card",
+        "36" => "Restricted Card, Retain Card",
+        "37" => "Contact Acquirer Security Department, Retain Card",
+        "38" => "PIN Tries Exceeded, Capture",
+        "39" => "No Credit Account",
+        "40" => "Function Not Supported",
+        "41" => "Lost Card",
+        "42" => "No Universal Account",
+        "43" => "Stolen Card",
+        "44" => "No Investment Account",
+        "51" => "Insufficient Funds",
+        "52" => "No Cheque Account",
+        "53" => "No Savings Account",
+        "54" => "Expired Card",
+        "55" => "Incorrect PIN",
+        "56" => "No Card Record",
+        "57" => "Function Not Permitted to Cardholder",
+        "58" => "Function Not Permitted to Terminal",
+        "59" => "Suspected Fraud",
+        "60" => "Acceptor Contact Acquirer",
+        "61" => "Exceeds Withdrawal Limit",
+        "62" => "Restricted Card",
+        "63" => "Security Violation",
+        "64" => "Original Amount Incorrect",
+        "66" => "Acceptor Contact Acquirer, Security",
+        "67" => "Capture Card",
+        "75" => "PIN Tries Exceeded",
+        "82" => "CVV Validation Error",
+        "90" => "Cutoff In Progress",
+        "91" => "Card Issuer Unavailable",
+        "92" => "Unable To Route Transaction",
+        "93" => "Cannot Complete, Violation Of The Law",
+        "94" => "Duplicate Transaction",
         "96" => "System Error"
       }
-      
-	    self.money_format = :cents
+
+      self.money_format = :cents
       self.supported_countries = ['AU']
       self.supported_cardtypes = [:visa, :master, :american_express, :diners_club]
       self.homepage_url = 'http://www.eway.com.au/'
       self.display_name = 'eWAY'
-	    
-    	def initialize(options = {})
+
+      def initialize(options = {})
         requires!(options, :login)
-        @options = options
         super
-    	end
+      end
 
       # ewayCustomerEmail, ewayCustomerAddress, ewayCustomerPostcode
       def purchase(money, creditcard, options = {})
@@ -152,20 +151,16 @@ module ActiveMerchant #:nodoc:
 
         post = {}
         add_creditcard(post, creditcard)
-        add_address(post, options)  
+        add_address(post, options)
         add_customer_data(post, options)
         add_invoice_data(post, options)
         # The request fails if all of the fields aren't present
         add_optional_data(post)
-    
+
         commit(money, post)
       end
-      
-      def test?
-        @options[:test] || super
-      end
-      
-      private                       
+
+      private
 
       def requires_address!(options)
         raise ArgumentError.new("Missing eWay required parameters: address or billing_address") unless (options.has_key?(:address) or options.has_key?(:billing_address))
@@ -178,9 +173,9 @@ module ActiveMerchant #:nodoc:
         post[:CustomerFirstName] = creditcard.first_name
         post[:CustomerLastName]  = creditcard.last_name
         post[:CardHoldersName] = creditcard.name
-              
+
         post[:CVN] = creditcard.verification_value if creditcard.verification_value?
-      end 
+      end
 
       def add_address(post, options)
         if address = options[:billing_address] || options[:address]
@@ -192,7 +187,7 @@ module ActiveMerchant #:nodoc:
       def add_customer_data(post, options)
         post[:CustomerEmail] = options[:email]
       end
-      
+
       def add_invoice_data(post, options)
         post[:CustomerInvoiceRef] = options[:order_id]
         post[:CustomerInvoiceDescription] = options[:description]
@@ -202,10 +197,10 @@ module ActiveMerchant #:nodoc:
         post[:TrxnNumber] = nil
         post[:Option1] = nil
         post[:Option2] = nil
-        post[:Option3] = nil     
+        post[:Option3] = nil
       end
 
-      def commit(money, parameters)       
+      def commit(money, parameters)
         parameters[:TotalAmount] = amount(money)
 
         response = parse( ssl_post(gateway_url(parameters[:CVN], test?), post_data(parameters)) )
@@ -213,13 +208,13 @@ module ActiveMerchant #:nodoc:
         Response.new(success?(response), message_from(response[:ewaytrxnerror]), response,
           :authorization => response[:ewayauthcode],
           :test => /\(Test( CVN)? Gateway\)/ === response[:ewaytrxnerror]
-        )      
+        )
       end
-      
+
       def success?(response)
         response[:ewaytrxnstatus] == "True"
       end
-                                             
+
       # Parse eway response xml into a convinient hash
       def parse(xml)
         #  "<?xml version=\"1.0\"?>".
@@ -233,10 +228,10 @@ module ActiveMerchant #:nodoc:
         #  <ewayReturnAmount>10</ewayReturnAmount>
         #  <ewayAuthCode>123456</ewayAuthCode>
         #  <ewayTrxnReference>987654321</ewayTrxnReference>
-        #  </ewayResponse>     
+        #  </ewayResponse>
 
         response = {}
-        xml = REXML::Document.new(xml)          
+        xml = REXML::Document.new(xml)
         xml.elements.each('//ewayResponse/*') do |node|
 
           response[node.name.downcase.to_sym] = normalize(node.text)
@@ -244,20 +239,20 @@ module ActiveMerchant #:nodoc:
         end unless xml.root.nil?
 
         response
-      end   
+      end
 
       def post_data(parameters = {})
         parameters[:CustomerID] = @options[:login]
-        
+
         xml   = REXML::Document.new
         root  = xml.add_element("ewaygateway")
-        
+
         parameters.each do |key, value|
           root.add_element("eway#{key}").text = value
-        end    
+        end
         xml.to_s
       end
-    
+
       def message_from(message)
         return '' if message.blank?
         MESSAGES[message[0,2]] || message
@@ -271,9 +266,9 @@ module ActiveMerchant #:nodoc:
         when ""       then nil
         when "null"   then nil
         else field
-        end        
+        end
       end
-      
+
       def gateway_url(cvn, test)
         if cvn
           test ? self.test_cvn_url : self.live_cvn_url
@@ -281,7 +276,7 @@ module ActiveMerchant #:nodoc:
           test ? self.test_url : self.live_url
         end
       end
-      
+
     end
   end
 end