activemerchant 1.79.2 → 1.80.0

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

@@ -122,7 +122,7 @@ module ActiveMerchant #:nodoc:
           post[:state]   = (address[:state].blank? ? 'NA' : address[:state].to_s)
           post[:zip]     = address[:zip].to_s
           post[:country] = address[:country].to_s
-          post[:phone]   = (address[:phone].to_s.gsub(/[^0-9]/, '') || "0000000")
+          post[:phone]   = (address[:phone].to_s.gsub(/[^0-9]/, '') || '0000000')
         end
 
         if address = options[:shipping_address]
@@ -153,8 +153,8 @@ module ActiveMerchant #:nodoc:
       def add_payment_cc(post, credit_card)
         post[:cc] = credit_card.number
         post[:cvv] = credit_card.verification_value if credit_card.verification_value?
-        year  = sprintf("%.4i", credit_card.year)
-        month = sprintf("%.2i", credit_card.month)
+        year  = sprintf('%.4i', credit_card.year)
+        month = sprintf('%.2i', credit_card.month)
         post[:expire] = "#{month}/#{year[2..3]}"
       end
 
@@ -166,10 +166,10 @@ module ActiveMerchant #:nodoc:
       def parse(body)
         if body =~ /^ID:/
           body.split('~').reduce(Hash.new) { |h,v|
-            m = v.match("(.*?):(.*)")
+            m = v.match('(.*?):(.*)')
             h.merge!(m[1].underscore.to_sym => m[2])
           }
-        elsif (m = body.match("(.*?):(.*)"))
+        elsif (m = body.match('(.*?):(.*)'))
           m[1] == 'OK' ?
             { status: 'success', id: m[2] } :
             { status: 'failure', message: m[2] }
@@ -195,12 +195,12 @@ module ActiveMerchant #:nodoc:
         identifier = (response[:id] || parameters[:init_transaction_id])
         authorization = [identifier]
         authorization << amount if amount
-        authorization.join("|")
+        authorization.join('|')
       end
 
       def split_authorization(authorization)
         if authorization =~ /|/
-          identifier, amount = authorization.split("|")
+          identifier, amount = authorization.split('|')
           [identifier, amount.to_i]
         else
           authorization

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

@@ -3,7 +3,7 @@ module ActiveMerchant #:nodoc:
     class TransnationalGateway < NetworkMerchantsGateway
       self.homepage_url = 'http://www.tnbci.com/'
       self.display_name = 'Transnational'
-      self.supported_countries = ["US"]
+      self.supported_countries = ['US']
     end
   end
 end

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

@@ -120,7 +120,7 @@ module ActiveMerchant #:nodoc:
       end
 
       def add_invoice(post, options)
-        post[:description] = options[:description] || "Active Merchant Purchase"
+        post[:description] = options[:description] || 'Active Merchant Purchase'
       end
 
       def add_creditcard(post, creditcard)
@@ -145,8 +145,8 @@ module ActiveMerchant #:nodoc:
 
       def headers(params = {})
         result = {
-          "Content-Type" => "application/json",
-          "Authorization" => "Basic #{Base64.strict_encode64(options[:api_key] + ':').strip}"
+          'Content-Type' => 'application/json',
+          'Authorization' => "Basic #{Base64.strict_encode64(options[:api_key] + ':').strip}"
         }
 
         result['X-Partner-Key'] = params[:partner_key] if params[:partner_key]
@@ -190,13 +190,13 @@ module ActiveMerchant #:nodoc:
       end
 
       def unparsable_response(raw_response)
-        message = "Invalid JSON response received from Trexle. Please contact support@trexle.com if you continue to receive this message."
+        message = 'Invalid JSON response received from Trexle. Please contact support@trexle.com if you continue to receive this message.'
         message += " (The raw response returned by the API was #{raw_response.inspect})"
         return Response.new(false, message)
       end
       
       def invalid_response
-        message = "Invalid response."
+        message = 'Invalid response.'
         return Response.new(false, message)
       end
 

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

@@ -67,38 +67,38 @@ module ActiveMerchant #:nodoc:
     class TrustCommerceGateway < Gateway
       self.live_url = self.test_url = 'https://vault.trustcommerce.com/trans/'
 
