fix

Author: recca0120

src/Message/CompletePurchaseRequest.php

@@ -2,6 +2,7 @@
 
 namespace Omnipay\MyCash\Message;
 
+use Omnipay\Common\Exception\InvalidRequestException;
 use Omnipay\Common\Exception\InvalidResponseException;
 use Omnipay\Common\Message\AbstractRequest;
 use Omnipay\MyCash\Traits\HasMyCash;
@@ -13,15 +14,13 @@ class CompletePurchaseRequest extends AbstractRequest
     /**
      * @return array
      *
-     * @throws InvalidResponseException
+     * @throws InvalidRequestException
      */
     public function getData()
     {
         $data = $this->httpRequest->request->all();
-        $validate = $data['Validate'];
-
-        if ($this->makeHash($data) !== $validate) {
-            throw new InvalidResponseException('Incorrect hash');
+        if (! hash_equals($this->makeHash($data), $data['Validate'])) {
+            throw new InvalidRequestException('Incorrect hash');
         }
 
         return $data;

src/Message/GetPaymentInfoRequest.php

@@ -2,6 +2,7 @@
 
 namespace Omnipay\MyCash\Message;
 
+use Omnipay\Common\Exception\InvalidRequestException;
 use Omnipay\Common\Exception\InvalidResponseException;
 use Omnipay\Common\Message\AbstractRequest;
 use Omnipay\MyCash\Traits\HasMyCash;
@@ -13,14 +14,13 @@ class GetPaymentInfoRequest extends AbstractRequest
     /**
      * @return array
      *
-     * @throws InvalidResponseException
+     * @throws InvalidRequestException
      */
     public function getData()
     {
         $data = $this->httpRequest->request->all();
-
-        if (! hash_equals($this->httpRequest->request->get('Validate'), $this->makeHash($data))) {
-            throw new InvalidResponseException('Incorrect hash');
+        if (! hash_equals($this->makeHash($data), $this->httpRequest->request->get('Validate'))) {
+            throw new InvalidRequestException('Incorrect hash');
         }
 
         return $data;

tests/Message/AcceptNotificationRequestTest.php

@@ -2,7 +2,7 @@
 
 namespace Omnipay\MyCash\Tests\Message;
 
-use Omnipay\Common\Exception\InvalidResponseException;
+use Omnipay\Common\Exception\InvalidRequestException;
 use Omnipay\Common\Message\NotificationInterface;
 use Omnipay\MyCash\Message\AcceptNotificationRequest;
 use Omnipay\MyCash\Message\CompletePurchaseRequest;
@@ -18,7 +18,7 @@ class AcceptNotificationRequestTest extends TestCase
 
     public function testValidateFails(): void
     {
-        $this->expectException(InvalidResponseException::class);
+        $this->expectException(InvalidRequestException::class);
 
         $options = [
             'RtnCode' => '1',

tests/Message/CompletePurchaseRequestTest.php

@@ -2,7 +2,7 @@
 
 namespace Omnipay\MyCash\Tests\Message;
 
-use Omnipay\Common\Exception\InvalidResponseException;
+use Omnipay\Common\Exception\InvalidRequestException;
 use Omnipay\MyCash\Message\CompletePurchaseRequest;
 use Omnipay\Tests\TestCase;
 
@@ -16,7 +16,7 @@ class CompletePurchaseRequestTest extends TestCase
 
     public function testValidateFails(): void
     {
-        $this->expectException(InvalidResponseException::class);
+        $this->expectException(InvalidRequestException::class);
         $this->expectExceptionMessage('Incorrect hash');
 
         $options = [