Skip to content

Commit

Permalink
allow empty address arguments when encoding abi params (#1047)
Browse files Browse the repository at this point in the history
* allow empty address arguments

* add test for address encoding
  • Loading branch information
ewingrj authored and frozeman committed Sep 19, 2017
1 parent 57b4c0b commit bacd46e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 1 addition & 1 deletion packages/web3-eth-abi/src/types/address.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ var SolidityType = require('../type');
var SolidityTypeAddress = function () {
this._inputFormatter = function(){
var args = Array.prototype.slice.call(arguments);
args[0] = formatters.inputAddressFormatter(args[0]);
args[0] = (!args[0] || args[0] === '0x0') ? '' : formatters.inputAddressFormatter(args[0]);
return f.formatInputInt.apply(this, args);
};
this._outputFormatter = f.formatOutputAddress;
Expand Down
3 changes: 3 additions & 0 deletions test/eth.abi.encodeParameters.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ var tests = [{
},{
params: [['uint8[]','bytes32'], [['34','434'], '0x324567fff']],
result: '0x0000000000000000000000000000000000000000000000000000000000000040324567fff00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000002000000000000000000000000000000000000000000000000000000000000002200000000000000000000000000000000000000000000000000000000000001b2'
},{
params: [['address','address','address', 'address'], ['0x90f8bf6a479f320ead074411a4b0e7944ea8c9c1','','0x0', null]],
result: '0x00000000000000000000000090f8bf6a479f320ead074411a4b0e7944ea8c9c1000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
}];

describe('encodeParameters', function () {
Expand Down

0 comments on commit bacd46e

Please sign in to comment.