You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If an object that has both a dependency and a modifier targeting the same value, the dependency check will behave differently depending on whether the object has already been added to the owner that will be checked.
Assuming that the new owner is a character that has the object character.secondary-attribute.reaction with a value of 9, the dependency check for the object character.ability.reflexes-1 will fail when the object has not yet been added (9 < 10), but will pass after the object has been added (13 > 10). This doesn't matter if the dependency check prevents the object from being added, but in general, dependency checks should be agnostic to whether they are ran before or after an object has been added to an owner.
If an object that has both a dependency and a modifier targeting the same value, the dependency check will behave differently depending on whether the object has already been added to the owner that will be checked.
An example:
Assuming that the new owner is a character that has the object
character.secondary-attribute.reaction
with a value of 9, the dependency check for the objectcharacter.ability.reflexes-1
will fail when the object has not yet been added (9 < 10), but will pass after the object has been added (13 > 10). This doesn't matter if the dependency check prevents the object from being added, but in general, dependency checks should be agnostic to whether they are ran before or after an object has been added to an owner.We could solve this by making sure that dependency checks ignore modifiers modifying the very same object, but this could pose some engineering challenges I'm not seeing yet. This is where such a check could be implemented: https://github.com/satellite-games/orbit/blob/main/lib/modifier/modifier.ts#L66
I'll leave this open until I know whether this becomes an actual issue.
The text was updated successfully, but these errors were encountered: