Clean code

Author: vuongxuongminh

src/AbstractGateway.php

@@ -24,11 +24,9 @@ abstract class AbstractGateway extends BaseAbstractGateway
      */
     public function initialize(array $parameters = [])
     {
-        parent::initialize(
+        return parent::initialize(
             $this->normalizeParameters($parameters)
         );
-
-        return $this;
     }
 
     /**

src/Message/AbstractPurchaseRequest.php

@@ -22,7 +22,6 @@ abstract class AbstractPurchaseRequest extends AbstractSignatureRequest
     public function initialize(array $parameters = [])
     {
         parent::initialize($parameters);
-        $this->setParameter('version', 2);
         $this->setAgainLink(
             $this->getAgainLink() ?? $this->httpRequest->getUri()
         );
@@ -32,4 +31,32 @@ public function initialize(array $parameters = [])
 
         return $this;
     }
+
+    /**
+     * {@inheritdoc}
+     * @throws \Omnipay\Common\Exception\InvalidRequestException
+     */
+    public function getData(): array
+    {
+        $this->validate('AgainLink', 'Title');
+
+        return parent::getData();
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    protected function getSignatureParameters(): array
+    {
+        $baseParameters = array_fill_keys([
+            'vpc_Version', 'vpc_Currency', 'vpc_Command', 'vpc_AccessCode', 'vpc_Merchant', 'vpc_Locale',
+            'vpc_ReturnURL', 'vpc_MerchTxnRef', 'vpc_OrderInfo', 'vpc_Amount', 'vpc_TicketNo',
+        ], true);
+        $parameters = array_merge($baseParameters, $this->getParameters());
+        $parameters = array_filter(array_keys($parameters), function ($parameter) {
+            return 0 === strpos($parameter, 'vpc_');
+        });
+
+        return $parameters;
+    }
 }

src/Message/AbstractRequest.php

@@ -25,10 +25,8 @@ abstract class AbstractRequest extends BaseAbstractRequest
      */
     public function initialize(array $parameters = [])
     {
-        parent::initialize(
+        return parent::initialize(
             $this->normalizeParameters($parameters)
         );
-
-        return $this;
     }
 }

src/Message/AbstractSignatureRequest.php

@@ -31,14 +31,4 @@ public function getData(): array
 
         return $parameters;
     }
-
-    /**
-     * {@inheritdoc}
-     */
-    protected function getSignatureParameters(): array
-    {
-        return array_filter(array_keys($this->getParameters()), function ($parameter) {
-            return 0 === strpos($parameter, 'vpc_');
-        });
-    }
 }

src/Message/Concerns/PurchaseParameters.php

@@ -101,6 +101,50 @@ public function setClientIp($value)
         return $this->setParameter('vpc_TicketNo', $value);
     }
 
+    /**
+     * Phương thức ánh xạ của [[getClientIp()]].
+     *
+     * @return null|string
+     * @see getAmount
+     */
+    public function getVpcAmount(): ?string
+    {
+        return $this->getAmount();
+    }
+
+    /**
+     * Phương thức ánh xạ của [[setClientIp()]].
+     *
+     * @param  string  $amount
+     * @return $this
+     * @see setAmount
+     */
+    public function setVpcAmount(string $amount)
+    {
+        return $this->setAmount($amount);
+    }
+
+    /**
+     * Trả về số tiền của đơn hàng.
+     *
+     * @return null|string
+     */
+    public function getAmount(): ?string
+    {
+        return $this->getParameter('vpc_Amount');
+    }
+
+    /**
+     * Thiết lập số tiền đơn hàng cần thanh toán.
+     *
+     * @param  string  $value
+     * @return $this
+     */
+    public function setAmount($value)
+    {
+        return $this->setParameter('vpc_Amount', $value);
+    }
+
     /**
      * Trả về thông tin đơn hàng.
      *
@@ -121,4 +165,25 @@ public function setVpcOrderInfo(string $info)
     {
         return $this->setParameter('vpc_OrderInfo', $info);
     }
+
+    /**
+     * Trả về mã đơn hàng cần thanh toán.
+     *
+     * @return null|string
+     */
+    public function getVpcMerchTxnRef(): ?string
+    {
+        return $this->getParameter('vpc_MerchTxnRef');
+    }
+
+    /**
+     * Thiết lập mã đơn hàng cần thanh toán tại hệ thống của bạn.
+     *
+     * @param  string  $info
+     * @return $this
+     */
+    public function setVpcMerchTxnRef(string $ref)
+    {
+        return $this->setParameter('vpc_MerchTxnRef', $ref);
+    }
 }