Fix data before send to Authorize

AndrewNovikof · Apr 16, 2018 · 17311ca · 17311ca
1 parent 57acdf2
commit 17311ca
Show file tree

Hide file tree

Showing 4 changed files with 49 additions and 6 deletions.
diff --git a/src/Message/AbstractRequest.php b/src/Message/AbstractRequest.php
@@ -2,6 +2,7 @@
 
 namespace Omnipay\Sberbank\Message;
 
+use Omnipay\Common\Exception\InvalidRequestException;
 use Omnipay\Common\Exception\RuntimeException;
 use Omnipay\Common\Message\AbstractRequest as BaseAbstractRequest;
 
@@ -165,11 +166,11 @@ public function sendData($data)
             ], $data)
         );
 
-        $responseClassName = str_replace('Request', 'Response', get_class($this));
+        $responseClassName = str_replace('Request', 'Response', \get_class($this));
         $reflection = new \ReflectionClass($responseClassName);
         if (!$reflection->isInstantiable()) {
             throw new RuntimeException(
-                'Class ' . str_replace('Request', 'Response', get_class($this)) . ' not found'
+                'Class ' . str_replace('Request', 'Response', \get_class($this)) . ' not found'
             );
         }
 

diff --git a/src/Message/AuthorizeRequest.php b/src/Message/AuthorizeRequest.php
@@ -2,6 +2,8 @@
 
 namespace Omnipay\Sberbank\Message;
 
+use Omnipay\Common\Exception\RuntimeException;
+
 /**
  * Class AuthorizeRequest
  * @package Omnipay\Sberbank\Message
@@ -226,4 +228,37 @@ public function setBindingId($value)
     {
         return $this->setParameter('bindingId', $value);
     }
+
+    /**
+     * @param mixed $data
+     * @return object|\Omnipay\Common\Message\ResponseInterface
+     * @throws \ReflectionException
+     * @throws \Omnipay\Common\Exception\InvalidRequestException
+     */
+    public function sendData($data)
+    {
+        $url = $this->getEndPoint() . $this->getMethod();
+        $this->validate('userName', 'password');
+        $data['currency'] = $this->getCurrencyNumeric();
+        $data['amount'] = $this->getAmountInteger();
+        $httpResponse = $this->httpClient->send(
+            $this->getHttpMethod(),
+            $url,
+            $this->getHeaders(),
+            array_merge([
+                'userName' => $this->getUserName(),
+                'password' => $this->getPassword()
+            ], $data)
+        );
+
+        $responseClassName = str_replace('Request', 'Response', \get_class($this));
+        $reflection = new \ReflectionClass($responseClassName);
+        if (!$reflection->isInstantiable()) {
+            throw new RuntimeException(
+                'Class ' . str_replace('Request', 'Response', \get_class($this)) . ' not found'
+            );
+        }
+
+        return $reflection->newInstance($this, json_decode($httpResponse->getBody(true), true));
+    }
 }
diff --git a/src/Message/BindCardRequest.php b/src/Message/BindCardRequest.php
@@ -24,11 +24,9 @@ public function getData()
     {
         $this->validate('bindingId');
 
-        $data = [
+        return [
             'bindingId' => $this->getBindingId()
         ];
-
-        return $data;
     }
 
     /**

diff --git a/tests/Message/AuthorizeRequestTest.php b/tests/Message/AuthorizeRequestTest.php
@@ -32,6 +32,13 @@ class AuthorizeRequestTest extends AbstractRequestTest
      */
     protected $orderNumber;
 
+    /**
+     * Currency
+     *
+     * @var string
+     */
+    protected $currency;
+
     /**
      * Sets up the fixture, for example, open a network connection.
      * This method is called before a test is executed.
@@ -41,6 +48,7 @@ public function setUp()
         $this->amount = mt_rand(1, 100);
         $this->returnUrl = 'https://test.com/' . uniqid('', true);
         $this->orderNumber = uniqid('order_number_', true);
+        $this->currency = 'RUB';
 
         parent::setUp();
     }
@@ -77,7 +85,8 @@ protected function getRequestParameters()
         return [
             'orderNumber' => $this->orderNumber,
             'amount' => $this->amount,
-            'returnUrl' => $this->returnUrl
+            'returnUrl' => $this->returnUrl,
+            'currency' => $this->currency
         ];
     }