fix: Use correct existing delegation amount (#752)

* use correct existing delegation amount * changelog
persistenceOne · Jan 25, 2024 · c7cc855 · c7cc855
1 parent 721ede7
commit c7cc855
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -49,6 +49,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
 
 ### Bug Fixes
 
+- [752](https://github.com/persistenceOne/pstake-native/pull/752) Use correct existing delegation amount.
 - [751](https://github.com/persistenceOne/pstake-native/pull/751) Set LSM bond factor as -1 by default.
 - [750](https://github.com/persistenceOne/pstake-native/pull/750) Shares to tokens.
 - [734](https://github.com/persistenceOne/pstake-native/pull/734) Host chain duplication check.

diff --git a/x/liquidstakeibc/keeper/icq.go b/x/liquidstakeibc/keeper/icq.go
@@ -96,6 +96,7 @@ func DelegationCallback(k Keeper, ctx sdk.Context, data []byte, query icqtypes.Q
 		)
 	}
 
+	existingDelegatedAmount := validator.DelegatedAmount
 	delegatedAmount := validator.ExchangeRate.Mul(delegation.Shares)
 	slashedAmount := sdk.NewDecFromInt(validator.DelegatedAmount).Sub(delegatedAmount)
 
@@ -115,7 +116,7 @@ func DelegationCallback(k Keeper, ctx sdk.Context, data []byte, query icqtypes.Q
 				types.EventTypeSlashing,
 				sdk.NewAttribute(types.AttributeChainID, hc.ChainId),
 				sdk.NewAttribute(types.AttributeValidatorAddress, validator.OperatorAddress),
-				sdk.NewAttribute(types.AttributeExistingDelegation, validator.DelegatedAmount.String()),
+				sdk.NewAttribute(types.AttributeExistingDelegation, existingDelegatedAmount.String()),
 				sdk.NewAttribute(types.AttributeUpdatedDelegation, delegatedAmount.String()),
 				sdk.NewAttribute(types.AttributeSlashedAmount, slashedAmount.String()),
 			),