fix: do not allow non-transferable nfts to transfer with nft-transfer…

… module
OmniFlix · Nov 20, 2023 · 4b4880e · 4b4880e
1 parent c1592b7
commit 4b4880e
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/x/ics721nft/keeper.go b/x/ics721nft/keeper.go
@@ -140,6 +140,14 @@ func (k Keeper) GetNFT(ctx sdk.Context, classID, tokenID string) (nfttransfer.NF
 	if !has {
 		return nil, false
 	}
+	nftMetadata, err := onfttypes.UnmarshalNFTMetadata(k.cdc, _nft.Data.GetValue())
+	if err != nil {
+		return nil, false
+	}
+	if nftMetadata.Transferable == false {
+		k.Logger(ctx).Error("non-transferable nft")
+		return nil, false
+	}
 	metadata, err := k.nb.BuildMetadata(_nft)
 	if err != nil {
 		k.Logger(ctx).Error("encode nft data failed")