-      SUCCESS_TYPES = ["approved", "accepted"]
+      SUCCESS_TYPES = ['approved', 'accepted']
 
       DECLINE_CODES = {
-        "decline"       => "The credit card was declined",
-        "avs"           => "AVS failed; the address entered does not match the billing address on file at the bank",
-        "cvv"           => "CVV failed; the number provided is not the correct verification number for the card",
-        "call"          => "The card must be authorized manually over the phone",
-        "expiredcard"   => "Issuer was not certified for card verification",
-        "carderror"     => "Card number is invalid",
-        "authexpired"   => "Attempt to postauth an expired (more than 14 days old) preauth",
-        "fraud"         => "CrediGuard fraud score was below requested threshold",
-        "blacklist"     => "CrediGuard blacklist value was triggered",
-        "velocity"      => "CrediGuard velocity control value was triggered",
-        "dailylimit"    => "Daily limit in transaction count or amount as been reached",
-        "weeklylimit"   => "Weekly limit in transaction count or amount as been reached",
-        "monthlylimit"  => "Monthly limit in transaction count or amount as been reached"
+        'decline'       => 'The credit card was declined',
+        'avs'           => 'AVS failed; the address entered does not match the billing address on file at the bank',
+        'cvv'           => 'CVV failed; the number provided is not the correct verification number for the card',
+        'call'          => 'The card must be authorized manually over the phone',
+        'expiredcard'   => 'Issuer was not certified for card verification',
+        'carderror'     => 'Card number is invalid',
+        'authexpired'   => 'Attempt to postauth an expired (more than 14 days old) preauth',
+        'fraud'         => 'CrediGuard fraud score was below requested threshold',
+        'blacklist'     => 'CrediGuard blacklist value was triggered',
+        'velocity'      => 'CrediGuard velocity control value was triggered',
+        'dailylimit'    => 'Daily limit in transaction count or amount as been reached',
+        'weeklylimit'   => 'Weekly limit in transaction count or amount as been reached',
+        'monthlylimit'  => 'Monthly limit in transaction count or amount as been reached'
       }
 
       BADDATA_CODES = {
-        "missingfields"       => "One or more parameters required for this transaction type were not sent",
-        "extrafields"         => "Parameters not allowed for this transaction type were sent",
-        "badformat"           => "A field was improperly formatted, such as non-digit characters in a number field",
-        "badlength"           => "A field was longer or shorter than the server allows",
-        "merchantcantaccept"  => "The merchant can't accept data passed in this field",
-        "mismatch"            => "Data in one of the offending fields did not cross-check with the other offending field"
+        'missingfields'       => 'One or more parameters required for this transaction type were not sent',
+        'extrafields'         => 'Parameters not allowed for this transaction type were sent',
+        'badformat'           => 'A field was improperly formatted, such as non-digit characters in a number field',
+        'badlength'           => 'A field was longer or shorter than the server allows',
+        'merchantcantaccept'  => "The merchant can't accept data passed in this field",
+        'mismatch'            => 'Data in one of the offending fields did not cross-check with the other offending field'
       }
 
       ERROR_CODES = {
-        "cantconnect"   => "Couldn't connect to the TrustCommerce gateway",
-        "dnsfailure"    => "The TCLink software was unable to resolve DNS hostnames",
-        "linkfailure"   => "The connection was established, but was severed before the transaction could complete",
-        "failtoprocess" => "The bank servers are offline and unable to authorize transactions"
+        'cantconnect'   => "Couldn't connect to the TrustCommerce gateway",
+        'dnsfailure'    => 'The TCLink software was unable to resolve DNS hostnames',
+        'linkfailure'   => 'The connection was established, but was severed before the transaction could complete',
+        'failtoprocess' => 'The bank servers are offline and unable to authorize transactions'
       }
 
       TEST_LOGIN = 'TestMerchant'
@@ -313,7 +313,7 @@ module ActiveMerchant #:nodoc:
       end
 
       def add_creditcard(params, creditcard)
-        params[:media]     = "cc"
+        params[:media]     = 'cc'
         params[:name]      = creditcard.name
         params[:cc]        = creditcard.number
         params[:exp]       = expdate(creditcard)
