From a2f5022b5fe234fe31534bbd9af8079a1a2a4703 Mon Sep 17 00:00:00 2001 From: Vritra4 Date: Mon, 30 Oct 2023 12:07:44 +0900 Subject: [PATCH] prev commit can't get rid of unncessary comma --- x/gov/keeper/proposal.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/x/gov/keeper/proposal.go b/x/gov/keeper/proposal.go index 8ae20781..d5abbcb2 100644 --- a/x/gov/keeper/proposal.go +++ b/x/gov/keeper/proposal.go @@ -31,13 +31,13 @@ func (keeper Keeper) SubmitProposal(ctx sdk.Context, messages []sdk.Msg, metadat return v1.Proposal{}, err } - // Will hold a comma-separated string of all Msg type URLs. - msgsStr := "" + // Will hold a string slice of all Msg type URLs. + msgs := []string{} // Loop through all messages and confirm that each has a handler and the gov module account // as the only signer for _, msg := range messages { - msgsStr = strings.Join([]string{msgsStr, sdk.MsgTypeURL(msg)}, ",") + msgs = append(msgs, sdk.MsgTypeURL(msg)) // perform a basic validation of the message if err := msg.ValidateBasic(); err != nil { @@ -100,7 +100,7 @@ func (keeper Keeper) SubmitProposal(ctx sdk.Context, messages []sdk.Msg, metadat sdk.NewEvent( types.EventTypeSubmitProposal, sdk.NewAttribute(types.AttributeKeyProposalID, fmt.Sprintf("%d", proposalID)), - sdk.NewAttribute(types.AttributeKeyProposalMessages, msgsStr), + sdk.NewAttribute(types.AttributeKeyProposalMessages, strings.Join(msgs, ",")), ), )