fix: Correct logic in ReplaceKey to check for forceCheck and window conditions

zackspear · zackspear · commit c13f0e4bdf08 · 2025-03-18T16:22:13.000-07:00
diff --git a/emhttp/plugins/dynamix/include/ReplaceKey.php b/emhttp/plugins/dynamix/include/ReplaceKey.php
@@ -175,7 +175,7 @@ public function check(bool $forceCheck = false)
 
         $isWithinWindow = ($now >= $sevenDaysBefore && $now <= $sevenDaysAfter);
 
-        if ($forceCheck || $isWithinWindow) {
+        if (!$forceCheck && !$isWithinWindow) {
             return;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,7 @@ public function check(bool $forceCheck = false)`
`175`	`175`
`176`	`176`	`$isWithinWindow = ($now >= $sevenDaysBefore && $now <= $sevenDaysAfter);`
`177`	`177`
`178`		`- if ($forceCheck \|\| $isWithinWindow) {`
	`178`	`+ if (!$forceCheck && !$isWithinWindow) {`
`179`	`179`	`return;`
`180`	`180`	`}`
`181`	`181`