Merge pull request #32 from beatrycze-volk/fix-setcookie

[BUGFIX] Fix calls to `setcookie`
slub · Oct 2, 2024 · 8bd0509 · 8bd0509
2 parents 8a0a680 + 4dc39b7
commit 8bd0509
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/Classes/Controller/BasketController.php b/Classes/Controller/BasketController.php
@@ -100,7 +100,7 @@ public function initializeAction()
                 foreach ($documents as $document) {
                     $kitodoDocuments[] = $document->getRecordId();
                 }
-                setcookie('dlf-requests', json_encode($kitodoDocuments), '', '/');
+                setcookie('dlf-requests', json_encode($kitodoDocuments), 0, '/');
             }
         }
 
@@ -234,7 +234,7 @@ public function requestAction()
                 }
             }
 
-            setcookie('dlf-requests', '[]', '', '/');
+            setcookie('dlf-requests', '[]', 0, '/');
 
             $message = !empty($this->requestParams['message']) ? $this->requestParams['message'] : '';
             $this->sendRequestAccessMail($documents, $message);

diff --git a/Classes/ViewHelpers/RedirectViewHelper.php b/Classes/ViewHelpers/RedirectViewHelper.php
@@ -85,7 +85,7 @@ protected static function setCookie(string $newKitodoRequestId) {
         // add ID to cookie 'dlf-request' if ID is not already contained
         if (!in_array($newKitodoRequestId, $kitodoRequestIds)) {
             array_push($kitodoRequestIds, $newKitodoRequestId);
-            setcookie('dlf-requests', json_encode($kitodoRequestIds), '', '/');
+            setcookie('dlf-requests', json_encode($kitodoRequestIds), 0, '/');
         }
     }
 }