[B+C] Add missing effects. Adds BUKKIT-5652. - #1074

I'd like to see all of these effects audited with links to the associated mc-dev code.

The code I've extracted the ids from isn't available on mc-dev, as the class is only available on the client binary.

Class name on 1.7.9 is "bmo", method "a(Lyy;IIIII)V".

Here is the said method: http://pastebin.com/CN9wHaMF

Since Searge isn't updating Minecraft Coder Pack anymore I've been working with the original obfuscated code.

They exist in MC dev... just not in the form you're thinking of
https://github.com/Bukkit/mc-dev/blob/5a9a0ae2b3e408a9e8bf4a3dc3247d95e61bd3a1/net/minecraft/server/EntityLiving.java#L119

IMHO the first (bmo.java) is more clear when it comes to their effect, as you can see the file they play or the particle they spawn.

There are some similarities between this and #1025. What are the (dis)advantages of either PR? Is there a need for both or can they be combined/adapted?

I also found the Sound enum, which is not updated in this PR. I haven't had a chance to look, but are there any further mismatched/missing cases from not only other parts of the API, but from the game itself?

Well IMHO #1025 and this one they are both mutually compatible. This adds new visual and sound effects while the other adds currently-unimplemented particle effects, which are the basis for visual effects.

However I consider that adding another class for Particle effects is redundant. I wanted to get this pulled first, and then add particle effects as is currently implemented in SportBukkit (https://github.com/OvercastNetwork/SportBukkit/blob/master/Bukkit/0011-Allow-playEffect-to-spawn-particles.patch): as a new Effect type (Type.PARTICLE), without actually making distinction based on how are they implemented by the current network protocol (ie integers for VISUAL and sound, and URIs for particles), allowing programmers to use the very same playEffect methods currently defined in World.