Fix behavior of ChatFormatter.unregisterTag() and unregisterModifier()

The method currently attempts to call remove(ChatTag) on a Map<String, ChatTag> - remove() only works on keys, not values, so this will never do anything ever.
(The second method is identical.)
The commit instead iterates the map, collects all keys with the given value, and removes each of them.