Fire PermissionRefreshEvent after the nodes have actually updated

bukkit/src/main/java/me/lucko/luckperms/users/BukkitUser.java

@@ -106,12 +106,13 @@ public class BukkitUser extends User {
             /* Must be called sync, as #recalculatePermissions is an unmodified Bukkit API call that is absolutely not thread safe.
                Shouldn't be too taxing on the server. This only gets called when permissions have actually changed,
                which is like once per user per login, assuming their permissions don't get modified. */
-            plugin.doSync(() -> attachment.getPermissible().recalculatePermissions());
+            plugin.doSync(() -> {
+                attachment.getPermissible().recalculatePermissions();
+                plugin.getApiProvider().fireEventAsync(new UserPermissionRefreshEvent(new UserLink(this)));
+            });
 
         } catch (Exception e) {
             e.printStackTrace();
         }
-
-        plugin.getApiProvider().fireEventAsync(new UserPermissionRefreshEvent(new UserLink(this)));
     }
 }

sponge/src/main/java/me/lucko/luckperms/users/SpongeUser.java

@@ -43,8 +43,6 @@ class SpongeUser extends User {
 
     @Override
     public void refreshPermissions() {
-        plugin.getApiProvider().fireEventAsync(new UserPermissionRefreshEvent(new UserLink(this)));
-
         // Do nothing. Should be grabbed from PermissionService.
         /*
         plugin.doSync(() -> {