chore: move pv try/catch inside vanish integration if statement

NEZNAMY · Nov 24, 2024 · ee5ff07 · ee5ff07
1 parent 80c63dd
commit ee5ff07
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/shared/src/main/java/me/neznamy/tab/shared/platform/Platform.java b/shared/src/main/java/me/neznamy/tab/shared/platform/Platform.java
@@ -200,13 +200,13 @@ public interface Platform {
      * @return  {@code true} if can see, {@code false} if not.
      */
     default boolean canSee(@NotNull TabPlayer viewer, @NotNull TabPlayer target) {
-        try {
-            if (!VanishIntegration.getHandlers().isEmpty()) {
+        if (!VanishIntegration.getHandlers().isEmpty()) {
+            try {
                 return VanishIntegration.getHandlers().stream().allMatch(integration -> integration.canSee(viewer, target));
+            } catch (ConcurrentModificationException e) {
+                // PV error, try again
+                return canSee(viewer, target);
             }
-        } catch (ConcurrentModificationException e) {
-            // PV error, try again
-            return canSee(viewer, target);
         }
         return !target.isVanished() || viewer.hasPermission(TabConstants.Permission.SEE_VANISHED);
     }