chore(bukkit): enable checkPermissionOnQuit and `checkPermissionOnJ…

…oin` by default in state feature
Syrent · Jun 5, 2024 · 2601065 · 2601065
1 parent 9f64c8e
commit 2601065
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/sayanvanish-api/src/main/kotlin/org/sayandev/sayanvanish/api/User.kt b/sayanvanish-api/src/main/kotlin/org/sayandev/sayanvanish/api/User.kt
@@ -60,6 +60,10 @@ interface User : BasicUser {
         SayanVanishAPI.getInstance().addUser(this)
     }
 
+    fun delete() {
+        SayanVanishAPI.getInstance().removeUser(uniqueId)
+    }
+
     override fun toJson(): String {
         val json = JsonObject()
         json.addProperty("unique-id", uniqueId.toString())

diff --git a/...h-bukkit/src/main/kotlin/org/sayandev/sayanvanish/bukkit/feature/features/FeatureState.kt b/...h-bukkit/src/main/kotlin/org/sayandev/sayanvanish/bukkit/feature/features/FeatureState.kt
@@ -23,8 +23,8 @@ class FeatureState(
     @Configurable val remember: Boolean = true,
     @Configurable val vanishOnJoin: Boolean = false,
     @Configurable val reappearOnQuit: Boolean = false,
-    @Configurable val checkPermissionOnQuit: Boolean = false,
-    @Configurable val checkPermissionOnJoin: Boolean = false,
+    @Configurable val checkPermissionOnQuit: Boolean = true,
+    @Configurable val checkPermissionOnJoin: Boolean = true,
 ) : ListenedFeature("state") {
 
     @EventHandler(priority = EventPriority.LOWEST)
@@ -54,7 +54,7 @@ class FeatureState(
 
         if (checkPermissionOnJoin && !user.hasPermission(Permission.VANISH)) {
             user.unVanish()
-            user.save()
+            user.delete()
             return
         }