From 534b30e4f9c9f3bec7e17bfd7ed2eeed4589538a Mon Sep 17 00:00:00 2001
From: mohammed jasem alaajel <xrambad@glomc.net>
Date: Sat, 18 May 2024 16:46:35 +0400
Subject: [PATCH] invalidate clarifcation, remove unified jedis method from
 ProxyDataManager

---
 .../minecraft/redisbungee/api/PlayerDataManager.java        | 6 +++++-
 .../minecraft/redisbungee/api/ProxyDataManager.java         | 4 ----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/api/src/main/java/com/imaginarycode/minecraft/redisbungee/api/PlayerDataManager.java b/api/src/main/java/com/imaginarycode/minecraft/redisbungee/api/PlayerDataManager.java
index 15e86b7..e2b2d34 100644
--- a/api/src/main/java/com/imaginarycode/minecraft/redisbungee/api/PlayerDataManager.java
+++ b/api/src/main/java/com/imaginarycode/minecraft/redisbungee/api/PlayerDataManager.java
@@ -49,7 +49,7 @@ public abstract class PlayerDataManager<P, LE, DE, PS extends IPubSubMessageEven
 
     public PlayerDataManager(RedisBungeePlugin<P> plugin) {
         this.plugin = plugin;
-        this.unifiedJedis = plugin.proxyDataManager().unifiedJedis();
+        this.unifiedJedis = plugin.getSummoner().obtainResource();
         this.proxyId = plugin.proxyDataManager().proxyId();
         this.networkId = plugin.proxyDataManager().networkId();
     }
@@ -80,6 +80,8 @@ public abstract class PlayerDataManager<P, LE, DE, PS extends IPubSubMessageEven
     }
 
     protected void handleNetworkPlayerQuit(IPlayerLeftNetworkEvent event) {
+        // reason we don't set the data here because in-case of failure in redis etc,
+        // events won't reach the proxy so data would be incorrect.
         this.proxyCache.invalidate(event.getUuid());
         this.serverCache.invalidate(event.getUuid());
         this.ipCache.invalidate(event.getUuid());
@@ -90,6 +92,8 @@ public abstract class PlayerDataManager<P, LE, DE, PS extends IPubSubMessageEven
     }
 
     protected void handleNetworkPlayerJoin(IPlayerJoinedNetworkEvent event) {
+        // reason we don't set the data here because in-case of failure in redis etc,
+        // events won't reach the proxy so data would be incorrect.
         this.proxyCache.invalidate(event.getUuid());
         this.serverCache.invalidate(event.getUuid());
         this.ipCache.invalidate(event.getUuid());
diff --git a/api/src/main/java/com/imaginarycode/minecraft/redisbungee/api/ProxyDataManager.java b/api/src/main/java/com/imaginarycode/minecraft/redisbungee/api/ProxyDataManager.java
index 07a5df3..ed0fecb 100644
--- a/api/src/main/java/com/imaginarycode/minecraft/redisbungee/api/ProxyDataManager.java
+++ b/api/src/main/java/com/imaginarycode/minecraft/redisbungee/api/ProxyDataManager.java
@@ -399,10 +399,6 @@ public abstract class ProxyDataManager implements Runnable {
         return proxyId;
     }
 
-    public UnifiedJedis unifiedJedis() {
-        return unifiedJedis;
-    }
-
     public String networkId() {
         return networkId;
     }