Fork/CitizensCMD
2
0
Fork 0
forked from Upstream/CitizensCMD
Code Pull Requests Actions Releases Activity

Merge pull request #17 from skykatik/master

Browse Source 
Fix memory leak
This commit is contained in:
HexedHero 2024-02-18 05:52:52 +00:00 committed by GitHub
commit 5ffb263484
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/main/java/me/mattstudios/citizenscmd/permissions/PermissionsManager.java
View File

@ -46,8 +46,7 @@ public class PermissionsManager {
public void setPermission(Player player, String permission) {
final PermissionAttachment permissionAttachment = player.addAttachment(plugin);
permissionsData.put(player.getUniqueId(), permissionAttachment);
final PermissionAttachment permissionAttachment1 = permissionsData.get(player.getUniqueId());
permissionAttachment1.setPermission(permission, true);
permissionAttachment.setPermission(permission, true);
}
/**
@ -57,7 +56,10 @@ public class PermissionsManager {
* @param permission The permission node to be removed
*/
public void unsetPermission(Player player, String permission) {
permissionsData.get(player.getUniqueId()).unsetPermission(permission);
PermissionAttachment attachment = permissionsData.remove(player.getUniqueId());
if (attachment != null) {
attachment.unsetPermission(permission);
}
}
}