Suggestion #5078

There seems to be an issue with the axes skill in mcmmo, or at the very least a misleading mishap. Let's separate all hit-able entities on the server into 3 categories: Mobs that give mcmmo xp, Mobs that don't give mcmmo xp, and players, which also don't give mcmmo xp for obvious boosting reasons. For mobs that do give mcmmo xp, the naturally spawned mobs with no custom spawners involved, the effects of Critical strike, and Greater Impact do indeed have a chance at going off, the evidence being the messages "CRITICAL HIT" and "STRUCK WITH GREAT FORCE" going off in the chat, as well as being able to one shot some of them with a non-enchanted axe when those messages go off. However, when attacking a player it was to my surprise where Critical strike, and Greater Impact weren't going off at all, no matter how many times I hit my friend, both of those skills weren't going off. I then tested this on a few mobs that don't give mcmmo xp, a villager from a villager spawner, an enderman from the end, and a zombie spawned out of a mob spawner. I killed 10 of each mob and no skills were going off, which doesn't really make any sense to me, they are still mobs, so even though I get no mcmmo xp from them, Critical Strike and Greater Impact should still be going off. So, from what I can see there seems to being one of two things going on here. Either scenario 1. The way that the plugin was coded, was to prevent skills such as Critical Strike and Greater Impact from going off on all entities that don't give mcmmo xp, and as a result because players don't give xp as well, thus the skills are not able to take effect on them, and it wasn't intended to be this way. Or scenario 2. It was intended so that these skills don't take effect on the player and Mobs that don't give mcmmo xp. This is an issue because the whole point of grinding Axes isn't so that you can beat up mobs easier. Clearly the intent of having this plugin on this server is so that you grind mobs for the xp so that you can use those skills on players. If scenario 1 is the case, then please change it so that all skills are able to be used on a player. *** Also from what it seems like, the skill Armor Impact does indeed affect the player, but im not sure that Axe Mastery does, so look into that as well. If scenario 2 is the case, then please remove all of the EFFECTS that /axes gives you, that don't effect players, such as Critical Strike, Greater Impact, Armor Impact, Axe Mastery. Just like how double drops was removed from /mining. This way there isn't a misconception, because it is very misleading to still have those skills there, when it cannot effect a player, where those skills were clearly indented to be used by players attacking other players.