From cc9dbbccb76f32066f153b7635193cbd07efdd16 Mon Sep 17 00:00:00 2001
From: Moritz Strohm <strohm@data-quest.de>
Date: Wed, 10 Sep 2025 11:18:51 +0000
Subject: TimedFolder: added one minute of time in the checks in isVisible,
 isReadable and isWritable, really fixes #5874

Closes #5874

Merge request studip/studip!4482
---
 lib/filesystem/TimedFolder.php                    | 26 +++++++++++++++++------
 templates/filesystem/timed_folder/description.php |  8 +++----
 2 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/lib/filesystem/TimedFolder.php b/lib/filesystem/TimedFolder.php
index 255c49c..ea72599 100644
--- a/lib/filesystem/TimedFolder.php
+++ b/lib/filesystem/TimedFolder.php
@@ -56,6 +56,19 @@ class TimedFolder extends PermissionEnabledFolder
     }
 
     /**
+     * A helper method to calculate the real end time that is used in time comparisons.
+     * Since the end time is a timestamp that only contains hours and minutes, the folder
+     * will be invisible, unreadable and unwritable in the minute in that timestamp unless
+     * one more minute is added.
+     *
+     * @returns int The real end time. This is the end timestamp + 60.
+     */
+    protected function getRealEndTime() : int
+    {
+        return $this->end_time + 60;
+    }
+
+    /**
      * Is the current folder visible for the given user?
      * That depends on parent folder visibility and time settings.
      *
@@ -65,9 +78,10 @@ class TimedFolder extends PermissionEnabledFolder
     public function isVisible($user_id = null)
     {
         $now = time();
+        $real_end_time = $this->getRealEndTime();
         return (
                 ($this->start_time == 0 || $this->start_time <= $now) &&
-                    ($this->end_time == 0 || $this->end_time >= $now)
+                    ($this->end_time == 0 || $real_end_time > $now)
                 ||
                 $GLOBALS['perm']->have_studip_perm($this->must_have_perm, $this->range_id, $user_id)) &&
             parent::isVisible($user_id);
@@ -76,9 +90,10 @@ class TimedFolder extends PermissionEnabledFolder
     public function isReadable($user_id = null)
     {
         $now = time();
+        $real_end_time = $this->getRealEndTime();
         return (
                 ($this->start_time == 0 || $this->start_time <= $now) &&
-                ($this->end_time == 0 || $this->end_time >= $now)
+                ($this->end_time == 0 || $real_end_time > $now)
                 ||
                 $GLOBALS['perm']->have_studip_perm($this->must_have_perm, $this->range_id, $user_id)) &&
             StandardFolder::isReadable($user_id);
@@ -87,9 +102,10 @@ class TimedFolder extends PermissionEnabledFolder
     public function isWritable($user_id = null)
     {
         $now = time();
+        $real_end_time = $this->getRealEndTime();
         return (
                 ($this->start_time == 0 || $this->start_time <= $now) &&
-                ($this->end_time == 0 || $this->end_time >= $now)
+                ($this->end_time == 0 || $real_end_time > $now)
                 ||
                 $GLOBALS['perm']->have_studip_perm($this->must_have_perm, $this->range_id, $user_id)) &&
             parent::isWritable($user_id);
@@ -185,10 +201,6 @@ class TimedFolder extends PermissionEnabledFolder
         $this->folderdata['data_content']['permission'] = $permvalue;
         $start = strtotime($request['start_time']);
         $end = strtotime($request['end_time']);
-        if (date('i', $end) === '59') {
-            //Add 59 seconds to $end to get to the last second of the minute:
-            $end += 59;
-        }
 
         if (!$start && !$end) {
 
diff --git a/templates/filesystem/timed_folder/description.php b/templates/filesystem/timed_folder/description.php
index 396a286..9142175 100644
--- a/templates/filesystem/timed_folder/description.php
+++ b/templates/filesystem/timed_folder/description.php
@@ -5,20 +5,20 @@
 <? if ($folder->start_time && !$folder->end_time) : ?>
     <strong>
         <?= sprintf(_('Sichtbar ab %s'),
-            strftime('%x %X', $folder->start_time)) ?>
+            date('d.m.Y H:i', $folder->start_time)) ?>
     </strong>
     <br>
 <? elseif (!$folder->start_time && $folder->end_time) : ?>
     <strong>
         <?= sprintf(_('Sichtbar bis %s'),
-            strftime('%x %X', $folder->end_time)) ?>
+            date('d.m.Y H:i', $folder->end_time)) ?>
     </strong>
     <br>
 <? elseif ($folder->start_time && $folder->end_time) : ?>
     <strong>
         <?= sprintf(_('Sichtbar von %s bis %s'),
-            strftime('%x %X', $folder->start_time),
-            strftime('%x %X', $folder->end_time)) ?>
+            date('d.m.Y H:i', $folder->start_time),
+            date('d.m.Y H:i', $folder->end_time)) ?>
     </strong>
     <br>
 <? endif ?>
-- 
cgit v1.0