Merge pull request #144 from axic/patch/show-web3-encoding-error

Display web3 encoding error

Merge pull request #144 from axic/patch/show-web3-encoding-error
0cafaeef · chriseth · e44ceb83 · fdeae6bd · 0cafaeef
Commit 0cafaeef authored Apr 18, 2016 by chriseth
Show whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

universal-dapp.js assets/js/universal-dapp.js +7 -1

No files found.
--- a/assets/js/universal-dapp.js
+++ b/assets/js/universal-dapp.js
@@ -287,8 +287,14 @@ UniversalDApp.prototype.getCallButton = function(args) {
            return;
        }
        var data = '';
-        if (!isConstructor || funArgs.length > 0)
+        if (!isConstructor || funArgs.length > 0) {
+            try {
                data = args.encode(funArgs);
+            } catch(e) {
+                replaceOutput($result, $('<span/>').text('Error encoding arguments: ' + e));
+                return;
+            }
+        }
        if (data.slice(0, 9) == 'undefined')
            data = data.slice(9);
        if (data.slice(0, 2) == '0x') data = data.slice(2);