Merge pull request #864 from ethereum/fixGetFunction

Fix resolving Function: take in account types

Merge pull request #864 from ethereum/fixGetFunction
a555335e · yann300 · GitHub · 3f0c31ec · ae36733c · a555335e
Unverified Commit a555335e authored Jul 27, 2018 by yann300 Committed by GitHub Jul 27, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

txListener.js remix-lib/src/execution/txListener.js +3 -3

No files found.
--- a/remix-lib/src/execution/txListener.js
+++ b/remix-lib/src/execution/txListener.js
@@ -356,10 +356,10 @@ function getConstructorInterface (abi) {
 }

 function getFunction (abi, fnName) {
-  fnName = fnName.split('(')[0]
  for (var i = 0; i < abi.length; i++) {
-    if (abi[i].name === fnName) {
-      return abi[i]
+    var fn = abi[i]
+    if (fnName === fn.name + '(' + fn.inputs.map((value) => { return value.type }).join(',') + ')') {
+      return fn
    }
  }
  return null