Skip to content

Conversation

@Sofistico
Copy link
Contributor

@Sofistico Sofistico commented Nov 15, 2025

on the specific case of force shove and equivalents

Summary

Mods "Fixing force shove center direction position crash"

Purpose of change

I saw that there is a hard crash when the user select the center direction of the knockback effect, i wanted to fix it

Describe the solution

I moved the random selector of position to another if when the target_pos is the same as the direction_pos

Describe alternatives you've considered

Do nothing, give a fail message when the center is selected

Testing

I tested that no crash happened and then i tested that the other directions works

…get_pos == direction_pos

on the specific case of force shove and equivalents
@github-actions github-actions bot added NPC / Factions NPCs, AI, Speech, Factions, Ownership [C++] Changes (can be) made in C++. Previously named `Code` labels Nov 15, 2025
@Sofistico Sofistico changed the title fixing the error that happens when the knockback effect is passed tar… fixing the error that happens when the knockback effect is passed target_pos == direction_pos Nov 15, 2025
@Sofistico Sofistico changed the title fixing the error that happens when the knockback effect is passed target_pos == direction_pos [MoM] fixing the error that happens when caster selects center position of Force Shove and equivalents Nov 15, 2025
@Sofistico Sofistico changed the title [MoM] fixing the error that happens when caster selects center position of Force Shove and equivalents [MoM] Fixing the error that happens when caster selects center position of Force Shove and equivalents Nov 15, 2025
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Nov 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astyled astyled PR, label is assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions NPC / Factions NPCs, AI, Speech, Factions, Ownership

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant