Difference in Fee structure in Result and auth.StdTx

[AdityaSripal]

Summary of Bug

auth.StdTx allows users to pass in sdk.Coins into StdFee. However the sdk.Result struct seems to expect using a single coin for the fee.

// Tx fee amount and denom.
FeeAmount int64
FeeDenom  string

Suggested Fix

Allow result to store multicoin fee

// Tx fee amount and denom.
FeeAmount []int64
FeeDenom  []string

Construct list such that amount list indices are correctly matched with denom list indices. May also want to sort the denoms in the same way that sdk.Coins does the sort.

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned

[ValarDragon]

Why should it expect:

FeeAmount []int64
FeeDenom  []string

and not

FeeCoins sdk.Coins

[AdityaSripal]

Yea, that works better. Just followed the initial model.

[AdityaSripal]

Dependent on: tendermint/tendermint#1997

[AdityaSripal]

Should be fixed in tendermint/tendermint#1861