@@ -370,7 +370,7 @@ module ActiveMerchant #:nodoc:
       end
 
       def post_data(parameters)
-        parameters.collect { |key, value| "#{key}=#{ CGI.escape(value.to_s)}" }.join("&")
+        parameters.collect { |key, value| "#{key}=#{ CGI.escape(value.to_s)}" }.join('&')
       end
 
       def commit(action, parameters)
@@ -388,13 +388,13 @@ module ActiveMerchant #:nodoc:
         end
 
         # to be considered successful, transaction status must be either "approved" or "accepted"
-        success = SUCCESS_TYPES.include?(data["status"])
+        success = SUCCESS_TYPES.include?(data['status'])
         message = message_from(data)
         Response.new(success, message, data,
           :test => test?,
-          :authorization => data["transid"],
-          :cvv_result => data["cvv"],
-          :avs_result => { :code => data["avs"] }
+          :authorization => data['transid'],
+          :cvv_result => data['cvv'],
+          :avs_result => { :code => data['avs'] }
         )
       end
 
@@ -410,15 +410,15 @@ module ActiveMerchant #:nodoc:
       end
 
       def message_from(data)
-        case data["status"]
-        when "decline"
-          return DECLINE_CODES[data["declinetype"]]
-        when "baddata"
-          return BADDATA_CODES[data["error"]]
-        when "error"
-          return ERROR_CODES[data["errortype"]]
+        case data['status']
+        when 'decline'
+          return DECLINE_CODES[data['declinetype']]
+        when 'baddata'
+          return BADDATA_CODES[data['error']]
+        when 'error'
+          return ERROR_CODES[data['errortype']]
         else
-          return "The transaction was successful"
+          return 'The transaction was successful'
         end
       end
 

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

@@ -14,10 +14,10 @@ module ActiveMerchant #:nodoc:
       # create an instance of UsaEpayAdvancedGateway.
       #
       def self.new(options={})
-        unless options.has_key?(:software_id) || options.has_key?(:live_url)
-          UsaEpayTransactionGateway.new(options)
-        else
+        if options.has_key?(:software_id) || options.has_key?(:live_url)
           UsaEpayAdvancedGateway.new(options)
+        else
+          UsaEpayTransactionGateway.new(options)
         end
       end
     end

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

@@ -62,7 +62,7 @@ module ActiveMerchant #:nodoc:
     # * {USA ePay Developer Login}[https://www.usaepay.com/developer/login]
     #
     class UsaEpayAdvancedGateway < Gateway
-      API_VERSION = "1.4"
+      API_VERSION = '1.4'
 
       TEST_URL_BASE = 'https://sandbox.usaepay.com/soap/gate/' #:nodoc:
       LIVE_URL_BASE = 'https://www.usaepay.com/soap/gate/' #:nodoc:
@@ -70,7 +70,7 @@ module ActiveMerchant #:nodoc:
       self.test_url = TEST_URL_BASE
       self.live_url = LIVE_URL_BASE
 
-      FAILURE_MESSAGE = "Default Failure" #:nodoc:
+      FAILURE_MESSAGE = 'Default Failure' #:nodoc:
 
       self.supported_countries = ['US']
       self.supported_cardtypes = [:visa, :master, :american_express, :discover, :diners_club, :jcb]
@@ -1032,14 +1032,14 @@ module ActiveMerchant #:nodoc:
       def build_request(action, options = {})
         soap = Builder::XmlMarkup.new
         soap.instruct!(:xml, :version => '1.0', :encoding => 'utf-8')
-        soap.tag! "SOAP-ENV:Envelope",
+        soap.tag! 'SOAP-ENV:Envelope',
           'xmlns:SOAP-ENV' => 'http://schemas.xmlsoap.org/soap/envelope/',
           'xmlns:ns1' => 'urn:usaepay',
           'xmlns:xsd' => 'http://www.w3.org/2001/XMLSchema',
           'xmlns:xsi' => 'http://www.w3.org/2001/XMLSchema-instance',
           'xmlns:SOAP-ENC' => 'http://schemas.xmlsoap.org/soap/encoding/',
           'SOAP-ENV:encodingStyle' => 'http://schemas.xmlsoap.org/soap/encoding/' do
