fixing the sequence related implementation

x/cwerrors/genesis.go

@@ -13,9 +13,6 @@ func InitGenesis(ctx sdk.Context, k keeper.Keeper, genState types.GenesisState)
 	if err := k.Params.Set(ctx, params); err != nil {
 		panic(err)
 	}
-	if err := k.ErrorID.Set(ctx, 0); err != nil {
-		panic(err)
-	}
 }
 
 // ExportGenesis exports the module genesis for the current block.

x/cwerrors/keeper/sudo_errors.go

@@ -35,7 +35,7 @@ func (k Keeper) SetError(ctx sdk.Context, sudoErr types.SudoError) error {
 // StoreErrorInState stores the error in the state and queues it for deletion after a certain block height
 func (k Keeper) StoreErrorInState(ctx sdk.Context, contractAddr sdk.AccAddress, sudoErr types.SudoError) error {
 	// just a unique identifier for the error
-	errorID, err := k.getNextErrorID(ctx)
+	errorID, err := k.ErrorID.Next(ctx)
 	if err != nil {
 		return err
 	}
@@ -70,7 +70,7 @@ func (k Keeper) StoreErrorInState(ctx sdk.Context, contractAddr sdk.AccAddress,
 }
 
 func (k Keeper) storeErrorCallback(ctx sdk.Context, sudoErr types.SudoError) error {
-	errorID, err := k.getNextErrorID(ctx)
+	errorID, err := k.ErrorID.Next(ctx)
 	if err != nil {
 		return err
 	}
@@ -79,17 +79,6 @@ func (k Keeper) storeErrorCallback(ctx sdk.Context, sudoErr types.SudoError) err
 	return nil
 }
 
-func (k Keeper) getNextErrorID(ctx sdk.Context) (uint64, error) {
-	errorID, err := k.ErrorID.Next(ctx)
-	if err != nil {
-		return 0, err
-	}
-	if err = k.ErrorID.Set(ctx, errorID); err != nil {
-		return 0, err
-	}
-	return errorID, nil
-}
-
 // GetErrosByContractAddress returns all errors (in state) for a given contract address
 func (k Keeper) GetErrorsByContractAddress(ctx sdk.Context, contractAddress []byte) (sudoErrs []types.SudoError, err error) {
 	rng := collections.NewPrefixedPairRange[[]byte, uint64](contractAddress)