resolve comments

swagger/static/swagger.yaml

@@ -3675,7 +3675,10 @@ paths:
                 properties:
                   operator:
                     type: string
-                    title: operator define the account address of object updater
+                    description: >-
+                      operator defines the account address of the operator,
+                      either the object owner or the updater with granted
+                      permission.
                   id:
                     type: string
                     title: id defines the object id
@@ -35703,7 +35706,9 @@ definitions:
         properties:
           operator:
             type: string
-            title: operator define the account address of object updater
+            description: >-
+              operator defines the account address of the operator, either the
+              object owner or the updater with granted permission.
           id:
             type: string
             title: id defines the object id
@@ -36886,7 +36891,9 @@ definitions:
     properties:
       operator:
         type: string
-        title: operator define the account address of object updater
+        description: >-
+          operator defines the account address of the operator, either the
+          object owner or the updater with granted permission.
       id:
         type: string
         title: id defines the object id

x/storage/keeper/keeper.go

@@ -738,7 +738,9 @@ func (k Keeper) GetShadowObjectInfo(ctx sdk.Context, bucketName, objectName stri
 		return nil, false
 	}
 
-	return k.GetShadowObjectInfoById(ctx, k.objectSeq.DecodeSequence(bz))
+	var objectInfo types.ShadowObjectInfo
+	k.cdc.MustUnmarshal(bz, &objectInfo)
+	return &objectInfo, true
 }
 
 func (k Keeper) MustGetShadowObjectInfo(ctx sdk.Context, bucketName, objectName string) *types.ShadowObjectInfo {
@@ -749,19 +751,6 @@ func (k Keeper) MustGetShadowObjectInfo(ctx sdk.Context, bucketName, objectName
 	return shadowObjectInfo
 }
 
-func (k Keeper) GetShadowObjectInfoById(ctx sdk.Context, objectId sdkmath.Uint) (*types.ShadowObjectInfo, bool) {
-	store := ctx.KVStore(k.storeKey)
-
-	bz := store.Get(types.GetShadowObjectByIDKey(objectId))
-	if bz == nil {
-		return nil, false
-	}
-
-	var objectInfo types.ShadowObjectInfo
-	k.cdc.MustUnmarshal(bz, &objectInfo)
-	return &objectInfo, true
-}
-
 type SealObjectOptions struct {
 	GlobalVirtualGroupId     uint32
 	SecondarySpBlsSignatures []byte
@@ -819,7 +808,6 @@ func (k Keeper) SealObject(
 		objectInfo.IsUpdating = false
 
 		store.Delete(types.GetShadowObjectKey(bucketInfo.BucketName, objectName))
-		store.Delete(types.GetShadowObjectByIDKey(shadowObjectInfo.Id))
 	} else if objectInfo.ObjectStatus != types.OBJECT_STATUS_CREATED {
 		return types.ErrObjectAlreadySealed
 	}
@@ -980,7 +968,6 @@ func (k Keeper) DeleteObject(
 			return err
 		}
 	}
-	_ = k.MustGetPrimarySPForBucket(ctx, bucketInfo)
 	internalBucketInfo := k.MustGetInternalBucketInfo(ctx, bucketInfo.Id)
 
 	err := k.UnChargeObjectStoreFee(ctx, bucketInfo, internalBucketInfo, objectInfo)
@@ -2498,9 +2485,7 @@ func (k Keeper) UpdateObjectContent(
 			UpdatedAt:   ctx.BlockTime().Unix(),
 			Version:     nextVersion,
 		}
-		store.Set(types.GetShadowObjectKey(bucketName, objectName), k.objectSeq.EncodeSequence(objectInfo.Id))
-		store.Set(types.GetShadowObjectByIDKey(shadowObjectInfo.Id), k.cdc.MustMarshal(shadowObjectInfo))
-
+		store.Set(types.GetShadowObjectKey(bucketName, objectName), k.cdc.MustMarshal(shadowObjectInfo))
 		err = k.LockShadowObjectStoreFee(ctx, bucketInfo, shadowObjectInfo, objectName)
 		if err != nil {
 			return err
@@ -2533,7 +2518,6 @@ func (k Keeper) UnlockShadowObjectFeeAndDeleteShadowObjectInfo(ctx sdk.Context,
 	}
 	store := ctx.KVStore(k.storeKey)
 	store.Delete(types.GetShadowObjectKey(bucketInfo.BucketName, objectName))
-	store.Delete(types.GetShadowObjectByIDKey(shadowObjectInfo.Id))
 	return
 }
 

x/storage/types/keys.go

@@ -44,8 +44,6 @@ var (
 	ObjectByIDPrefix = []byte{0x22}
 	GroupByIDPrefix  = []byte{0x23}
 
-	ShadowObjectByIDPrefix = []byte{0x24}
-
 	BucketSequencePrefix = []byte{0x31}
 	ObjectSequencePrefix = []byte{0x32}
 	GroupSequencePrefix  = []byte{0x33}
@@ -112,12 +110,6 @@ func GetObjectByIDKey(objectId math.Uint) []byte {
 	return append(ObjectByIDPrefix, seq.EncodeSequence(objectId)...)
 }
 
-// GetShadowObjectByIDKey return the shadowObjectId store key
-func GetShadowObjectByIDKey(objectId math.Uint) []byte {
-	var seq sequence.Sequence[math.Uint]
-	return append(ShadowObjectByIDPrefix, seq.EncodeSequence(objectId)...)
-}
-
 // GetGroupByIDKey return the groupId store key
 func GetGroupByIDKey(groupId math.Uint) []byte {
 	var seq sequence.Sequence[math.Uint]