Merge pull request #214 from TransbankDevelopers/feat/add-token-valid…

…ation-web-pay Feat/add token validation web pay
TransbankDevelopers · Jul 1, 2021 · cd5aa51 · cd5aa51
2 parents 8201fa4 + 4c292e4
commit cd5aa51
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/src/Webpay/WebpayPlus/MallTransaction.php b/src/Webpay/WebpayPlus/MallTransaction.php
@@ -77,6 +77,13 @@ public function create($buyOrder, $sessionId, $returnUrl, $details)
      */
     public function commit($token)
     {
+        if (!is_string($token)) {
+            throw new InvalidArgumentException('Token parameter given is not string.');
+        }
+        if (!isset($token) || trim($token) === '') {
+            throw new InvalidArgumentException('Token parameter given is empty.');
+        }
+
         try {
             $response = $this->sendRequest(
                 'PUT',

diff --git a/src/Webpay/WebpayPlus/Transaction.php b/src/Webpay/WebpayPlus/Transaction.php
@@ -70,6 +70,13 @@ public function create($buyOrder, $sessionId, $amount, $returnUrl)
      */
     public function commit($token)
     {
+        if (!is_string($token)) {
+            throw new InvalidArgumentException('Token parameter given is not string.');
+        }
+        if (!isset($token) || trim($token) === '') {
+            throw new InvalidArgumentException('Token parameter given is empty.');
+        }
+
         try {
             $response = $this->sendRequest(
                 'PUT',