lenient by default for PlaceholderValueProvider

Topper-MC · Oct 30, 2024 · b0e9155 · b0e9155
1 parent 084dbc4
commit b0e9155
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/spigot/plugin/src/main/java/me/hsgamer/topper/spigot/plugin/TopperPlugin.java b/spigot/plugin/src/main/java/me/hsgamer/topper/spigot/plugin/TopperPlugin.java
@@ -111,7 +111,7 @@ private void migrateConfig() {
                     map.put("placeholder", placeholder);
                     map.put("online", prefix.contains("[online]"));
                     map.put("async", prefix.contains("[async]"));
-                    map.put("lenient", prefix.contains("[lenient]"));
+                    map.put("show-errors", !prefix.contains("[lenient]"));
                 } else {
                     map.put("placeholder", value);
                 }

diff --git a/...gin/src/main/java/me/hsgamer/topper/spigot/plugin/hook/papi/PlaceholderValueProvider.java b/...gin/src/main/java/me/hsgamer/topper/spigot/plugin/hook/papi/PlaceholderValueProvider.java
@@ -18,7 +18,7 @@ public class PlaceholderValueProvider implements ValueProvider {
     private final String placeholder;
     private final boolean isOnlineOnly;
     private final boolean isAsync;
-    private final boolean isLenient;
+    private final boolean showErrors;
 
     public PlaceholderValueProvider(TopperPlugin plugin, Map<String, Object> map) {
         this.plugin = plugin;
@@ -33,7 +33,7 @@ public PlaceholderValueProvider(TopperPlugin plugin, Map<String, Object> map) {
                 .map(String::toLowerCase)
                 .map(Boolean::parseBoolean)
                 .orElse(false);
-        isLenient = Optional.ofNullable(map.get("lenient"))
+        showErrors = Optional.ofNullable(map.get("show-errors"))
                 .map(Object::toString)
                 .map(String::toLowerCase)
                 .map(Boolean::parseBoolean)
@@ -56,14 +56,14 @@ public CompletableFuture<Optional<Double>> getValue(UUID uuid) {
             try {
                 String parsed = PlaceholderAPI.setPlaceholders(player, placeholder).trim();
                 if (parsed.isEmpty()) {
-                    if (!isLenient) {
+                    if (showErrors) {
                         plugin.getLogger().warning("The placeholder " + placeholder + " returns empty");
                     }
                     return Optional.empty();
                 }
                 return Optional.of(Double.parseDouble(parsed));
             } catch (Exception e) {
-                if (!isLenient) {
+                if (showErrors) {
                     plugin.getLogger().log(Level.WARNING, "There is an error while parsing the placeholder: " + placeholder, e);
                 }
                 return Optional.empty();