Let all access to monster groups go through MonsterGroupManager::GetMonsterGroup

Instead of accessing an entry in the map directly. The function reports invalid group idents, accessing the map directly returns a default created (empty) group with no warning.

Removes some outdated comments, printf is not used here at all, monster groups are defined completely with json.