Added test cases

Author: vuongxuongminh

src/Message/AbstractRequest.php

@@ -7,15 +7,40 @@
 
 namespace Omnipay\VNPay\Message;
 
+use Omnipay\VNPay\Concerns\Parameters;
+use Omnipay\Common\Message\AbstractRequest;
+use Omnipay\VNPay\Concerns\ParametersNormalization;
+
 /**
  * @author Vuong Minh <vuongxuongminh@gmail.com>
  * @since 1.0.0
  */
 abstract class AbstractSignatureRequest extends AbstractRequest
 {
+    use Parameters;
+    use ParametersNormalization;
     use Concerns\RequestEndpoint;
     use Concerns\RequestSignature;
 
+    /**
+     * {@inheritdoc}
+     */
+    public function initialize(array $parameters = [])
+    {
+        parent::initialize(
+            $this->normalizeParameters($parameters)
+        );
+
+        $this->setVnpIpAddr(
+            $this->getVnpIpAddr() ?? $this->httpRequest->getClientIp()
+        );
+        $this->setVnpCreateDate(
+            $this->getVnpCreateDate() ?? date('Ymdhis')
+        );
+
+        return $this;
+    }
+
     /**
      * {@inheritdoc}
      */
@@ -35,4 +60,133 @@ public function getData(): array
 
         return $parameters;
     }
+
+    /**
+     * Trả về mã đơn hàng cần thực thi tác vụ.
+     * Đây là phương thức ánh xạ của [[getTransactionId()]].
+     *
+     * @return null|string
+     * @see getTransactionId
+     */
+    public function getVnpTxnRef(): ?string
+    {
+        return $this->getTransactionId();
+    }
+
+    /**
+     * Thiết lập mã đơn hàng cần thực thi tác vụ.
+     * Đây là phương thức ánh xạ của [[setTransactionId()]].
+     *
+     * @param  null|string  $ref
+     *
+     * @return $this
+     * @see setTransactionId
+     */
+    public function setVnpTxnRef(?string $ref)
+    {
+        return $this->setTransactionId($ref);
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function getTransactionId(): ?string
+    {
+        return $this->getParameter('vnp_TxnRef');
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function setTransactionId($value)
+    {
+        return $this->setParameter('vnp_TxnRef', $value);
+    }
+
+    /**
+     * Trả về thông tin đơn hàng hay lý do truy vấn đến VNPay.
+     *
+     * @return null|string
+     */
+    public function getVnpOrderInfo(): ?string
+    {
+        return $this->getParameter('vnp_OrderInfo');
+    }
+
+    /**
+     * Thiết lập thông tin đơn hàng hay lý do truy vấn đến VNPay.
+     *
+     * @param  null|string  $info
+     * @return $this
+     */
+    public function setVnpOrderInfo(?string $info)
+    {
+        return $this->setParameter('vnp_OrderInfo', $info);
+    }
+
+    /**
+     * Trả về thời gian khởi tạo truy vấn đến VNPay.
+     *
+     * @return null|string
+     * @see getVnpReturnUrl
+     */
+    public function getVnpCreateDate(): ?string
+    {
+        return $this->getParameter('vnp_CreateDate');
+    }
+
+    /**
+     * Thiết lập thời gian khởi tạo truy vấn đến VNPay.
+     * Mặc định sẽ là thời gian hiện tại.
+     *
+     * @param  null|string  $date
+     * @return $this
+     * @see setReturnUrl
+     */
+    public function setVnpCreateDate(?string $date)
+    {
+        return $this->setParameter('vnp_CreateDate', $date);
+    }
+
+    /**
+     * Trả về ip của khách dùng để thanh toán.
+     * Đây là phương thức ánh xạ của [[getClientIp()]].
+     *
+     * @return null|string
+     * @see getClientIp
+     */
+    public function getVnpIpAddr(): ?string
+    {
+        return $this->getClientIp();
+    }
+
+    /**
+     * Thiết lập ip của khách dùng để thanh toán.
+     * Đây là phương thức ánh xạ của [[setClientIp()]].
+     * Mặc định nếu không thiết lập sẽ là IP của khách.
+     *
+     * @param  null|string  $ip
+     * @return $this
+     * @see setClientIp
+     */
+    public function setVnpIpAddr(?string $ip)
+    {
+        return $this->setClientIp($ip);
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function getClientIp(): ?string
+    {
+        return $this->getParameter('vnp_IpAddr');
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function setClientIp($value)
+    {
+        return $this->setParameter('vnp_IpAddr', $value);
+    }
 }

src/Message/AbstractSignatureRequest.php

@@ -25,18 +25,19 @@ protected function validateSignature(): void
     {
         $data = $this->getData();
 
-        if (! isset($data['vnp_SecureHash'], $data['vnp_SecureHashType'])) {
+        if (! isset($data['vnp_SecureHash'])) {
             throw new InvalidResponseException('Response from VNPay is invalid!');
         }
 
         $dataSignature = array_filter($this->getData(), function ($parameter) {
             return 0 === strpos($parameter, 'vnp_')
-                && ! in_array($parameter, ['vnp_SecureHash', 'vnp_SecureHashType']);
+                && 'vnp_SecureHash' !== $parameter
+                && 'vnp_SecureHashType' !== $parameter;
         }, ARRAY_FILTER_USE_KEY);
 
         $signature = new Signature(
             $this->getRequest()->getVnpHashSecret(),
-            $data['vnp_SecureHashType']
+            $data['vnp_SecureHashType'] ?? 'md5'
         );
 
         if (! $signature->validate($dataSignature, $data['vnp_SecureHash'])) {

src/Message/Concerns/ResponseSignatureValidation.php

@@ -8,12 +8,19 @@
 
 namespace Omnipay\VNPay\Message;
 
+use Omnipay\VNPay\Concerns\Parameters;
+use Omnipay\Common\Message\AbstractRequest;
+use Omnipay\VNPay\Concerns\ParametersNormalization;
+
 /**
  * @author Vuong Minh <vuongxuongminh@gmail.com>
  * @since 1.0.0
  */
 class IncomingRequest extends AbstractRequest
 {
+    use Parameters;
+    use ParametersNormalization;
+
     /**
      * {@inheritdoc}
      */
@@ -41,7 +48,9 @@ public function sendData($data): SignatureResponse
      */
     public function initialize(array $parameters = [])
     {
-        parent::initialize($parameters);
+        parent::initialize(
+            $this->normalizeParameters($parameters)
+        );
 
         foreach ($this->getIncomingParameters() as $parameter => $value) {
             $this->setParameter($parameter, $value);

src/Message/IncomingRequest.php

@@ -50,7 +50,7 @@ public function initialize(array $parameters = [])
      */
     public function sendData($data): PurchaseResponse
     {
-        $query = http_build_query($data, null, '&', PHP_QUERY_RFC3986);
+        $query = http_build_query($data);
         $redirectUrl = $this->getEndpoint().'?'.$query;
 
         return $this->response = new PurchaseResponse($this, $data, $redirectUrl);