Allow calling iface.functions.contractFn() without params, even when …

…contractFn() takes params. In this case, omit result.data. #44
ethers-io · Oct 27, 2017 · e31bdc4 · e31bdc4 · ricmoo · Oct 27, 2017
1 parent aeec6d6
commit e31bdc4
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/contracts/interface.js b/contracts/interface.js
@@ -392,15 +392,17 @@ function Interface(abi) {
 
                         var params = Array.prototype.slice.call(arguments, 0);
 
-                        if (params.length < inputTypes.length) {
+                        if (params.length < inputTypes.length && params.length) {
                             throwError('missing parameter');
                         } else if (params.length > inputTypes.length) {
                             throwError('too many parameters');
                         }
 
-                        signature = utils.keccak256(utils.toUtf8Bytes(signature)).substring(0, 10);
+                        if (params.length || !inputTypes.length) {
+                            signature = utils.keccak256(utils.toUtf8Bytes(signature)).substring(0, 10);
+                            result.data = signature + Interface.encodeParams(inputTypes, params).substring(2);
+                        }
 
-                        result.data = signature + Interface.encodeParams(inputTypes, params).substring(2);
                         if (method.constant) {
                             result.parse = function(data) {
                                 return Interface.decodeParams(