Fixed Criticals not working

[tyrannus00]

Type of change

• Bug fix
• New feature

Description

Fixes Criticals packet/bypass mode.
The bug was introduced in the update to 1.20.5/6 (653eadc)

The issue is that when sending a packet from within the PacketEvent.Send stack, that due to the singleton nature of the event, the packet variable would be changed to the newly sent packet, so that this packet effectively gets sent twice while the original packet, which triggered the event, doesn't get sent.
This caused Criticals in packet or bypass mode to cancel the attack packet and send an additional movement packet instead, and potentially caused issues in other places as well.

How Has This Been Tested?

I tested it myself, and since I am basically reverting to the state before the update this has been tested for years, if you want.

Checklist:

• My code follows the style guidelines of this project.
• I have added comments to my code in more complex areas.
• I have tested the code in both development and production environments.

[19MisterX98]

dammit. I thought replacing the packet would solve the serverspoof dilemma.
anyways, you should probably revert the rest of the mentioned commit too. Mixin extras is no longer necessary and the comment is simply wrong now.

the brand channel can stay removed. Ill fix serverspoof in another way.