Fix the path to the main class + correct the event priority

Author: TuxCoding

README.md

@@ -4,3 +4,5 @@ McCombatLevel
 Bukkit plugin that creates and displays a combat level for mcMMO.
 
 See http://dev.bukkit.org/bukkit-plugins/mccombatlevel/
+
+    http://www.curse.com/bukkit-plugins/minecraft/mccombatlevel

src/main/java/com/gmail/mrphpfan/mccombatlevel/McCombatLevel.java

@@ -146,6 +146,7 @@ public int calculateLevel(McMMOPlayer mcMMOPlayer) {
         } catch (Exception ex) {
             getLogger().log(Level.WARNING, "Exception occured falling back", ex);
             if (levelCalculator instanceof DefaultCalculator) {
+                //nulling the calculator, because event the default replacer created an exception
                 levelCalculator = null;
             }
 

src/main/java/com/gmail/mrphpfan/mccombatlevel/PlayerListener.java

@@ -50,7 +50,8 @@ public void onPlayerLogout(PlayerQuitEvent event) {
         plugin.removeCachedLevels(event.getPlayer());
     }
 
-    @EventHandler(ignoreCancelled = true)
+    //set it to low in order to update the level before other plugins want to get that value
+    @EventHandler(ignoreCancelled = true, priority = EventPriority.LOW)
     public void onPlayerLevelUp(McMMOPlayerLevelUpEvent event) {
         SkillType skill = event.getSkill();
 
@@ -64,7 +65,7 @@ public void onPlayerLevelUp(McMMOPlayerLevelUpEvent event) {
 
     //some chat plugins listen and change stuff on the default priority. In order
     //to see these changes we need an higher priority.
-    @EventHandler(ignoreCancelled = true, priority = EventPriority.HIGH)
+    @EventHandler(ignoreCancelled = true, priority = EventPriority.LOW)
     public void onChat(AsyncPlayerChatEvent event) {
         if (!plugin.isPrefixEnabled()) {
             //check if prefix is enabled

src/main/java/com/gmail/mrphpfan/mccombatlevel/calculator/DefaultCalculator.java

@@ -7,6 +7,9 @@
 
 public class DefaultCalculator implements LevelCalculator {
 
+    //max of 1000 level
+    private static final int MAX_LEVEL = 1000;
+
     @Override
     public int calculateLevel(McMMOPlayer mcMMOPlayer) {
         int swords = getLevel(mcMMOPlayer, SkillType.SWORDS);
@@ -22,7 +25,10 @@ public int calculateLevel(McMMOPlayer mcMMOPlayer) {
 
     private int getLevel(McMMOPlayer mcMMOPlayer, SkillType skillType) {
         int skillLevel = mcMMOPlayer.getSkillLevel(skillType);
-        //max of 1000 level
-        return skillLevel <= 1000 ? skillLevel : 1000;
+        if (skillLevel <= MAX_LEVEL) {
+            return skillLevel;
+        }
+
+        return MAX_LEVEL;
     }
 }

src/main/java/com/gmail/mrphpfan/mccombatlevel/calculator/JavaScriptCalculator.java

@@ -33,7 +33,7 @@ public int calculateLevel(McMMOPlayer mcMMOPlayer) {
                 throw new RuntimeException("Formula doesn't returned a number");
             }
         } catch (ScriptException ex) {
-            throw new RuntimeException("Combat level cannot be calculated", ex);
+            throw new RuntimeException("Combat level cannot be calculated", ex.getCause());
         }
     }
 

src/main/resources/plugin.yml

@@ -1,5 +1,5 @@
 name: ${project.name}
-main: com.gmail.mrphpfan.McCombatLevel
+main: ${project.groupId}.${project.artifactId}.${project.name}
 version: ${project.version}
 
 # meta informations for plugin managers and auto updaters