-          soap.tag! "SOAP-ENV:Body" do
+          soap.tag! 'SOAP-ENV:Body' do
             send("build_#{action}", soap, options)
           end
         end
@@ -1058,9 +1058,9 @@ module ActiveMerchant #:nodoc:
         soap.Token 'xsi:type' => 'ns1:ueSecurityToken' do
           build_tag soap, :string, 'ClientIP', options[:client_ip]
           soap.PinHash 'xsi:type' => 'ns1:ueHash' do
-            build_tag soap, :string, "HashValue", hash
-            build_tag soap, :string, "Seed", seed
-            build_tag soap, :string, "Type", 'sha1'
+            build_tag soap, :string, 'HashValue', hash
+            build_tag soap, :string, 'Seed', seed
+            build_tag soap, :string, 'Type', 'sha1'
           end
           build_tag soap, :string, 'SourceKey', @options[:login]
         end
@@ -1069,12 +1069,12 @@ module ActiveMerchant #:nodoc:
       # Customer ======================================================
 
       def build_add_customer(soap, options)
-        soap.tag! "ns1:addCustomer" do
+        soap.tag! 'ns1:addCustomer' do
           build_token soap, options
           build_customer_data soap, options
           build_tag soap, :double, 'Amount', amount(options[:amount])
           build_tag soap, :double, 'Tax', amount(options[:tax])
-          build_tag soap, :string, 'Next', options[:next].strftime("%Y-%m-%d") if options[:next]
+          build_tag soap, :string, 'Next', options[:next].strftime('%Y-%m-%d') if options[:next]
         end
       end
 
@@ -1103,15 +1103,15 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_quick_update_customer(soap, options)
-        soap.tag! "ns1:quickUpdateCustomer" do
+        soap.tag! 'ns1:quickUpdateCustomer' do
           build_token soap, options
           build_tag soap, :integer, 'CustNum', options[:customer_number]
-          build_field_value_array soap, "UpdateData", "FieldValue", options[:update_data], CUSTOMER_UPDATE_DATA_FIELDS
+          build_field_value_array soap, 'UpdateData', 'FieldValue', options[:update_data], CUSTOMER_UPDATE_DATA_FIELDS
         end
       end
 
       def build_add_customer_payment_method(soap, options)
-        soap.tag! "ns1:addCustomerPaymentMethod" do
+        soap.tag! 'ns1:addCustomerPaymentMethod' do
           build_token soap, options
           build_tag soap, :integer, 'CustNum', options[:customer_number]
           build_customer_payment_methods soap, options
@@ -1141,7 +1141,7 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_delete_customer_payment_method(soap, options)
-        soap.tag! "ns1:deleteCustomerPaymentMethod" do
+        soap.tag! 'ns1:deleteCustomerPaymentMethod' do
           build_token soap, options
           build_tag soap, :integer, 'Custnum', options[:customer_number]
           build_tag soap, :integer, 'PaymentMethodID', options[:method_id]
@@ -1149,7 +1149,7 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_run_customer_transaction(soap, options)
-        soap.tag! "ns1:runCustomerTransaction" do
+        soap.tag! 'ns1:runCustomerTransaction' do
           build_token soap, options
           build_tag soap, :integer, 'CustNum', options[:customer_number]
           build_tag soap, :integer, 'PaymentMethodID', options[:method_id] || 0
@@ -1226,21 +1226,21 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_get_transaction(soap, options)
-        soap.tag! "ns1:getTransaction" do
+        soap.tag! 'ns1:getTransaction' do
           build_token soap, options
           build_tag soap, :integer, 'RefNum', options[:reference_number]
         end
       end
 
       def build_get_transaction_status(soap, options)
-        soap.tag! "ns1:getTransactionStatus" do
+        soap.tag! 'ns1:getTransactionStatus' do
           build_token soap, options
           build_tag soap, :integer, 'RefNum', options[:reference_number]
         end
       end
 
       def build_get_transaction_custom(soap, options)
-        soap.tag! "ns1:getTransactionCustom" do
+        soap.tag! 'ns1:getTransactionCustom' do
           build_token soap, options
           build_tag soap, :integer, 'RefNum', options[:reference_number]
           build_transaction_field_array soap, options
