From 396630821f99bdbfa7bff5ac18b96e1b727fbaf3 Mon Sep 17 00:00:00 2001
From: William278 <will27528@gmail.com>
Date: Wed, 18 Jun 2025 22:19:54 +0100
Subject: [PATCH] fix: return false if user is checked out if CIPs are off

---
 .../net/william278/husksync/sync/LockstepDataSyncer.java   | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/common/src/main/java/net/william278/husksync/sync/LockstepDataSyncer.java b/common/src/main/java/net/william278/husksync/sync/LockstepDataSyncer.java
index 26c76fc8..57379594 100644
--- a/common/src/main/java/net/william278/husksync/sync/LockstepDataSyncer.java
+++ b/common/src/main/java/net/william278/husksync/sync/LockstepDataSyncer.java
@@ -53,9 +53,10 @@ public class LockstepDataSyncer extends DataSyncer {
 
             // If they are checked out, ask the server to check them back in and return false
             final Optional<String> server = getRedis().getUserCheckedOut(user);
-            if (plugin.getSettings().getSynchronization().isCheckinPetitions()
-                    && server.isPresent() && !server.get().equals(plugin.getServerName())) {
-                getRedis().petitionServerCheckin(server.get(), user);
+            if (server.isPresent() && !server.get().equals(plugin.getServerName())) {
+                if (plugin.getSettings().getSynchronization().isCheckinPetitions()) {
+                    getRedis().petitionServerCheckin(server.get(), user);
+                }
                 return false;
             }