v7.3.53 - properly reset Toggle varibale when toggling functionality …

…back on - affects "Remember Toggle Status"
slipcor · Feb 17, 2024 · ca65ef9 · ca65ef9
1 parent 8707668
commit ca65ef9
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/doc/changelog.md b/doc/changelog.md
@@ -2,6 +2,7 @@
 
 ## v7.3 - API Expansion
 
+- v7.3.53 - properly reset Toggle varibale when toggling functionality back on - affects "Remember Toggle Status"
 - v7.3.52 - [core update] - Support servers without Spigot base unless they want to use hex codes
 - v7.3.51 - update for 1.20.4, also make updating minecraft-version-dependent to not break servers by introducing unknown blocks or tree definitions
 - v7.3.50 - print correct permission message for "treeconfig" command

diff --git a/readme.md b/readme.md
@@ -74,7 +74,7 @@ It also will take down an entire tree when it is enabled in the config.
 
 ## Changelog
 
-- v7.3.52 - [core update] - Support servers without Spigot base unless they want to use hex codes
+- v7.3.53 - properly reset Toggle varibale when toggling functionality back on - affects "Remember Toggle Status"
 - [read more](doc/changelog.md)
 
 ***

diff --git a/src/main/java/net/slipcor/treeassist/TreeAssist.java b/src/main/java/net/slipcor/treeassist/TreeAssist.java
@@ -520,10 +520,11 @@ public boolean toggleGlobal(String player) {
      * @return whether the player is able to use the plugin in that world now
      */
     public boolean toggleWorld(String world, String player) {
+        boolean result = false;
         if (disabledMap.containsKey(world)) {
             if (disabledMap.get(world).contains(player)) {
                 disabledMap.get(world).remove(player);
-                return true;
+                result = true;
             } else {
                 disabledMap.get(world).add(player);
             }
@@ -538,7 +539,7 @@ public boolean toggleWorld(String world, String player) {
             }
             config.save();
         }
-        return false;
+        return result;
     }
 
     /**