@@ -1248,14 +1248,14 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_get_check_trace(soap, options)
-        soap.tag! "ns1:getCheckTrace" do
+        soap.tag! 'ns1:getCheckTrace' do
           build_token soap, options
           build_tag soap, :integer, 'RefNum', options[:reference_number]
         end
       end
 
       def build_capture_transaction(soap, options)
-        soap.tag! "ns1:captureTransaction" do
+        soap.tag! 'ns1:captureTransaction' do
           build_token soap, options
           build_tag soap, :integer, 'RefNum', options[:reference_number]
           build_tag soap, :double, 'Amount', amount(options[:amount])
@@ -1263,14 +1263,14 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_void_transaction(soap, options)
-        soap.tag! "ns1:voidTransaction" do
+        soap.tag! 'ns1:voidTransaction' do
           build_token soap, options
           build_tag soap, :integer, 'RefNum', options[:reference_number]
         end
       end
 
       def build_refund_transaction(soap, options)
-        soap.tag! "ns1:refundTransaction" do
+        soap.tag! 'ns1:refundTransaction' do
           build_token soap, options
           build_tag soap, :integer, 'RefNum', options[:reference_number]
           build_tag soap, :integer, 'Amount', amount(options[:amount])
@@ -1278,7 +1278,7 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_override_transaction(soap, options)
-        soap.tag! "ns1:overrideTransaction" do
+        soap.tag! 'ns1:overrideTransaction' do
           build_token soap, options
           build_tag soap, :integer, 'RefNum', options[:reference_number]
           build_tag soap, :string, 'Reason', options[:reason]
@@ -1288,7 +1288,7 @@ module ActiveMerchant #:nodoc:
       # Account =======================================================
 
       def build_get_account_details(soap, options)
-        soap.tag! "ns1:getAccountDetails" do
+        soap.tag! 'ns1:getAccountDetails' do
           build_token soap, options
         end
       end
@@ -1310,7 +1310,7 @@ module ActiveMerchant #:nodoc:
         if options[:payment_methods]
           length = options[:payment_methods].length
           soap.PaymentMethods 'SOAP-ENC:arrayType' => "ns1:PaymentMethod[#{length}]",
-            'xsi:type' =>"ns1:PaymentMethodArray" do
+            'xsi:type' =>'ns1:PaymentMethodArray' do
             build_customer_payment_methods soap, options
           end
         end
@@ -1357,7 +1357,7 @@ module ActiveMerchant #:nodoc:
       def build_customer_payment_methods(soap, options)
         payment_methods, tag_name = extract_methods_and_tag(options)
         payment_methods.each do |payment_method|
-          soap.tag! tag_name, 'xsi:type' => "ns1:PaymentMethod" do
+          soap.tag! tag_name, 'xsi:type' => 'ns1:PaymentMethod' do
             build_tag soap, :integer, 'MethodID', payment_method[:method_id]
             build_tag soap, :string, 'MethodType', payment_method[:type]
             build_tag soap, :string, 'MethodName', payment_method[:name]
@@ -1368,7 +1368,7 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_customer_transaction(soap, options)
-        soap.Parameters 'xsi:type' => "ns1:CustomerTransactionRequest" do
+        soap.Parameters 'xsi:type' => 'ns1:CustomerTransactionRequest' do
           build_transaction_detail soap, options
           CUSTOMER_TRANSACTION_REQUEST_OPTIONS.each do |k,v|
             build_tag soap, v[0], v[1], options[k]
@@ -1381,7 +1381,7 @@ module ActiveMerchant #:nodoc:
       # Transaction Helpers ===========================================
 
       def build_transaction_request_object(soap, options, name='Params')
-        soap.tag! name, 'xsi:type' => "ns1:TransactionRequestObject" do
+        soap.tag! name, 'xsi:type' => 'ns1:TransactionRequestObject' do
           TRANSACTION_REQUEST_OBJECT_OPTIONS.each do |k,v|
             build_tag soap, v[0], v[1], options[k]
           end
