Remove unused import and simplify conditional in NameChecker

Removed extra import 'Permission' and simplified a conditional statement in the NameChecker class. This refactoring streamlines the logic for setting the name during PreShopCreationEvent, making the code more straightforward without changing functionality.
Brokkonaut · Jan 1, 2024 · 76053f4 · 76053f4
1 parent a439c2d
commit 76053f4
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/src/main/java/com/Acrobot/ChestShop/Listeners/PreShopCreation/NameChecker.java b/src/main/java/com/Acrobot/ChestShop/Listeners/PreShopCreation/NameChecker.java
@@ -5,7 +5,6 @@
 import org.bukkit.event.EventPriority;
 import org.bukkit.event.Listener;
 
-import com.Acrobot.ChestShop.Permission;
 import com.Acrobot.ChestShop.Configuration.Properties;
 import com.Acrobot.ChestShop.Events.PreShopCreationEvent;
 import com.Acrobot.ChestShop.UUIDs.NameManager;
@@ -22,8 +21,7 @@ public static void onPreShopCreation(PreShopCreationEvent event) {
         Player player = event.getPlayer();
         if (Properties.ADMIN_SHOP_NAME.equalsIgnoreCase(name)) {
             name = Properties.ADMIN_SHOP_NAME;
-        }
-        if (name.isEmpty() || (!NameManager.canUseName(player, name) && !Permission.has(player, Permission.ADMIN))) {
+        } else {
             name = NameManager.getNameFor(player);
         }
         event.setOwnerName(name);