Merge pull request #74 from twiking/incorrect-data-sent-when-updating…

…-card Incorrect data sent when updating card
thephpleague · May 12, 2017 · e11b40d · e11b40d
2 parents 6b05b89 + e145173
commit e11b40d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 30 deletions.
diff --git a/src/Message/UpdateCardRequest.php b/src/Message/UpdateCardRequest.php
@@ -27,19 +27,11 @@ public function getData()
         $this->validate('cardReference');
         $this->validate('customerReference');
 
-        $data = array();
-        $data['description'] = $this->getDescription();
-
-        if ($this->getSource()) {
-            $data['source'] = $this->getSource();
-        } elseif ($this->getToken()) {
-            $data['source'] = $this->getToken();
-        } elseif ($this->getCard()) {
-            $data['source'] = $this->getCardData();
-            $data['email'] = $this->getCard()->getEmail();
+        if ($this->getCard()) {
+            return $this->getCardData();
+        } else {
+            return array();
         }
-
-        return $data;
     }
 
     public function getEndpoint()

diff --git a/tests/Message/UpdateCardRequestTest.php b/tests/Message/UpdateCardRequestTest.php
@@ -18,30 +18,14 @@ public function testEndpoint()
         $this->assertSame('https://api.stripe.com/v1/customers/cus_1MZSEtqSghKx99/cards/card_15Wg7vIobxWFFmzdvC5fVY67', $this->request->getEndpoint());
     }
 
-    public function testDataWithToken()
-    {
-        $this->request->setToken('xyz');
-        $data = $this->request->getData();
-
-        $this->assertSame('xyz', $data['source']);
-    }
-
-    public function testDataWithSource()
-    {
-        $this->request->setSource('xyz');
-        $data = $this->request->getData();
-
-        $this->assertSame('xyz', $data['source']);
-    }
-
     public function testDataWithCard()
     {
         $card = $this->getValidCard();
         $this->request->setCard($card);
         $data = $this->request->getData();
 
-        $this->assertSame($card['billingAddress1'], $data['source']['address_line1']);
-        $this->assertSame($card['number'], $data['source']['number']);
+        $this->assertSame($card['billingAddress1'], $data['address_line1']);
+        $this->assertSame($card['number'], $data['number']);
     }
 
     public function testSendSuccess()