Damage user on trigger and Step Trigger refactor [Part 1]

[TokenStyle]

About the PR

Two step's PR with the goal that I've doin, that players can squash cockroaches by stepping/walk on them, and add red blooded cockroaches with gib sound on death, and add hauberoach. This refactor adds a lot of possibilities to step triggers, like walking on plushies will make some sound.

• Refactor DamageUserOnTrigger into DamageOnTrigger and add StepTriggerOnSizeComponent.
• Make cockroach die when player's stepped on it (or something big enough).

And my wishes:

• Transfer player's DNA to a cockroach when squashed and if the player's without shoes.
• Blood cockroaches that drops red blood on death (yml only tho).
• Transfer fibers to a cockroach that's squashed by the player's with shoes. Dunno how to do it tho', probably long-term thing. Maybe related to this issue transferring dna to clothes when worn #14744 .
• Pacified entities can't squash cockroaches.

Why / Balance

Cockroaches have to die when player's walk on 'em.

Technical details

• Add reliable comments for DamageUserOnTriggerSystem and DamageUserOnTriggerComponent.
• Add DamageOnTriggerComponent and System that deals damage to triggering entity.
• Add ForceIgnoreTriggerContinue to StepTriggerComponent so it's able to be used by yml devs. By default, StepTriggerAttemptEvent.Continue is never true (i.e., the mousetraps are manually from code sets StepTriggerAttemptEvent.Continue to true), so I've added this workaround.
• Add StepTriggerOnSizeComponent and System that checks size of the tripper/stepped/walked entity.

Media

Requirements

• I have read and am following the Pull Request and Changelog Guidelines.
• I have added media to this PR or it does not require an ingame showcase.

Breaking changes

StepTriggerComponent now has ForceIgnoreTriggerContinue field, that's when true skips StepTriggerAttemptEvent.Continue check.

Changelog

no cl, no fun

[TokenStyle]

CollisionMask and CollisionLayer works not as intended, don't know how to fix rn, to fix StepTriggerOnSizeSystem logic, to check actual size of the entity, it should pass Mobs and LargeMobs, but Continue false for SmallMobs

[iaada]

I've had a similar desire for step triggers to crunch glass shards underfoot. However I was waiting for #39034, as it's a much larger trigger refactor. In particular it moves the triggers to shared.

[TokenStyle]

Oh I gotcha, then I will upstream after #39034 , thanks!

[keronshb]

Just going to move this to DNM while the trigger refactor is reviewed.
If it doesn't get resolved in a month or so I'll remove the tag. It's only so the refactor PR won't have to make too many changes while it's being reviewed.

[TokenStyle]

Closed because of new PR that's follows Trigger System refactor #39267