@@ -77,7 +77,14 @@ module ActiveMerchant #:nodoc: def name "#{@first_name} #{@last_name}" end - + + def name=(string) + first_name_match = string.match(/^([a-zA-Z]*) /) + @first_name = first_name_match[1] if first_name_match + last_name_match = string.match(/^[a-zA-Z]* (.*)/) + @last_name = last_name_match[1] if last_name_match + end + def verification_value? !@verification_value.blank? end lib/active_merchant/billing/credit_card.rb @@ -257,7 +257,34 @@ class CreditCardTest < Test::Unit::TestCase c = CreditCard.new(:first_name => 'James', :last_name => 'Herdman') assert c.name? end - + + def test_should_work_with_a_single_name_attribute + c = CreditCard.new(:name => 'John Smith') + assert c.name? + assert_equal "John", c.first_name + assert_equal "Smith", c.last_name + end + + def test_should_work_with_a_single_name_attribute_that_is_double_barrelled + c = CreditCard.new(:name => 'John Hammer Smith') + assert c.name? + assert_equal "John", c.first_name + assert_equal "Hammer Smith", c.last_name + end + + def test_should_work_with_a_single_name_attribute_that_is_double_barrell_hyphenated + c = CreditCard.new(:name => 'John Hammer-Smith') + assert c.name? + assert_equal "John", c.first_name + assert_equal "Hammer-Smith", c.last_name + end + + def test_should_work_with_a_single_blank_name_attribute + c = CreditCard.new(:name => '') + assert_false c.first_name? + assert_false c.last_name? + end + # The following is a regression for a bug that raised an exception when # a new credit card was validated def test_validate_new_card test/unit/credit_card_test.rb b8945dae1de380422b193a3886ef826243145df9 David Rice davidjrice@gmail.com http://github.com/davidjrice/active_merchant/commit/ae7150a18d79c1ac18ada6e9f8530b252c20bcbc ae7150a18d79c1ac18ada6e9f8530b252c20bcbc

2009-08-27T05:34:58-07:00

Adding handling for a single :name attribute to be given to a credit card, it is then split into first name / last name via the magic of regex 730db680879be9075ccc438013e967480ec1eb32 David Rice davidjrice@gmail.com