Merge pull request #91 from Arul-/patch-2

Fix: method validation in both send and call
web3p · Jun 14, 2018 · d0559b8 · d0559b8
2 parents be7403a + 3d8a567
commit d0559b8
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/Contract.php b/src/Contract.php
@@ -422,8 +422,8 @@ public function send()
             $method = array_splice($arguments, 0, 1)[0];
             $callback = array_pop($arguments);
 
-            if (!is_string($method) && !isset($this->functions[$method])) {
-                throw new InvalidArgumentException('Please make sure the method is existed.');
+            if (!is_string($method) || !isset($this->functions[$method])) {
+                throw new InvalidArgumentException('Please make sure the method exists.');
             }
             $function = $this->functions[$method];
 
@@ -468,8 +468,8 @@ public function call()
             $method = array_splice($arguments, 0, 1)[0];
             $callback = array_pop($arguments);
 
-            if (!is_string($method) && !isset($this->functions[$method])) {
-                throw new InvalidArgumentException('Please make sure the method is existed.');
+            if (!is_string($method) || !isset($this->functions[$method])) {
+                throw new InvalidArgumentException('Please make sure the method exists.');
             }
             $function = $this->functions[$method];
 
@@ -621,4 +621,4 @@ public function getData()
             return $functionData;
         }
     }
-}
+}