bugfix/KIP-84 -> release/v0.4.2

types/constants.go

@@ -3,6 +3,6 @@ package types
 const (
 	// we set page iteration limit for safety
 	PageIterationLimit = 512
-	SekaiVersion       = "v0.4.1"
+	SekaiVersion       = "v0.4.2"
 	CosmosVersion      = "v0.47.6"
 )

x/layer2/keeper/msg_server.go

@@ -81,19 +81,20 @@ func (k msgServer) BondDappProposal(goCtx context.Context, msg *types.MsgBondDap
 		return nil, types.ErrInvalidDappBondDenom
 	}
 
+	dapp.TotalBond = dapp.TotalBond.Add(msg.Bond)
+
+	properties := k.keeper.gk.GetNetworkProperties(ctx)
+	if dapp.TotalBond.Amount.GT(sdk.NewInt(int64(properties.MaxDappBond)).Mul(sdk.NewInt(1000_000))) {
+		return nil, types.ErrMaxDappBondReached
+	}
+
 	// send initial bond to module account
 	addr := sdk.MustAccAddressFromBech32(msg.Sender)
 	err := k.keeper.bk.SendCoinsFromAccountToModule(ctx, addr, types.ModuleName, sdk.Coins{msg.Bond})
 	if err != nil {
 		return nil, err
 	}
 
-	properties := k.keeper.gk.GetNetworkProperties(ctx)
-	if dapp.TotalBond.Amount.GTE(sdk.NewInt(int64(properties.MaxDappBond)).Mul(sdk.NewInt(1000_000))) {
-		return nil, types.ErrMaxDappBondReached
-	}
-
-	dapp.TotalBond = dapp.TotalBond.Add(msg.Bond)
 	k.keeper.SetDapp(ctx, dapp)
 
 	userDappBond := k.keeper.GetUserDappBond(ctx, msg.DappName, msg.Sender)
@@ -114,6 +115,11 @@ func (k msgServer) BondDappProposal(goCtx context.Context, msg *types.MsgBondDap
 func (k msgServer) ReclaimDappBondProposal(goCtx context.Context, msg *types.MsgReclaimDappBondProposal) (*types.MsgReclaimDappBondProposalResponse, error) {
 	ctx := sdk.UnwrapSDKContext(goCtx)
 
+	dapp := k.keeper.GetDapp(ctx, msg.DappName)
+	if dapp.Name == "" {
+		return nil, types.ErrDappDoesNotExist
+	}
+
 	userDappBond := k.keeper.GetUserDappBond(ctx, msg.DappName, msg.Sender)
 	if userDappBond.DappName == "" {
 		return nil, types.ErrUserDappBondDoesNotExist
@@ -125,8 +131,7 @@ func (k msgServer) ReclaimDappBondProposal(goCtx context.Context, msg *types.Msg
 		return nil, types.ErrNotEnoughUserDappBond
 	}
 
-	userDappBond.Bond.Amount = userDappBond.Bond.Amount.Sub(msg.Bond.Amount)
-	k.keeper.SetUserDappBond(ctx, userDappBond)
+	dapp.TotalBond = dapp.TotalBond.Sub(msg.Bond)
 
 	// send tokens back to user
 	addr := sdk.MustAccAddressFromBech32(msg.Sender)
@@ -135,6 +140,11 @@ func (k msgServer) ReclaimDappBondProposal(goCtx context.Context, msg *types.Msg
 		return nil, err
 	}
 
+	k.keeper.SetDapp(ctx, dapp)
+
+	userDappBond.Bond.Amount = userDappBond.Bond.Amount.Sub(msg.Bond.Amount)
+	k.keeper.SetUserDappBond(ctx, userDappBond)
+
 	return &types.MsgReclaimDappBondProposalResponse{}, nil
 }