TotalFreedomMC/PlayerParticles
1
0
Fork 0
mirror of https://github.com/TotalFreedomMC/PlayerParticles.git synced 2025-07-30 17:31:52 +00:00
Code Issues Projects Releases Wiki Activity

Add strictOrdering parameter to setting isDefault check

Browse source
This commit is contained in:
Ethan P 2020-10-01 21:28:06 -07:00
parent 63ccc28c8e
commit 5f6ee80f1c
No known key found for this signature in database
GPG key ID: 6963FD04F6CF35EA
1 changed files with 12 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

15
src/main/java/dev/esophose/playerparticles/manager/ConfigurationManager.java
View file

@ -273,7 +273,7 @@ public class ConfigurationManager extends Manager {
this.configuration.save();
// Legacy nag: WorldGuard Regions
if (!isDefault(Setting.WORLDGUARD_ALLOWED_REGIONS) || !isDefault(Setting.WORLDGUARD_DISALLOWED_REGIONS)) {
if (!isDefault(Setting.WORLDGUARD_ALLOWED_REGIONS, false) || !isDefault(Setting.WORLDGUARD_DISALLOWED_REGIONS, false)) {
Arrays.asList(
"It looks like you're using the 'allowed-regions' or 'disallowed-regions' settings.",
"These settings are deprecated from PlayerParticles, and will be removed in a future update.",
@ -281,9 +281,18 @@ public class ConfigurationManager extends Manager {
).forEach(PlayerParticles.getInstance().getLogger()::warning);
}
}
private boolean isDefault(Setting setting) {
/**
* Checks if a setting is the default value.
*
* @param setting The setting to check.
* @param strictOrdering If true, compares lists with strict ordering.
* @return True if the setting is default.
*/
private boolean isDefault(Setting setting, boolean strictOrdering) {
if (setting.defaultValue instanceof List) {
if (strictOrdering) return setting.defaultValue.equals(setting.value);
HashSet<String> currentSet = new HashSet<>(setting.getStringList());
Set<String> defaultSet = ((List<?>) setting.defaultValue).stream()
.map(Object::toString)