Merge pull request #23 from musement/master

add 'cardholderName' to AuthorizeRequest and CreatePaymentMethodRequest

Author: delatbabel

src/Message/AuthorizeRequest.php

@@ -48,6 +48,12 @@ public function getData()
             return ! is_null($value);
         });
 
+        if ($this->getCardholderName()) {
+            $data['creditCard'] = array(
+                'cardholderName' => $this->getCardholderName(),
+            );
+        }
+
         $data += $this->getOptionData();
         $data += $this->getCardData();
         $data['options']['submitForSettlement'] = false;
@@ -67,4 +73,24 @@ public function sendData($data)
 
         return $this->createResponse($response);
     }
+
+    /**
+     * [optional] The cardholder name associated with the credit card. 175 character maximum.
+     * Required for iOS integration because its missing in "tokenizeCard" function there.
+     * See: https://developers.braintreepayments.com/reference/request/transaction/sale/php#credit_card.cardholder_name
+     *
+     * @param $value
+     * @return mixed
+     */
+    public function setCardholderName($value)
+    {
+        $cardholderName = trim($value);
+        $cardholderName = strlen($cardholderName)>0 ? $cardholderName : null;
+        return $this->setParameter('cardholderName', $cardholderName);
+    }
+
+    public function getCardholderName()
+    {
+        return $this->getParameter('cardholderName');
+    }
 }

src/Message/CreatePaymentMethodRequest.php

@@ -17,6 +17,9 @@ public function getData()
             'customerId' => $this->getCustomerId(),
             'paymentMethodNonce' => $this->getToken(),
         );
+        if ($cardholderName = $this->getCardholderName()) {
+            $data['cardholderName'] = $cardholderName;
+        }
         $data += $this->getOptionData();
 
         return $data;
@@ -34,4 +37,24 @@ public function sendData($data)
 
         return $this->createResponse($response);
     }
+
+    /**
+     * [optional] The cardholder name associated with the credit card. 175 character maximum.
+     * Required for iOS integration because its missing in "tokenizeCard" function there.
+     * See: https://developers.braintreepayments.com/reference/request/payment-method/create/php#cardholder_name
+     *
+     * @param $value
+     * @return mixed
+     */
+    public function setCardholderName($value)
+    {
+        $cardholderName = trim($value);
+        $cardholderName = strlen($cardholderName)>0 ? $cardholderName : null;
+        return $this->setParameter('cardholderName', $cardholderName);
+    }
+
+    public function getCardholderName()
+    {
+        return $this->getParameter('cardholderName');
+    }
 }

tests/Message/CreatePaymentMethodRequestTest.php

@@ -6,34 +6,59 @@
 
 class CreatePaymentMethodRequestTest extends TestCase
 {
-    /**
-     * @var CreatePaymentMethodRequest
-     */
-    private $request;
-
-    public function setUp()
+    public function testGetData()
     {
-        $this->request = new CreatePaymentMethodRequest($this->getHttpClient(), $this->getHttpRequest(), \Braintree_Configuration::gateway());
-        $this->request->initialize(
+        $request = $this->createPaymentMethodRequest();
+        $request->initialize(
             array(
                 'customerId' => '4815162342',
                 'token' => 'abc123',
                 'verifyCard' => true,
                 'verificationMerchantAccountId' => '123581321',
             )
         );
+
+        $expectedData = array(
+            'customerId' => '4815162342',
+            'paymentMethodNonce' => 'abc123',
+            'options' => array(
+                'verifyCard' => true,
+                'verificationMerchantAccountId' => '123581321',
+            )
+        );
+        $this->assertSame($expectedData, $request->getData());
     }
 
-    public function testGetData()
+    public function testGetDataWithCardholderName()
     {
+        $request = $this->createPaymentMethodRequest();
+        $request->initialize(
+            array(
+                'customerId' => '4815162342',
+                'token' => 'abc123',
+                'cardholderName' => 'John Yolo',
+                'verifyCard' => true,
+                'verificationMerchantAccountId' => '123581321',
+            )
+        );
+
         $expectedData = array(
             'customerId' => '4815162342',
             'paymentMethodNonce' => 'abc123',
+            'cardholderName' => 'John Yolo',
             'options' => array(
                 'verifyCard' => true,
                 'verificationMerchantAccountId' => '123581321',
             )
         );
-        $this->assertSame($expectedData, $this->request->getData());
+        $this->assertSame($expectedData, $request->getData());
+    }
+
+    /**
+     * @return CreatePaymentMethodRequest
+     */
+    private function createPaymentMethodRequest()
+    {
+        return new CreatePaymentMethodRequest($this->getHttpClient(), $this->getHttpRequest(), \Braintree_Configuration::gateway());
     }
 }