Remove NumTokens query handlers and associated code from NFT query service

- Deleted request and local_request functions for NumTokens.
- Removed HTTP handler registration for NumTokens in both server and client.
- Cleaned up related patterns and forwarding functions.

Author: angelorc

proto/bitsong/nft/v1beta1/query.proto

@@ -22,11 +22,6 @@ service Query {
     option (google.api.http).get = "/bitsong/nft/v1beta1/collections/{collection}/{token_id}/owner";
   }
 
-  rpc NumTokens(QueryNumTokensRequest) returns (QueryNumTokensResponse) {
-    option (cosmos.query.v1.module_query_safe) = true;
-    option (google.api.http).get = "/bitsong/nft/v1beta1/collections/{collection}/num_tokens";
-  }
-
   rpc NftInfo(QueryNftInfoRequest) returns (QueryNftInfoResponse) {
     option (cosmos.query.v1.module_query_safe) = true;
     option (google.api.http).get = "/bitsong/nft/v1beta1/collections/{collection}/{token_id}";
@@ -41,6 +36,12 @@ service Query {
     option (cosmos.query.v1.module_query_safe) = true;
     option (google.api.http).get = "/bitsong/nft/v1beta1/nfts_by_owner/{owner}";
   }
+
+  // Edition returns a specific edition of an nft
+  // OwnerOfEdition returns the owner of a specific edition
+  // NumEditions returns the total number of editions
+  // NftEditions returns all editions of a specific nft
+  // AllNftEditionsByOwner returns all nft editions owned by the owner
 }
 
 message QueryCollectionRequest {
@@ -72,20 +73,6 @@ message QueryOwnerOfResponse {
   string owner = 1;
 }
 
-message QueryNumTokensRequest {
-  option (gogoproto.equal)           = false;
-  option (gogoproto.goproto_getters) = false;
-
-  string collection = 1;
-}
-
-message QueryNumTokensResponse {
-  option (gogoproto.equal)           = false;
-  option (gogoproto.goproto_getters) = false;
-
-  uint64 count = 1;
-}
-
 message QueryNftInfoRequest {
   option (gogoproto.equal)           = false;
   option (gogoproto.goproto_getters) = false;

x/nft/keeper/grpc_query.go

@@ -65,23 +65,6 @@ func (k Keeper) OwnerOf(ctx context.Context, req *types.QueryOwnerOfRequest) (*t
 	}, nil
 }
 
-func (k Keeper) NumTokens(ctx context.Context, req *types.QueryNumTokensRequest) (*types.QueryNumTokensResponse, error) {
-	if req == nil {
-		return nil, status.Error(codes.InvalidArgument, "empty request")
-	}
-	if req.Collection == "" {
-		return nil, status.Error(codes.InvalidArgument, "collection cannot be empty")
-	}
-
-	sdkCtx := sdk.UnwrapSDKContext(ctx)
-
-	supply := k.GetSupply(sdkCtx, req.Collection)
-
-	return &types.QueryNumTokensResponse{
-		Count: supply.Uint64(),
-	}, nil
-}
-
 func (k Keeper) NftInfo(ctx context.Context, req *types.QueryNftInfoRequest) (*types.QueryNftInfoResponse, error) {
 	if req == nil {
 		return nil, status.Error(codes.InvalidArgument, "empty request")
@@ -133,6 +116,36 @@ func (k Keeper) Nfts(ctx context.Context, req *types.QueryNftsRequest) (*types.Q
 	}, nil
 }
 
+func (k Keeper) GetNftsByOwner(ctx context.Context, owner sdk.AccAddress, pagination *query.PageRequest) ([]types.Nft, *query.PageResponse, error) {
+	var nfts []types.Nft
+
+	sdkCtx := sdk.UnwrapSDKContext(ctx)
+
+	store := prefix.NewStore(
+		sdkCtx.KVStore(k.storeKey),
+		append(types.NFTsByOwnerPrefix, address.MustLengthPrefix(owner)...),
+	)
+
+	pageRes, err := query.Paginate(store, pagination, func(key []byte, value []byte) error {
+		denom, tokenId := types.MustSplitNftLengthPrefixedKey(key)
+
+		nft, err := k.NFTs.Get(ctx, collections.Join(string(denom), string(tokenId)))
+		if err != nil {
+			return err
+		}
+
+		nfts = append(nfts, nft)
+
+		return nil
+	})
+
+	if err != nil {
+		return nil, nil, err
+	}
+
+	return nfts, pageRes, nil
+}
+
 func (k Keeper) AllNftsByOwner(ctx context.Context, req *types.QueryAllNftsByOwnerRequest) (*types.QueryAllNftsByOwnerResponse, error) {
 	if req == nil {
 		return nil, status.Error(codes.InvalidArgument, "empty request")
@@ -169,16 +182,6 @@ func (k Keeper) AllNftsByOwner(ctx context.Context, req *types.QueryAllNftsByOwn
 		return nil, status.Error(codes.Internal, err.Error())
 	}
 
-	/*iter, err := k.NFTs.Indexes.Owner.MatchExact(ctx, owner)
-	if err != nil {
-		return nil, status.Error(codes.Internal, err.Error())
-	}
-	defer iter.Close()
-
-	nfts, err := indexes.CollectValues(ctx, k.NFTs, iter)
-	if err != nil {
-		return nil, status.Error(codes.Internal, err.Error())
-	}*/
 	return &types.QueryAllNftsByOwnerResponse{
 		Nfts:       nfts,
 		Pagination: pageRes,

x/nft/keeper/grpc_query_test.go

@@ -59,58 +59,6 @@ func (suite *KeeperTestSuite) TestQueryOwnerOf() {
 	suite.Equal(owner1.String(), res.Owner)
 }
 
-func (suite *KeeperTestSuite) TestQueryNumTokens() {
-	collectionDenom, err := suite.keeper.CreateCollection(
-		suite.ctx,
-		creator1,
-		minter1,
-		testCollection1.Symbol,
-		testCollection1.Name,
-		testCollection1.Description,
-		testCollection1.Uri,
-	)
-	suite.NoError(err)
-
-	supply := suite.keeper.GetSupply(suite.ctx, collectionDenom)
-	suite.Equal(uint64(0), supply.Uint64())
-
-	err = suite.keeper.MintNFT(
-		suite.ctx,
-		minter1,
-		owner1,
-		collectionDenom,
-		testNft1.TokenId,
-		testNft1.Name,
-		testNft1.Description,
-		testNft1.Uri,
-	)
-	suite.NoError(err)
-
-	supply = suite.keeper.GetSupply(suite.ctx, collectionDenom)
-	suite.Equal(uint64(1), supply.Uint64())
-
-	err = suite.keeper.MintNFT(
-		suite.ctx,
-		minter1,
-		owner1,
-		collectionDenom,
-		testNft2.TokenId,
-		testNft2.Name,
-		testNft2.Description,
-		testNft2.Uri,
-	)
-	suite.NoError(err)
-
-	supply = suite.keeper.GetSupply(suite.ctx, collectionDenom)
-	suite.Equal(uint64(2), supply.Uint64())
-
-	res, err := suite.keeper.NumTokens(suite.ctx, &types.QueryNumTokensRequest{
-		Collection: collectionDenom,
-	})
-	suite.NoError(err)
-	suite.Equal(uint64(2), res.Count)
-}
-
 func (suite *KeeperTestSuite) TestQueryNftInfo() {
 	collectionDenom, err := suite.keeper.CreateCollection(
 		suite.ctx,