@@ -1404,7 +1404,7 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_transaction_detail(soap, options)
-        soap.Details 'xsi:type' => "ns1:TransactionDetail" do
+        soap.Details 'xsi:type' => 'ns1:TransactionDetail' do
           TRANSACTION_DETAIL_OPTIONS.each do |k,v|
             build_tag soap, v[0], v[1], options[k]
           end
@@ -1415,7 +1415,7 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_credit_card_data(soap, options)
-        soap.CreditCardData 'xsi:type' => "ns1:CreditCardData" do
+        soap.CreditCardData 'xsi:type' => 'ns1:CreditCardData' do
           build_tag soap, :string, 'CardNumber', options[:payment_method].number
           build_tag soap, :string, 'CardExpiration', build_card_expiration(options)
           if options[:billing_address]
@@ -1439,7 +1439,7 @@ module ActiveMerchant #:nodoc:
       end
 
       def build_check_data(soap, options)
-        soap.CheckData 'xsi:type' => "ns1:CheckData" do
+        soap.CheckData 'xsi:type' => 'ns1:CheckData' do
           build_tag soap, :integer, 'CheckNumber', options[:payment_method].number
           build_tag soap, :string, 'Account', options[:payment_method].account_number
           build_tag soap, :string, 'Routing', options[:payment_method].routing_number
@@ -1452,10 +1452,10 @@ module ActiveMerchant #:nodoc:
 
       def build_recurring_billing(soap, options)
         if options[:recurring]
-          soap.RecurringBilling 'xsi:type' => "ns1:RecurringBilling" do
+          soap.RecurringBilling 'xsi:type' => 'ns1:RecurringBilling' do
             build_tag soap, :double, 'Amount', amount(options[:recurring][:amount])
-            build_tag soap, :string, 'Next', options[:recurring][:next].strftime("%Y-%m-%d") if options[:recurring][:next]
-            build_tag soap, :string, 'Expire', options[:recurring][:expire].strftime("%Y-%m-%d") if options[:recurring][:expire]
+            build_tag soap, :string, 'Next', options[:recurring][:next].strftime('%Y-%m-%d') if options[:recurring][:next]
+            build_tag soap, :string, 'Expire', options[:recurring][:expire].strftime('%Y-%m-%d') if options[:recurring][:expire]
             RECURRING_BILLING_OPTIONS.each do |k,v|
               build_tag soap, v[0], v[1], options[:recurring][k]
             end
@@ -1478,7 +1478,7 @@ module ActiveMerchant #:nodoc:
           if options[:billing_address][:name]
             options[:billing_address][:first_name], options[:billing_address][:last_name] = split_names(options[:billing_address][:name])
           end
-          soap.BillingAddress 'xsi:type' => "ns1:Address" do
+          soap.BillingAddress 'xsi:type' => 'ns1:Address' do
             ADDRESS_OPTIONS.each do |k,v|
               build_tag soap, v[0], v[1], options[:billing_address][k]
             end
@@ -1491,7 +1491,7 @@ module ActiveMerchant #:nodoc:
           if options[:shipping_address][:name]
             options[:shipping_address][:first_name], options[:shipping_address][:last_name] = split_names(options[:shipping_address][:name])
           end
-          soap.ShippingAddress 'xsi:type' => "ns1:Address" do
+          soap.ShippingAddress 'xsi:type' => 'ns1:Address' do
             ADDRESS_OPTIONS.each do |k,v|
               build_tag soap, v[0], v[1], options[:shipping_address][k]
             end
@@ -1526,7 +1526,7 @@ module ActiveMerchant #:nodoc:
         url = test? ? test_url : live_url
 
         begin
-          soap = ssl_post(url, request, "Content-Type" => "text/xml")
+          soap = ssl_post(url, request, 'Content-Type' => 'text/xml')
         rescue ActiveMerchant::ResponseError => error
           soap = error.response.body
         end
@@ -1558,7 +1558,7 @@ module ActiveMerchant #:nodoc:
 
       def parse(action, soap)
         xml = REXML::Document.new(soap)
-        root = REXML::XPath.first(xml, "//SOAP-ENV:Body")
+        root = REXML::XPath.first(xml, '//SOAP-ENV:Body')
         response = root ? parse_element(root[0]) : { :response => soap }
 
         success, message, authorization, avs, cvv = false, FAILURE_MESSAGE, nil, nil, nil