chore: state breaking barberry fix (#478) (#479)

(cherry picked from commit 77b364b) Co-authored-by: Roman <[email protected]>
osmosis-labs · Jul 3, 2023 · ed4eb88 · ed4eb88
1 parent fe548cf
commit ed4eb88
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/x/auth/vesting/msg_server.go b/x/auth/vesting/msg_server.go
@@ -128,15 +128,13 @@ func (s msgServer) CreateClawbackVestingAccount(goCtx context.Context, msg *type
 		lockupCoins = lockupCoins.Add(period.Amount...)
 	}
 
-	if ctx.IsCheckTx() {
-		if err := lockupCoins.Validate(); err != nil {
-			return nil, sdkerrors.Wrapf(
-				sdkerrors.ErrInvalidCoins, "lockup coins %s are invalid: %v", lockupCoins, err)
-		}
-		if err := vestingCoins.Validate(); err != nil {
-			return nil, sdkerrors.Wrapf(
-				sdkerrors.ErrInvalidCoins, "vesting coins %s are invalid: %v", vestingCoins, err)
-		}
+	if err := lockupCoins.Validate(); err != nil {
+		return nil, sdkerrors.Wrapf(
+			sdkerrors.ErrInvalidCoins, "lockup coins %s are invalid: %v", lockupCoins, err)
+	}
+	if err := vestingCoins.Validate(); err != nil {
+		return nil, sdkerrors.Wrapf(
+			sdkerrors.ErrInvalidCoins, "vesting coins %s are invalid: %v", vestingCoins, err)
 	}
 
 	// if lockup absent, default to an instant unlock schedule