@@ -4,13 +4,19 @@ module ActiveMerchant #:nodoc: # of necessary attributes such as checkholder's name, routing and account numbers, but it is # not backed by any database. # - # You may use Check in place of CreditCard with any gateway that supports it. Currently, only - # +BrainTreeGateway+ supports the Check object. + # You may use Check in place of CreditCard with any gateway that supports it. + # Currently the following gateways support the check object. + # +BrainTreeGateway+ + # +Authorize.net (AIM)+ + class Check include Validateable attr_accessor :first_name, :last_name, :routing_number, :account_number, :account_holder_type, :account_type, :number + # Required for eCheck.net + attr_accessor :account_name, :bank_name, :bank_account_name, :echeck_type + # Used for Canadian bank accounts attr_accessor :institution_number, :transit_number @@ -28,10 +34,12 @@ module ActiveMerchant #:nodoc: end def validate - [:name, :routing_number, :account_number].each do |attr| + [:name, :routing_number, :account_number, :account_name, :echeck_type].each do |attr| errors.add(attr, "cannot be empty") if self.send(attr).blank? end + errors.add(:echeck_type, "is invalid") unless valid_echeck_type? + errors.add(:routing_number, "is invalid") unless valid_routing_number? errors.add(:account_holder_type, "must be personal or business") if @@ -45,6 +53,10 @@ module ActiveMerchant #:nodoc: 'check' end + def valid_echeck_type? + AuthorizeNetCimGateway::ECHECK_TYPES.values.include?(echeck_type) + end + # Routing numbers may be validated by calculating a checksum and dividing it by 10. The # formula is: # (3(d1 + d4 + d7) + 7(d2 + d5 + d8) + 1(d3 + d6 + d9))mod 10 = 0 lib/active_merchant/billing/check.rb @@ -226,6 +226,20 @@ module ActiveMerchant #:nodoc: request = build_recurring_request(:cancel, :subscription_id => subscription_id) recurring_commit(:cancel, request) end + + # Perform an eCheck.net purchase, which is essentially an authorization and capture in a single operation. + # + # ==== Parameters + # + # * <tt>money</tt> -- The amount to be purchased. Either an Integer value in cents or a Money object. + # * <tt>check</tt> -- The Check object for the transaction. + # * <tt>options</tt> -- A hash of optional parameters. + def echeck(money, check, options = {}) + post = {:method => "ECHECK"} + add_check(post, check) + post.merge!(options) + commit("AUTH_CAPTURE", money, post) + end private @@ -295,7 +309,17 @@ module ActiveMerchant #:nodoc: request = post.merge(parameters).collect { |key, value| "x_#{key}=#{CGI.escape(value.to_s)}" }.join("&") request end - + + def add_check(post, check) + post[:bank_aba_code] = check.routing_number + post[:bank_acct_num] = check.account_number + post[:bank_acct_type] = check.account_type + post[:bank_name] = check.bank_name + post[:bank_acct_name] = check.bank_account_name + post[:echeck_type] = check.echeck_type + post[:bank_check_number] = check.number + end + def add_invoice(post, options) post[:invoice_num] = options[:order_id] post[:description] = options[:description] @@ -615,7 +639,7 @@ module ActiveMerchant #:nodoc: def recurring_commit(action, request) url = test? ? arb_test_url : arb_live_url - xml = ssl_post(url, request, "Content-Type" => "text/xml") + xml = ssl_post(url, request, "Content-Type" => "application/x-www-form-urlencoded") response = recurring_parse(action, xml) @@ -651,7 +675,9 @@ module ActiveMerchant #:nodoc: end end end - + + # eCheck.net ------ + AuthorizedNetGateway = AuthorizeNetGateway end end lib/active_merchant/billing/gateways/authorize_net.rb @@ -70,8 +70,12 @@ module ActiveMerchant #:nodoc: } ECHECK_TYPES = { + :arc => 'ARC', + :boc => 'BOC', :ccd => 'CCD', - :ppd => 'PPD' + :ppd => 'PPD', + :tel => 'TEL', + :web => 'WEB' } self.homepage_url = 'http://www.authorize.net/' lib/active_merchant/billing/gateways/authorize_net_cim.rb @@ -32,7 +32,9 @@ class CheckTest < Test::Unit::TestCase :routing_number => VALID_ABA, :account_number => ACCOUNT_NUMBER, :account_holder_type => 'personal', - :account_type => 'checking') + :account_type => 'checking', + :account_name => 'Rick James', + :echeck_type => 'PPD') assert c.valid? end @@ -85,4 +87,30 @@ class CheckTest < Test::Unit::TestCase c.valid? assert !c.errors.on(:account_type) end + + def test_account_name + c = Check.new + c.account_name = 'Rick James' + c.valid? + assert !c.errors.on(:account_name) + c.account_name = nil + c.valid? + assert c.errors.on(:account_name) + c.account_name = "" + c.valid? + assert c.errors.on(:account_name) + end + + def test_echeck_type + c = Check.new + AuthorizeNetCimGateway::ECHECK_TYPES.map do |name, value| + c.echeck_type = value + c.valid? + assert !c.errors.on(:echeck_type) + end + c.echeck_type = 'moo' + c.valid? + assert c.errors.on(:echeck_type) + assert_equal c.errors.on(:echeck_type), "is invalid" + end end test/unit/check_test.rb @@ -6,7 +6,7 @@ class AuthorizeNetTest < Test::Unit::TestCase :login => 'X', :password => 'Y' ) - @amount = 100 + @amount = 0 @credit_card = credit_card @subscription_id = '100748' end @@ -211,6 +211,14 @@ class AuthorizeNetTest < Test::Unit::TestCase assert_equal '2013-11', @gateway.send(:arb_expdate, credit_card('4111111111111111', :month => "11", :year => "2013")) end + def test_echeck + check = Check.new(:routing_number => "111000025", :account_number => "123456789012") + response = @gateway.send(:echeck, 100, check) + assert_instance_of Response, response + assert response.success? + assert response.test? + end + private def post_data_fixture 'x_encap_char=%24&x_card_num=4242424242424242&x_exp_date=0806&x_card_code=123&x_type=AUTH_ONLY&x_first_name=Longbob&x_version=3.1&x_login=X&x_last_name=Longsen&x_tran_key=Y&x_relay_response=FALSE&x_delim_data=TRUE&x_delim_char=%2C&x_amount=1.01' test/unit/gateways/authorize_net_test.rb a3d18bd755595ba78b897237a116bafc464bed55 Tim Matheson me@tim-mathesons-macbook.local http://github.com/timmatheson/active_merchant/commit/21b1971b76c7fc233030fde50f17d5cedbe2599f 21b1971b76c7fc233030fde50f17d5cedbe2599f 2009-10-16T09:43:53-07:00 2009-10-16T09:43:53-07:00 adding eCheck.net support to Authorize.net 61066e7aceece8dad46c5ef25ab65b67744f2a44 Tim Matheson me@tim-mathesons-macbook.local