Merge pull request #287 from irisnet/optimization

apply some comments
irisnet · Mar 10, 2022 · e57d01c · e57d01c
2 parents 67c1090 + f954466
commit e57d01c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/modules/nft/keeper/grpc_query.go b/modules/nft/keeper/grpc_query.go
@@ -65,6 +65,9 @@ func (k Keeper) Owner(c context.Context, request *types.QueryOwnerRequest) (*typ
 		}
 		return nil
 	})
+	if err != nil {
+		return nil, err
+	}
 	for i := 0; i < len(owner.IDCollections); i++ {
 		owner.IDCollections[i].TokenIds = idsMap[owner.IDCollections[i].DenomId]
 	}

diff --git a/modules/nft/keeper/msg_server.go b/modules/nft/keeper/msg_server.go
@@ -72,7 +72,7 @@ func (m msgServer) MintNFT(goCtx context.Context, msg *types.MsgMintNFT) (*types
 	}
 
 	if denom.MintRestricted && denom.Creator != sender.String() {
-		return nil, sdkerrors.Wrapf(sdkerrors.ErrUnauthorized, "%s is not allowed to mint NFT of denom %s", denom.Creator, msg.DenomId)
+		return nil, sdkerrors.Wrapf(sdkerrors.ErrUnauthorized, "%s is not allowed to mint NFT of denom %s", msg.Sender, msg.DenomId)
 	}
 
 	if err := m.Keeper.MintNFT(ctx, msg.DenomId, msg.Id,