Cleanup simple packet handlers

[tal5]

Actionbar event packet handlers

• Renamed baseComponent to actionbarText.
• Marked the message and raw json as plain-text.
• Removed redundant variables when returning the new packet in favor of constructing in-line.
• Updated the method to take in & return the proper packet type.
• Moved the variable for the script event to above the first if check & used it in that if check instead of getting the instance twice.

Block light packet handlers

• Split it up into 2 methods for each packet type.
• Changed to !BlockLight.lightsByChunk.isEmpty() -> call the method instead of an (in this case redundant) early return.

Fake players packet handlers

• Updated the method to take in & return the proper packet type.
• Removed processFakePlayerSpawn in favor of having the logic directly in the handler method.
• Replaced Collections#singletonList with List#of.
• Note: is there a reason for the id != -1 check? some internal Denizen thing? removed it, seems to be legacy handling from when that packet had several other functions.

Hide particles packet handlers

• Removed the redundant ParticleOptions particle variable in favor of using the return value directly.
• Updated the method to take in & return the proper packet type.

[mcmonkey4eva]

Note: is there a reason for the id != -1 check? some internal Denizen thing? I think that probably sources from the old version of the packet being an add/remove/modify multi-packet where it's -1 when it's not actually an add

[tal5]

Note: is there a reason for the id != -1 check? some internal Denizen thing? I think that probably sources from the old version of the packet being an add/remove/modify multi-packet where it's -1 when it's not actually an add

I see, removed it then - technically it'd be covered by getEntity returning null either way