From: Tim Düsterhus <duesterhus@woltlab.com>
Date: Tue, 26 Apr 2022 12:23:17 +0000 (+0200)
Subject: Remove the intransparent doubling of the visit tracker lifetime if existing data... 
X-Git-Tag: 5.5.0_Alpha_4~2^2~3
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=25fc44b768e04ff28fce68bd628013874eeb2d27;p=GitHub%2FWoltLab%2FWCF.git

Remove the intransparent doubling of the visit tracker lifetime if existing data exists

In case of com.woltlab.wbb this only triggers if and only if a user *ever*
marked all boards as read. With the increase of the default lifetime in the
previous commit it becomes obsolete.
---

diff --git a/wcfsetup/install/files/lib/system/visitTracker/VisitTracker.class.php b/wcfsetup/install/files/lib/system/visitTracker/VisitTracker.class.php
index bc19615381..2fbeee68bd 100644
--- a/wcfsetup/install/files/lib/system/visitTracker/VisitTracker.class.php
+++ b/wcfsetup/install/files/lib/system/visitTracker/VisitTracker.class.php
@@ -112,10 +112,6 @@ class VisitTracker extends SingletonFactory
         }
 
         if (isset($this->userVisits[$objectTypeID])) {
-            // double times the lifetime period for existing visit data;
-            // equals 2 weeks for the default lifetime of 7 days
-            $minimum -= $lifetime;
-
             // using `max()` here will yield the most recent point in time
             return \max($this->userVisits[$objectTypeID], $minimum);
         }