Feature/2314 save playlist and regions in screen processor

.php-cs-fixer.dist.php

@@ -16,6 +16,7 @@
         'phpdoc_align' => false,
         'no_superfluous_phpdoc_tags' => false,
         'array_syntax' => ['syntax' => 'short'],
+        'phpdoc_to_comment' => false,
         'declare_strict_types' => true,
         ConstructorEmptyBracesFixer::name() => true,
         MultilinePromotedPropertiesFixer::name() => true,

CHANGELOG.md

@@ -6,6 +6,18 @@ All notable changes to this project will be documented in this file.
 
 - [#214](https://github.com/os2display/display-api-service/pull/214)
   - Updated endSessionUrl to be nullable.
+
+- [#213](https://github.com/os2display/display-api-service/pull/213)
+  - Set `phpdoc_to_comment` to `false`in `.php-cs-fixer.dist.php` to avoid breaking psalm ignore
+  - Add regions and groups to `ScreenInput.php`
+  - Add "cascade: persist remove" to PlaylistScreenRegion
+  - Save playlist/regions in `ScreenProcessor.php` and in `src/entity/ScreenLayoutRegions` (as an alternative to sending
+    multiple requests)
+  - Save groups in `ScreenProcessor.php` and in `src/entity/tenant/Screen.php`
+  - Update psalm baseline
+  - Add regions/playlists and groups to POST screen test
+  - `composer update symfony/* --with-dependencies`
+
 - [#193](https://github.com/os2display/display-api-service/pull/193)
   - Adds support for interactive slides.
   - Adds interactivity for creating quick bookings from a slide through Microsoft Graph.
@@ -62,8 +74,8 @@ All notable changes to this project will be documented in this file.
   - Changed route prefix to v2.
 - [#197](https://github.com/os2display/display-api-service/pull/197)
   - Fixed weight issue when assigning slides to playlist.
-- [#194](https://github.com/os2display/display-api-service/pull/194)
-  Updated test run documentation and added test for `rrule` in playlist.
+- [#194](https://github.com/os2display/display-api-service/pull/194) Updated test run documentation and added test for
+  `rrule` in playlist.
 - Fixed issue with PlaylistSlide transaction.
 - Fixed issues with feed following api platform upgrade.
 - [#192](https://github.com/os2display/display-api-service/pull/192)
@@ -85,8 +97,8 @@ All notable changes to this project will be documented in this file.
 - [#184](https://github.com/os2display/display-api-service/pull/184)
   - Added RelationsModifiedTrait to serialization groups.
 - [#182](https://github.com/os2display/display-api-service/pull/182)
-  - Changed "Theme" api output to have "Logo" embedded to avoid 404 errors when fetching logo from other shared slide
-  w. foreign tenant.
+  - Changed "Theme" api output to have "Logo" embedded to avoid 404 errors when fetching logo from other shared slide w.
+  foreign tenant.
 - [#181](https://github.com/os2display/display-api-service/pull/181)
   - Update minimum PHP version to 8.2 to support trait constants
   - Add 'relationsModified' timestamps on relevant entities and API resources.
@@ -107,16 +119,11 @@ All notable changes to this project will be documented in this file.
   - Switch from doctrine annotations to attributes
   - Add rector as dev dependency and apply rules
   - Handle doctrine deprecations
-- [#173](https://github.com/os2display/display-api-service/pull/173)
-  Upgraded to API Platform 3
-- [#172](https://github.com/os2display/display-api-service/pull/172)
-  Linted YAML API resources
-- [#171](https://github.com/os2display/display-api-service/pull/171)
-  Fixed slide playlists collection operation.
-- [#170](https://github.com/os2display/display-api-service/pull/170)
-  Updated Symfony development packages.
-- [#165](https://github.com/os2display/display-api-service/pull/165)
-  Symfony 6.3
+- [#173](https://github.com/os2display/display-api-service/pull/173) Upgraded to API Platform 3
+- [#172](https://github.com/os2display/display-api-service/pull/172) Linted YAML API resources
+- [#171](https://github.com/os2display/display-api-service/pull/171) Fixed slide playlists collection operation.
+- [#170](https://github.com/os2display/display-api-service/pull/170) Updated Symfony development packages.
+- [#165](https://github.com/os2display/display-api-service/pull/165) Symfony 6.3
 - [#162](https://github.com/os2display/display-api-service/pull/162)
   - Adds "external" openid-connect provider.
   - Renamed "oidc" openid-connect provider to "internal".
@@ -126,9 +133,9 @@ All notable changes to this project will be documented in this file.
   - Upgrades openid-connect bundle to 3.1 to support multiple providers.
   - Changes php requirement in composer.json to >= 8.1.
   - Removed PHP Upgrade coding standards github actions check.
-  - Changed user identifier from email to providerId. Made email nullable. Copied value from email to providerId in migration.
-- [#161](https://github.com/os2display/display-api-service/pull/161)
-  Fixed non-entity related psalm errors.
+  - Changed user identifier from email to providerId. Made email nullable. Copied value from email to providerId in
+    migration.
+- [#161](https://github.com/os2display/display-api-service/pull/161) Fixed non-entity related psalm errors.
 
 ## [1.5.0] - 2023-10-26
 
@@ -205,7 +212,8 @@ All notable changes to this project will be documented in this file.
 - [#138](https://github.com/os2display/display-api-service/pull/138)
   - Fixed Tenant and command to allow for empty fallbackImageUrl.
 - [#139](https://github.com/os2display/display-api-service/pull/139)
-  - Changed from service decoration to event listeners to re-enable setting `tenants` on the response from `/v1/authentication/token`.
+  - Changed from service decoration to event listeners to re-enable setting `tenants` on the response from
+    `/v1/authentication/token`.
   - Ensure same response data from both `/v1/authentication/token` and `/v1/authentication/token/refresh`endpoints.
   - Added `user` and `tenants` to JWT payload.
 

README.md

@@ -258,16 +258,16 @@ the coding standard for the project.
 
 - PHP files [PHP Coding Standards Fixer](https://cs.symfony.com/)
 
-    ```shell
-    docker compose exec phpfpm composer coding-standards-check
-    ```
+```shell
+docker compose exec phpfpm composer coding-standards-check
+```
 
 - Markdown files (markdownlint standard rules)
 
-    ```shell
-    docker run --rm -v .:/app --workdir=/app node:20 npm install
-    docker run --rm -v .:/app --workdir=/app node:20 npm run coding-standards-check
-    ```
+```shell
+docker run --rm -v .:/app --workdir=/app node:20 npm install
+docker run --rm -v .:/app --workdir=/app node:20 npm run coding-standards-check
+```
 
 #### YAML
 
@@ -281,23 +281,24 @@ To attempt to automatically fix coding style issues
 
 - PHP files [PHP Coding Standards Fixer](https://cs.symfony.com/)
 
-    ```sh
-    docker compose exec phpfpm composer coding-standards-apply
-    ```
+```sh
+docker compose exec phpfpm composer coding-standards-apply
+```
 
 - Markdown files (markdownlint standard rules)
 
-    ```shell
-    docker run --rm -v .:/app --workdir=/app node:18 npm install
-    docker run --rm -v .:/app --workdir=/app node:18 npm run coding-standards-apply
-    ```
+```shell
+docker run --rm -v .:/app --workdir=/app node:18 npm install
+docker run --rm -v .:/app --workdir=/app node:18 npm run coding-standards-apply
+```
 
 ## Tests
 
 Run automated tests:
 
 ```shell
-docker compose exec phpfpm composer tests
+docker compose exec phpfpm composer test setup
+docker compose exec phpfpm composer test
 ```
 
 Disable or hide deprecation warnings using the [`SYMFONY_DEPRECATIONS_HELPER` environment