feat: enable caching for for camps/{campId}/checklists

api/config/packages/http_cache.yaml

@@ -1,5 +1,5 @@
 parameters:
-    app.httpCache.matchPath: '^/?($|index.jsonhal$|content_types|camps/[0-9a-f]*/categories)'
+    app.httpCache.matchPath: '^/?($|index.jsonhal$|content_types|camps/[0-9a-f]*/categories|periods/[0-9a-f]*/schedule_entries|camps/[0-9a-f]*/activities|camps/[0-9a-f]*/checklists)'
 
 fos_http_cache:
     debug:

api/migrations/dev-data/Version202504121559.php

@@ -16,14 +16,6 @@ public function getDescription(): string {
 
     public function up(Schema $schema): void {
         // START PHP CODE
-        $this->addSql(createTruncateDatabaseCommand());
-
-        $statements = getStatementsForMigrationFile();
-        foreach ($statements as $statement) {
-            if (trim($statement)) {
-                $this->addSql($statement);
-            }
-        }
         // END PHP CODE
     }
 

api/migrations/dev-data/Version202504130046.php

@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DataMigrations;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+require_once __DIR__.'/helpers.php';
+
+final class Version202504130046 extends AbstractMigration {
+    public function getDescription(): string {
+        return 'Add 2nd period to Harry Potter camp';
+    }
+
+    public function up(Schema $schema): void {
+        // START PHP CODE
+        $this->addSql(createTruncateDatabaseCommand());
+
+        $statements = getStatementsForMigrationFile();
+        foreach ($statements as $statement) {
+            if (trim($statement)) {
+                $this->addSql($statement);
+            }
+        }
+        // END PHP CODE
+    }
+
+    public function down(Schema $schema): void {}
+}

api/migrations/dev-data/data.sql

@@ -1976,7 +1976,8 @@ INSERT INTO public.period (id, description, start, "end", createtime, updatetime
 	('c085d1d5ddfa', 'Die Jedi-Akademie', '2026-07-14', '2026-07-16', '2023-08-13 06:32:29', '2023-08-13 06:32:29', '0969e3c95dfc'),
 	('7fa4564a5d5d', 'Main', '2031-01-24', '2031-01-30', '2023-09-29 23:24:38', '2023-09-29 23:24:38', '70ca971c992f'),
 	('88f1f55a69d7', 'Hauptlager', '2025-07-13', '2025-07-20', '2023-08-08 09:22:58', '2023-08-08 09:48:01', '5d28f99890bc'),
-	('05938f2a5372', 'Hauptlager', '2022-01-02', '2022-01-02', '2024-09-28 21:19:13', '2024-09-28 21:22:24', '25a82475e0b7');
+	('05938f2a5372', 'Hauptlager', '2022-01-02', '2022-01-02', '2024-09-28 21:19:13', '2024-09-28 21:22:24', '25a82475e0b7'),
+	('c550b8707c26', 'Nachweekend', '2025-08-09', '2025-08-10', '2025-04-12 22:30:55', '2025-04-12 22:30:55', '6973c230d6b1');
 
 
 
@@ -2020,7 +2021,9 @@ INSERT INTO public.day (id, dayoffset, createtime, updatetime, periodid) VALUES
 	('273358ce9d70', 5, '2023-08-08 09:22:58', '2023-08-08 09:22:58', '88f1f55a69d7'),
 	('40e3a286dc08', 6, '2023-08-08 09:22:58', '2023-08-08 09:22:58', '88f1f55a69d7'),
 	('485e190f4852', 7, '2023-08-08 09:22:58', '2023-08-08 09:22:58', '88f1f55a69d7'),
-	('fb406d22f8b9', 0, '2024-09-28 21:19:13', '2024-09-28 21:19:13', '05938f2a5372');
+	('fb406d22f8b9', 0, '2024-09-28 21:19:13', '2024-09-28 21:19:13', '05938f2a5372'),
+	('be09cc00bfb5', 0, '2025-04-12 22:30:55', '2025-04-12 22:30:55', 'c550b8707c26'),
+	('ee1a47697fd0', 1, '2025-04-12 22:30:55', '2025-04-12 22:30:55', 'c550b8707c26');
 
 
 

api/src/State/PeriodPersistProcessor.php

@@ -8,13 +8,15 @@
 use App\Entity\Period;
 use App\State\Util\AbstractPersistProcessor;
 use App\Util\DateTimeUtil;
+use FOS\HttpCacheBundle\CacheManager;
 
 /**
  * @template-extends AbstractPersistProcessor<Period>
  */
 class PeriodPersistProcessor extends AbstractPersistProcessor {
     public function __construct(
-        ProcessorInterface $decorated
+        ProcessorInterface $decorated,
+        private readonly CacheManager $cacheManager
     ) {
         parent::__construct($decorated);
     }
@@ -23,26 +25,29 @@ public function __construct(
      * @param Period $data
      */
     public function onBefore($data, Operation $operation, array $uriVariables = [], array $context = []): Period {
-        self::moveDaysAndScheduleEntries($data, $context['previous_data'] ?? null);
+        $this->moveDaysAndScheduleEntries($data, $context['previous_data'] ?? null);
         self::removeExtraDays($data);
         self::addMissingDays($data);
 
         return $data;
     }
 
-    public static function moveDaysAndScheduleEntries(Period $period, ?Period $originalPeriod = null) {
+    public function moveDaysAndScheduleEntries(Period $period, ?Period $originalPeriod = null) {
         if (!$originalPeriod) {
             return;
         }
 
-        // moveScheduleEntries === true: scheduleEntries move relative to the start date (no change of offset needed -> return)
-        // moveScheduleEntries === false: scheduleEntries stay absolutely on the scheduled calendar date (change of offset needed)
-        if ($period->moveScheduleEntries) {
+        $deltaMinutes = DateTimeUtil::differenceInMinutes($originalPeriod->start, $period->start);
+        if (0 === $deltaMinutes) {
             return;
         }
 
-        $deltaMinutes = DateTimeUtil::differenceInMinutes($originalPeriod->start, $period->start);
-        if (0 === $deltaMinutes) {
+        // start date shifts --> purge schedule_entries subresource to reflect changes in dates + numbers
+        $this->cacheManager->invalidateTags(["/api/periods/{$period->getId()}/schedule_entries"]);
+
+        // moveScheduleEntries === true: scheduleEntries move relative to the start date (no change of offset needed -> return)
+        // moveScheduleEntries === false: scheduleEntries stay absolutely on the scheduled calendar date (change of offset needed)
+        if ($period->moveScheduleEntries) {
             return;
         }
 

api/tests/State/PeriodPersistProcessorTest.php

@@ -10,8 +10,7 @@
 use App\Entity\Period;
 use App\Entity\ScheduleEntry;
 use App\State\PeriodPersistProcessor;
-use Doctrine\ORM\EntityManagerInterface;
-use PHPUnit\Framework\MockObject\MockObject;
+use FOS\HttpCacheBundle\CacheManager;
 use PHPUnit\Framework\TestCase;
 
 /**
@@ -23,8 +22,6 @@ class PeriodPersistProcessorTest extends TestCase {
     private ScheduleEntry $scheduleEntry;
     private DayResponsible $dayResponsible;
 
-    private EntityManagerInterface|MockObject $em;
-
     private PeriodPersistProcessor $processor;
 
     /**
@@ -53,11 +50,11 @@ protected function setUp(): void {
         $day2->addDayResponsible($this->dayResponsible);
 
         $decoratedProcessor = $this->createMock(ProcessorInterface::class);
-        $this->em = $this->createMock(EntityManagerInterface::class);
+        $cacheManager = $this->createMock(CacheManager::class);
 
         $this->processor = new PeriodPersistProcessor(
             $decoratedProcessor,
-            $this->em
+            $cacheManager
         );
     }
 

e2e/specs/constants.js

@@ -2,6 +2,14 @@ export const cachedEndpoint = Cypress.env('API_ROOT_URL_CACHED')
 
 export const grgrCampId = '3c79b99ab424'
 export const loremIpsumCampId = '9c2447aefe38'
+export const skilagerCampId = '70ca971c992f'
+export const basiskursCampId = '5d28f99890bc'
+
+export const skilagerPeriodId = '7fa4564a5d5d'
+export const grgrPeriodId = '76be24bce434'
+
+export const harryMainPeriodId = 'fe47dfd2b541'
+export const harrySecondPeriodId = 'c550b8707c26'
 
 export const bipiUser = '[email protected]'
 export const castorUser = '[email protected]'