fix: still set duplicate redis values to prevent them from expiring early

Author: Sammster10

core/src/main/java/net/staticstudios/data/impl/h2/H2DataAccessor.java

@@ -561,9 +561,7 @@ public void postDDL() throws SQLException {
     @Override
     public void setRedisValue(String holderSchema, String holderTable, String identifier, ColumnValuePairs idColumns, String value, int expirationSeconds, int delay) {
         String prev = getAndSetRedisValueCache(holderSchema, holderTable, identifier, idColumns, value);
-        if (Objects.equals(prev, value)) {
-            return; // no change
-        }
+        //note: even if the prev value = the new value we need to update redis since we want to reset the expiration time if one is present
         String key = RedisUtils.buildRedisKey(holderSchema, holderTable, identifier, idColumns);
 
         Runnable runnable = () -> {
@@ -595,6 +593,10 @@ public void setRedisValue(String holderSchema, String holderTable, String identi
             }
         }
 
+        if (Objects.equals(prev, value)) {
+            return; // no need to call handlers if the value didn't actually change
+        }
+
         RedisUtils.DeconstructedKey deconstructedKey = RedisUtils.deconstruct(key);
         dataManager.callCachedValueUpdateHandlers(deconstructedKey.partialKey(), deconstructedKey.encodedIdNames(), deconstructedKey.encodedIdValues(), prev, value);
     }