Skip to content

Converts artifact triggers to datums

Rob Nelson requested to merge pull/13856/datumtrigger into Bleeding-Edge

Created by: BobdaBiscuit

At the request of PJB I have turned the shitty old code into shitty current code, all effect triggering has been moved outside of the artifact object into a trigger datum.

The artifact has a series of events that can be invoked and passed onto the trigger datum along with information needed to decide if the artifact should activate. This is easily expanded upon so new triggers can be easily added.

I have tried to not remove any features/change functionality as best as I could but a few things have changed (that I can remember to write here).

So current behaviour differences that I know of.

  • Artifact icon no longer decides what trigger it could have
  • Rates of different triggers appearing has changed obviously
  • Secondary effects take a few ticks to toggle if they are environment based
  • Any ontouch based effects automatically have the touch trigger
  • Forcefield automatically has the aura effect mode, and a touch trigger
  • Admin custom artifacts now have the correct amount of effects, however the effect IDs are incorrect (does not matter at all)

I doubt people will really care.

Done some tests and solved all runtimes I could find.

🆑

  • tweak: Xenoarch large artifact trigger behaviour has been slightly tweaked as some changes to the back end have been done. Report anything that seems off from usual.

Merge request reports