Allow enchants on fireworks/charges/potions/banners

[imDaniX]

This PR fixes #3879
Also added p as a short name for power for fireworks meta.

[JRoy]

Needs conflict resolving from latest commits (maybe my previous review comment should also make sense given my recent PR to fix some firework meta stuff)

[JRoy]

While this would allow fireworks/potions/etc to parse enchantments, it would not, for example, parse a firework with firework meta and enchantment meta. If that can be added (maybe by splitting meta into a hashmap of tokens?)

[imDaniX]

While this would allow fireworks/potions/etc to parse enchantments, it would not, for example, parse a firework with firework meta and enchantment meta. If that can be added (maybe by splitting meta into a hashmap of tokens?)

Wait, but isn't addStringMeta method works word-by-word? This method can accept something like unbreakable:1 effect:trail, but it can't properly parse it anyway because it just splits the string into 1-2 pieces(so it will become unbreakable and 1 effect:trail), and even parseStringMeta calls the method for each argument.

[JRoy]

While this would allow fireworks/potions/etc to parse enchantments, it would not, for example, parse a firework with firework meta and enchantment meta. If that can be added (maybe by splitting meta into a hashmap of tokens?)

Wait, but isn't addStringMeta method works word-by-word? This method can accept something like unbreakable:1 effect:trail, but it can't properly parse it anyway because it just splits the string into 1-2 pieces(so it will become unbreakable and 1 effect:trail), and even parseStringMeta calls the method for each argument.

That should be fixed ideally then.

[imDaniX]

I mean, right now it works as-is. Here's the result of /i fireworkrocket 1 efficiency:1 name:Angry_Creeper color:red fade:green type:creeper power:1 durability:1