Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

X2Effect_Persistent::GetAttackingDamageModifier() does not get the damage effect during damage preview #1305

Open
Iridar opened this issue Feb 27, 2024 · 1 comment · Fixed by #1306
Assignees

Comments

@Iridar
Copy link
Contributor

Iridar commented Feb 27, 2024

This means that persistent effects that add contextual damage bonuses depending on the damage effect properties do not actually get said context until the damage effect will be applied, so they cannot properly display their damage bonuses during damage preview.

This, for example, causes the bug described in #612.

Solution: add a GetAttackingDamageModifier_CH() method that will have the damage effect passed to it as an argument.

@Iridar Iridar added this to the 1.28.0 milestone Feb 27, 2024
@Iridar Iridar self-assigned this Feb 27, 2024
Iridar pushed a commit to Iridar/X2WOTCCommunityHighlander that referenced this issue Feb 27, 2024
@Iridar Iridar removed this from the 1.28.0 milestone May 2, 2024
@Iridar Iridar reopened this May 4, 2024
@Iridar
Copy link
Contributor Author

Iridar commented May 4, 2024

Something weird, getting the following errors in the Launch log now:

[0006.83] Error: (AddNetObject) Objects Function XComGame.X2Effect_Persistent:GetAttackingDamageModifier_CH and IntProperty XComGame.X2Effect_Persistent:GetDefendingDamageModifier.ReturnValue have duplicate NetIndex 51097
[0006.83] Error: (AddNetObject) Objects ObjectProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier_CH.EffectState and Function XComGame.X2Effect_Persistent:GetAttackingDamageModifier have duplicate NetIndex 51096
[0006.83] Error: (AddNetObject) Objects ObjectProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier_CH.Attacker and ObjectProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier.EffectState have duplicate NetIndex 51095
[0006.84] Error: (AddNetObject) Objects InterfaceProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier_CH.TargetDamageable and ObjectProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier.Attacker have duplicate NetIndex 51094
[0006.84] Error: (AddNetObject) Objects ObjectProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier_CH.AbilityState and InterfaceProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier.TargetDamageable have duplicate NetIndex 51093
[0006.84] Error: (AddNetObject) Objects StructProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier_CH.AppliedData and ObjectProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier.AbilityState have duplicate NetIndex 51092
[0006.84] Error: (AddNetObject) Objects IntProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier_CH.CurrentDamage and StructProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier.AppliedData have duplicate NetIndex 51091
[0006.84] Error: (AddNetObject) Objects ObjectProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier_CH.WeaponDamageEffect and IntProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier.CurrentDamage have duplicate NetIndex 51090
[0006.84] Error: (AddNetObject) Objects ObjectProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier_CH.NewGameState and ObjectProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier.NewGameState have duplicate NetIndex 51089
[0006.84] Error: (AddNetObject) Objects IntProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier_CH.ReturnValue and IntProperty XComGame.X2Effect_Persistent:GetAttackingDamageModifier.ReturnValue have duplicate NetIndex 51088

BlackDog86 pushed a commit to BlackDog86/X2WOTCCommunityHighlander that referenced this issue Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant