[fixed] "on fire" state wasn't copied when changing class (new)

[mugg91]

Status

• READY: this PR is (to the best of your knowledge) ready to be incorporated into the game.

Description

Fixes #1300

This is the same as #1905 except for some minor changes.

• A test print was left in, commented out, in IsFlammable.as
• removed or added some empty lines etc.

I redid this PR because solving a file conflict in the old PR with StandardRespawnCommand.as caused some confusing file change diffs.

Tested in offline and online, works.

[mugg91]

The string burn_hitter doesn't really serve any purpose. If we remove it, we could save two instances of #include "Hitters.as" - in StandardRespawnCommand.as and FireCommon.as, cutting down on loading time.

In IsFlammable.as, this line
blob.server_Hit(this, pos, Vec2f(0, 0), 0.25, this.get_u8(burn_hitter), true); could change to blob.server_Hit(this, pos, Vec2f(0, 0), 0.25, Hitters::burn, true);

[mugg91]

Made a commit that does the following changes:

• The change I explained in my previous post - removed tag "burn_hitter", allowing me to remove #include "Hitters.as" in two files.
• Moved function isIgniteHitter() from FireCommon.as to Hitters.as. This allowed me to remove #include "FireCommon.as" from Fireplace.as.

No more changes planned. Ready for review.