From 16d4aa78599b5a7e5f751647e358d0532da969d5 Mon Sep 17 00:00:00 2001 From: Harsh Mehta Date: Sat, 22 Nov 2025 12:13:09 +0530 Subject: [PATCH 1/2] [BUG] Resolved Auto reload case sensitive issue. Signed-off-by: Harsh Mehta --- .../ch/njol/skript/structures/StructAutoReload.java | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/main/java/ch/njol/skript/structures/StructAutoReload.java b/src/main/java/ch/njol/skript/structures/StructAutoReload.java index 191a15e9dc6..e338236861e 100644 --- a/src/main/java/ch/njol/skript/structures/StructAutoReload.java +++ b/src/main/java/ch/njol/skript/structures/StructAutoReload.java @@ -191,8 +191,12 @@ public static final class AutoReload implements ScriptData { // private constructor to prevent instantiation. private AutoReload(long lastReload, @Nullable String permission, @Nullable String... recipients) { - if (recipients != null) - this.recipients.addAll(Lists.newArrayList(recipients)); + if (recipients != null) { + for (String recipient : recipients) { + if (recipient != null) + this.recipients.add(recipient.toLowerCase()); + } + } this.permission = permission; this.lastReload = lastReload; @@ -208,7 +212,7 @@ private AutoReload(long lastReload, @Nullable String permission, @Nullable Strin List senders = Lists.newArrayList(Bukkit.getConsoleSender()); if (!recipients.isEmpty()) { Bukkit.getOnlinePlayers().stream() - .filter(p -> recipients.contains(p.getName()) || recipients.contains(p.getUniqueId().toString())) + .filter(p -> recipients.contains(p.getName().toLowerCase()) || recipients.contains(p.getUniqueId().toString())) .forEach(senders::add); return Collections.unmodifiableList(senders); } From aaf49c9ca046dcb7a4f8db544960c4508a5b00e3 Mon Sep 17 00:00:00 2001 From: Harsh Mehta Date: Sat, 22 Nov 2025 13:04:12 +0530 Subject: [PATCH 2/2] [BUG] Review comments resolved. Signed-off-by: Harsh Mehta --- src/main/java/ch/njol/skript/structures/StructAutoReload.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/ch/njol/skript/structures/StructAutoReload.java b/src/main/java/ch/njol/skript/structures/StructAutoReload.java index e338236861e..5c7c3772f95 100644 --- a/src/main/java/ch/njol/skript/structures/StructAutoReload.java +++ b/src/main/java/ch/njol/skript/structures/StructAutoReload.java @@ -194,7 +194,7 @@ private AutoReload(long lastReload, @Nullable String permission, @Nullable Strin if (recipients != null) { for (String recipient : recipients) { if (recipient != null) - this.recipients.add(recipient.toLowerCase()); + this.recipients.add(recipient.toLowerCase(Locale.ENGLISH)); } } @@ -212,7 +212,7 @@ private AutoReload(long lastReload, @Nullable String permission, @Nullable Strin List senders = Lists.newArrayList(Bukkit.getConsoleSender()); if (!recipients.isEmpty()) { Bukkit.getOnlinePlayers().stream() - .filter(p -> recipients.contains(p.getName().toLowerCase()) || recipients.contains(p.getUniqueId().toString())) + .filter(p -> recipients.contains(p.getName().toLowerCase(Locale.ENGLISH)) || recipients.contains(p.getUniqueId().toString())) .forEach(senders::add); return Collections.unmodifiableList(senders); }