Upstream
/
WorldGuard
mirror of https://github.com/EngineHub/WorldGuard.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Handle non-existent region group flags smarter

This commit is contained in:
stonar96 2022-10-19 03:05:46 +02:00
parent 36d4483353
commit 1901692178
1 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
worldguard-core/src/main/java/com/sk89q/worldguard/protection/flags/registry/SimpleFlagRegistry.java
View File

@ -24,7 +24,10 @@ import com.google.common.collect.Lists;
import com.google.common.collect.Maps;
import com.sk89q.worldguard.protection.flags.Flag;
import com.sk89q.worldguard.protection.flags.Flags;
import com.sk89q.worldguard.protection.flags.RegionGroup;
import com.sk89q.worldguard.protection.flags.RegionGroupFlag;
import com.sk89q.worldguard.protection.flags.StateFlag;
import com.sk89q.worldguard.protection.flags.StateFlag.State;
import javax.annotation.Nullable;
import java.util.Collection;
@ -167,6 +170,18 @@ public class SimpleFlagRegistry implements FlagRegistry {
if (regionGroupFlag != null) {
values.put(regionGroupFlag, regionGroupFlag.unmarshal(entry.getValue()));
} else {
log.warning("Found non-existent region group flag '" + entry.getKey() + "' with value '" + entry.getValue() + "' for flag '" + parent.getName() + "'");
log.warning("The region group flag '" + entry.getKey() + "' with value '" + entry.getValue() + "' will be removed");
regionGroupFlag = new RegionGroupFlag("dummy", RegionGroup.ALL);
RegionGroup regionGroup = regionGroupFlag.unmarshal(entry.getValue());
if (regionGroup != null && regionGroup != RegionGroup.ALL) {
if (!(parent instanceof StateFlag) || values.get(parent) == State.ALLOW) {
log.warning("For safety reasons the flag '" + parent.getName() + "' with value '" + values.get(parent) + "' will also be removed");
values.remove(parent);
}
}
}
}
}