Upstream/Plan
1
0
Fork 0
mirror of https://github.com/plan-player-analytics/Plan.git synced 2025-01-26 10:01:23 +01:00
Code Issues Projects Releases Wiki Activity

Fixed ConfigNode#remove ConcurrentModificationException

Browse Source
This commit is contained in:
Rsl1122 2019-01-08 11:06:36 +02:00
parent c4a0e18dd9
commit 2b4281ee53
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
Plan/common/src/main/java/com/djrapitops/plan/system/settings/config/ConfigNode.java
View File

@ -126,11 +126,12 @@ public class ConfigNode {
parent.nodeOrder.remove(key); parent.nodeOrder.remove(key);
updateParent(null); updateParent(null);
for (String key : nodeOrder) { // Remove children recursively to avoid memory leaks
ConfigNode child = childNodes.get(key); nodeOrder.stream()
if (child != null) child.remove(); .sorted() // will use internal state and prevent Concurrent modification of underlying list
} .map(childNodes::get)
.filter(Objects::nonNull)
.forEach(ConfigNode::remove);
} }
/** /**