Browse Source

Made changes to the tree feller threshold

nossr50 13 years ago
parent
commit
b512fe2003
2 changed files with 7 additions and 2 deletions
  1. 2 1
      Changelog.txt
  2. 5 1
      src/main/java/com/gmail/nossr50/skills/WoodCutting.java

+ 2 - 1
Changelog.txt

@@ -16,7 +16,8 @@ Version 1.3.06-dev
  = Fixed bug with repairing wooden tools
  = Fixed bug with Nether Wart not awarding XP
  = Fixed bug with fishing treasures when treasures list is empty
- = Fixed bug with only getting one level when there was enough XP for multiple levels.
+ = Fixed bug with only getting one level when there was enough XP for multiple levels.
+ ! Changed how the tree feller threshold worked for the better
  ! Changed mcremove to no longer kick players when they are removed from database
  ! Changed mcremove to work on offline users for FlatFile
  ! Changed PlayerProfile constructor to always take a boolean

+ 5 - 1
src/main/java/com/gmail/nossr50/skills/WoodCutting.java

@@ -53,7 +53,7 @@ public class WoodCutting {
      * @param PP The PlayerProfile of the player
      */
     private static void removeBlocks(ArrayList<Block> toBeFelled, Player player, PlayerProfile PP) {
-        if (toBeFelled.size() > LoadProperties.treeFellerThreshold) {
+        if (toBeFelled.size() >= LoadProperties.treeFellerThreshold) {
             player.sendMessage(mcLocale.getString("Skills.Woodcutting.TreeFellerThreshold"));
             return;
         }
@@ -186,6 +186,10 @@ public class WoodCutting {
      */
     private static void processTreeFelling(Block currentBlock, ArrayList<Block> toBeFelled) {
         Material type = currentBlock.getType();
+        
+        if(toBeFelled.size() >= LoadProperties.treeFellerThreshold) {
+            return;
+        }
 
         if (type.equals(Material.LOG) || type.equals(Material.LEAVES)) {
             toBeFelled.add(currentBlock);