Changeable Conditions

[APickledWalrus]

Problem

For some time, we have been recommending a combination of Condition+Effect in favor of a boolean Expression. However, there is some loss of ease of use:

• Losing support for EffToggle: toggle the player's gliding state
• Losing support for one-line changing: set the player's gliding state to {_boolean}

If Conditions were to support changing, these features could be restored in some manner, preserving the ease of use options provided by boolean Expressions.

Solution

This PR proposes a simple API for changing the values represented by conditions. This API is further extended by PropertyCondition to make implementing even easier.

I decided to allow any changer to be used, though some modes are likely pointless. We could instead have specific methods for whether SET or RESET are allowed (as an example as those are probably the only realistic modes to support). I'd like opinions to tweak this.

This API is then used by ExprWhether to allow changing conditions:

• toggle whether player is gliding
• set whether the player is gliding to {_boolean}

There is a large number of files changed as I implemented this for most conditions where it made sense.

Testing Completed

Some in-game testing. I added at least 40 new tests covering many of the updated conditions.

Supporting Information

n/a

---

Completes: none
Related: none
AI assistance: none

[sovdeeth]

looking good

[Efnilite]

almost there 🙏