mirror of
				https://gitlab.com/Chill-Projet/chill-bundles.git
				synced 2025-10-31 01:08:26 +00:00 
			
		
		
		
	Compare commits
	
		
			62 Commits
		
	
	
		
			v4.2.0
			...
			452-workfl
		
	
	| Author | SHA1 | Date | |
|---|---|---|---|
| 2221d17930 | |||
| 9c2abb2dfa | |||
| 94744b9542 | |||
| f42bb498e4 | |||
| 01889ac671 | |||
| 62e5842311 | |||
| 8ad6f397a8 | |||
| d713704633 | |||
| b1fa9242a0 | |||
| 6ac554f93a | |||
| 372d8e5825 | |||
| 10f05e5559 | |||
| ddb2a65419 | |||
| 8d40a8089f | |||
| e1bf4a24d2 | |||
| 208a378185 | |||
| 9089c8959b | |||
| 1b9b581c31 | |||
| aa1abe4c88 | |||
| d82c9cc9a7 | |||
| a7e3b1c5d2 | |||
| 84cf11933d | |||
| bc2fbee5c6 | |||
| ebd10ca522 | |||
| d3a31be412 | |||
| d159a82f88 | |||
| c2d9c73fd4 | |||
| 0d6d15fcf7 | |||
| f9ad96c78b | |||
| fcc9529a20 | |||
| 955cb817c4 | |||
| 823f9546b9 | |||
| be39fa16e7 | |||
| c8bb7575e7 | |||
|  | 80a3734171 | ||
| ab98f3a102 | |||
| 7516e68d77 | |||
| 7b60b7a8af | |||
| d984dec7db | |||
| 46a4dedab8 | |||
| db98519e65 | |||
| c39637180a | |||
| 15f9409bc8 | |||
| 5b90d23367 | |||
| c48625d1cd | |||
| 1195b54a68 | |||
| 2a280b814f | |||
| 230c758255 | |||
| eafda987ae | |||
| 7db8a371fc | |||
| 0d0649dd31 | |||
| ac12b8cdcf | |||
| 9c1611d052 | |||
| 90e3043c3d | |||
| af13bf9088 | |||
| 4aa65d69c7 | |||
| 9e33aec594 | |||
| f88bc7e9f0 | |||
| 8e78c41549 | |||
| 6e36771349 | |||
| dfab223391 | |||
| 539752485c | 
							
								
								
									
										7
									
								
								.changes/unreleased/DX-20251027-150053.yaml
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										7
									
								
								.changes/unreleased/DX-20251027-150053.yaml
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,7 @@ | ||||
| kind: DX | ||||
| body: | | ||||
|     Send notifications log to dedicated channel, if it exists | ||||
| time: 2025-10-27T15:00:53.309372316+01:00 | ||||
| custom: | ||||
|     Issue: "" | ||||
|     SchemaChange: No schema change | ||||
							
								
								
									
										6
									
								
								.changes/unreleased/UX-20251029-110804.yaml
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										6
									
								
								.changes/unreleased/UX-20251029-110804.yaml
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,6 @@ | ||||
| kind: UX | ||||
| body: Improve the ux for selecting whether user wants to be notified of the final step of a workflow or all steps | ||||
| time: 2025-10-29T11:08:04.077020411+01:00 | ||||
| custom: | ||||
|     Issue: "542" | ||||
|     SchemaChange: No schema change | ||||
| @@ -1,9 +1,10 @@ | ||||
| ## v4.2.0 - 2025-09-02 | ||||
| ### Feature | ||||
| * ([#64](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/64)) Add external identifier for a Person    | ||||
| * ([#64](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/64)) Add external identifier for a Person | ||||
|  | ||||
|   **Schema Change**: Add columns or tables | ||||
| * ([#330](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/330) Allow users to choose for which notifications they want to receive an email | ||||
| ### Fixed | ||||
| * ([#422](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/422)) Fixed html layout of pages for recovering password    | ||||
| * Fix typo in 'uncheckAll' script for centers selection    | ||||
| * Fix incorrect parameter name in event details link    | ||||
| * ([#422](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/422)) Fixed html layout of pages for recovering password | ||||
| * Fix typo in 'uncheckAll' script for centers selection | ||||
| * Fix incorrect parameter name in event details link | ||||
|   | ||||
							
								
								
									
										6
									
								
								.changes/v4.2.1.md
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										6
									
								
								.changes/v4.2.1.md
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,6 @@ | ||||
| ## v4.2.1 - 2025-09-03 | ||||
| ### Fixed | ||||
| * Fix exports to work with DirectExportInterface    | ||||
| ### DX | ||||
| * Improve error message when a stored object cannot be written on local disk | ||||
|     | ||||
							
								
								
									
										10
									
								
								.changes/v4.3.0.md
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										10
									
								
								.changes/v4.3.0.md
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,10 @@ | ||||
| ## v4.3.0 - 2025-09-08 | ||||
| ### Feature | ||||
| * ([#409](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/409)) Add 45 and 60 min calendar ranges    | ||||
| * Add a command to generate a list of permissions    | ||||
| * ([#412](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/412)) Add an absence end date    | ||||
|  | ||||
|   **Schema Change**: Add columns or tables | ||||
| ### Fixed | ||||
| * fix date formatting in calendar range display    | ||||
| * Change route URL to avoid clash with person duplicate controller method    | ||||
							
								
								
									
										8
									
								
								.changes/v4.4.0.md
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										8
									
								
								.changes/v4.4.0.md
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,8 @@ | ||||
| ## v4.4.0 - 2025-09-11 | ||||
| ### Feature | ||||
| * ([#359](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/359)) Allow the merge of two accompanying period works    | ||||
| * ([#369](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/369)) Duplication of a document to another accompanying period work evaluation    | ||||
| * ([#359](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/359)) Fusion of two accompanying period works    | ||||
| ### Fixed | ||||
| * Fix display of 'duplicate' and 'merge' buttons in CRUD templates    | ||||
| * Fix saving notification preferences in user's profile    | ||||
							
								
								
									
										3
									
								
								.changes/v4.4.1.md
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										3
									
								
								.changes/v4.4.1.md
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,3 @@ | ||||
| ## v4.4.1 - 2025-09-11 | ||||
| ### Fixed | ||||
| * fix translations in duplicate evaluation document modal and realign close modal button    | ||||
							
								
								
									
										3
									
								
								.changes/v4.4.2.md
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										3
									
								
								.changes/v4.4.2.md
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,3 @@ | ||||
| ## v4.4.2 - 2025-09-12 | ||||
| ### Fixed | ||||
| * Fix document generation and workflow generation do not work on accompanying period work documents    | ||||
							
								
								
									
										13
									
								
								.changes/v4.5.0.md
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										13
									
								
								.changes/v4.5.0.md
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,13 @@ | ||||
| ## v4.5.0 - 2025-10-03 | ||||
| ### Feature | ||||
| * Only allow delete of attachment on workflows that are not final    | ||||
| * Move up signature buttons on index workflow page for easier access    | ||||
| * Filter out document from attachment list if it is the same as the workflow document    | ||||
| * Block edition on attached document on workflow, if the workflow is finalized or sent external    | ||||
| * Convert workflow's attached document to pdf while sending them external    | ||||
| * After a signature is canceled or rejected, going to a waiting page until the post-process routines apply a workflow transition    | ||||
| ### Fixed | ||||
| * ([#426](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/426)) Increased the number of required characters when setting a new password in Chill from 9 to 14 - GDPR compliance    | ||||
| * Fix permissions on storedObject which are subject by a workflow    | ||||
| ### DX | ||||
| * Introduce a WaitingScreen component to display a waiting screen    | ||||
							
								
								
									
										4
									
								
								.changes/v4.5.1.md
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										4
									
								
								.changes/v4.5.1.md
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,4 @@ | ||||
| ## v4.5.1 - 2025-10-03 | ||||
| ### Fixed | ||||
| * Add missing javascript dependency    | ||||
| * Add exception handling for conversion of attachment on sending external, when documens are already in pdf    | ||||
							
								
								
									
										14
									
								
								.changes/v4.6.0.md
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										14
									
								
								.changes/v4.6.0.md
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,14 @@ | ||||
| ## v4.6.0 - 2025-10-15 | ||||
| ### Feature | ||||
| * ([#423](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/423)) Create environment banner that can be activated and configured depending on the image deployed    | ||||
| * ([#394](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/394)) Only show active workflow on the page "my tracked workflow"    | ||||
| ### Fixed | ||||
| * Fix loading of classLists in SocialIssuesAcc.vue, ensure elements are present    | ||||
| * Fix the rendering of list of StoredObjectVersions, where there are kept version (before converting to pdf) and intermediate versions deleted    | ||||
| * ([#434](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/434)) Notification: fix editing of sent notification by removing form.addressesEmails, a field that no longer exists    | ||||
| * Fix loading of social issues and social actions within vue component    | ||||
| * ([#446](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/446)) Add unique condition on stored object filename, with cleaning step on existing duplicate filenames    | ||||
|  | ||||
|   **Schema Change**: Drop or rename table or columns, or enforce new constraint that must be manually fixed | ||||
| * [workflow] take permissions into account to delete the workflow attachment    | ||||
| * ([#448](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/448)) Fix the execution of daily cronjob notification, when the previous last execution storage was invalid    | ||||
							
								
								
									
										3
									
								
								.changes/v4.6.1.md
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										3
									
								
								.changes/v4.6.1.md
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,3 @@ | ||||
| ## v4.6.1 - 2025-10-27 | ||||
| ### Fixed | ||||
| * Fix export case where no 'reason' is picked within the PersonHavingActivityBetweenDateFilter.php    | ||||
| @@ -27,11 +27,11 @@ Chill is a comprehensive web application built as a set of Symfony bundles. It i | ||||
|  | ||||
| ## Project Structure | ||||
|  | ||||
| Note: This is a project which exists from a long time ago, and we found multiple structure inside each bundle. When having the choice, the developers should choose the new structure. | ||||
| Note: This is a project that's existed for a long time, and throughout the years we've used multiple structures inside each bundle. When having the choice, the developers should choose the new structure. | ||||
|  | ||||
| The project follows a standard Symfony bundle structure: | ||||
| - `/src/Bundle/`: Contains all the Chill bundles. The code is either at the root of the bundle directory, or within a `src/` directory (preferred). See psr4 mapping at the root's `composer.json`. | ||||
| - each bundle come with his own tests, either in the `Tests` directory (when the code is directly within the bundle directory (for instance `src/Bundle/ChillMainBundle/Tests`, `src/Bundle/ChillPersonBundle/Tests`)), or inside the `tests` directory, alongside to the `src/` sub-directory (example: `src/Bundle/ChillWopiBundle/tests`) (this is the preferred way). | ||||
| - each bundle comes with its own tests, either in the `Tests` directory (when the code is directly within the bundle directory (for instance `src/Bundle/ChillMainBundle/Tests`, `src/Bundle/ChillPersonBundle/Tests`)), or inside the `tests` directory, alongside the `src/` sub-directory (example: `src/Bundle/ChillWopiBundle/tests`) (this is the preferred way). | ||||
| - `/docs/`: Contains project documentation | ||||
|  | ||||
| Each bundle typically has the following structure: | ||||
| @@ -46,13 +46,13 @@ Each bundle typically has the following structure: | ||||
|  | ||||
| ### A special word about TicketBundle | ||||
|  | ||||
| The ticket bundle is developed using a kind of "Command" pattern. The controller fill a "Command", and a "CommandHandler" handle this command. They are savec in the `src/Bundle/ChillTicketBundle/src/Action` directory. | ||||
| The ticket bundle is developed using a kind of "Command" pattern. The controller fills a "Command," and a "CommandHandler" handles this command. They are saved in the `src/Bundle/ChillTicketBundle/src/Action` directory. | ||||
|  | ||||
| ## Development Guidelines | ||||
|  | ||||
| ### Building and Configuration Instructions | ||||
|  | ||||
| All the command should be run through the `symfony` command, which will configure the required variables. | ||||
| All the commands should be run through the `symfony` command, which will configure the required variables. | ||||
|  | ||||
| For assets, we must ensure that we use node at version `^20.0.0`. This is done using `nvm use 20`. | ||||
|  | ||||
| @@ -87,7 +87,7 @@ For assets, we must ensure that we use node at version `^20.0.0`. This is done u | ||||
|    docker compose up -d | ||||
|    ``` | ||||
|  | ||||
| 5. **Set Up the Database**: | ||||
| 6. **Set Up the Database**: | ||||
|    ```bash | ||||
|    # Create the database | ||||
|    symfony console doctrine:database:create | ||||
| @@ -99,20 +99,20 @@ For assets, we must ensure that we use node at version `^20.0.0`. This is done u | ||||
|    symfony console doctrine:fixtures:load | ||||
|    ``` | ||||
|  | ||||
| 6. **Build Assets**: | ||||
| 7. **Build Assets**: | ||||
|    ```bash | ||||
|    nvm use 20 | ||||
|    yarn run encore dev | ||||
|    ``` | ||||
|  | ||||
| 7. **Start the Development Server**: | ||||
| 8. **Start the Development Server**: | ||||
|    ```bash | ||||
|    symfony server:start -d | ||||
|    ``` | ||||
|  | ||||
| #### Docker Setup | ||||
|  | ||||
| The project includes Docker configuration for easier development: | ||||
| The project includes a Docker configuration for easier development: | ||||
|  | ||||
| 1. **Start Docker Services**: | ||||
|    ```bash | ||||
| @@ -153,9 +153,9 @@ Key configuration files: | ||||
|  | ||||
| Each time a doctrine entity is created, we generate migration to adapt the database. | ||||
|  | ||||
| The migration are created using the command `symfony console doctrine:migrations:diff --no-interaction --namespace <namespace>`, where the namespace is the relevant namespace for migration. As this is a bash script, do not forget to quote the `\` (`\` must become `\\` in your command). | ||||
| The migration is created using the command `symfony console doctrine:migrations:diff --no-interaction --namespace <namespace>`, where the namespace is the relevant namespace for migration. As this is a bash script, remember to quote the `\` (`\` must become `\\` in your command). | ||||
|  | ||||
| Each bundle has his own namespace for migration (always ask me to confirm that command, with a list of updated / created entities so that I can confirm you that it is ok): | ||||
| Each bundle has his own namespace for migration (always ask me to confirm that command with a list of updated / created entities so that I can confirm to you that it is ok): | ||||
|  | ||||
| - `Chill\Bundle\ActivityBundle` writes migrations to `Chill\Migrations\Activity`; | ||||
| - `Chill\Bundle\BudgetBundle` writes migrations to `Chill\Migrations\Budget`; | ||||
| @@ -183,7 +183,7 @@ Once created the, comment's classes should be removed and a description of the c | ||||
|  | ||||
| When we need to use a DateTime or DateTimeImmutable that need to express "now", we prefer the usage of | ||||
| `Symfony\Component\Clock\ClockInterface`, where possible. This is usually not possible in doctrine entities, | ||||
| where injection does not work when restoring an entity from database, but usually possible in services. | ||||
| where injection does not work when restoring an entity from a database, but usually possible in services. | ||||
|  | ||||
| In test, we use `\Symfony\Component\Clock\MockClock` which is an implementation of `Symfony\Component\Clock\ClockInterface` | ||||
| where we have full and easy control of the date. | ||||
| @@ -198,9 +198,9 @@ The project uses PHPUnit for testing. Each bundle has its own test suite, and th | ||||
|  | ||||
| For creating mock, we prefer using prophecy (library phpspec/prophecy). | ||||
|  | ||||
| ##### Useful helpers and tips that avoid create a mock | ||||
| ##### Useful helpers and tips that avoid creating a mock | ||||
|  | ||||
| Some notable implementations that are tests helper, and avoid to create a mock: | ||||
| Some notable implementations that are test helpers and avoid creating a mock: | ||||
|  | ||||
| - `\Psr\Log\NullLogger`, an implementation of `\Psr\Log\LoggerInterface`; | ||||
| - `\Symfony\Component\Clock\MockClock`, an implementation of `Symfony\Component\Clock\ClockInterface` (already mentioned above); | ||||
| @@ -240,9 +240,6 @@ The tests are run from the project's root (not from the bundle's root). | ||||
| # Run all tests | ||||
| vendor/bin/phpunit | ||||
|  | ||||
| # Run tests for a specific bundle | ||||
| vendor/bin/phpunit --testsuite NameBundle | ||||
|  | ||||
| # Run a specific test file | ||||
| vendor/bin/phpunit path/to/TestFile.php | ||||
|  | ||||
| @@ -250,6 +247,9 @@ vendor/bin/phpunit path/to/TestFile.php | ||||
| vendor/bin/phpunit --filter methodName path/to/TestFile.php | ||||
| ``` | ||||
|  | ||||
| When writing tests, only test specific files. Do not run all tests or the full | ||||
| test suite. | ||||
|  | ||||
| #### Test Structure | ||||
|  | ||||
| Tests are organized by bundle and follow the same structure as the bundle itself: | ||||
| @@ -297,7 +297,7 @@ class TicketTest extends TestCase | ||||
|  | ||||
| #### Test Database | ||||
|  | ||||
| For tests that require a database, the project uses postgresql database filled by fixtures (usage of doctrine-fixtures). You can configure a different database for testing in the `.env.test` file. | ||||
| For tests that require a database, the project uses a postgresql database filled with fixtures (usage of doctrine-fixtures). You can configure a different database for testing in the `.env.test` file. | ||||
|  | ||||
| ### Code Quality Tools | ||||
|  | ||||
|   | ||||
							
								
								
									
										84
									
								
								CHANGELOG.md
									
									
									
									
									
								
							
							
						
						
									
										84
									
								
								CHANGELOG.md
									
									
									
									
									
								
							| @@ -6,15 +6,89 @@ adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html), | ||||
| and is generated by [Changie](https://github.com/miniscruff/changie). | ||||
|  | ||||
|  | ||||
| ## v4.2.0 - 2025-09-02 | ||||
| ## v4.6.1 - 2025-10-27 | ||||
| ### Fixed | ||||
| * Fix export case where no 'reason' is picked within the PersonHavingActivityBetweenDateFilter.php    | ||||
|  | ||||
| ## v4.6.0 - 2025-10-15 | ||||
| ### Feature | ||||
| * ([#64](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/64)) Add external identifier for a Person    | ||||
| * ([#423](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/423)) Create environment banner that can be activated and configured depending on the image deployed    | ||||
| * ([#394](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/394)) Only show active workflow on the page "my tracked workflow"    | ||||
| ### Fixed | ||||
| * Fix loading of classLists in SocialIssuesAcc.vue, ensure elements are present    | ||||
| * Fix the rendering of list of StoredObjectVersions, where there are kept version (before converting to pdf) and intermediate versions deleted    | ||||
| * ([#434](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/434)) Notification: fix editing of sent notification by removing form.addressesEmails, a field that no longer exists    | ||||
| * Fix loading of social issues and social actions within vue component    | ||||
| * ([#446](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/446)) Add unique condition on stored object filename, with cleaning step on existing duplicate filenames    | ||||
|  | ||||
|   **Schema Change**: Drop or rename table or columns, or enforce new constraint that must be manually fixed | ||||
| * [workflow] take permissions into account to delete the workflow attachment    | ||||
| * ([#448](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/448)) Fix the execution of daily cronjob notification, when the previous last execution storage was invalid    | ||||
|  | ||||
| ## v4.5.1 - 2025-10-03 | ||||
| ### Fixed | ||||
| * Add missing javascript dependency    | ||||
| * Add exception handling for conversion of attachment on sending external, when documens are already in pdf    | ||||
|  | ||||
| ## v4.5.0 - 2025-10-03 | ||||
| ### Feature | ||||
| * Only allow delete of attachment on workflows that are not final    | ||||
| * Move up signature buttons on index workflow page for easier access    | ||||
| * Filter out document from attachment list if it is the same as the workflow document    | ||||
| * Block edition on attached document on workflow, if the workflow is finalized or sent external    | ||||
| * Convert workflow's attached document to pdf while sending them external    | ||||
| * After a signature is canceled or rejected, going to a waiting page until the post-process routines apply a workflow transition    | ||||
| ### Fixed | ||||
| * ([#426](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/426)) Increased the number of required characters when setting a new password in Chill from 9 to 14 - GDPR compliance    | ||||
| * Fix permissions on storedObject which are subject by a workflow    | ||||
| ### DX | ||||
| * Introduce a WaitingScreen component to display a waiting screen    | ||||
|  | ||||
| ## v4.4.2 - 2025-09-12 | ||||
| ### Fixed | ||||
| * Fix document generation and workflow generation do not work on accompanying period work documents    | ||||
|  | ||||
| ## v4.4.1 - 2025-09-11 | ||||
| ### Fixed | ||||
| * fix translations in duplicate evaluation document modal and realign close modal button    | ||||
|  | ||||
| ## v4.4.0 - 2025-09-11 | ||||
| ### Feature | ||||
| * ([#359](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/359)) Allow the merge of two accompanying period works    | ||||
| * ([#369](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/369)) Duplication of a document to another accompanying period work evaluation    | ||||
| * ([#359](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/359)) Fusion of two accompanying period works    | ||||
| ### Fixed | ||||
| * Fix display of 'duplicate' and 'merge' buttons in CRUD templates    | ||||
| * Fix saving notification preferences in user's profile    | ||||
|  | ||||
| ## v4.3.0 - 2025-09-08 | ||||
| ### Feature | ||||
| * ([#409](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/409)) Add 45 and 60 min calendar ranges    | ||||
| * Add a command to generate a list of permissions    | ||||
| * ([#412](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/412)) Add an absence end date    | ||||
|  | ||||
|   **Schema Change**: Add columns or tables | ||||
| ### Fixed | ||||
| * ([#422](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/422)) Fixed html layout of pages for recovering password    | ||||
| * Fix typo in 'uncheckAll' script for centers selection    | ||||
| * Fix incorrect parameter name in event details link    | ||||
| * fix date formatting in calendar range display    | ||||
| * Change route URL to avoid clash with person duplicate controller method    | ||||
|  | ||||
| ## v4.2.1 - 2025-09-03 | ||||
| ### Fixed | ||||
| * Fix exports to work with DirectExportInterface    | ||||
| ### DX | ||||
| * Improve error message when a stored object cannot be written on local disk | ||||
|     | ||||
|  | ||||
| ## v4.2.0 - 2025-09-02 | ||||
| ### Feature | ||||
| * ([#64](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/64)) Add external identifier for a Person | ||||
|  | ||||
|   **Schema Change**: Add columns or tables | ||||
| * ([#330](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/330) Allow users to choose for which notifications they want to receive an email | ||||
| ### Fixed | ||||
| * ([#422](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/422)) Fixed html layout of pages for recovering password | ||||
| * Fix typo in 'uncheckAll' script for centers selection | ||||
| * Fix incorrect parameter name in event details link | ||||
|  | ||||
| ## v4.1.0 - 2025-08-26 | ||||
| ### Feature | ||||
|   | ||||
| @@ -1,6 +1,13 @@ | ||||
| chill_main: | ||||
|     available_languages: [ '%env(resolve:LOCALE)%', 'en' ] | ||||
|     available_countries: ['BE', 'FR'] | ||||
|     top_banner: | ||||
|         visible: false | ||||
|         text: | ||||
|             fr: 'Vous travaillez actuellement avec la version de PRÉ-PRODUCTION.' | ||||
|             nl: 'Je werkt momenteel in de PRE-PRODUCTIE versie' | ||||
|         color: '#353535' | ||||
|         background_color: '#d8bb48' | ||||
|     notifications: | ||||
|         from_email: '%env(resolve:NOTIFICATION_FROM_EMAIL)%' | ||||
|         from_name: '%env(resolve:NOTIFICATION_FROM_NAME)%' | ||||
|   | ||||
| @@ -55,6 +55,7 @@ | ||||
|     "@tsconfig/node20": "^20.1.4", | ||||
|     "@types/dompurify": "^3.0.5", | ||||
|     "@types/leaflet": "^1.9.3", | ||||
|     "@vueuse/core": "^13.9.0", | ||||
|     "bootstrap-icons": "^1.11.3", | ||||
|     "dropzone": "^5.7.6", | ||||
|     "es6-promise": "^4.2.8", | ||||
|   | ||||
| @@ -90,7 +90,9 @@ class ActivityReasonFilter implements ExportElementValidatedInterface, FilterInt | ||||
|  | ||||
|     public function getFormDefaultData(): array | ||||
|     { | ||||
|         return []; | ||||
|         return [ | ||||
|             'reasons' => [], | ||||
|         ]; | ||||
|     } | ||||
|  | ||||
|     public function describeAction($data, ExportGenerationContext $context): string|\Symfony\Contracts\Translation\TranslatableInterface|array | ||||
|   | ||||
| @@ -42,6 +42,8 @@ final readonly class PersonHavingActivityBetweenDateFilter implements ExportElem | ||||
|  | ||||
|     public function alterQuery(QueryBuilder $qb, $data, ExportGenerationContext $exportGenerationContext): void | ||||
|     { | ||||
|         error_log('alterQuery called with data: '.json_encode(array_keys($data))); | ||||
|  | ||||
|         // create a subquery for activity | ||||
|         $sqb = $qb->getEntityManager()->createQueryBuilder(); | ||||
|         $sqb->select('1') | ||||
| @@ -59,7 +61,6 @@ final readonly class PersonHavingActivityBetweenDateFilter implements ExportElem | ||||
|         if (\in_array('activity', $qb->getAllAliases(), true)) { | ||||
|             $sqb->andWhere('activity_person_having_activity.id = activity.id'); | ||||
|         } | ||||
|  | ||||
|         if (isset($data['reasons']) && [] !== $data['reasons']) { | ||||
|             // add clause activity reason | ||||
|             $sqb->join('activity_person_having_activity.reasons', 'reasons_person_having_activity'); | ||||
| @@ -124,12 +125,38 @@ final readonly class PersonHavingActivityBetweenDateFilter implements ExportElem | ||||
|  | ||||
|     public function normalizeFormData(array $formData): array | ||||
|     { | ||||
|         return ['date_from_rolling' => $formData['date_from_rolling']->normalize(), 'date_to_rolling' => $formData['date_to_rolling']->normalize()]; | ||||
|         $normalized = [ | ||||
|             'date_from_rolling' => $formData['date_from_rolling']->normalize(), | ||||
|             'date_to_rolling' => $formData['date_to_rolling']->normalize(), | ||||
|             'reasons' => [], | ||||
|         ]; | ||||
|  | ||||
|         if (isset($formData['reasons']) && [] !== $formData['reasons']) { | ||||
|             $normalized['reasons'] = array_map( | ||||
|                 fn (ActivityReason $reason) => $reason->getId(), | ||||
|                 $formData['reasons'] | ||||
|             ); | ||||
|         } | ||||
|  | ||||
|         return $normalized; | ||||
|     } | ||||
|  | ||||
|     public function denormalizeFormData(array $formData, int $fromVersion): array | ||||
|     { | ||||
|         return ['date_from_rolling' => RollingDate::fromNormalized($formData['date_from_rolling']), 'date_to_rolling' => RollingDate::fromNormalized($formData['date_to_rolling'])]; | ||||
|         $denormalized = [ | ||||
|             'date_from_rolling' => RollingDate::fromNormalized($formData['date_from_rolling']), | ||||
|             'date_to_rolling' => RollingDate::fromNormalized($formData['date_to_rolling']), | ||||
|             'reasons' => [], | ||||
|         ]; | ||||
|  | ||||
|         if (isset($formData['reasons']) && [] !== $formData['reasons']) { | ||||
|             $denormalized['reasons'] = array_map( | ||||
|                 fn ($id) => $this->activityReasonRepository->find($id), | ||||
|                 $formData['reasons'] | ||||
|             ); | ||||
|         } | ||||
|  | ||||
|         return $denormalized; | ||||
|     } | ||||
|  | ||||
|     public function getFormDefaultData(): array | ||||
| @@ -143,10 +170,12 @@ final readonly class PersonHavingActivityBetweenDateFilter implements ExportElem | ||||
|  | ||||
|     public function describeAction($data, ExportGenerationContext $context): array | ||||
|     { | ||||
|         $reasons = $data['reasons'] ?? []; | ||||
|  | ||||
|         return [ | ||||
|             [] === $data['reasons'] ? | ||||
|                 'export.filter.person_between_dates.describe_action_with_no_subject' | ||||
|                 : 'export.filter.person_between_dates.describe_action_with_subject', | ||||
|             [] === $reasons ? | ||||
|                 'export.filter.activity.describe_action_with_no_subject' | ||||
|                 : 'export.filter.activity.describe_action_with_subject', | ||||
|             [ | ||||
|                 'date_from' => $this->rollingDateConverter->convert($data['date_from_rolling']), | ||||
|                 'date_to' => $this->rollingDateConverter->convert($data['date_to_rolling']), | ||||
| @@ -154,7 +183,7 @@ final readonly class PersonHavingActivityBetweenDateFilter implements ExportElem | ||||
|                     ', ', | ||||
|                     array_map( | ||||
|                         fn (ActivityReason $r): string => '"'.$this->translatableStringHelper->localize($r->getName()).'"', | ||||
|                         $data['reasons'] | ||||
|                         $reasons | ||||
|                     ) | ||||
|                 ), | ||||
|             ], | ||||
| @@ -168,6 +197,7 @@ final readonly class PersonHavingActivityBetweenDateFilter implements ExportElem | ||||
|  | ||||
|     public function validateForm($data, ExecutionContextInterface $context): void | ||||
|     { | ||||
|         error_log('validateForm called with data: '.json_encode(array_keys($data))); | ||||
|         if ($this->rollingDateConverter->convert($data['date_from_rolling']) | ||||
|             >= $this->rollingDateConverter->convert($data['date_to_rolling'])) { | ||||
|             $context->buildViolation('export.filter.activity.person_between_dates.date mismatch') | ||||
|   | ||||
| @@ -136,8 +136,14 @@ export default { | ||||
|             issueIsLoading: false, | ||||
|             actionIsLoading: false, | ||||
|             actionAreLoaded: false, | ||||
|             socialIssuesClassList: `col-form-label ${document.querySelector("input#chill_activitybundle_activity_socialIssues").getAttribute("required") ? "required" : ""}`, | ||||
|             socialActionsClassList: `col-form-label ${document.querySelector("input#chill_activitybundle_activity_socialActions").getAttribute("required") ? "required" : ""}`, | ||||
|             socialIssuesClassList: { | ||||
|                 "col-form-label": true, | ||||
|                 required: false, | ||||
|             }, | ||||
|             socialActionsClassList: { | ||||
|                 "col-form-label": true, | ||||
|                 required: false, | ||||
|             }, | ||||
|         }; | ||||
|     }, | ||||
|     computed: { | ||||
| @@ -158,6 +164,21 @@ export default { | ||||
|         }, | ||||
|     }, | ||||
|     mounted() { | ||||
|         /* Load classNames after element is present */ | ||||
|         const socialActionsEl = document.querySelector( | ||||
|             "input#chill_activitybundle_activity_socialActions", | ||||
|         ); | ||||
|         if (socialActionsEl && socialActionsEl.hasAttribute("required")) { | ||||
|             this.socialActionsClassList.required = true; | ||||
|         } | ||||
|  | ||||
|         const socialIssuesEl = document.querySelector( | ||||
|             "input#chill_activitybundle_activity_socialIssues", | ||||
|         ); | ||||
|         if (socialIssuesEl && socialIssuesEl.hasAttribute("required")) { | ||||
|             this.socialIssuesClassList.required = true; | ||||
|         } | ||||
|  | ||||
|         /* Load other issues in multiselect */ | ||||
|         this.issueIsLoading = true; | ||||
|         this.actionAreLoaded = false; | ||||
|   | ||||
| @@ -55,5 +55,6 @@ | ||||
| 			</dl> | ||||
|  | ||||
| 		{% endblock %} | ||||
|         {% block content_view_actions_duplicate_link %}{% endblock %} | ||||
| 	{% endembed %} | ||||
| {% endblock %} | ||||
|   | ||||
| @@ -70,6 +70,8 @@ | ||||
|                         <option value="00:10:00">10 minutes</option> | ||||
|                         <option value="00:15:00">15 minutes</option> | ||||
|                         <option value="00:30:00">30 minutes</option> | ||||
|                         <option value="00:45:00">45 minutes</option> | ||||
|                         <option value="00:60:00">60 minutes</option> | ||||
|                     </select> | ||||
|                     <label class="input-group-text" for="slotMinTime">De</label> | ||||
|                     <select | ||||
|   | ||||
| @@ -32,6 +32,8 @@ | ||||
|                     <option value="00:10:00">10 minutes</option> | ||||
|                     <option value="00:15:00">15 minutes</option> | ||||
|                     <option value="00:30:00">30 minutes</option> | ||||
|                     <option value="00:45:00">45 minutes</option> | ||||
|                     <option value="00:60:00">60 minutes</option> | ||||
|                 </select> | ||||
|                 <label class="input-group-text" for="slotMinTime">De</label> | ||||
|                 <select | ||||
| @@ -102,7 +104,8 @@ | ||||
|                     event.title | ||||
|                 }}</b> | ||||
|                 <b v-else-if="event.extendedProps.is === 'range'" | ||||
|                     >{{ formatDate(event.startStr) }} - | ||||
|                     >{{ formatDate(event.startStr, "time") }} - | ||||
|                     {{ formatDate(event.endStr, "time") }}: | ||||
|                     {{ event.extendedProps.locationName }}</b | ||||
|                 > | ||||
|                 <b v-else-if="event.extendedProps.is === 'local'">{{ | ||||
| @@ -294,9 +297,26 @@ const nextWeeks = computed((): Weeks[] => | ||||
|     }), | ||||
| ); | ||||
|  | ||||
| const formatDate = (datetime: string) => { | ||||
|     console.log(typeof datetime); | ||||
|     return ISOToDate(datetime); | ||||
| const formatDate = (datetime: string, format: null | "time" = null) => { | ||||
|     const date = ISOToDate(datetime); | ||||
|     if (!date) return ""; | ||||
|  | ||||
|     if (format === "time") { | ||||
|         return date.toLocaleTimeString("fr-FR", { | ||||
|             hour: "2-digit", | ||||
|             minute: "2-digit", | ||||
|         }); | ||||
|     } | ||||
|  | ||||
|     // French date formatting | ||||
|     return date.toLocaleDateString("fr-FR", { | ||||
|         weekday: "short", | ||||
|         year: "numeric", | ||||
|         month: "short", | ||||
|         day: "numeric", | ||||
|         hour: "2-digit", | ||||
|         minute: "2-digit", | ||||
|     }); | ||||
| }; | ||||
|  | ||||
| const baseOptions = ref<CalendarOptions>({ | ||||
|   | ||||
| @@ -18,6 +18,7 @@ use Chill\DocStoreBundle\Exception\StoredObjectManagerException; | ||||
| use Chill\DocStoreBundle\Service\Cryptography\KeyGenerator; | ||||
| use Chill\DocStoreBundle\Service\StoredObjectManagerInterface; | ||||
| use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface; | ||||
| use Symfony\Component\Filesystem\Exception\IOExceptionInterface; | ||||
| use Symfony\Component\Filesystem\Filesystem; | ||||
| use Symfony\Component\Filesystem\Path; | ||||
|  | ||||
| @@ -147,16 +148,11 @@ class StoredObjectManager implements StoredObjectManagerInterface | ||||
|     public function writeContent(string $filename, string $encryptedContent): void | ||||
|     { | ||||
|         $fullPath = $this->buildPath($filename); | ||||
|         $dir = Path::getDirectory($fullPath); | ||||
|  | ||||
|         if (!$this->filesystem->exists($dir)) { | ||||
|             $this->filesystem->mkdir($dir); | ||||
|         } | ||||
|  | ||||
|         $result = file_put_contents($fullPath, $encryptedContent); | ||||
|  | ||||
|         if (false === $result) { | ||||
|             throw StoredObjectManagerException::unableToStoreDocumentOnDisk(); | ||||
|         try { | ||||
|             $this->filesystem->dumpFile($fullPath, $encryptedContent); | ||||
|         } catch (IOExceptionInterface $exception) { | ||||
|             throw StoredObjectManagerException::unableToStoreDocumentOnDisk($exception); | ||||
|         } | ||||
|     } | ||||
|  | ||||
|   | ||||
| @@ -59,7 +59,7 @@ final readonly class StoredObjectVersionApiController | ||||
|  | ||||
|         return new JsonResponse( | ||||
|             $this->serializer->serialize( | ||||
|                 new Collection($items, $paginator), | ||||
|                 new Collection(array_values($items->toArray()), $paginator), | ||||
|                 'json', | ||||
|                 [AbstractNormalizer::GROUPS => ['read', StoredObjectVersionNormalizer::WITH_POINT_IN_TIMES_CONTEXT, StoredObjectVersionNormalizer::WITH_RESTORED_CONTEXT]] | ||||
|             ), | ||||
|   | ||||
| @@ -23,10 +23,14 @@ use Random\RandomException; | ||||
|  * Store each version of StoredObject's. | ||||
|  * | ||||
|  * A version should not be created manually: use the method @see{StoredObject::registerVersion} instead. | ||||
|  * | ||||
|  * Each filename must be unique within the same StoredObject. We add a condition on id to apply this condition only for | ||||
|  * newly created versions when this new index is applied. | ||||
|  */ | ||||
| #[ORM\Entity] | ||||
| #[ORM\Table('chill_doc.stored_object_version')] | ||||
| #[ORM\UniqueConstraint(name: 'chill_doc_stored_object_version_unique_by_object', columns: ['stored_object_id', 'version'])] | ||||
| #[ORM\UniqueConstraint(name: 'chill_doc_stored_object_version_unique_by_filename', columns: ['filename'], options: ['where' => '(id > 0)'])] | ||||
| class StoredObjectVersion implements TrackCreationInterface | ||||
| { | ||||
|     use TrackCreationTrait; | ||||
|   | ||||
| @@ -0,0 +1,20 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\DocStoreBundle\Exception; | ||||
|  | ||||
| class ConversionWithSameMimeTypeException extends \RuntimeException | ||||
| { | ||||
|     public function __construct(string $mimeType, ?\Throwable $previous = null) | ||||
|     { | ||||
|         parent::__construct("Conversion to same MIME type '{$mimeType}' is not allowed: already at the same MIME type", 0, $previous); | ||||
|     } | ||||
| } | ||||
| @@ -25,7 +25,7 @@ export interface GenericDoc { | ||||
|     type: "doc_store_generic_doc"; | ||||
|     uniqueKey: string; | ||||
|     key: string; | ||||
|     identifiers: object; | ||||
|     identifiers: { id: number }; | ||||
|     context: "person" | "accompanying-period"; | ||||
|     doc_date: DateTime; | ||||
|     metadata: GenericDocMetadata; | ||||
| @@ -36,6 +36,18 @@ export interface GenericDocForAccompanyingPeriod extends GenericDoc { | ||||
|     context: "accompanying-period"; | ||||
| } | ||||
|  | ||||
| export function isGenericDocForAccompanyingPeriod( | ||||
|     doc: GenericDoc, | ||||
| ): doc is GenericDocForAccompanyingPeriod { | ||||
|     return doc.context === "accompanying-period"; | ||||
| } | ||||
|  | ||||
| export function isGenericDocWithStoredObject( | ||||
|     doc: GenericDoc, | ||||
| ): doc is GenericDoc & { storedObject: StoredObject } { | ||||
|     return doc.storedObject !== null; | ||||
| } | ||||
|  | ||||
| interface BaseMetadataWithHtml extends BaseMetadata { | ||||
|     html: string; | ||||
| } | ||||
| @@ -44,28 +56,33 @@ export interface GenericDocForAccompanyingCourseDocument | ||||
|     extends GenericDocForAccompanyingPeriod { | ||||
|     key: "accompanying_course_document"; | ||||
|     metadata: BaseMetadataWithHtml; | ||||
|     storedObject: StoredObject; | ||||
| } | ||||
|  | ||||
| export interface GenericDocForAccompanyingCourseActivityDocument | ||||
|     extends GenericDocForAccompanyingPeriod { | ||||
|     key: "accompanying_course_activity_document"; | ||||
|     metadata: BaseMetadataWithHtml; | ||||
|     storedObject: StoredObject; | ||||
| } | ||||
|  | ||||
| export interface GenericDocForAccompanyingCourseCalendarDocument | ||||
|     extends GenericDocForAccompanyingPeriod { | ||||
|     key: "accompanying_course_calendar_document"; | ||||
|     metadata: BaseMetadataWithHtml; | ||||
|     storedObject: StoredObject; | ||||
| } | ||||
|  | ||||
| export interface GenericDocForAccompanyingCoursePersonDocument | ||||
|     extends GenericDocForAccompanyingPeriod { | ||||
|     key: "person_document"; | ||||
|     metadata: BaseMetadataWithHtml; | ||||
|     storedObject: StoredObject; | ||||
| } | ||||
|  | ||||
| export interface GenericDocForAccompanyingCourseWorkEvaluationDocument | ||||
|     extends GenericDocForAccompanyingPeriod { | ||||
|     key: "accompanying_period_work_evaluation_document"; | ||||
|     metadata: BaseMetadataWithHtml; | ||||
|     storedObject: StoredObject; | ||||
| } | ||||
|   | ||||
| @@ -4,6 +4,7 @@ import { StoredObject, StoredObjectVersion } from "../../types"; | ||||
| import DropFileWidget from "ChillDocStoreAssets/vuejs/DropFileWidget/DropFileWidget.vue"; | ||||
| import { computed, reactive } from "vue"; | ||||
| import { useToast } from "vue-toast-notification"; | ||||
| import { DOCUMENT_ADD, trans } from "translator"; | ||||
|  | ||||
| interface DropFileConfig { | ||||
|     allowRemove: boolean; | ||||
| @@ -75,11 +76,9 @@ function closeModal(): void { | ||||
|         @click="openModal" | ||||
|         class="btn btn-create" | ||||
|     > | ||||
|         Ajouter un document | ||||
|     </button> | ||||
|     <button v-else @click="openModal" class="btn btn-edit"> | ||||
|         Remplacer le document | ||||
|         {{ trans(DOCUMENT_ADD) }} | ||||
|     </button> | ||||
|     <button v-else @click="openModal" class="btn btn-edit"></button> | ||||
|     <modal | ||||
|         v-if="state.showModal" | ||||
|         :modal-dialog-class="modalClasses" | ||||
|   | ||||
| @@ -3,9 +3,9 @@ import { | ||||
|     StoredObject, | ||||
|     StoredObjectPointInTime, | ||||
|     StoredObjectVersionWithPointInTime, | ||||
| } from "./../../../types"; | ||||
| } from "ChillDocStoreAssets/types"; | ||||
| import UserRenderBoxBadge from "ChillMainAssets/vuejs/_components/Entity/UserRenderBoxBadge.vue"; | ||||
| import { ISOToDatetime } from "./../../../../../../ChillMainBundle/Resources/public/chill/js/date"; | ||||
| import { ISOToDatetime } from "ChillMainAssets/chill/js/date"; | ||||
| import FileIcon from "ChillDocStoreAssets/vuejs/FileIcon.vue"; | ||||
| import RestoreVersionButton from "ChillDocStoreAssets/vuejs/StoredObjectButton/HistoryButton/RestoreVersionButton.vue"; | ||||
| import DownloadButton from "ChillDocStoreAssets/vuejs/StoredObjectButton/DownloadButton.vue"; | ||||
|   | ||||
| @@ -46,6 +46,16 @@ abstract class AbstractStoredObjectVoter implements StoredObjectVoterInterface | ||||
|  | ||||
|     public function voteOnAttribute(StoredObjectRoleEnum $attribute, StoredObject $subject, TokenInterface $token): bool | ||||
|     { | ||||
|         // we first try to get the permission from the workflow, as attachement (this is the less intensive query) | ||||
|         $workflowPermissionAsAttachment = match ($attribute) { | ||||
|             StoredObjectRoleEnum::SEE => $this->workflowDocumentService->isAllowedByWorkflowForReadOperation($subject), | ||||
|             StoredObjectRoleEnum::EDIT => $this->workflowDocumentService->isAllowedByWorkflowForWriteOperation($subject), | ||||
|         }; | ||||
|  | ||||
|         if (WorkflowRelatedEntityPermissionHelper::FORCE_DENIED === $workflowPermissionAsAttachment) { | ||||
|             return false; | ||||
|         } | ||||
|  | ||||
|         // Retrieve the related entity | ||||
|         $entity = $this->getRepository()->findAssociatedEntityToStoredObject($subject); | ||||
|  | ||||
| @@ -66,7 +76,7 @@ abstract class AbstractStoredObjectVoter implements StoredObjectVoterInterface | ||||
|         return match ($workflowPermission) { | ||||
|             WorkflowRelatedEntityPermissionHelper::FORCE_GRANT => true, | ||||
|             WorkflowRelatedEntityPermissionHelper::FORCE_DENIED => false, | ||||
|             WorkflowRelatedEntityPermissionHelper::ABSTAIN => $regularPermission, | ||||
|             WorkflowRelatedEntityPermissionHelper::ABSTAIN => WorkflowRelatedEntityPermissionHelper::FORCE_GRANT === $workflowPermissionAsAttachment || $regularPermission, | ||||
|         }; | ||||
|     } | ||||
| } | ||||
|   | ||||
| @@ -14,6 +14,12 @@ namespace Chill\DocStoreBundle\Security\Authorization; | ||||
| use Chill\DocStoreBundle\Entity\StoredObject; | ||||
| use Symfony\Component\Security\Core\Authentication\Token\TokenInterface; | ||||
|  | ||||
| /** | ||||
|  * Interface for voting on stored object permissions. | ||||
|  * | ||||
|  * Each time a stored object is attached to a document, the voter is responsible for determining | ||||
|  * whether the user has the necessary permissions to access or modify the stored object. | ||||
|  */ | ||||
| interface StoredObjectVoterInterface | ||||
| { | ||||
|     public function supports(StoredObjectRoleEnum $attribute, StoredObject $subject): bool; | ||||
|   | ||||
| @@ -15,6 +15,7 @@ use Chill\DocStoreBundle\Entity\StoredObject; | ||||
| use Chill\DocStoreBundle\Entity\StoredObjectPointInTime; | ||||
| use Chill\DocStoreBundle\Entity\StoredObjectPointInTimeReasonEnum; | ||||
| use Chill\DocStoreBundle\Entity\StoredObjectVersion; | ||||
| use Chill\DocStoreBundle\Exception\ConversionWithSameMimeTypeException; | ||||
| use Chill\DocStoreBundle\Exception\StoredObjectManagerException; | ||||
| use Chill\WopiBundle\Service\WopiConverter; | ||||
| use Symfony\Component\Mime\MimeTypesInterface; | ||||
| @@ -41,9 +42,10 @@ class StoredObjectToPdfConverter | ||||
|      * | ||||
|      * @return array{0: StoredObjectPointInTime, 1: StoredObjectVersion, 2?: string} contains the point in time before conversion and the new version of the stored object. The converted content is included in the response if $includeConvertedContent is true | ||||
|      * | ||||
|      * @throws \UnexpectedValueException    if the preferred mime type for the conversion is not found | ||||
|      * @throws \RuntimeException            if the conversion or storage of the new version fails | ||||
|      * @throws \UnexpectedValueException           if the preferred mime type for the conversion is not found | ||||
|      * @throws \RuntimeException                   if the conversion or storage of the new version fails | ||||
|      * @throws StoredObjectManagerException | ||||
|      * @throws ConversionWithSameMimeTypeException if the document has already the same mime type79* | ||||
|      */ | ||||
|     public function addConvertedVersion(StoredObject $storedObject, string $lang, $convertTo = 'pdf', bool $includeConvertedContent = false): array | ||||
|     { | ||||
| @@ -56,7 +58,7 @@ class StoredObjectToPdfConverter | ||||
|         $currentVersion = $storedObject->getCurrentVersion(); | ||||
|  | ||||
|         if ($currentVersion->getType() === $newMimeType) { | ||||
|             throw new \UnexpectedValueException('Already at the same mime type'); | ||||
|             throw new ConversionWithSameMimeTypeException($newMimeType); | ||||
|         } | ||||
|  | ||||
|         $content = $this->storedObjectManager->read($currentVersion); | ||||
|   | ||||
| @@ -40,6 +40,10 @@ class StoredObjectVersionApiControllerTest extends \PHPUnit\Framework\TestCase | ||||
|             $storedObject->registerVersion(); | ||||
|         } | ||||
|  | ||||
|         // remove one version in the history | ||||
|         $v5 = $storedObject->getVersions()->get(5); | ||||
|         $storedObject->removeVersion($v5); | ||||
|  | ||||
|         $security = $this->prophesize(Security::class); | ||||
|         $security->isGranted(StoredObjectRoleEnum::SEE->value, $storedObject) | ||||
|             ->willReturn(true) | ||||
| @@ -53,6 +57,7 @@ class StoredObjectVersionApiControllerTest extends \PHPUnit\Framework\TestCase | ||||
|         self::assertEquals($response->getStatusCode(), 200); | ||||
|         self::assertIsArray($body); | ||||
|         self::assertArrayHasKey('results', $body); | ||||
|         self::assertIsList($body['results']); | ||||
|         self::assertCount(10, $body['results']); | ||||
|     } | ||||
|  | ||||
|   | ||||
| @@ -86,9 +86,165 @@ class AbstractStoredObjectVoterTest extends TestCase | ||||
|     } | ||||
|  | ||||
|     /** | ||||
|      * @dataProvider dataProviderVoteOnAttribute | ||||
|      * @dataProvider dataProviderVoteOnAttributeWithStoredObjectPermission | ||||
|      */ | ||||
|     public function testVoteOnAttribute( | ||||
|     public function testVoteOnAttributeWithStoredObjectPermission( | ||||
|         StoredObjectRoleEnum $attribute, | ||||
|         bool $expected, | ||||
|         bool $isGrantedRegularPermission, | ||||
|         string $isGrantedWorkflowPermission, | ||||
|         string $isGrantedStoredObjectAttachment, | ||||
|     ): void { | ||||
|         $storedObject = new StoredObject(); | ||||
|         $repository = new DummyRepository($related = new \stdClass()); | ||||
|         $token = new UsernamePasswordToken(new User(), 'dummy'); | ||||
|  | ||||
|         $security = $this->prophesize(Security::class); | ||||
|         $security->isGranted('SOME_ROLE', $related)->willReturn($isGrantedRegularPermission); | ||||
|  | ||||
|         $workflowRelatedEntityPermissionHelper = $this->prophesize(WorkflowRelatedEntityPermissionHelper::class); | ||||
|  | ||||
|         $security = $this->prophesize(Security::class); | ||||
|         $security->isGranted('SOME_ROLE', $related)->willReturn($isGrantedRegularPermission); | ||||
|  | ||||
|         if (StoredObjectRoleEnum::SEE === $attribute) { | ||||
|             $workflowRelatedEntityPermissionHelper->isAllowedByWorkflowForReadOperation($storedObject) | ||||
|                 ->shouldBeCalled() | ||||
|                 ->willReturn($isGrantedStoredObjectAttachment); | ||||
|             $workflowRelatedEntityPermissionHelper->isAllowedByWorkflowForReadOperation($related) | ||||
|                 ->willReturn($isGrantedWorkflowPermission); | ||||
|         } elseif (StoredObjectRoleEnum::EDIT === $attribute) { | ||||
|             $workflowRelatedEntityPermissionHelper->isAllowedByWorkflowForWriteOperation($storedObject) | ||||
|                 ->shouldBeCalled() | ||||
|                 ->willReturn($isGrantedStoredObjectAttachment); | ||||
|             $workflowRelatedEntityPermissionHelper->isAllowedByWorkflowForWriteOperation($related) | ||||
|                 ->willReturn($isGrantedWorkflowPermission); | ||||
|         } else { | ||||
|             throw new \LogicException('Invalid attribute for StoredObjectVoter'); | ||||
|         } | ||||
|  | ||||
|         $storedObjectVoter = new class ($repository, $workflowRelatedEntityPermissionHelper->reveal(), $security->reveal()) extends AbstractStoredObjectVoter { | ||||
|             public function __construct(private $repository, $helper, $security) | ||||
|             { | ||||
|                 parent::__construct($security, $helper); | ||||
|             } | ||||
|  | ||||
|             protected function getRepository(): AssociatedEntityToStoredObjectInterface | ||||
|             { | ||||
|                 return $this->repository; | ||||
|             } | ||||
|  | ||||
|             protected function getClass(): string | ||||
|             { | ||||
|                 return \stdClass::class; | ||||
|             } | ||||
|  | ||||
|             protected function attributeToRole(StoredObjectRoleEnum $attribute): string | ||||
|             { | ||||
|                 return 'SOME_ROLE'; | ||||
|             } | ||||
|  | ||||
|             protected function canBeAssociatedWithWorkflow(): bool | ||||
|             { | ||||
|                 return true; | ||||
|             } | ||||
|         }; | ||||
|  | ||||
|         $actual = $storedObjectVoter->voteOnAttribute($attribute, $storedObject, $token); | ||||
|  | ||||
|         self::assertEquals($expected, $actual); | ||||
|     } | ||||
|  | ||||
|     public static function dataProviderVoteOnAttributeWithStoredObjectPermission(): iterable | ||||
|     { | ||||
|         foreach (['read' => StoredObjectRoleEnum::SEE, 'write' => StoredObjectRoleEnum::EDIT] as $action => $attribute) { | ||||
|             yield 'Not related to any workflow nor attachment ('.$action.')' => [ | ||||
|                 $attribute, | ||||
|                 true, | ||||
|                 true, | ||||
|                 WorkflowRelatedEntityPermissionHelper::ABSTAIN, | ||||
|                 WorkflowRelatedEntityPermissionHelper::ABSTAIN, | ||||
|             ]; | ||||
|  | ||||
|             yield 'Not related to any workflow nor attachment (refuse) ('.$action.')' => [ | ||||
|                 $attribute, | ||||
|                 false, | ||||
|                 false, | ||||
|                 WorkflowRelatedEntityPermissionHelper::ABSTAIN, | ||||
|                 WorkflowRelatedEntityPermissionHelper::ABSTAIN, | ||||
|             ]; | ||||
|  | ||||
|             yield 'Is granted by a workflow takes precedence (workflow) ('.$action.')' => [ | ||||
|                 $attribute, | ||||
|                 false, | ||||
|                 true, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_DENIED, | ||||
|                 WorkflowRelatedEntityPermissionHelper::ABSTAIN, | ||||
|             ]; | ||||
|  | ||||
|             yield 'Is granted by a workflow takes precedence (stored object) ('.$action.')' => [ | ||||
|                 $attribute, | ||||
|                 false, | ||||
|                 true, | ||||
|                 WorkflowRelatedEntityPermissionHelper::ABSTAIN, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_DENIED, | ||||
|             ]; | ||||
|  | ||||
|             yield 'Is granted by a workflow takes precedence (workflow) although grant ('.$action.')' => [ | ||||
|                 $attribute, | ||||
|                 false, | ||||
|                 true, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_DENIED, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_GRANT, | ||||
|             ]; | ||||
|  | ||||
|             yield 'Is granted by a workflow takes precedence (stored object) although grant ('.$action.')' => [ | ||||
|                 $attribute, | ||||
|                 false, | ||||
|                 true, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_GRANT, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_DENIED, | ||||
|             ]; | ||||
|  | ||||
|             yield 'Is granted by a workflow takes precedence (initially refused) (workflow) although grant ('.$action.')' => [ | ||||
|                 $attribute, | ||||
|                 false, | ||||
|                 false, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_DENIED, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_GRANT, | ||||
|             ]; | ||||
|  | ||||
|             yield 'Is granted by a workflow takes precedence (initially refused) (stored object) although grant ('.$action.')' => [ | ||||
|                 $attribute, | ||||
|                 false, | ||||
|                 false, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_GRANT, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_DENIED, | ||||
|             ]; | ||||
|  | ||||
|             yield 'Force grant inverse the regular permission (workflow) ('.$action.')' => [ | ||||
|                 $attribute, | ||||
|                 true, | ||||
|                 false, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_GRANT, | ||||
|                 WorkflowRelatedEntityPermissionHelper::ABSTAIN, | ||||
|             ]; | ||||
|  | ||||
|             yield 'Force grant inverse the regular permission (so) ('.$action.')' => [ | ||||
|                 $attribute, | ||||
|                 true, | ||||
|                 false, | ||||
|                 WorkflowRelatedEntityPermissionHelper::ABSTAIN, | ||||
|                 WorkflowRelatedEntityPermissionHelper::FORCE_GRANT, | ||||
|             ]; | ||||
|  | ||||
|         } | ||||
|     } | ||||
|  | ||||
|     /** | ||||
|      * @dataProvider dataProviderVoteOnAttributeWithoutStoredObjectPermission | ||||
|      */ | ||||
|     public function testVoteOnAttributeWithoutStoredObjectPermission( | ||||
|         StoredObjectRoleEnum $attribute, | ||||
|         bool $expected, | ||||
|         bool $canBeAssociatedWithWorkflow, | ||||
| @@ -105,6 +261,10 @@ class AbstractStoredObjectVoterTest extends TestCase | ||||
|         $security->isGranted('SOME_ROLE', $related)->willReturn($isGrantedRegularPermission); | ||||
|  | ||||
|         $workflowRelatedEntityPermissionHelper = $this->prophesize(WorkflowRelatedEntityPermissionHelper::class); | ||||
|  | ||||
|         $workflowRelatedEntityPermissionHelper->isAllowedByWorkflowForReadOperation($storedObject)->willReturn(WorkflowRelatedEntityPermissionHelper::ABSTAIN); | ||||
|         $workflowRelatedEntityPermissionHelper->isAllowedByWorkflowForWriteOperation($storedObject)->willReturn(WorkflowRelatedEntityPermissionHelper::ABSTAIN); | ||||
|  | ||||
|         if (null !== $isGrantedWorkflowPermissionRead) { | ||||
|             $workflowRelatedEntityPermissionHelper->isAllowedByWorkflowForReadOperation($related) | ||||
|                 ->willReturn($isGrantedWorkflowPermissionRead)->shouldBeCalled(); | ||||
| @@ -123,7 +283,7 @@ class AbstractStoredObjectVoterTest extends TestCase | ||||
|         self::assertEquals($expected, $voter->voteOnAttribute($attribute, $storedObject, $token), $message); | ||||
|     } | ||||
|  | ||||
|     public static function dataProviderVoteOnAttribute(): iterable | ||||
|     public static function dataProviderVoteOnAttributeWithoutStoredObjectPermission(): iterable | ||||
|     { | ||||
|         // not associated on a workflow | ||||
|         yield [StoredObjectRoleEnum::SEE, true, false, true, null, null, 'not associated on a workflow, granted by regular access, must not rely on helper']; | ||||
|   | ||||
| @@ -0,0 +1,63 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\Migrations\DocStore; | ||||
|  | ||||
| use Doctrine\DBAL\Schema\Schema; | ||||
| use Doctrine\Migrations\AbstractMigration; | ||||
|  | ||||
| final class Version20251013094414 extends AbstractMigration | ||||
| { | ||||
|     public function getDescription(): string | ||||
|     { | ||||
|         return 'DocStore: Enforce filename uniqueness on chill_doc.stored_object_version; clean duplicates and add partial unique index on filename (for new rows only).'; | ||||
|     } | ||||
|  | ||||
|     public function up(Schema $schema): void | ||||
|     { | ||||
|         // 1) Clean duplicates: for each (stored_object_id, filename, key, iv), keep only the last inserted row | ||||
|         //    and delete all others. Use ROW_NUMBER over id DESC to define the last one. | ||||
|         $this->addSql(<<<'SQL' | ||||
|             WITH ranked AS ( | ||||
|                 SELECT id, | ||||
|                        rank() OVER ( | ||||
|                            PARTITION BY stored_object_id, filename, "key"::jsonb, iv::jsonb | ||||
|                            ORDER BY id DESC | ||||
|                        ) AS rn | ||||
|                 FROM chill_doc.stored_object_version | ||||
|             ) | ||||
|             DELETE FROM chill_doc.stored_object_version sov | ||||
|             USING ranked r | ||||
|             WHERE sov.id = r.id | ||||
|               AND r.rn > 1 | ||||
|         SQL); | ||||
|  | ||||
|         // 2) Create a partial unique index on filename that applies only to subsequently inserted rows. | ||||
|         //    Per user's instruction, compute the cutoff using the stored_object_id sequence value. | ||||
|         $nextVal = (int) $this->connection->fetchOne("SELECT nextval('chill_doc.stored_object_version_id_seq')"); | ||||
|  | ||||
|         // Safety: if somehow sequence is not available, fallback to current max id from the table | ||||
|         if ($nextVal <= 0) { | ||||
|             $nextVal = (int) $this->connection->fetchOne('SELECT COALESCE(MAX(id), 0) FROM chill_doc.stored_object_version'); | ||||
|         } | ||||
|  | ||||
|         $this->addSql(sprintf( | ||||
|             'CREATE UNIQUE INDEX chill_doc_stored_object_version_unique_by_filename ON chill_doc.stored_object_version (filename) WHERE id > %d', | ||||
|             $nextVal | ||||
|         )); | ||||
|     } | ||||
|  | ||||
|     public function down(Schema $schema): void | ||||
|     { | ||||
|         // Drop the partial unique index; data cleanup is irreversible. | ||||
|         $this->addSql('DROP INDEX IF EXISTS chill_doc_stored_object_version_unique_by_filename'); | ||||
|     } | ||||
| } | ||||
| @@ -23,6 +23,8 @@ See the document: Voir le document | ||||
|  | ||||
| document: | ||||
|     Any title: Aucun titre | ||||
|     replace: Remplacer | ||||
|     Add: Ajouter un document | ||||
|  | ||||
| generic_doc: | ||||
|     filter: | ||||
|   | ||||
| @@ -118,7 +118,7 @@ | ||||
|  | ||||
|         {{ entity.notes|chill_print_or_message("Aucune note", 'blockquote') }} | ||||
|     {% endblock crud_content_view_details %} | ||||
|  | ||||
|     {% block content_view_actions_duplicate_link %}{% endblock %} | ||||
|     {% block content_view_actions_back %} | ||||
|         <li class="cancel"> | ||||
|             <a class="btn btn-cancel" href="{{ chill_return_path_or('chill_job_report_index', { 'person': entity.person.id }) }}"> | ||||
|   | ||||
| @@ -46,6 +46,7 @@ | ||||
|             </dd> | ||||
|         </dl> | ||||
|     {% endblock crud_content_view_details %} | ||||
|     {% block content_view_actions_duplicate_link %}{% endblock %} | ||||
|  | ||||
|     {% block content_view_actions_back %} | ||||
|         <li class="cancel"> | ||||
|   | ||||
| @@ -206,6 +206,8 @@ | ||||
|             </a> | ||||
|         </li> | ||||
|     {% endblock %} | ||||
|     {% block content_view_actions_duplicate_link %}{% endblock %} | ||||
|  | ||||
|     {% block content_view_actions_after %} | ||||
|         <li> | ||||
|             <a class="btn btn-misc" href="{{ chill_return_path_or('chill_crud_immersion_bilan', { 'id': entity.id, 'person_id': entity.person.id }) }}"> | ||||
|   | ||||
| @@ -94,6 +94,7 @@ | ||||
|  | ||||
|  | ||||
|     {% endblock crud_content_view_details %} | ||||
|     {% block content_view_actions_duplicate_link %}{% endblock %} | ||||
|  | ||||
|     {% block content_view_actions_back %} | ||||
|         <li class="cancel"> | ||||
|   | ||||
| @@ -0,0 +1,64 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\MainBundle\Action\User\UpdateProfile; | ||||
|  | ||||
| use Chill\MainBundle\Entity\User; | ||||
| use Chill\MainBundle\Notification\NotificationFlagManager; | ||||
| use Chill\MainBundle\Validation\Constraint\PhonenumberConstraint; | ||||
| use libphonenumber\PhoneNumber; | ||||
|  | ||||
| final class UpdateProfileCommand | ||||
| { | ||||
|     public array $notificationFlags = []; | ||||
|  | ||||
|     public function __construct( | ||||
|         #[PhonenumberConstraint] | ||||
|         public ?PhoneNumber $phonenumber, | ||||
|     ) {} | ||||
|  | ||||
|     public static function create(User $user, NotificationFlagManager $flagManager): self | ||||
|     { | ||||
|         $updateProfileCommand = new self($user->getPhonenumber()); | ||||
|  | ||||
|         foreach ($flagManager->getAllNotificationFlagProviders() as $provider) { | ||||
|             $updateProfileCommand->setNotificationFlag( | ||||
|                 $provider->getFlag(), | ||||
|                 User::NOTIF_FLAG_IMMEDIATE_EMAIL, | ||||
|                 $user->isNotificationSendImmediately($provider->getFlag()) | ||||
|             ); | ||||
|             $updateProfileCommand->setNotificationFlag( | ||||
|                 $provider->getFlag(), | ||||
|                 User::NOTIF_FLAG_DAILY_DIGEST, | ||||
|                 $user->isNotificationDailyDigest($provider->getFlag()) | ||||
|             ); | ||||
|         } | ||||
|  | ||||
|         return $updateProfileCommand; | ||||
|     } | ||||
|  | ||||
|     /** | ||||
|      * @param User::NOTIF_FLAG_IMMEDIATE_EMAIL|User::NOTIF_FLAG_DAILY_DIGEST $kind | ||||
|      */ | ||||
|     private function setNotificationFlag(string $type, string $kind, bool $value): void | ||||
|     { | ||||
|         if (!array_key_exists($type, $this->notificationFlags)) { | ||||
|             $this->notificationFlags[$type] = ['immediate_email' => true, 'daily_digest' => false]; | ||||
|         } | ||||
|  | ||||
|         $k = match ($kind) { | ||||
|             User::NOTIF_FLAG_IMMEDIATE_EMAIL => 'immediate_email', | ||||
|             User::NOTIF_FLAG_DAILY_DIGEST => 'daily_digest', | ||||
|         }; | ||||
|  | ||||
|         $this->notificationFlags[$type][$k] = $value; | ||||
|     } | ||||
| } | ||||
| @@ -0,0 +1,27 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\MainBundle\Action\User\UpdateProfile; | ||||
|  | ||||
| use Chill\MainBundle\Entity\User; | ||||
|  | ||||
| final readonly class UpdateProfileCommandHandler | ||||
| { | ||||
|     public function updateProfile(User $user, UpdateProfileCommand $command): void | ||||
|     { | ||||
|         $user->setPhonenumber($command->phonenumber); | ||||
|  | ||||
|         foreach ($command->notificationFlags as $flag => $values) { | ||||
|             $user->setNotificationImmediately($flag, $values['immediate_email']); | ||||
|             $user->setNotificationDailyDigest($flag, $values['daily_digest']); | ||||
|         } | ||||
|     } | ||||
| } | ||||
| @@ -334,7 +334,7 @@ class ChillImportUsersCommand extends Command | ||||
|  | ||||
|     protected function loadUsers() | ||||
|     { | ||||
|         $reader = Reader::createFromPath($this->tempInput->getArgument('csvfile')); | ||||
|         $reader = Reader::from($this->tempInput->getArgument('csvfile')); | ||||
|         $reader->setHeaderOffset(0); | ||||
|  | ||||
|         foreach ($reader->getRecords() as $line => $r) { | ||||
| @@ -362,7 +362,7 @@ class ChillImportUsersCommand extends Command | ||||
|  | ||||
|     protected function prepareGroupingCenters() | ||||
|     { | ||||
|         $reader = Reader::createFromPath($this->tempInput->getOption('grouping-centers')); | ||||
|         $reader = Reader::from($this->tempInput->getOption('grouping-centers')); | ||||
|         $reader->setHeaderOffset(0); | ||||
|  | ||||
|         foreach ($reader->getRecords() as $r) { | ||||
| @@ -378,7 +378,7 @@ class ChillImportUsersCommand extends Command | ||||
|  | ||||
|     protected function prepareWriter() | ||||
|     { | ||||
|         $this->output = $output = Writer::createFromPath($this->tempInput | ||||
|         $this->output = $output = Writer::from($this->tempInput | ||||
|             ->getOption('csv-dump'), 'a+'); | ||||
|  | ||||
|         $output->insertOne([ | ||||
|   | ||||
| @@ -119,7 +119,7 @@ class ChillUserSendRenewPasswordCodeCommand extends Command | ||||
|     protected function getReader() | ||||
|     { | ||||
|         try { | ||||
|             $reader = Reader::createFromPath($this->input->getArgument('csvfile')); | ||||
|             $reader = Reader::from($this->input->getArgument('csvfile')); | ||||
|         } catch (\Exception $e) { | ||||
|             $this->logger->error('The csv file could not be read', [ | ||||
|                 'path' => $this->input->getArgument('csvfile'), | ||||
|   | ||||
| @@ -0,0 +1,35 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\MainBundle\Command; | ||||
|  | ||||
| use Chill\MainBundle\Security\RoleDumper; | ||||
| use Symfony\Component\Console\Attribute\AsCommand; | ||||
| use Symfony\Component\Console\Command\Command; | ||||
| use Symfony\Component\Console\Input\InputInterface; | ||||
| use Symfony\Component\Console\Output\OutputInterface; | ||||
|  | ||||
| #[AsCommand(name: 'chill:main:dump-list-permissions', description: 'Print a markdown reference of permissions (roles) grouped by title with dependencies).')] | ||||
| final class DumpListPermissionsCommand extends Command | ||||
| { | ||||
|     public function __construct(private readonly RoleDumper $roleDumper) | ||||
|     { | ||||
|         parent::__construct(); | ||||
|     } | ||||
|  | ||||
|     protected function execute(InputInterface $input, OutputInterface $output): int | ||||
|     { | ||||
|         $markdown = $this->roleDumper->dumpAsMarkdown(); | ||||
|         $output->writeln($markdown); | ||||
|  | ||||
|         return Command::SUCCESS; | ||||
|     } | ||||
| } | ||||
| @@ -48,6 +48,7 @@ class AbsenceController extends AbstractController | ||||
|         $user = $this->security->getUser(); | ||||
|  | ||||
|         $user->setAbsenceStart(null); | ||||
|         $user->setAbsenceEnd(null); | ||||
|         $em = $this->managerRegistry->getManager(); | ||||
|         $em->flush(); | ||||
|  | ||||
|   | ||||
| @@ -345,7 +345,7 @@ class ExportController extends AbstractController | ||||
|      * @param array  $dataExport    Raw data from export step | ||||
|      * @param array  $dataFormatter Raw data from formatter step | ||||
|      */ | ||||
|     private function buildExportDataForNormalization(string $alias, ?array $dataCenters, array $dataExport, array $dataFormatter, ?SavedExport $savedExport): array | ||||
|     private function buildExportDataForNormalization(string $alias, ?array $dataCenters, array $dataExport, ?array $dataFormatter, ?SavedExport $savedExport): array | ||||
|     { | ||||
|         if ($this->filterStatsByCenters) { | ||||
|             $formCenters = $this->createCreateFormExport($alias, 'generate_centers', [], null); | ||||
| @@ -365,7 +365,7 @@ class ExportController extends AbstractController | ||||
|         $formExport->submit($dataExport); | ||||
|         $dataExport = $formExport->getData(); | ||||
|  | ||||
|         if (\count($dataFormatter) > 0) { | ||||
|         if (is_array($dataFormatter) && \count($dataFormatter) > 0) { | ||||
|             $formFormatter = $this->createCreateFormExport( | ||||
|                 $alias, | ||||
|                 'generate_formatter', | ||||
| @@ -381,7 +381,7 @@ class ExportController extends AbstractController | ||||
|             'export' => $dataExport['export']['export'] ?? [], | ||||
|             'filters' => $dataExport['export']['filters'] ?? [], | ||||
|             'aggregators' => $dataExport['export']['aggregators'] ?? [], | ||||
|             'pick_formatter' => $dataExport['export']['pick_formatter']['alias'], | ||||
|             'pick_formatter' => ($dataExport['export']['pick_formatter'] ?? [])['alias'] ?? '', | ||||
|             'formatter' => $dataFormatter['formatter'] ?? [], | ||||
|         ]; | ||||
|     } | ||||
|   | ||||
| @@ -43,7 +43,7 @@ final readonly class UserExportController | ||||
|  | ||||
|         $users = $this->userRepository->findAllAsArray($request->getLocale()); | ||||
|  | ||||
|         $csv = Writer::createFromPath('php://temp', 'r+'); | ||||
|         $csv = Writer::from('php://temp', 'r+'); | ||||
|         $csv->insertOne( | ||||
|             array_map( | ||||
|                 fn (string $e) => $this->translator->trans('admin.users.export.'.$e), | ||||
| @@ -104,7 +104,7 @@ final readonly class UserExportController | ||||
|  | ||||
|         $userPermissions = $this->userRepository->findAllUserACLAsArray(); | ||||
|  | ||||
|         $csv = Writer::createFromPath('php://temp', 'r+'); | ||||
|         $csv = Writer::from('php://temp', 'r+'); | ||||
|         $csv->insertOne( | ||||
|             array_map( | ||||
|                 fn (string $e) => $this->translator->trans('admin.users.export.'.$e), | ||||
|   | ||||
| @@ -1,63 +0,0 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\MainBundle\Controller; | ||||
|  | ||||
| use Chill\MainBundle\Form\UserProfileType; | ||||
| use Chill\MainBundle\Security\ChillSecurity; | ||||
| use Symfony\Bundle\FrameworkBundle\Controller\AbstractController; | ||||
| use Symfony\Component\HttpFoundation\Request; | ||||
| use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException; | ||||
| use Symfony\Contracts\Translation\TranslatorInterface; | ||||
| use Symfony\Component\Routing\Annotation\Route; | ||||
|  | ||||
| final class UserProfileController extends AbstractController | ||||
| { | ||||
|     public function __construct( | ||||
|         private readonly TranslatorInterface $translator, | ||||
|         private readonly ChillSecurity $security, | ||||
|         private readonly \Doctrine\Persistence\ManagerRegistry $managerRegistry, | ||||
|     ) {} | ||||
|  | ||||
|     /** | ||||
|      * User profile that allows editing of phonenumber and visualization of certain data. | ||||
|      */ | ||||
|     #[Route(path: '/{_locale}/main/user/my-profile', name: 'chill_main_user_profile')] | ||||
|     public function __invoke(Request $request) | ||||
|     { | ||||
|         if (!$this->security->isGranted('ROLE_USER')) { | ||||
|             throw new AccessDeniedHttpException(); | ||||
|         } | ||||
|  | ||||
|         $user = $this->security->getUser(); | ||||
|         $editForm = $this->createForm(UserProfileType::class, $user); | ||||
|  | ||||
|         $editForm->get('notificationFlags')->setData($user->getNotificationFlags()); | ||||
|  | ||||
|         $editForm->handleRequest($request); | ||||
|  | ||||
|         if ($editForm->isSubmitted() && $editForm->isValid()) { | ||||
|             $notificationFlagsData = $editForm->get('notificationFlags')->getData(); | ||||
|             $user->setNotificationFlags($notificationFlagsData); | ||||
|  | ||||
|             $em = $this->managerRegistry->getManager(); | ||||
|             $em->flush(); | ||||
|             $this->addFlash('success', $this->translator->trans('user.profile.Profile successfully updated!')); | ||||
|  | ||||
|             return $this->redirectToRoute('chill_main_user_profile'); | ||||
|         } | ||||
|  | ||||
|         return $this->render('@ChillMain/User/profile.html.twig', [ | ||||
|             'user' => $user, | ||||
|             'form' => $editForm->createView(), | ||||
|         ]); | ||||
|     } | ||||
| } | ||||
| @@ -0,0 +1,75 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\MainBundle\Controller; | ||||
|  | ||||
| use Chill\MainBundle\Action\User\UpdateProfile\UpdateProfileCommand; | ||||
| use Chill\MainBundle\Action\User\UpdateProfile\UpdateProfileCommandHandler; | ||||
| use Chill\MainBundle\Entity\User; | ||||
| use Chill\MainBundle\Form\UpdateProfileType; | ||||
| use Chill\MainBundle\Notification\NotificationFlagManager; | ||||
| use Chill\MainBundle\Security\ChillSecurity; | ||||
| use Doctrine\ORM\EntityManagerInterface; | ||||
| use Symfony\Component\Form\FormFactoryInterface; | ||||
| use Symfony\Component\HttpFoundation\RedirectResponse; | ||||
| use Symfony\Component\HttpFoundation\Request; | ||||
| use Symfony\Component\HttpFoundation\Response; | ||||
| use Symfony\Component\HttpFoundation\Session\Session; | ||||
| use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException; | ||||
| use Symfony\Component\Routing\Generator\UrlGeneratorInterface; | ||||
| use Symfony\Contracts\Translation\TranslatorInterface; | ||||
| use Symfony\Component\Routing\Annotation\Route; | ||||
| use Twig\Environment; | ||||
|  | ||||
| final readonly class UserUpdateProfileController | ||||
| { | ||||
|     public function __construct( | ||||
|         private TranslatorInterface $translator, | ||||
|         private ChillSecurity $security, | ||||
|         private EntityManagerInterface $entityManager, | ||||
|         private NotificationFlagManager $notificationFlagManager, | ||||
|         private FormFactoryInterface $formFactory, | ||||
|         private UrlGeneratorInterface $urlGenerator, | ||||
|         private Environment $twig, | ||||
|         private UpdateProfileCommandHandler $updateProfileCommandHandler, | ||||
|     ) {} | ||||
|  | ||||
|     /** | ||||
|      * User profile that allows editing of phonenumber and visualization of certain data. | ||||
|      */ | ||||
|     #[Route(path: '/{_locale}/main/user/my-profile', name: 'chill_main_user_profile')] | ||||
|     public function __invoke(Request $request, Session $session) | ||||
|     { | ||||
|         if (!$this->security->isGranted('ROLE_USER')) { | ||||
|             throw new AccessDeniedHttpException(); | ||||
|         } | ||||
|  | ||||
|         $user = $this->security->getUser(); | ||||
|  | ||||
|         $command = UpdateProfileCommand::create($user, $this->notificationFlagManager); | ||||
|         $editForm = $this->formFactory->create(UpdateProfileType::class, $command); | ||||
|  | ||||
|         $editForm->handleRequest($request); | ||||
|  | ||||
|         if ($editForm->isSubmitted() && $editForm->isValid()) { | ||||
|             $this->updateProfileCommandHandler->updateProfile($user, $command); | ||||
|             $this->entityManager->flush(); | ||||
|             $session->getFlashBag()->add('success', $this->translator->trans('user.profile.Profile successfully updated!')); | ||||
|  | ||||
|             return new RedirectResponse($this->urlGenerator->generate('chill_main_user_profile')); | ||||
|         } | ||||
|  | ||||
|         return new Response($this->twig->render('@ChillMain/User/profile.html.twig', [ | ||||
|             'user' => $user, | ||||
|             'form' => $editForm->createView(), | ||||
|         ])); | ||||
|     } | ||||
| } | ||||
| @@ -11,6 +11,7 @@ declare(strict_types=1); | ||||
|  | ||||
| namespace Chill\MainBundle\Controller; | ||||
|  | ||||
| use Chill\MainBundle\CRUD\Controller\ApiController; | ||||
| use Chill\MainBundle\Entity\User; | ||||
| use Chill\MainBundle\Entity\Workflow\EntityWorkflow; | ||||
| use Chill\MainBundle\Pagination\PaginatorFactory; | ||||
| @@ -27,7 +28,7 @@ use Symfony\Component\Security\Core\Exception\AccessDeniedException; | ||||
| use Symfony\Component\Security\Core\Security; | ||||
| use Symfony\Component\Serializer\SerializerInterface; | ||||
|  | ||||
| class WorkflowApiController | ||||
| class WorkflowApiController extends ApiController | ||||
| { | ||||
|     public function __construct(private readonly EntityManagerInterface $entityManager, private readonly EntityWorkflowRepository $entityWorkflowRepository, private readonly PaginatorFactory $paginatorFactory, private readonly Security $security, private readonly SerializerInterface $serializer) {} | ||||
|  | ||||
|   | ||||
| @@ -264,11 +264,12 @@ class WorkflowController extends AbstractController | ||||
|     { | ||||
|         $this->denyAccessUnlessGranted('IS_AUTHENTICATED_REMEMBERED'); | ||||
|  | ||||
|         $total = $this->entityWorkflowRepository->countBySubscriber($this->security->getUser()); | ||||
|         $total = $this->entityWorkflowRepository->countBySubscriber($this->security->getUser(), false); | ||||
|         $paginator = $this->paginatorFactory->create($total); | ||||
|  | ||||
|         $workflows = $this->entityWorkflowRepository->findBySubscriber( | ||||
|             $this->security->getUser(), | ||||
|             false, | ||||
|             ['createdAt' => 'DESC'], | ||||
|             $paginator->getItemsPerPage(), | ||||
|             $paginator->getCurrentPageFirstItemNumber() | ||||
|   | ||||
| @@ -44,7 +44,7 @@ final readonly class WorkflowSignatureStateChangeController | ||||
|             $signature, | ||||
|             $request, | ||||
|             EntityWorkflowStepSignatureVoter::CANCEL, | ||||
|             function (EntityWorkflowStepSignature $signature) {$this->signatureStepStateChanger->markSignatureAsCanceled($signature); }, | ||||
|             fn (EntityWorkflowStepSignature $signature): string => $this->signatureStepStateChanger->markSignatureAsCanceled($signature), | ||||
|             '@ChillMain/WorkflowSignature/cancel.html.twig', | ||||
|         ); | ||||
|     } | ||||
| @@ -56,11 +56,18 @@ final readonly class WorkflowSignatureStateChangeController | ||||
|             $signature, | ||||
|             $request, | ||||
|             EntityWorkflowStepSignatureVoter::REJECT, | ||||
|             function (EntityWorkflowStepSignature $signature) {$this->signatureStepStateChanger->markSignatureAsRejected($signature); }, | ||||
|             fn (EntityWorkflowStepSignature $signature): string => $this->signatureStepStateChanger->markSignatureAsRejected($signature), | ||||
|             '@ChillMain/WorkflowSignature/reject.html.twig', | ||||
|         ); | ||||
|     } | ||||
|  | ||||
|     /** | ||||
|      * @param callable(EntityWorkflowStepSignature): string $markSignature | ||||
|      * | ||||
|      * @throws \Twig\Error\LoaderError | ||||
|      * @throws \Twig\Error\RuntimeError | ||||
|      * @throws \Twig\Error\SyntaxError | ||||
|      */ | ||||
|     private function markSignatureAction( | ||||
|         EntityWorkflowStepSignature $signature, | ||||
|         Request $request, | ||||
| @@ -79,12 +86,13 @@ final readonly class WorkflowSignatureStateChangeController | ||||
|         $form->handleRequest($request); | ||||
|  | ||||
|         if ($form->isSubmitted() && $form->isValid()) { | ||||
|             $this->entityManager->wrapInTransaction(function () use ($signature, $markSignature) { | ||||
|                 $markSignature($signature); | ||||
|             }); | ||||
|             $expectedStep = $this->entityManager->wrapInTransaction(fn () => $markSignature($signature)); | ||||
|  | ||||
|             return new RedirectResponse( | ||||
|                 $this->chillUrlGenerator->returnPathOr('chill_main_workflow_show', ['id' => $signature->getStep()->getEntityWorkflow()->getId()]) | ||||
|                 $this->chillUrlGenerator->forwardReturnPath( | ||||
|                     'chill_main_workflow_wait', | ||||
|                     ['id' => $signature->getStep()->getEntityWorkflow()->getId(), 'expectedStep' => $expectedStep] | ||||
|                 ) | ||||
|             ); | ||||
|         } | ||||
|  | ||||
|   | ||||
| @@ -0,0 +1,41 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\MainBundle\Controller; | ||||
|  | ||||
| use Chill\MainBundle\Entity\Workflow\EntityWorkflow; | ||||
| use Chill\MainBundle\Routing\ChillUrlGeneratorInterface; | ||||
| use Symfony\Component\HttpFoundation\RedirectResponse; | ||||
| use Symfony\Component\HttpFoundation\Response; | ||||
| use Symfony\Component\Routing\Annotation\Route; | ||||
| use Twig\Environment; | ||||
|  | ||||
| final readonly class WorkflowWaitStepChangeController | ||||
| { | ||||
|     public function __construct( | ||||
|         private ChillUrlGeneratorInterface $chillUrlGenerator, | ||||
|         private Environment $twig, | ||||
|     ) {} | ||||
|  | ||||
|     #[Route('/{_locale}/main/workflow/{id}/wait/{expectedStep}', name: 'chill_main_workflow_wait', methods: ['GET'])] | ||||
|     public function waitForSignatureChange(EntityWorkflow $entityWorkflow, string $expectedStep): Response | ||||
|     { | ||||
|         if ($entityWorkflow->getStep() === $expectedStep) { | ||||
|             return new RedirectResponse( | ||||
|                 $this->chillUrlGenerator->returnPathOr('chill_main_workflow_show', ['id' => $entityWorkflow->getId()]) | ||||
|             ); | ||||
|         } | ||||
|  | ||||
|         return new Response( | ||||
|             $this->twig->render('@ChillMain/Workflow/waiting.html.twig', ['workflow' => $entityWorkflow, 'expectedStep' => $expectedStep]) | ||||
|         ); | ||||
|     } | ||||
| } | ||||
| @@ -30,6 +30,7 @@ use Chill\MainBundle\Controller\UserGroupAdminController; | ||||
| use Chill\MainBundle\Controller\UserGroupApiController; | ||||
| use Chill\MainBundle\Controller\UserJobApiController; | ||||
| use Chill\MainBundle\Controller\UserJobController; | ||||
| use Chill\MainBundle\Controller\WorkflowApiController; | ||||
| use Chill\MainBundle\DependencyInjection\Widget\Factory\WidgetFactoryInterface; | ||||
| use Chill\MainBundle\Doctrine\DQL\Age; | ||||
| use Chill\MainBundle\Doctrine\DQL\Extract; | ||||
| @@ -66,6 +67,7 @@ use Chill\MainBundle\Entity\Regroupment; | ||||
| use Chill\MainBundle\Entity\User; | ||||
| use Chill\MainBundle\Entity\UserGroup; | ||||
| use Chill\MainBundle\Entity\UserJob; | ||||
| use Chill\MainBundle\Entity\Workflow\EntityWorkflow; | ||||
| use Chill\MainBundle\Form\CenterType; | ||||
| use Chill\MainBundle\Form\CivilityType; | ||||
| use Chill\MainBundle\Form\CountryType; | ||||
| @@ -79,6 +81,7 @@ use Chill\MainBundle\Form\UserGroupType; | ||||
| use Chill\MainBundle\Form\UserJobType; | ||||
| use Chill\MainBundle\Form\UserType; | ||||
| use Chill\MainBundle\Security\Authorization\ChillExportVoter; | ||||
| use Chill\MainBundle\Security\Authorization\EntityWorkflowVoter; | ||||
| use Misd\PhoneNumberBundle\Doctrine\DBAL\Types\PhoneNumberType; | ||||
| use Ramsey\Uuid\Doctrine\UuidType; | ||||
| use Symfony\Component\Config\FileLocator; | ||||
| @@ -202,6 +205,11 @@ class ChillMainExtension extends Extension implements | ||||
|                 [] | ||||
|         ); | ||||
|  | ||||
|         $container->setParameter( | ||||
|             'chill_main.top_banner', | ||||
|             $config['top_banner'] ?? [] | ||||
|         ); | ||||
|  | ||||
|         $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../config')); | ||||
|         $loader->load('services.yaml'); | ||||
|         $loader->load('services/doctrine.yaml'); | ||||
| @@ -247,6 +255,7 @@ class ChillMainExtension extends Extension implements | ||||
|                     'name' => $config['installation_name'], ], | ||||
|                 'available_languages' => $config['available_languages'], | ||||
|                 'add_address' => $config['add_address'], | ||||
|                 'chill_main_config' => $config, | ||||
|             ], | ||||
|             'form_themes' => ['@ChillMain/Form/fields.html.twig'], | ||||
|         ]; | ||||
| @@ -940,6 +949,21 @@ class ChillMainExtension extends Extension implements | ||||
|                         ], | ||||
|                     ], | ||||
|                 ], | ||||
|                 [ | ||||
|                     'class' => EntityWorkflow::class, | ||||
|                     'name' => 'workflow', | ||||
|                     'base_path' => '/api/1.0/main/workflow', | ||||
|                     'base_role' => EntityWorkflowVoter::SEE, | ||||
|                     'controller' => WorkflowApiController::class, | ||||
|                     'actions' => [ | ||||
|                         '_entity' => [ | ||||
|                             'methods' => [ | ||||
|                                 Request::METHOD_GET => true, | ||||
|                                 Request::METHOD_HEAD => true, | ||||
|                             ], | ||||
|                         ], | ||||
|                     ], | ||||
|                 ], | ||||
|             ], | ||||
|         ]); | ||||
|     } | ||||
|   | ||||
| @@ -168,6 +168,20 @@ class Configuration implements ConfigurationInterface | ||||
|             ->end() | ||||
|             ->end() | ||||
|             ->end() | ||||
|             ->arrayNode('top_banner') | ||||
|             ->canBeUnset() | ||||
|             ->children() | ||||
|             ->booleanNode('visible') | ||||
|             ->defaultFalse() | ||||
|             ->end() | ||||
|             ->arrayNode('text') | ||||
|             ->useAttributeAsKey('lang') | ||||
|             ->scalarPrototype()->end() | ||||
|             ->end() // end of text | ||||
|             ->scalarNode('color')->defaultNull()->end() | ||||
|             ->scalarNode('background_color')->defaultNull()->end() | ||||
|             ->end() // end of top_banner children | ||||
|             ->end() // end of top_banner | ||||
|             ->arrayNode('widgets') | ||||
|             ->canBeEnabled() | ||||
|             ->canBeUnset() | ||||
|   | ||||
| @@ -24,6 +24,7 @@ use Symfony\Component\Security\Core\User\UserInterface; | ||||
| use Symfony\Component\Serializer\Annotation as Serializer; | ||||
| use Symfony\Component\Validator\Context\ExecutionContextInterface; | ||||
| use Chill\MainBundle\Validation\Constraint\PhonenumberConstraint; | ||||
| use Symfony\Component\Validator\Constraints as Assert; | ||||
|  | ||||
| /** | ||||
|  * User. | ||||
| @@ -45,6 +46,8 @@ class User implements UserInterface, \Stringable, PasswordAuthenticatedUserInter | ||||
|     #[ORM\Column(type: \Doctrine\DBAL\Types\Types::DATETIME_IMMUTABLE, nullable: true)] | ||||
|     private ?\DateTimeImmutable $absenceStart = null; | ||||
|  | ||||
|     #[ORM\Column(type: \Doctrine\DBAL\Types\Types::DATETIME_IMMUTABLE, nullable: true)] | ||||
|     private ?\DateTimeImmutable $absenceEnd = null; | ||||
|     /** | ||||
|      * Array where SAML attributes's data are stored. | ||||
|      */ | ||||
| @@ -157,6 +160,11 @@ class User implements UserInterface, \Stringable, PasswordAuthenticatedUserInter | ||||
|         return $this->absenceStart; | ||||
|     } | ||||
|  | ||||
|     public function getAbsenceEnd(): ?\DateTimeImmutable | ||||
|     { | ||||
|         return $this->absenceEnd; | ||||
|     } | ||||
|  | ||||
|     /** | ||||
|      * Get attributes. | ||||
|      * | ||||
| @@ -336,7 +344,13 @@ class User implements UserInterface, \Stringable, PasswordAuthenticatedUserInter | ||||
|  | ||||
|     public function isAbsent(): bool | ||||
|     { | ||||
|         return null !== $this->getAbsenceStart() && $this->getAbsenceStart() <= new \DateTimeImmutable('now'); | ||||
|         $now = new \DateTimeImmutable('now'); | ||||
|         $absenceStart = $this->getAbsenceStart(); | ||||
|         $absenceEnd = $this->getAbsenceEnd(); | ||||
|  | ||||
|         return null !== $absenceStart | ||||
|             && $absenceStart <= $now | ||||
|             && (null === $absenceEnd || $now <= $absenceEnd); | ||||
|     } | ||||
|  | ||||
|     /** | ||||
| @@ -410,6 +424,11 @@ class User implements UserInterface, \Stringable, PasswordAuthenticatedUserInter | ||||
|         $this->absenceStart = $absenceStart; | ||||
|     } | ||||
|  | ||||
|     public function setAbsenceEnd(?\DateTimeImmutable $absenceEnd): void | ||||
|     { | ||||
|         $this->absenceEnd = $absenceEnd; | ||||
|     } | ||||
|  | ||||
|     public function setAttributeByDomain(string $domain, string $key, $value): self | ||||
|     { | ||||
|         $this->attributes[$domain][$key] = $value; | ||||
| @@ -633,46 +652,82 @@ class User implements UserInterface, \Stringable, PasswordAuthenticatedUserInter | ||||
|         return true; | ||||
|     } | ||||
|  | ||||
|     public function getNotificationFlags(): array | ||||
|     private function getNotificationFlagData(string $flag): array | ||||
|     { | ||||
|         return $this->notificationFlags; | ||||
|     } | ||||
|  | ||||
|     public function setNotificationFlags(array $notificationFlags) | ||||
|     { | ||||
|         $this->notificationFlags = $notificationFlags; | ||||
|     } | ||||
|  | ||||
|     public function getNotificationFlagData(string $flag): array | ||||
|     { | ||||
|         return $this->notificationFlags[$flag] ?? []; | ||||
|     } | ||||
|  | ||||
|     public function setNotificationFlagData(string $flag, array $data): void | ||||
|     { | ||||
|         $this->notificationFlags[$flag] = $data; | ||||
|         return $this->notificationFlags[$flag] ?? [self::NOTIF_FLAG_IMMEDIATE_EMAIL]; | ||||
|     } | ||||
|  | ||||
|     public function isNotificationSendImmediately(string $type): bool | ||||
|     { | ||||
|         if ([] === $this->getNotificationFlagData($type) || in_array(User::NOTIF_FLAG_IMMEDIATE_EMAIL, $this->getNotificationFlagData($type), true)) { | ||||
|             return true; | ||||
|         return $this->isNotificationForElement($type, self::NOTIF_FLAG_IMMEDIATE_EMAIL); | ||||
|     } | ||||
|  | ||||
|     public function setNotificationImmediately(string $type, bool $active): void | ||||
|     { | ||||
|         $this->setNotificationFlagElement($type, $active, self::NOTIF_FLAG_IMMEDIATE_EMAIL); | ||||
|     } | ||||
|  | ||||
|     public function setNotificationDailyDigest(string $type, bool $active): void | ||||
|     { | ||||
|         $this->setNotificationFlagElement($type, $active, self::NOTIF_FLAG_DAILY_DIGEST); | ||||
|     } | ||||
|  | ||||
|     /** | ||||
|      * @param self::NOTIF_FLAG_IMMEDIATE_EMAIL|self::NOTIF_FLAG_DAILY_DIGEST $kind | ||||
|      */ | ||||
|     private function setNotificationFlagElement(string $type, bool $active, string $kind): void | ||||
|     { | ||||
|         $notificationFlags = [...$this->notificationFlags]; | ||||
|         $changed = false; | ||||
|  | ||||
|         if (!isset($notificationFlags[$type])) { | ||||
|             $notificationFlags[$type] = [self::NOTIF_FLAG_IMMEDIATE_EMAIL]; | ||||
|             $changed = true; | ||||
|         } | ||||
|  | ||||
|         return false; | ||||
|         if ($active) { | ||||
|             if (!in_array($kind, $notificationFlags[$type], true)) { | ||||
|                 $notificationFlags[$type] = [...$notificationFlags[$type], $kind]; | ||||
|                 $changed = true; | ||||
|             } | ||||
|         } else { | ||||
|             if (in_array($kind, $notificationFlags[$type], true)) { | ||||
|                 $notificationFlags[$type] = array_values( | ||||
|                     array_filter($notificationFlags[$type], static fn ($k) => $k !== $kind) | ||||
|                 ); | ||||
|                 $changed = true; | ||||
|             } | ||||
|         } | ||||
|  | ||||
|         if ($changed) { | ||||
|             $this->notificationFlags = [...$notificationFlags]; | ||||
|         } | ||||
|     } | ||||
|  | ||||
|     private function isNotificationForElement(string $type, string $kind): bool | ||||
|     { | ||||
|         return in_array($kind, $this->getNotificationFlagData($type), true); | ||||
|     } | ||||
|  | ||||
|     public function isNotificationDailyDigest(string $type): bool | ||||
|     { | ||||
|         if (in_array(User::NOTIF_FLAG_DAILY_DIGEST, $this->getNotificationFlagData($type), true)) { | ||||
|             return true; | ||||
|         } | ||||
|  | ||||
|         return false; | ||||
|         return $this->isNotificationForElement($type, self::NOTIF_FLAG_DAILY_DIGEST); | ||||
|     } | ||||
|  | ||||
|     public function getLocale(): string | ||||
|     { | ||||
|         return 'fr'; | ||||
|     } | ||||
|  | ||||
|     #[Assert\Callback] | ||||
|     public function validateAbsenceDates(ExecutionContextInterface $context): void | ||||
|     { | ||||
|         if (null !== $this->getAbsenceEnd() && null === $this->getAbsenceStart()) { | ||||
|             $context->buildViolation( | ||||
|                 'user.absence_end_requires_start' | ||||
|             ) | ||||
|                 ->atPath('absenceEnd') | ||||
|                 ->addViolation(); | ||||
|         } | ||||
|     } | ||||
| } | ||||
|   | ||||
| @@ -20,7 +20,7 @@ use Chill\MainBundle\Repository\CenterRepositoryInterface; | ||||
| use Chill\MainBundle\Repository\RegroupmentRepositoryInterface; | ||||
|  | ||||
| /** | ||||
|  * @phpstan-type NormalizedData array{centers: array{centers: list<int>, regroupments: list<int>}, export: array{form: array<string, mixed>, version: int}, filters: array<string, array{enabled: boolean, form: array<string, mixed>, version: int}>, aggregators: array<string, array{enabled: boolean, form: array<string, mixed>, version: int}>, pick_formatter: string, formatter: array{form: array<string, mixed>, version: int}} | ||||
|  * @phpstan-type NormalizedData array{centers: array{centers: list<int>, regroupments: list<int>}, export: array{form: array<string, mixed>, version: int}, filters: array<string, array{enabled: boolean, form: array<string, mixed>, version: int}>, aggregators: array<string, array{enabled: boolean, form: array<string, mixed>, version: int}>, pick_formatter?: string, formatter: array{form: array<string, mixed>, version: int}} | ||||
|  */ | ||||
| class ExportConfigNormalizer | ||||
| { | ||||
| @@ -72,10 +72,14 @@ class ExportConfigNormalizer | ||||
|         } | ||||
|         $serialized['aggregators'] = $aggregatorsSerialized; | ||||
|  | ||||
|         $serialized['pick_formatter'] = $formData['pick_formatter']; | ||||
|         $formatter = $this->exportManager->getFormatter($formData['pick_formatter']); | ||||
|         $serialized['formatter']['form'] = $formatter->normalizeFormData($formData['formatter']); | ||||
|         $serialized['formatter']['version'] = $formatter->getNormalizationVersion(); | ||||
|         if ($export instanceof ExportInterface) { | ||||
|             $serialized['pick_formatter'] = $formData['pick_formatter']; | ||||
|             $formatter = $this->exportManager->getFormatter($formData['pick_formatter']); | ||||
|             $serialized['formatter']['form'] = $formatter->normalizeFormData($formData['formatter']); | ||||
|             $serialized['formatter']['version'] = $formatter->getNormalizationVersion(); | ||||
|         } elseif ($export instanceof DirectExportInterface) { | ||||
|             $serialized['formatter'] = ['form' => [], 'version' => 0]; | ||||
|         } | ||||
|  | ||||
|         return $serialized; | ||||
|     } | ||||
| @@ -87,7 +91,12 @@ class ExportConfigNormalizer | ||||
|     public function denormalizeConfig(string $exportAlias, array $serializedData, bool $replaceDisabledByDefaultData = false): array | ||||
|     { | ||||
|         $export = $this->exportManager->getExport($exportAlias); | ||||
|         $formater = $this->exportManager->getFormatter($serializedData['pick_formatter']); | ||||
|  | ||||
|         if ($export instanceof ExportInterface) { | ||||
|             $formatter = $this->exportManager->getFormatter($serializedData['pick_formatter']); | ||||
|         } else { | ||||
|             $formatter = null; | ||||
|         } | ||||
|  | ||||
|         $filtersConfig = []; | ||||
|         foreach ($serializedData['filters'] as $alias => $filterData) { | ||||
| @@ -117,8 +126,8 @@ class ExportConfigNormalizer | ||||
|             'export' => $export->denormalizeFormData($serializedData['export']['form'], $serializedData['export']['version']), | ||||
|             'filters' => $filtersConfig, | ||||
|             'aggregators' => $aggregatorsConfig, | ||||
|             'pick_formatter' => $serializedData['pick_formatter'], | ||||
|             'formatter' => $formater->denormalizeFormData($serializedData['formatter']['form'], $serializedData['formatter']['version']), | ||||
|             'pick_formatter' => $serializedData['pick_formatter'] ?? '', | ||||
|             'formatter' => $formatter?->denormalizeFormData($serializedData['formatter']['form'], $serializedData['formatter']['version']), | ||||
|             'centers' => [ | ||||
|                 'centers' => array_values(array_filter(array_map(fn (int $id) => $this->centerRepository->find($id), $serializedData['centers']['centers']), fn ($item) => null !== $item)), | ||||
|                 'regroupments' => array_values(array_filter(array_map(fn (int $id) => $this->regroupmentRepository->find($id), $serializedData['centers']['regroupments']), fn ($item) => null !== $item)), | ||||
|   | ||||
| @@ -23,9 +23,14 @@ class AbsenceType extends AbstractType | ||||
|     { | ||||
|         $builder | ||||
|             ->add('absenceStart', ChillDateType::class, [ | ||||
|                 'required' => true, | ||||
|                 'required' => false, | ||||
|                 'input' => 'datetime_immutable', | ||||
|                 'label' => 'absence.Absence start', | ||||
|             ]) | ||||
|             ->add('absenceEnd', ChillDateType::class, [ | ||||
|                 'required' => false, | ||||
|                 'input' => 'datetime_immutable', | ||||
|                 'label' => 'absence.Absence end', | ||||
|             ]); | ||||
|     } | ||||
|  | ||||
|   | ||||
| @@ -1,75 +0,0 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\MainBundle\Form\DataMapper; | ||||
|  | ||||
| use Chill\MainBundle\Entity\User; | ||||
| use Symfony\Component\Form\DataMapperInterface; | ||||
|  | ||||
| final readonly class NotificationFlagDataMapper implements DataMapperInterface | ||||
| { | ||||
|     public function __construct(private array $notificationFlagProviders) {} | ||||
|  | ||||
|     public function mapDataToForms($viewData, $forms): void | ||||
|     { | ||||
|         if (null === $viewData) { | ||||
|             $viewData = []; | ||||
|         } | ||||
|  | ||||
|         $formsArray = iterator_to_array($forms); | ||||
|  | ||||
|         foreach ($this->notificationFlagProviders as $flagProvider) { | ||||
|             $flag = $flagProvider->getFlag(); | ||||
|  | ||||
|             if (isset($formsArray[$flag])) { | ||||
|                 $flagForm = $formsArray[$flag]; | ||||
|  | ||||
|                 $immediateEmailChecked = in_array(User::NOTIF_FLAG_IMMEDIATE_EMAIL, $viewData[$flag] ?? [], true) | ||||
|                     || !array_key_exists($flag, $viewData); | ||||
|                 $dailyEmailChecked = in_array(User::NOTIF_FLAG_DAILY_DIGEST, $viewData[$flag] ?? [], true); | ||||
|  | ||||
|                 if ($flagForm->has('immediate_email')) { | ||||
|                     $flagForm->get('immediate_email')->setData($immediateEmailChecked); | ||||
|                 } | ||||
|                 if ($flagForm->has('daily_email')) { | ||||
|                     $flagForm->get('daily_email')->setData($dailyEmailChecked); | ||||
|                 } | ||||
|             } | ||||
|         } | ||||
|     } | ||||
|  | ||||
|     public function mapFormsToData($forms, &$viewData): void | ||||
|     { | ||||
|         $formsArray = iterator_to_array($forms); | ||||
|         $viewData = []; | ||||
|  | ||||
|         foreach ($this->notificationFlagProviders as $flagProvider) { | ||||
|             $flag = $flagProvider->getFlag(); | ||||
|  | ||||
|             if (isset($formsArray[$flag])) { | ||||
|                 $flagForm = $formsArray[$flag]; | ||||
|                 $viewData[$flag] = []; | ||||
|  | ||||
|                 if (true === $flagForm['immediate_email']->getData()) { | ||||
|                     $viewData[$flag][] = User::NOTIF_FLAG_IMMEDIATE_EMAIL; | ||||
|                 } | ||||
|  | ||||
|                 if (true === $flagForm['daily_email']->getData()) { | ||||
|                     $viewData[$flag][] = User::NOTIF_FLAG_DAILY_DIGEST; | ||||
|                 } | ||||
|  | ||||
|                 if ([] === $viewData[$flag]) { | ||||
|                     $viewData[$flag][] = User::NOTIF_FLAG_IMMEDIATE_EMAIL; | ||||
|                 } | ||||
|             } | ||||
|         } | ||||
|     } | ||||
| } | ||||
| @@ -11,11 +11,9 @@ declare(strict_types=1); | ||||
|  | ||||
| namespace Chill\MainBundle\Form\Type; | ||||
|  | ||||
| use Chill\MainBundle\Form\DataMapper\NotificationFlagDataMapper; | ||||
| use Chill\MainBundle\Notification\NotificationFlagManager; | ||||
| use Symfony\Component\Form\AbstractType; | ||||
| use Symfony\Component\Form\Extension\Core\Type\CheckboxType; | ||||
| use Symfony\Component\Form\Extension\Core\Type\FormType; | ||||
| use Symfony\Component\Form\FormBuilderInterface; | ||||
| use Symfony\Component\OptionsResolver\OptionsResolver; | ||||
|  | ||||
| @@ -30,27 +28,24 @@ class NotificationFlagsType extends AbstractType | ||||
|  | ||||
|     public function buildForm(FormBuilderInterface $builder, array $options): void | ||||
|     { | ||||
|         $builder->setDataMapper(new NotificationFlagDataMapper($this->notificationFlagProviders)); | ||||
|  | ||||
|         foreach ($this->notificationFlagProviders as $flagProvider) { | ||||
|             $flag = $flagProvider->getFlag(); | ||||
|             $builder->add($flag, FormType::class, [ | ||||
|             $flagBuilder = $builder->create($flag, options: [ | ||||
|                 'label' => $flagProvider->getLabel(), | ||||
|                 'required' => false, | ||||
|                 'compound' => true, | ||||
|             ]); | ||||
|  | ||||
|             $builder->get($flag) | ||||
|             $flagBuilder | ||||
|                 ->add('immediate_email', CheckboxType::class, [ | ||||
|                     'label' => false, | ||||
|                     'required' => false, | ||||
|                     'mapped' => false, | ||||
|                 ]) | ||||
|                 ->add('daily_email', CheckboxType::class, [ | ||||
|                 ->add('daily_digest', CheckboxType::class, [ | ||||
|                     'label' => false, | ||||
|                     'required' => false, | ||||
|                     'mapped' => false, | ||||
|                 ]) | ||||
|             ; | ||||
|             $builder->add($flagBuilder); | ||||
|         } | ||||
|     } | ||||
|  | ||||
| @@ -58,6 +53,7 @@ class NotificationFlagsType extends AbstractType | ||||
|     { | ||||
|         $resolver->setDefaults([ | ||||
|             'data_class' => null, | ||||
|             'compound' => true, | ||||
|         ]); | ||||
|     } | ||||
| } | ||||
|   | ||||
| @@ -11,31 +11,29 @@ declare(strict_types=1); | ||||
| 
 | ||||
| namespace Chill\MainBundle\Form; | ||||
| 
 | ||||
| use Chill\MainBundle\Action\User\UpdateProfile\UpdateProfileCommand; | ||||
| use Chill\MainBundle\Form\Type\ChillPhoneNumberType; | ||||
| use Chill\MainBundle\Form\Type\NotificationFlagsType; | ||||
| use Symfony\Component\Form\AbstractType; | ||||
| use Symfony\Component\Form\FormBuilderInterface; | ||||
| use Symfony\Component\OptionsResolver\OptionsResolver; | ||||
| 
 | ||||
| class UserProfileType extends AbstractType | ||||
| class UpdateProfileType extends AbstractType | ||||
| { | ||||
|     public function buildForm(FormBuilderInterface $builder, array $options) | ||||
|     public function buildForm(FormBuilderInterface $builder, array $options): void | ||||
|     { | ||||
|         $builder | ||||
|             ->add('phonenumber', ChillPhoneNumberType::class, [ | ||||
|                 'required' => false, | ||||
|             ]) | ||||
|             ->add('notificationFlags', NotificationFlagsType::class, [ | ||||
|                 'label' => false, | ||||
|                 'mapped' => false, | ||||
|             ]) | ||||
|             ->add('notificationFlags', NotificationFlagsType::class) | ||||
|         ; | ||||
|     } | ||||
| 
 | ||||
|     public function configureOptions(OptionsResolver $resolver) | ||||
|     public function configureOptions(OptionsResolver $resolver): void | ||||
|     { | ||||
|         $resolver->setDefaults([ | ||||
|             'data_class' => \Chill\MainBundle\Entity\User::class, | ||||
|             'data_class' => UpdateProfileCommand::class, | ||||
|         ]); | ||||
|     } | ||||
| } | ||||
| @@ -59,7 +59,7 @@ class UserPasswordType extends AbstractType | ||||
|                 'invalid_message' => 'The password fields must match', | ||||
|                 'constraints' => [ | ||||
|                     new Length([ | ||||
|                         'min' => 9, | ||||
|                         'min' => 14, | ||||
|                         'minMessage' => 'The password must be greater than {{ limit }} characters', | ||||
|                     ]), | ||||
|                     new NotBlank(), | ||||
|   | ||||
| @@ -105,6 +105,11 @@ class UserType extends AbstractType | ||||
|                 'required' => false, | ||||
|                 'input' => 'datetime_immutable', | ||||
|                 'label' => 'absence.Absence start', | ||||
|             ]) | ||||
|             ->add('absenceEnd', ChillDateType::class, [ | ||||
|                 'required' => false, | ||||
|                 'input' => 'datetime_immutable', | ||||
|                 'label' => 'absence.Absence end', | ||||
|             ]); | ||||
|  | ||||
|         // @phpstan-ignore-next-line | ||||
|   | ||||
| @@ -53,11 +53,16 @@ readonly class DailyNotificationDigestCronjob implements CronJobInterface | ||||
|     public function run(array $lastExecutionData): ?array | ||||
|     { | ||||
|         $now = $this->clock->now(); | ||||
|  | ||||
|         if (isset($lastExecutionData['last_execution'])) { | ||||
|             $lastExecution = \DateTimeImmutable::createFromFormat( | ||||
|                 \DateTimeImmutable::ATOM, | ||||
|                 $lastExecutionData['last_execution'] | ||||
|             ); | ||||
|  | ||||
|             if (false === $lastExecution) { | ||||
|                 $lastExecution = $now->sub(new \DateInterval('P1D')); | ||||
|             } | ||||
|         } else { | ||||
|             $lastExecution = $now->sub(new \DateInterval('P1D')); | ||||
|         } | ||||
| @@ -96,7 +101,7 @@ readonly class DailyNotificationDigestCronjob implements CronJobInterface | ||||
|         ]); | ||||
|  | ||||
|         return [ | ||||
|             'last_execution' => $now->format('Y-m-d-H:i:s.u e'), | ||||
|             'last_execution' => $now->format(\DateTimeInterface::ATOM), | ||||
|         ]; | ||||
|     } | ||||
| } | ||||
|   | ||||
| @@ -57,9 +57,15 @@ class EntityWorkflowRepository implements ObjectRepository | ||||
|         return (int) $qb->getQuery()->getSingleScalarResult(); | ||||
|     } | ||||
|  | ||||
|     public function countBySubscriber(User $user): int | ||||
|     /** | ||||
|      * @param bool|null $isFinal true to get only the entityWorkflow which is finalized, false to get the workflows that are not finalized, and null to ignore | ||||
|      * | ||||
|      * @throws \Doctrine\ORM\NoResultException | ||||
|      * @throws \Doctrine\ORM\NonUniqueResultException | ||||
|      */ | ||||
|     public function countBySubscriber(User $user, ?bool $isFinal = null): int | ||||
|     { | ||||
|         $qb = $this->buildQueryBySubscriber($user)->select('count(ew)'); | ||||
|         $qb = $this->buildQueryBySubscriber($user, $isFinal)->select('count(ew)'); | ||||
|  | ||||
|         return (int) $qb->getQuery()->getSingleScalarResult(); | ||||
|     } | ||||
| @@ -182,9 +188,14 @@ class EntityWorkflowRepository implements ObjectRepository | ||||
|         return $qb->getQuery()->getResult(); | ||||
|     } | ||||
|  | ||||
|     public function findBySubscriber(User $user, ?array $orderBy = null, $limit = null, $offset = null): array | ||||
|     /** | ||||
|      * @param bool|null  $isFinal true to get only the entityWorkflow which is finalized, false to get the workflows that are not finalized, and null to ignore | ||||
|      * @param mixed|null $limit | ||||
|      * @param mixed|null $offset | ||||
|      */ | ||||
|     public function findBySubscriber(User $user, ?bool $isFinal = null, ?array $orderBy = null, $limit = null, $offset = null): array | ||||
|     { | ||||
|         $qb = $this->buildQueryBySubscriber($user)->select('ew'); | ||||
|         $qb = $this->buildQueryBySubscriber($user, $isFinal)->select('ew'); | ||||
|  | ||||
|         foreach ($orderBy as $key => $sort) { | ||||
|             $qb->addOrderBy('ew.'.$key, $sort); | ||||
| @@ -312,7 +323,7 @@ class EntityWorkflowRepository implements ObjectRepository | ||||
|         return $qb; | ||||
|     } | ||||
|  | ||||
|     private function buildQueryBySubscriber(User $user): QueryBuilder | ||||
|     private function buildQueryBySubscriber(User $user, ?bool $isFinal): QueryBuilder | ||||
|     { | ||||
|         $qb = $this->repository->createQueryBuilder('ew'); | ||||
|  | ||||
| @@ -325,6 +336,14 @@ class EntityWorkflowRepository implements ObjectRepository | ||||
|  | ||||
|         $qb->setParameter('user', $user); | ||||
|  | ||||
|         if (null !== $isFinal) { | ||||
|             if ($isFinal) { | ||||
|                 $qb->andWhere(sprintf('EXISTS (SELECT 1 FROM %s step WHERE step.isFinal = true AND ew = step.entityWorkflow)', EntityWorkflowStep::class)); | ||||
|             } else { | ||||
|                 $qb->andWhere(sprintf('NOT EXISTS (SELECT 1 FROM %s step WHERE step.isFinal = true AND ew = step.entityWorkflow)', EntityWorkflowStep::class)); | ||||
|             } | ||||
|         } | ||||
|  | ||||
|         return $qb; | ||||
|     } | ||||
| } | ||||
|   | ||||
| @@ -37,8 +37,13 @@ export const ISOToDate = (str: string | null): Date | null => { | ||||
|         return null; | ||||
|     } | ||||
|  | ||||
|     const [year, month, day] = str.split("-").map((p) => parseInt(p)); | ||||
|     // If the string already contains time info, use it directly | ||||
|     if (str.includes("T") || str.includes(" ")) { | ||||
|         return new Date(str); | ||||
|     } | ||||
|  | ||||
|     // Otherwise, parse date only | ||||
|     const [year, month, day] = str.split("-").map((p) => parseInt(p)); | ||||
|     return new Date(year, month - 1, day, 0, 0, 0, 0); | ||||
| }; | ||||
|  | ||||
| @@ -69,20 +74,19 @@ export const ISOToDatetime = (str: string | null): Date | null => { | ||||
|  * | ||||
|  */ | ||||
| export const datetimeToISO = (date: Date): string => { | ||||
|     let cal, time, offset; | ||||
|     cal = [ | ||||
|     const cal = [ | ||||
|         date.getFullYear(), | ||||
|         (date.getMonth() + 1).toString().padStart(2, "0"), | ||||
|         date.getDate().toString().padStart(2, "0"), | ||||
|     ].join("-"); | ||||
|  | ||||
|     time = [ | ||||
|     const time = [ | ||||
|         date.getHours().toString().padStart(2, "0"), | ||||
|         date.getMinutes().toString().padStart(2, "0"), | ||||
|         date.getSeconds().toString().padStart(2, "0"), | ||||
|     ].join(":"); | ||||
|  | ||||
|     offset = [ | ||||
|     const offset = [ | ||||
|         date.getTimezoneOffset() <= 0 ? "+" : "-", | ||||
|         Math.abs(Math.floor(date.getTimezoneOffset() / 60)) | ||||
|             .toString() | ||||
|   | ||||
| @@ -0,0 +1,13 @@ | ||||
| /** | ||||
|  * Extracts the "returnPath" parameter from the current URL's query string and returns it. | ||||
|  * If the parameter is not present, returns the provided fallback path. | ||||
|  * | ||||
|  * @param {string} fallbackPath - The fallback path to use if "returnPath" is not found in the query string. | ||||
|  * @return {string} The "returnPath" from the query string, or the fallback path if "returnPath" is not present. | ||||
|  */ | ||||
| export function returnPathOr(fallbackPath: string): string { | ||||
|     const urlParams = new URLSearchParams(window.location.search); | ||||
|     const returnPath = urlParams.get("returnPath"); | ||||
|  | ||||
|     return returnPath ?? fallbackPath; | ||||
| } | ||||
| @@ -0,0 +1,16 @@ | ||||
| import { EntityWorkflow } from "ChillMainAssets/types"; | ||||
| import { makeFetch } from "ChillMainAssets/lib/api/apiMethods"; | ||||
|  | ||||
| export const fetchWorkflow = async ( | ||||
|     workflowId: number, | ||||
| ): Promise<EntityWorkflow> => { | ||||
|     try { | ||||
|         return await makeFetch<null, EntityWorkflow>( | ||||
|             "GET", | ||||
|             `/api/1.0/main/workflow/${workflowId}.json`, | ||||
|         ); | ||||
|     } catch (error) { | ||||
|         console.error(`Failed to fetch workflow ${workflowId}:`, error); | ||||
|         throw error; | ||||
|     } | ||||
| }; | ||||
| @@ -1,5 +1,9 @@ | ||||
| import { GenericDoc } from "ChillDocStoreAssets/types/generic_doc"; | ||||
| import { | ||||
|     GenericDoc, | ||||
|     isGenericDocWithStoredObject, | ||||
| } from "ChillDocStoreAssets/types/generic_doc"; | ||||
| import { StoredObject, StoredObjectStatus } from "ChillDocStoreAssets/types"; | ||||
| import { Person } from "../../../ChillPersonBundle/Resources/public/types"; | ||||
|  | ||||
| export interface DateTime { | ||||
|     datetime: string; | ||||
| @@ -202,6 +206,77 @@ export interface WorkflowAttachment { | ||||
|     genericDoc: null | GenericDoc; | ||||
| } | ||||
|  | ||||
| export type AttachmentWithDocAndStored = WorkflowAttachment & { | ||||
|     genericDoc: GenericDoc & { storedObject: StoredObject }; | ||||
| }; | ||||
|  | ||||
| export function isAttachmentWithDocAndStored( | ||||
|     a: WorkflowAttachment, | ||||
| ): a is AttachmentWithDocAndStored { | ||||
|     return ( | ||||
|         isWorkflowAttachmentWithGenericDoc(a) && | ||||
|         isGenericDocWithStoredObject(a.genericDoc) | ||||
|     ); | ||||
| } | ||||
|  | ||||
| export function isWorkflowAttachmentWithGenericDoc( | ||||
|     attachment: WorkflowAttachment, | ||||
| ): attachment is WorkflowAttachment & { genericDoc: GenericDoc } { | ||||
|     return attachment.genericDoc !== null; | ||||
| } | ||||
|  | ||||
| export interface Workflow { | ||||
|     name: string; | ||||
|     text: string; | ||||
| } | ||||
|  | ||||
| export interface EntityWorkflowStep { | ||||
|     type: "entity_workflow_step"; | ||||
|     id: number; | ||||
|     comment: string; | ||||
|     currentStep: StepDefinition; | ||||
|     isFinal: boolean; | ||||
|     isFreezed: boolean; | ||||
|     isFinalized: boolean; | ||||
|     transitionPrevious: Transition | null; | ||||
|     transitionAfter: Transition | null; | ||||
|     previousId: number | null; | ||||
|     nextId: number | null; | ||||
|     transitionPreviousBy: User | null; | ||||
|     transitionPreviousAt: DateTime | null; | ||||
| } | ||||
|  | ||||
| export interface Transition { | ||||
|     name: string; | ||||
|     text: string; | ||||
|     isForward: boolean; | ||||
| } | ||||
|  | ||||
| export interface StepDefinition { | ||||
|     name: string; | ||||
|     text: string; | ||||
| } | ||||
|  | ||||
| export interface EntityWorkflow { | ||||
|     type: "entity_workflow"; | ||||
|     id: number; | ||||
|     relatedEntityClass: string; | ||||
|     relatedEntityId: number; | ||||
|     workflow: Workflow; | ||||
|     currentStep: EntityWorkflowStep; | ||||
|     steps: EntityWorkflowStep[]; | ||||
|     datas: WorkflowData; | ||||
|     title: string; | ||||
|     isOnHoldAtCurrentStep: boolean; | ||||
|     _permissions: { | ||||
|         CHILL_MAIN_WORKFLOW_ATTACHMENT_EDIT: boolean; | ||||
|     }; | ||||
| } | ||||
|  | ||||
| export interface WorkflowData { | ||||
|     persons: Person[]; | ||||
| } | ||||
|  | ||||
| export interface ExportGeneration { | ||||
|     id: string; | ||||
|     type: "export_generation"; | ||||
| @@ -215,3 +290,8 @@ export interface ExportGeneration { | ||||
| export interface PrivateCommentEmbeddable { | ||||
|     comments: Record<number, string>; | ||||
| } | ||||
|  | ||||
| /** | ||||
|  * Possible states for the WaitingScreen Component. | ||||
|  */ | ||||
| export type WaitingScreenState = "pending" | "failure" | "stopped" | "ready"; | ||||
|   | ||||
| @@ -10,7 +10,8 @@ import { computed, onMounted, ref } from "vue"; | ||||
| import { StoredObject, StoredObjectStatus } from "ChillDocStoreAssets/types"; | ||||
| import { fetchExportGenerationStatus } from "ChillMainAssets/lib/api/export"; | ||||
| import DocumentActionButtonsGroup from "ChillDocStoreAssets/vuejs/DocumentActionButtonsGroup.vue"; | ||||
| import { ExportGeneration } from "ChillMainAssets/types"; | ||||
| import WaitingScreen from "../_components/WaitingScreen.vue"; | ||||
| import { ExportGeneration, WaitingScreenState } from "ChillMainAssets/types"; | ||||
|  | ||||
| interface AppProps { | ||||
|     exportGenerationId: string; | ||||
| @@ -34,13 +35,16 @@ const storedObject = computed<null | StoredObject>(() => { | ||||
| }); | ||||
|  | ||||
| const isPending = computed<boolean>(() => status.value === "pending"); | ||||
| const isFetching = computed<boolean>( | ||||
|     () => tryiesForReady.value < maxTryiesForReady, | ||||
| ); | ||||
| const isReady = computed<boolean>(() => status.value === "ready"); | ||||
| const isFailure = computed<boolean>(() => status.value === "failure"); | ||||
| const filename = computed<string>(() => `${props.title}-${props.createdDate}`); | ||||
|  | ||||
| const state = computed<WaitingScreenState>((): WaitingScreenState => { | ||||
|     if (status.value === "empty") { | ||||
|         return "pending"; | ||||
|     } | ||||
|  | ||||
|     return status.value; | ||||
| }); | ||||
|  | ||||
| /** | ||||
|  * counter for the number of times that we check for a new status | ||||
|  */ | ||||
| @@ -85,57 +89,36 @@ onMounted(() => { | ||||
| </script> | ||||
|  | ||||
| <template> | ||||
|     <div id="waiting-screen"> | ||||
|         <div | ||||
|             v-if="isPending && isFetching" | ||||
|             class="alert alert-danger text-center" | ||||
|         > | ||||
|             <div> | ||||
|                 <p> | ||||
|                     {{ trans(EXPORT_GENERATION_EXPORT_GENERATION_IS_PENDING) }} | ||||
|                 </p> | ||||
|             </div> | ||||
|     <WaitingScreen :state="state"> | ||||
|         <template v-slot:pending> | ||||
|             <p> | ||||
|                 {{ trans(EXPORT_GENERATION_EXPORT_GENERATION_IS_PENDING) }} | ||||
|             </p> | ||||
|         </template> | ||||
|  | ||||
|             <div> | ||||
|                 <i class="fa fa-cog fa-spin fa-3x fa-fw"></i> | ||||
|                 <span class="sr-only">Loading...</span> | ||||
|             </div> | ||||
|         </div> | ||||
|         <div v-if="isPending && !isFetching" class="alert alert-info"> | ||||
|             <div> | ||||
|                 <p> | ||||
|                     {{ trans(EXPORT_GENERATION_TOO_MANY_RETRIES) }} | ||||
|                 </p> | ||||
|             </div> | ||||
|         </div> | ||||
|         <div v-if="isFailure" class="alert alert-danger text-center"> | ||||
|             <div> | ||||
|                 <p> | ||||
|                     {{ trans(EXPORT_GENERATION_ERROR_WHILE_GENERATING_EXPORT) }} | ||||
|                 </p> | ||||
|             </div> | ||||
|         </div> | ||||
|         <div v-if="isReady" class="alert alert-success text-center"> | ||||
|             <div> | ||||
|                 <p> | ||||
|                     {{ trans(EXPORT_GENERATION_EXPORT_READY) }} | ||||
|                 </p> | ||||
|         <template v-slot:stopped> | ||||
|             <p> | ||||
|                 {{ trans(EXPORT_GENERATION_TOO_MANY_RETRIES) }} | ||||
|             </p> | ||||
|         </template> | ||||
|  | ||||
|                 <p v-if="storedObject !== null"> | ||||
|                     <document-action-buttons-group | ||||
|                         :stored-object="storedObject" | ||||
|                         :filename="filename" | ||||
|                     ></document-action-buttons-group> | ||||
|                 </p> | ||||
|             </div> | ||||
|         </div> | ||||
|     </div> | ||||
|         <template v-slot:failure> | ||||
|             <p> | ||||
|                 {{ trans(EXPORT_GENERATION_ERROR_WHILE_GENERATING_EXPORT) }} | ||||
|             </p> | ||||
|         </template> | ||||
|  | ||||
|         <template v-slot:ready> | ||||
|             <p> | ||||
|                 {{ trans(EXPORT_GENERATION_EXPORT_READY) }} | ||||
|             </p> | ||||
|  | ||||
|             <p v-if="storedObject !== null"> | ||||
|                 <document-action-buttons-group | ||||
|                     :stored-object="storedObject" | ||||
|                     :filename="filename" | ||||
|                 ></document-action-buttons-group> | ||||
|             </p> | ||||
|         </template> | ||||
|     </WaitingScreen> | ||||
| </template> | ||||
|  | ||||
| <style scoped lang="scss"> | ||||
| #waiting-screen { | ||||
|     > .alert { | ||||
|         min-height: 350px; | ||||
|     } | ||||
| } | ||||
| </style> | ||||
|   | ||||
| @@ -0,0 +1,75 @@ | ||||
| <script setup lang="ts"> | ||||
| import { useIntervalFn } from "@vueuse/core"; | ||||
| import { fetchWorkflow } from "ChillMainAssets/lib/workflow/api"; | ||||
| import { returnPathOr } from "ChillMainAssets/lib/return_path/returnPathHelper"; | ||||
| import { ref } from "vue"; | ||||
| import WaitingScreen from "ChillMainAssets/vuejs/_components/WaitingScreen.vue"; | ||||
| import { WaitingScreenState } from "ChillMainAssets/types"; | ||||
| import { | ||||
|     trans, | ||||
|     WORKFLOW_WAIT_TITLE, | ||||
|     WORKFLOW_WAIT_ERROR_WHILE_WAITING, | ||||
|     WORKFLOW_WAIT_SUCCESS, | ||||
| } from "translator"; | ||||
|  | ||||
| interface WaitPostProcessWorkflowComponentProps { | ||||
|     workflowId: number; | ||||
|     expectedStep: string; | ||||
| } | ||||
|  | ||||
| const props = defineProps<WaitPostProcessWorkflowComponentProps>(); | ||||
| const counter = ref<number>(0); | ||||
| const MAX_TRYIES = 50; | ||||
|  | ||||
| const state = ref<WaitingScreenState>("pending"); | ||||
|  | ||||
| const { pause, resume } = useIntervalFn( | ||||
|     async () => { | ||||
|         try { | ||||
|             const workflow = await fetchWorkflow(props.workflowId); | ||||
|             counter.value++; | ||||
|             if (workflow.currentStep.currentStep.name === props.expectedStep) { | ||||
|                 window.location.assign( | ||||
|                     returnPathOr("/fr/main/workflow" + workflow.id + "/show"), | ||||
|                 ); | ||||
|                 resume(); | ||||
|                 state.value = "ready"; | ||||
|             } | ||||
|  | ||||
|             if (counter.value > MAX_TRYIES) { | ||||
|                 pause(); | ||||
|                 state.value = "failure"; | ||||
|             } | ||||
|         } catch (error) { | ||||
|             console.error(error); | ||||
|             pause(); | ||||
|         } | ||||
|     }, | ||||
|     2000, | ||||
|     { immediate: true }, | ||||
| ); | ||||
| </script> | ||||
|  | ||||
| <template> | ||||
|     <div class="container"> | ||||
|         <WaitingScreen :state="state"> | ||||
|             <template v-slot:pending> | ||||
|                 <p> | ||||
|                     {{ trans(WORKFLOW_WAIT_TITLE) }} | ||||
|                 </p> | ||||
|             </template> | ||||
|             <template v-slot:failure> | ||||
|                 <p> | ||||
|                     {{ trans(WORKFLOW_WAIT_ERROR_WHILE_WAITING) }} | ||||
|                 </p> | ||||
|             </template> | ||||
|             <template v-slot:ready> | ||||
|                 <p> | ||||
|                     {{ trans(WORKFLOW_WAIT_SUCCESS) }} | ||||
|                 </p> | ||||
|             </template> | ||||
|         </WaitingScreen> | ||||
|     </div> | ||||
| </template> | ||||
|  | ||||
| <style scoped lang="scss"></style> | ||||
| @@ -0,0 +1,51 @@ | ||||
| import { createApp } from "vue"; | ||||
| import App from "./App.vue"; | ||||
|  | ||||
| function mountApp(): void { | ||||
|     const el = document.querySelector<HTMLDivElement>(".screen-wait"); | ||||
|     if (!el) { | ||||
|         console.error( | ||||
|             "WaitPostProcessWorkflow: mount element .screen-wait not found", | ||||
|         ); | ||||
|         return; | ||||
|     } | ||||
|  | ||||
|     const workflowIdAttr = el.getAttribute("data-workflow-id"); | ||||
|     const expectedStep = el.getAttribute("data-expected-step") || ""; | ||||
|  | ||||
|     if (!workflowIdAttr) { | ||||
|         console.error( | ||||
|             "WaitPostProcessWorkflow: data-workflow-id attribute missing on mount element", | ||||
|         ); | ||||
|         return; | ||||
|     } | ||||
|  | ||||
|     if (!expectedStep) { | ||||
|         console.error( | ||||
|             "WaitPostProcessWorkflow: data-expected-step attribute missing on mount element", | ||||
|         ); | ||||
|         return; | ||||
|     } | ||||
|  | ||||
|     const workflowId = Number(workflowIdAttr); | ||||
|     if (Number.isNaN(workflowId)) { | ||||
|         console.error( | ||||
|             "WaitPostProcessWorkflow: data-workflow-id is not a valid number:", | ||||
|             workflowIdAttr, | ||||
|         ); | ||||
|         return; | ||||
|     } | ||||
|  | ||||
|     const app = createApp(App, { | ||||
|         workflowId, | ||||
|         expectedStep, | ||||
|     }); | ||||
|  | ||||
|     app.mount(el); | ||||
| } | ||||
|  | ||||
| if (document.readyState === "loading") { | ||||
|     document.addEventListener("DOMContentLoaded", mountApp); | ||||
| } else { | ||||
|     mountApp(); | ||||
| } | ||||
| @@ -1,10 +1,12 @@ | ||||
| <script setup lang="ts"> | ||||
| import { computed, useTemplateRef } from "vue"; | ||||
| import type { WorkflowAttachment } from "ChillMainAssets/types"; | ||||
| import { computed, onMounted, ref, useTemplateRef } from "vue"; | ||||
| import type { EntityWorkflow, WorkflowAttachment } from "ChillMainAssets/types"; | ||||
| import PickGenericDocModal from "ChillMainAssets/vuejs/WorkflowAttachment/Component/PickGenericDocModal.vue"; | ||||
| import { GenericDocForAccompanyingPeriod } from "ChillDocStoreAssets/types/generic_doc"; | ||||
| import AttachmentList from "ChillMainAssets/vuejs/WorkflowAttachment/Component/AttachmentList.vue"; | ||||
| import { GenericDoc } from "ChillDocStoreAssets/types"; | ||||
| import { fetchWorkflow } from "ChillMainAssets/lib/workflow/api"; | ||||
| import { trans, WORKFLOW_ATTACHMENTS_ADD_AN_ATTACHMENT } from "translator"; | ||||
|  | ||||
| interface AppConfig { | ||||
|     workflowId: number; | ||||
| @@ -34,6 +36,13 @@ const attachedGenericDoc = computed<GenericDocForAccompanyingPeriod[]>( | ||||
|             ) as GenericDocForAccompanyingPeriod[], | ||||
| ); | ||||
|  | ||||
| const workflow = ref<EntityWorkflow | null>(null); | ||||
|  | ||||
| onMounted(async () => { | ||||
|     workflow.value = await fetchWorkflow(Number(props.workflowId)); | ||||
|     console.log("workflow", workflow.value); | ||||
| }); | ||||
|  | ||||
| const openModal = function () { | ||||
|     pickDocModal.value?.openModal(); | ||||
| }; | ||||
| @@ -49,23 +58,33 @@ const onPickGenericDoc = ({ | ||||
| const onRemoveAttachment = (payload: { attachment: WorkflowAttachment }) => { | ||||
|     emit("removeAttachment", payload); | ||||
| }; | ||||
|  | ||||
| const canEditAttachement = computed<boolean>(() => { | ||||
|     if (null === workflow.value) { | ||||
|         return false; | ||||
|     } | ||||
|  | ||||
|     return workflow.value._permissions.CHILL_MAIN_WORKFLOW_ATTACHMENT_EDIT; | ||||
| }); | ||||
| </script> | ||||
|  | ||||
| <template> | ||||
|     <pick-generic-doc-modal | ||||
|         :workflow="workflow" | ||||
|         :accompanying-period-id="props.accompanyingPeriodId" | ||||
|         :to-remove="attachedGenericDoc" | ||||
|         ref="pickDocModal" | ||||
|         @pickGenericDoc="onPickGenericDoc" | ||||
|     ></pick-generic-doc-modal> | ||||
|     <attachment-list | ||||
|         :workflow="workflow" | ||||
|         :attachments="props.attachments" | ||||
|         @removeAttachment="onRemoveAttachment" | ||||
|     ></attachment-list> | ||||
|     <ul class="record_actions"> | ||||
|     <ul v-if="canEditAttachement" class="record_actions"> | ||||
|         <li> | ||||
|             <button type="button" class="btn btn-create" @click="openModal"> | ||||
|                 Ajouter une pièce jointe | ||||
|                 {{ trans(WORKFLOW_ATTACHMENTS_ADD_AN_ATTACHMENT) }} | ||||
|             </button> | ||||
|         </li> | ||||
|     </ul> | ||||
|   | ||||
| @@ -1,10 +1,18 @@ | ||||
| <script setup lang="ts"> | ||||
| import { WorkflowAttachment } from "ChillMainAssets/types"; | ||||
| import { | ||||
|     AttachmentWithDocAndStored, | ||||
|     EntityWorkflow, | ||||
|     isAttachmentWithDocAndStored, | ||||
|     WorkflowAttachment, | ||||
| } from "ChillMainAssets/types"; | ||||
| import GenericDocItemBox from "ChillMainAssets/vuejs/WorkflowAttachment/Component/GenericDocItemBox.vue"; | ||||
| import DocumentActionButtonsGroup from "ChillDocStoreAssets/vuejs/DocumentActionButtonsGroup.vue"; | ||||
| import { computed } from "vue"; | ||||
| import { trans, WORKFLOW_ATTACHMENTS_NO_ATTACHMENT } from "translator"; | ||||
|  | ||||
| interface AttachmentListProps { | ||||
|     attachments: WorkflowAttachment[]; | ||||
|     workflow: EntityWorkflow | null; | ||||
| } | ||||
|  | ||||
| const emit = defineEmits<{ | ||||
| @@ -13,30 +21,43 @@ const emit = defineEmits<{ | ||||
| }>(); | ||||
|  | ||||
| const props = defineProps<AttachmentListProps>(); | ||||
|  | ||||
| const notNullAttachments = computed<AttachmentWithDocAndStored[]>(() => | ||||
|     props.attachments.filter( | ||||
|         (a: WorkflowAttachment): a is AttachmentWithDocAndStored => | ||||
|             isAttachmentWithDocAndStored(a), | ||||
|     ), | ||||
| ); | ||||
|  | ||||
| const canRemove = computed<boolean>((): boolean => { | ||||
|     if (null === props.workflow) { | ||||
|         return false; | ||||
|     } | ||||
|  | ||||
|     return props.workflow._permissions.CHILL_MAIN_WORKFLOW_ATTACHMENT_EDIT; | ||||
| }); | ||||
| </script> | ||||
|  | ||||
| <template> | ||||
|     <p | ||||
|         v-if="props.attachments.length === 0" | ||||
|         v-if="notNullAttachments.length === 0" | ||||
|         class="chill-no-data-statement text-center" | ||||
|     > | ||||
|         Aucune pièce jointe | ||||
|         {{ trans(WORKFLOW_ATTACHMENTS_NO_ATTACHMENT) }} | ||||
|     </p> | ||||
|     <!-- TODO translate --> | ||||
|     <div else class="flex-table"> | ||||
|         <div v-for="a in props.attachments" :key="a.id" class="item-bloc"> | ||||
|     <div v-else class="flex-table"> | ||||
|         <div v-for="a in notNullAttachments" :key="a.id" class="item-bloc"> | ||||
|             <generic-doc-item-box | ||||
|                 v-if="a.genericDoc !== null" | ||||
|                 :generic-doc="a.genericDoc" | ||||
|             ></generic-doc-item-box> | ||||
|             <div class="item-row separator"> | ||||
|                 <ul class="record_actions"> | ||||
|                     <li v-if="a.genericDoc?.storedObject !== null"> | ||||
|                     <li> | ||||
|                         <document-action-buttons-group | ||||
|                             :stored-object="a.genericDoc.storedObject" | ||||
|                         ></document-action-buttons-group> | ||||
|                     </li> | ||||
|                     <li> | ||||
|                     <li v-if="canRemove"> | ||||
|                         <button | ||||
|                             type="button" | ||||
|                             class="btn btn-delete" | ||||
|   | ||||
| @@ -1,8 +1,8 @@ | ||||
| <script setup lang="ts"> | ||||
| import { GenericDocForAccompanyingPeriod } from "ChillDocStoreAssets/types/generic_doc"; | ||||
| import { GenericDoc } from "ChillDocStoreAssets/types/generic_doc"; | ||||
|  | ||||
| interface GenericDocItemBoxProps { | ||||
|     genericDoc: GenericDocForAccompanyingPeriod; | ||||
|     genericDoc: GenericDoc; | ||||
| } | ||||
|  | ||||
| const props = defineProps<GenericDocItemBoxProps>(); | ||||
|   | ||||
| @@ -6,8 +6,10 @@ import { | ||||
| import PickGenericDocItem from "ChillMainAssets/vuejs/WorkflowAttachment/Component/PickGenericDocItem.vue"; | ||||
| import { fetch_generic_docs_by_accompanying_period } from "ChillDocStoreAssets/js/generic-doc-api"; | ||||
| import { computed, onMounted, ref } from "vue"; | ||||
| import { EntityWorkflow } from "ChillMainAssets/types"; | ||||
|  | ||||
| interface PickGenericDocProps { | ||||
|     workflow: EntityWorkflow | null; | ||||
|     accompanyingPeriodId: number; | ||||
|     pickedList: GenericDocForAccompanyingPeriod[]; | ||||
|     toRemove: GenericDocForAccompanyingPeriod[]; | ||||
| @@ -36,9 +38,21 @@ const isPicked = (genericDoc: GenericDocForAccompanyingPeriod): boolean => | ||||
|     ) !== -1; | ||||
|  | ||||
| onMounted(async () => { | ||||
|     genericDocs.value = await fetch_generic_docs_by_accompanying_period( | ||||
|     const fetchedGenericDocs = await fetch_generic_docs_by_accompanying_period( | ||||
|         props.accompanyingPeriodId, | ||||
|     ); | ||||
|     const documentClasses = [ | ||||
|         "Chill\\DocStoreBundle\\Entity\\AccompanyingCourseDocument", | ||||
|         "Chill\\PersonBundle\\Entity\\AccompanyingPeriod\\AccompanyingPeriodWorkEvaluationDocument", | ||||
|         "Chill\\DocStoreBundle\\Entity\\PersonDocument", | ||||
|     ]; | ||||
|  | ||||
|     genericDocs.value = fetchedGenericDocs.filter( | ||||
|         (doc) => | ||||
|             !documentClasses.includes( | ||||
|                 props.workflow?.relatedEntityClass || "", | ||||
|             ) || props.workflow?.relatedEntityId !== doc.identifiers.id, | ||||
|     ); | ||||
|     loaded.value = true; | ||||
| }); | ||||
|  | ||||
|   | ||||
| @@ -3,8 +3,10 @@ import Modal from "ChillMainAssets/vuejs/_components/Modal.vue"; | ||||
| import { computed, ref, useTemplateRef } from "vue"; | ||||
| import PickGenericDoc from "ChillMainAssets/vuejs/WorkflowAttachment/Component/PickGenericDoc.vue"; | ||||
| import { GenericDocForAccompanyingPeriod } from "ChillDocStoreAssets/types/generic_doc"; | ||||
| import { EntityWorkflow } from "ChillMainAssets/types"; | ||||
|  | ||||
| interface PickGenericDocModalProps { | ||||
|     workflow: EntityWorkflow | null; | ||||
|     accompanyingPeriodId: number; | ||||
|     toRemove: GenericDocForAccompanyingPeriod[]; | ||||
| } | ||||
| @@ -80,6 +82,7 @@ defineExpose({ openModal, closeModal }); | ||||
|         </template> | ||||
|         <template v-slot:body> | ||||
|             <pick-generic-doc | ||||
|                 :workflow="props.workflow" | ||||
|                 :accompanying-period-id="props.accompanyingPeriodId" | ||||
|                 :to-remove="props.toRemove" | ||||
|                 :picked-list="pickeds" | ||||
|   | ||||
| @@ -1,40 +1,38 @@ | ||||
| <template> | ||||
|     <div class="d-grid gap-2 my-3"> | ||||
|         <button | ||||
|             class="btn btn-misc" | ||||
|             class="btn btn-outline-primary text-start d-flex align-items-center" | ||||
|             :class="{ active: subscriberFinal }" | ||||
|             type="button" | ||||
|             v-if="!subscriberFinal" | ||||
|             @click="subscribeTo('subscribe', 'final')" | ||||
|             @click=" | ||||
|                 subscribeTo( | ||||
|                     subscriberFinal ? 'unsubscribe' : 'subscribe', | ||||
|                     'final', | ||||
|                 ) | ||||
|             " | ||||
|         > | ||||
|             <i class="fa fa-check fa-fw"></i> | ||||
|             {{ trans(WORKFLOW_SUBSCRIBE_FINAL) }} | ||||
|             <i | ||||
|                 class="fa fa-fw me-2" | ||||
|                 :class="subscriberFinal ? 'fa-check-square-o' : 'fa-square-o'" | ||||
|             ></i> | ||||
|             <span>{{ trans(WORKFLOW_SUBSCRIBE_FINAL) }}</span> | ||||
|         </button> | ||||
|         <button | ||||
|             class="btn btn-misc" | ||||
|             class="btn btn-outline-primary text-start d-flex align-items-center" | ||||
|             :class="{ active: subscriberStep }" | ||||
|             type="button" | ||||
|             v-if="subscriberFinal" | ||||
|             @click="subscribeTo('unsubscribe', 'final')" | ||||
|             @click=" | ||||
|                 subscribeTo( | ||||
|                     subscriberStep ? 'unsubscribe' : 'subscribe', | ||||
|                     'step', | ||||
|                 ) | ||||
|             " | ||||
|         > | ||||
|             <i class="fa fa-times fa-fw"></i> | ||||
|             {{ trans(WORKFLOW_UNSUBSCRIBE_FINAL) }} | ||||
|         </button> | ||||
|         <button | ||||
|             class="btn btn-misc" | ||||
|             type="button" | ||||
|             v-if="!subscriberStep" | ||||
|             @click="subscribeTo('subscribe', 'step')" | ||||
|         > | ||||
|             <i class="fa fa-check fa-fw"></i> | ||||
|             {{ trans(WORKFLOW_SUBSCRIBE_ALL_STEPS) }} | ||||
|         </button> | ||||
|         <button | ||||
|             class="btn btn-misc" | ||||
|             type="button" | ||||
|             v-if="subscriberStep" | ||||
|             @click="subscribeTo('unsubscribe', 'step')" | ||||
|         > | ||||
|             <i class="fa fa-times fa-fw"></i> | ||||
|             {{ trans(WORKFLOW_UNSUBSCRIBE_ALL_STEPS) }} | ||||
|             <i | ||||
|                 class="fa fa-fw me-2" | ||||
|                 :class="subscriberStep ? 'fa-check-square-o' : 'fa-square-o'" | ||||
|             ></i> | ||||
|             <span>{{ trans(WORKFLOW_SUBSCRIBE_ALL_STEPS) }}</span> | ||||
|         </button> | ||||
|     </div> | ||||
| </template> | ||||
| @@ -45,9 +43,7 @@ import { defineProps, defineEmits } from "vue"; | ||||
| import { | ||||
|     trans, | ||||
|     WORKFLOW_SUBSCRIBE_FINAL, | ||||
|     WORKFLOW_UNSUBSCRIBE_FINAL, | ||||
|     WORKFLOW_SUBSCRIBE_ALL_STEPS, | ||||
|     WORKFLOW_UNSUBSCRIBE_ALL_STEPS, | ||||
| } from "translator"; | ||||
|  | ||||
| // props | ||||
|   | ||||
| @@ -84,6 +84,8 @@ const emits = defineEmits<{ | ||||
| } | ||||
| .modal-header .close { | ||||
|     border-top-right-radius: 0.3rem; | ||||
|     margin-right: 0; | ||||
|     margin-left: auto; | ||||
| } | ||||
| /* | ||||
|    * The following styles are auto-applied to elements with | ||||
|   | ||||
| @@ -0,0 +1,62 @@ | ||||
| <script setup lang="ts"> | ||||
| import { WaitingScreenState } from "ChillMainAssets/types"; | ||||
|  | ||||
| interface Props { | ||||
|     state: WaitingScreenState; | ||||
| } | ||||
|  | ||||
| const props = defineProps<Props>(); | ||||
| </script> | ||||
|  | ||||
| <template> | ||||
|     <div id="waiting-screen"> | ||||
|         <div | ||||
|             v-if="props.state === 'pending' && !!$slots.pending" | ||||
|             class="alert alert-danger text-center" | ||||
|         > | ||||
|             <div> | ||||
|                 <slot name="pending"></slot> | ||||
|             </div> | ||||
|  | ||||
|             <div> | ||||
|                 <i class="fa fa-cog fa-spin fa-3x fa-fw"></i> | ||||
|                 <span class="sr-only">Loading...</span> | ||||
|             </div> | ||||
|         </div> | ||||
|  | ||||
|         <div | ||||
|             v-if="props.state === 'stopped' && !!$slots.stopped" | ||||
|             class="alert alert-info" | ||||
|         > | ||||
|             <div> | ||||
|                 <slot name="stopped"></slot> | ||||
|             </div> | ||||
|         </div> | ||||
|  | ||||
|         <div | ||||
|             v-if="props.state === 'failure' && !!$slots.failure" | ||||
|             class="alert alert-danger text-center" | ||||
|         > | ||||
|             <div> | ||||
|                 <slot name="failure"></slot> | ||||
|             </div> | ||||
|         </div> | ||||
|  | ||||
|         <div | ||||
|             v-if="props.state === 'ready' && !!$slots.ready" | ||||
|             class="alert alert-success text-center" | ||||
|         > | ||||
|             <div> | ||||
|                 <slot name="ready"></slot> | ||||
|             </div> | ||||
|         </div> | ||||
|     </div> | ||||
| </template> | ||||
|  | ||||
| <style scoped lang="scss"> | ||||
| #waiting-screen { | ||||
|     > .alert { | ||||
|         min-height: 350px; | ||||
|     } | ||||
| } | ||||
| </style> | ||||
| @@ -44,17 +44,7 @@ | ||||
|                 {% endif %} | ||||
|                 {% endif %} | ||||
|             {% endblock content_view_actions_duplicate_link %} | ||||
|             {% block content_view_actions_merge %} | ||||
|                 <li> | ||||
|                     <a href="{{ chill_path_add_return_path('chill_thirdparty_find_duplicate', | ||||
|                         { 'thirdparty_id': entity.id }) }}" | ||||
|                        title="{{ 'Merge'|trans }}" | ||||
|                        class="btn btn-misc"> | ||||
|                         <i class="bi bi-chevron-contract"></i> | ||||
|                         {{ 'Merge'|trans }} | ||||
|                     </a> | ||||
|                 </li> | ||||
|             {% endblock %} | ||||
|             {% block content_view_actions_merge %}{% endblock %} | ||||
|             {% block content_view_actions_edit_link %} | ||||
|                 {% if chill_crud_action_exists(crud_name, 'edit') %} | ||||
|                 {% if is_granted(chill_crud_config('role', crud_name, 'edit'), entity) %} | ||||
|   | ||||
| @@ -280,11 +280,17 @@ | ||||
|     </div> | ||||
| {% endblock %} | ||||
|  | ||||
| {% block pick_linked_entities_widget %} | ||||
|     <input type="hidden" {{ block('widget_attributes') }} {% if value is not empty %}value="{{ value|escape('html_attr') }}" {% endif %} data-input-uniqid="{{ form.vars['uniqid'] }}" /> | ||||
|     <div data-input-uniqid="{{ form.vars['uniqid'] }}" data-module="pick-linked-entities" data-pick-entities-type="{{ form.vars['pick-entities-type'] }}" | ||||
|     ></div> | ||||
|  | ||||
| {% block pick_linked_entities_widget %} | ||||
|     <input type="hidden" {{ block('widget_attributes') }} | ||||
|         {% if value is not empty %}value="{{ value|escape('html_attr') }}" {% endif %} | ||||
|        data-input-uniqid="{{ form.vars['uniqid'] }}"/> | ||||
|     <div | ||||
|         data-input-uniqid="{{ form.vars['uniqid'] }}" | ||||
|         data-module="pick-linked-entities" | ||||
|         data-pick-entities-type="{{ form.vars['pick-entities-type'] }}" | ||||
|          data-suggested="{{ form.vars['suggested']|json_encode|escape('html_attr') }}" | ||||
|     ></div> | ||||
| {% endblock %} | ||||
|  | ||||
| {% block pick_postal_code_widget %} | ||||
|   | ||||
| @@ -0,0 +1,17 @@ | ||||
| {% if chill_main_config.top_banner is defined and chill_main_config.top_banner.text is defined %} | ||||
|     {% set banner_text = '' %} | ||||
|     {% set current_locale = app.request.locale %} | ||||
|  | ||||
|     {% if chill_main_config.top_banner.text[current_locale] is defined %} | ||||
|         {% set banner_text = chill_main_config.top_banner.text[current_locale] %} | ||||
|     {% else %} | ||||
|         {% set banner_text = chill_main_config.top_banner.text|first %} | ||||
|     {% endif %} | ||||
|  | ||||
|     {% if banner_text %} | ||||
|         <div class="top-banner w-100 text-center py-2" | ||||
|              style="{% if chill_main_config.top_banner.color is defined %}color: {{ chill_main_config.top_banner.color }};{% endif %}{% if chill_main_config.top_banner.background_color is defined %}background-color: {{ chill_main_config.top_banner.background_color }};{% endif %}"> | ||||
|             {{ banner_text }} | ||||
|         </div> | ||||
|     {% endif %} | ||||
| {% endif %} | ||||
| @@ -8,36 +8,36 @@ | ||||
|  | ||||
|     <div class="col-md-10"> | ||||
|         <h2>{{ 'absence.My absence'|trans }}</h2> | ||||
|         <div> | ||||
|             {% if user.absenceStart is not null %} | ||||
|                 <div class="alert alert-success flash_message">{{ 'absence.You are listed as absent, as of {date, date, short}'|trans({ | ||||
|                         date: user.absenceStart | ||||
|                     }) }} | ||||
|                     {% if user.absenceEnd is not null %} | ||||
|                    {{ 'until %date%'|trans({'%date%': user.absenceEnd|format_date('short') }) }} | ||||
|                 {% endif %} | ||||
|                 </div> | ||||
|             {% else %} | ||||
|                 <div class="alert alert-warning flash_message">{{ 'absence.No absence listed'|trans }}</div> | ||||
|             {% endif %} | ||||
|         </div> | ||||
|         <div> | ||||
|             {{ form_start(form) }} | ||||
|             {{ form_row(form.absenceStart) }} | ||||
|             {{ form_row(form.absenceEnd) }} | ||||
|  | ||||
|         {% if user.absenceStart is not null %} | ||||
|             <div> | ||||
|                 <p>{{ 'absence.You are listed as absent, as of'|trans }} {{ user.absenceStart|format_date('long') }}</p> | ||||
|                 <ul class="record_actions sticky-form-buttons"> | ||||
|                     <li> | ||||
|                         <a href="{{ path('chill_main_user_absence_unset') }}" | ||||
|                            class="btn btn-delete">{{ 'absence.Unset absence'|trans }}</a> | ||||
|                     </li> | ||||
|                 </ul> | ||||
|             </div> | ||||
|         {% else %} | ||||
|             <div> | ||||
|                 <p class="chill-no-data-statement">{{ 'absence.No absence listed'|trans }}</p> | ||||
|             </div> | ||||
|             <div> | ||||
|                 {{ form_start(form) }} | ||||
|                 {{ form_row(form.absenceStart) }} | ||||
|  | ||||
|                 <ul class="record_actions sticky-form-buttons"> | ||||
|                     <li> | ||||
|                         <button class="btn btn-save" type="submit"> | ||||
|                             {{ 'Save'|trans }} | ||||
|                         </button> | ||||
|                     </li> | ||||
|                 </ul> | ||||
|  | ||||
|                 {{ form_end(form) }} | ||||
|             </div> | ||||
|         {% endif %} | ||||
|             <ul class="record_actions sticky-form-buttons"> | ||||
|                 <li> | ||||
|                     <a class="btn btn-delete" title="Modifier" href="{{ path('chill_main_user_absence_unset') }}">{{ 'absence.Unset absence'|trans }}</a> | ||||
|                 </li> | ||||
|                 <li> | ||||
|                     <button class="btn btn-save" type="submit"> | ||||
|                         {{ 'Save'|trans }} | ||||
|                     </button> | ||||
|                 </li> | ||||
|             </ul> | ||||
|             {{ form_end(form) }} | ||||
|         </div> | ||||
|     </div> | ||||
|  | ||||
| {% endblock %} | ||||
|   | ||||
| @@ -5,7 +5,7 @@ | ||||
|         role="button" | ||||
|         data-bs-toggle="dropdown" | ||||
|         aria-expanded="false"> | ||||
|         <i class="fa fa-flash"></i> | ||||
|         <i class="bi bi-lightning-fill"></i> | ||||
|     </a> | ||||
|     <div class="dropdown-menu"> | ||||
|         {% for menu in menus %} | ||||
|   | ||||
| @@ -21,8 +21,6 @@ | ||||
|     {{ form_row(form.title, { 'label': 'notification.subject'|trans }) }} | ||||
|     {{ form_row(form.addressees, { 'label': 'notification.sent_to'|trans }) }} | ||||
|  | ||||
|     {{ form_row(form.addressesEmails) }} | ||||
|  | ||||
|     {% include handler.template(notification) with handler.templateData(notification) %} | ||||
|  | ||||
|     <div class="mb-3 row"> | ||||
|   | ||||
| @@ -64,7 +64,7 @@ | ||||
|                             {{ form_widget(flag.immediate_email, {'label_attr': { 'class': 'checkbox-inline checkbox-switch'}}) }} | ||||
|                         </td> | ||||
|                         <td class="text-center"> | ||||
|                             {{ form_widget(flag.daily_email, {'label_attr': { 'class': 'checkbox-inline checkbox-switch'}}) }} | ||||
|                             {{ form_widget(flag.daily_digest, {'label_attr': { 'class': 'checkbox-inline checkbox-switch'}}) }} | ||||
|                         </td> | ||||
|                     </tr> | ||||
|                 {% endfor %} | ||||
|   | ||||
| @@ -58,12 +58,14 @@ | ||||
|         {% endif %} | ||||
|     </section> | ||||
|  | ||||
|     {% if signatures|length > 0 %} | ||||
|     <section class="step my-4">{% include '@ChillMain/Workflow/_signature.html.twig' %}</section> | ||||
|     {% endif %} | ||||
|  | ||||
|     <section class="step my-4">{% include '@ChillMain/Workflow/_attachment.html.twig' %}</section> | ||||
|  | ||||
|     <section class="step my-4">{% include '@ChillMain/Workflow/_follow.html.twig' %}</section> | ||||
|     {% if signatures|length > 0 %} | ||||
|         <section class="step my-4">{% include '@ChillMain/Workflow/_signature.html.twig' %}</section> | ||||
|     {% elseif entity_workflow.currentStep.sends|length > 0 %} | ||||
|     {% if entity_workflow.currentStep.sends|length > 0 %} | ||||
|         <section class="step my-4"> | ||||
|             <h2>{{ 'workflow.external_views.title'|trans({'numberOfSends': entity_workflow.currentStep.sends|length }) }}</h2> | ||||
|             {% include '@ChillMain/Workflow/_send_views_list.html.twig' with {'sends': entity_workflow.currentStep.sends} %} | ||||
|   | ||||
| @@ -0,0 +1,18 @@ | ||||
| {% extends '@ChillMain/layout.html.twig' %} | ||||
|  | ||||
| {% block title %}{{ 'workflow.signature.waiting_for'|trans }}{% endblock %} | ||||
|  | ||||
| {% block css %} | ||||
|     {{ encore_entry_link_tags('page_workflow_waiting_post_process') }} | ||||
| {% endblock %} | ||||
|  | ||||
| {% block js %} | ||||
|     {{ encore_entry_script_tags('page_workflow_waiting_post_process') }} | ||||
| {% endblock %} | ||||
|  | ||||
| {% block content %} | ||||
|     <h1>{{ block('title') }}</h1> | ||||
|  | ||||
|     <div class="screen-wait" data-workflow-id="{{ workflow.id|e('html_attr') }}" data-expected-step="{{ expectedStep|e('html_attr') }}"></div> | ||||
|  | ||||
| {% endblock %} | ||||
| @@ -26,6 +26,10 @@ | ||||
| </head> | ||||
|  | ||||
| <body> | ||||
|     {% if chill_main_config.top_banner is defined and chill_main_config.top_banner.visible is true %} | ||||
|         {{ include('@ChillMain/Layout/_top_banner.html.twig') }} | ||||
|     {% endif %} | ||||
|  | ||||
|     {% if responsive_debug is defined and responsive_debug == 1 %} | ||||
|         {{ include('@ChillMain/Layout/_debug.html.twig') }} | ||||
|     {% endif %} | ||||
| @@ -79,7 +83,7 @@ | ||||
|                             <div class="d-flex flex-row mb-5 alert alert-warning" role="alert"> | ||||
|                                 <p class="m-2">{{'absence.You are marked as being absent'|trans }}</p> | ||||
|                                 <span class="ms-auto"> | ||||
|                                     <a class="btn btn-remove" title="Modifier" href="{{ path('chill_main_user_absence_index') }}">{{ 'absence.Unset absence'|trans }}</a> | ||||
|                                     <a class="btn btn-delete" title="Modifier" href="{{ path('chill_main_user_absence_unset') }}">{{ 'absence.Unset absence'|trans }}</a> | ||||
|                                 </span> | ||||
|                             </div> | ||||
|                         {% endif %} | ||||
|   | ||||
| @@ -0,0 +1,53 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\MainBundle\Security\Authorization; | ||||
|  | ||||
| use Chill\MainBundle\Entity\Workflow\EntityWorkflow; | ||||
| use Symfony\Component\Security\Core\Authentication\Token\TokenInterface; | ||||
| use Symfony\Component\Security\Core\Authorization\Voter\Voter; | ||||
| use Symfony\Component\Workflow\Registry; | ||||
|  | ||||
| final class EntityWorkflowAttachmentVoter extends Voter | ||||
| { | ||||
|     public function __construct( | ||||
|         private readonly Registry $registry, | ||||
|     ) {} | ||||
|     public const EDIT = 'CHILL_MAIN_WORKFLOW_ATTACHMENT_EDIT'; | ||||
|  | ||||
|     protected function supports(string $attribute, $subject): bool | ||||
|     { | ||||
|         return $subject instanceof EntityWorkflow && self::EDIT === $attribute; | ||||
|     } | ||||
|  | ||||
|     protected function voteOnAttribute(string $attribute, $subject, TokenInterface $token): bool | ||||
|     { | ||||
|         if (!$subject instanceof EntityWorkflow) { | ||||
|             throw new \UnexpectedValueException('Subject must be an instance of EntityWorkflow'); | ||||
|         } | ||||
|  | ||||
|         if ($subject->isFinal()) { | ||||
|             return false; | ||||
|         } | ||||
|  | ||||
|         $workflow = $this->registry->get($subject, $subject->getWorkflowName()); | ||||
|  | ||||
|         $marking = $workflow->getMarking($subject); | ||||
|         foreach ($marking->getPlaces() as $place => $int) { | ||||
|             $placeMetadata = $workflow->getMetadataStore()->getPlaceMetadata($place); | ||||
|             if ($placeMetadata['isSentExternal'] ?? false) { | ||||
|                 return false; | ||||
|             } | ||||
|         } | ||||
|  | ||||
|         return true; | ||||
|     } | ||||
| } | ||||
							
								
								
									
										86
									
								
								src/Bundle/ChillMainBundle/Security/RoleDumper.php
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										86
									
								
								src/Bundle/ChillMainBundle/Security/RoleDumper.php
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,86 @@ | ||||
| <?php | ||||
|  | ||||
| declare(strict_types=1); | ||||
|  | ||||
| /* | ||||
|  * Chill is a software for social workers | ||||
|  * | ||||
|  * For the full copyright and license information, please view | ||||
|  * the LICENSE file that was distributed with this source code. | ||||
|  */ | ||||
|  | ||||
| namespace Chill\MainBundle\Security; | ||||
|  | ||||
| use Symfony\Component\Security\Core\Role\RoleHierarchyInterface; | ||||
| use Symfony\Contracts\Translation\TranslatorInterface; | ||||
|  | ||||
| final readonly class RoleDumper | ||||
| { | ||||
|     public function __construct( | ||||
|         private RoleProvider $roleProvider, | ||||
|         private RoleHierarchyInterface $roleHierarchy, | ||||
|         private TranslatorInterface $translator, | ||||
|     ) {} | ||||
|  | ||||
|     public function dumpAsMarkdown(): string | ||||
|     { | ||||
|         $roles = $this->roleProvider->getRoles(); | ||||
|         $rolesWithoutScopes = $this->roleProvider->getRolesWithoutScopes(); | ||||
|  | ||||
|         // Group roles by title | ||||
|         $groups = []; | ||||
|         foreach ($roles as $role) { | ||||
|             $title = $this->roleProvider->getRoleTitle($role); | ||||
|             $title ??= 'Other'; | ||||
|             $groups[$title][] = $role; | ||||
|         } | ||||
|  | ||||
|         // Sort groups by title | ||||
|         ksort($groups, SORT_NATURAL | SORT_FLAG_CASE); | ||||
|  | ||||
|         $lines = []; | ||||
|         foreach ($groups as $title => $roleList) { | ||||
|             // Sort roles by translated label for deterministic output | ||||
|             usort($roleList, function (string $a, string $b): int { | ||||
|                 $ta = $this->translator->trans($a); | ||||
|                 $tb = $this->translator->trans($b); | ||||
|  | ||||
|                 return strcasecmp($ta, $tb); | ||||
|             }); | ||||
|  | ||||
|             $translatedTitle = $this->translator->trans($title); | ||||
|             $lines[] = '## '.$translatedTitle; | ||||
|  | ||||
|             foreach ($roleList as $role) { | ||||
|                 // Translate primary role | ||||
|                 $translatedRole = $this->translator->trans($role); | ||||
|  | ||||
|                 // Scope marker: (S) if needs scope, (~~S~~) if no scope required | ||||
|                 $needsScope = !in_array($role, $rolesWithoutScopes, true); | ||||
|                 $scopeMarker = $needsScope ? '(S)' : '(~~S~~)'; | ||||
|  | ||||
|                 // Compute dependent roles from hierarchy (exclude itself) | ||||
|                 $reachable = $this->roleHierarchy->getReachableRoleNames([$role]); | ||||
|                 $dependents = array_values(array_filter($reachable, static fn (string $r): bool => $r !== $role)); | ||||
|  | ||||
|                 // Translate dependents and sort deterministically | ||||
|                 $translatedDependents = array_map(fn (string $r) => $this->translator->trans($r), $dependents); | ||||
|                 sort($translatedDependents, SORT_NATURAL | SORT_FLAG_CASE); | ||||
|  | ||||
|                 if (count($translatedDependents) > 0) { | ||||
|                     $lines[] = sprintf('- **%s** %s: %s', $translatedRole, $scopeMarker, implode(', ', $translatedDependents)); | ||||
|                 } else { | ||||
|                     $lines[] = sprintf('- **%s** %s', $translatedRole, $scopeMarker); | ||||
|                 } | ||||
|             } | ||||
|  | ||||
|             // Add a blank line between groups | ||||
|             $lines[] = ''; | ||||
|         } | ||||
|  | ||||
|         // Trim possible trailing blank line | ||||
|         $markdown = rtrim(implode("\n", $lines)); | ||||
|  | ||||
|         return $markdown."\n"; // End with newline for POSIX friendliness | ||||
|     } | ||||
| } | ||||
| @@ -52,12 +52,8 @@ class RoleProvider | ||||
|  | ||||
|     /** | ||||
|      * Get the title for each role. | ||||
|      * | ||||
|      * @param string $role | ||||
|      * | ||||
|      * @return string the title of the role | ||||
|      */ | ||||
|     public function getRoleTitle($role) | ||||
|     public function getRoleTitle(string $role): ?string | ||||
|     { | ||||
|         $this->initializeRolesTitlesCache(); | ||||
|  | ||||
| @@ -73,7 +69,7 @@ class RoleProvider | ||||
|     /** | ||||
|      * initialize the array for caching role and titles. | ||||
|      */ | ||||
|     private function initializeRolesTitlesCache() | ||||
|     private function initializeRolesTitlesCache(): void | ||||
|     { | ||||
|         // break if already initialized | ||||
|         if (null !== $this->rolesTitlesCache) { | ||||
|   | ||||
| @@ -12,18 +12,25 @@ declare(strict_types=1); | ||||
| namespace Chill\MainBundle\Serializer\Normalizer; | ||||
|  | ||||
| use Chill\MainBundle\Entity\Workflow\EntityWorkflow; | ||||
| use Chill\MainBundle\Security\Authorization\EntityWorkflowAttachmentVoter; | ||||
| use Chill\MainBundle\Workflow\EntityWorkflowManager; | ||||
| use Chill\MainBundle\Workflow\Helper\MetadataExtractor; | ||||
| use Symfony\Component\Security\Core\Security; | ||||
| use Symfony\Component\Serializer\Normalizer\NormalizerAwareInterface; | ||||
| use Symfony\Component\Serializer\Normalizer\NormalizerAwareTrait; | ||||
| use Symfony\Component\Serializer\Normalizer\NormalizerInterface; | ||||
| use Symfony\Component\Workflow\Registry; | ||||
|  | ||||
| class EntityWorkflowNormalizer implements NormalizerInterface, NormalizerAwareInterface | ||||
| final class EntityWorkflowNormalizer implements NormalizerInterface, NormalizerAwareInterface | ||||
| { | ||||
|     use NormalizerAwareTrait; | ||||
|  | ||||
|     public function __construct(private readonly EntityWorkflowManager $entityWorkflowManager, private readonly MetadataExtractor $metadataExtractor, private readonly Registry $registry) {} | ||||
|     public function __construct( | ||||
|         private readonly EntityWorkflowManager $entityWorkflowManager, | ||||
|         private readonly MetadataExtractor $metadataExtractor, | ||||
|         private readonly Registry $registry, | ||||
|         private readonly Security $security, | ||||
|     ) {} | ||||
|  | ||||
|     /** | ||||
|      * @param EntityWorkflow $object | ||||
| @@ -46,6 +53,9 @@ class EntityWorkflowNormalizer implements NormalizerInterface, NormalizerAwareIn | ||||
|             'datas' => $this->normalizer->normalize($handler->getEntityData($object), $format, $context), | ||||
|             'title' => $handler->getEntityTitle($object), | ||||
|             'isOnHoldAtCurrentStep' => $object->isOnHoldAtCurrentStep(), | ||||
|             '_permissions' => [ | ||||
|                 EntityWorkflowAttachmentVoter::EDIT => $this->security->isGranted(EntityWorkflowAttachmentVoter::EDIT, $object), | ||||
|             ], | ||||
|         ]; | ||||
|     } | ||||
|  | ||||
|   | ||||
| @@ -39,6 +39,8 @@ class UserNormalizer implements ContextAwareNormalizerInterface, NormalizerAware | ||||
|         'label' => '', | ||||
|         'email' => '', | ||||
|         'isAbsent' => false, | ||||
|         'absenceStart' => null, | ||||
|         'absenceEnd' => null, | ||||
|     ]; | ||||
|  | ||||
|     public function __construct(private readonly UserRender $userRender, private readonly ClockInterface $clock) {} | ||||
| @@ -77,6 +79,11 @@ class UserNormalizer implements ContextAwareNormalizerInterface, NormalizerAware | ||||
|             ['docgen:expects' => PhoneNumber::class, 'groups' => 'docgen:read'] | ||||
|         ); | ||||
|  | ||||
|         $absenceDatesContext = array_merge( | ||||
|             $context, | ||||
|             ['docgen:expects' => \DateTimeImmutable::class, 'groups' => 'docgen:read'] | ||||
|         ); | ||||
|  | ||||
|         if (null === $object && 'docgen' === $format) { | ||||
|             return [...self::NULL_USER, 'phonenumber' => $this->normalizer->normalize(null, $format, $phonenumberContext), 'civility' => $this->normalizer->normalize(null, $format, $civilityContext), 'user_job' => $this->normalizer->normalize(null, $format, $userJobContext), 'main_center' => $this->normalizer->normalize(null, $format, $centerContext), 'main_scope' => $this->normalizer->normalize(null, $format, $scopeContext), 'current_location' => $this->normalizer->normalize(null, $format, $locationContext), 'main_location' => $this->normalizer->normalize(null, $format, $locationContext)]; | ||||
|         } | ||||
| @@ -99,6 +106,8 @@ class UserNormalizer implements ContextAwareNormalizerInterface, NormalizerAware | ||||
|             'main_center' => $this->normalizer->normalize($object->getMainCenter(), $format, $centerContext), | ||||
|             'main_scope' => $this->normalizer->normalize($object->getMainScope($at), $format, $scopeContext), | ||||
|             'isAbsent' => $object->isAbsent(), | ||||
|             'absenceStart' => $this->normalizer->normalize($object->getAbsenceStart(), $format, $absenceDatesContext), | ||||
|             'absenceEnd' => $this->normalizer->normalize($object->getAbsenceEnd(), $format, $absenceDatesContext), | ||||
|         ]; | ||||
|  | ||||
|         if ('docgen' === $format) { | ||||
|   | ||||
| @@ -64,7 +64,7 @@ class AddressReferenceBEFromBestAddress | ||||
|  | ||||
|         $uncompressedStream = gzopen($tmpname, 'r'); | ||||
|  | ||||
|         $csv = Reader::createFromStream($uncompressedStream); | ||||
|         $csv = Reader::from($uncompressedStream); | ||||
|         $csv->setDelimiter(','); | ||||
|         $csv->setHeaderOffset(0); | ||||
|  | ||||
|   | ||||
| @@ -287,7 +287,7 @@ final class AddressReferenceBaseImporter | ||||
|  | ||||
|             $filename = sprintf('%s-%s.csv', (new \DateTimeImmutable())->format('Ymd-His'), uniqid()); | ||||
|             $path = Path::normalize(sprintf('%s%s%s', sys_get_temp_dir(), DIRECTORY_SEPARATOR, $filename)); | ||||
|             $writer = Writer::createFromPath($path, 'w+'); | ||||
|             $writer = Writer::from($path, 'w+'); | ||||
|             // insert headers | ||||
|             $writer->insertOne([ | ||||
|                 'postalcode', | ||||
|   | ||||
| @@ -53,7 +53,7 @@ class AddressReferenceFromBAN | ||||
|         // re-open it to read it | ||||
|         $csvDecompressed = gzopen($path, 'r'); | ||||
|  | ||||
|         $csv = Reader::createFromStream($csvDecompressed); | ||||
|         $csv = Reader::from($csvDecompressed); | ||||
|         $csv->setDelimiter(';')->setHeaderOffset(0); | ||||
|         $stmt = new Statement(); | ||||
|         $stmt = $stmt->process($csv, [ | ||||
|   | ||||
| @@ -41,7 +41,7 @@ class AddressReferenceFromBano | ||||
|  | ||||
|         fseek($file, 0); | ||||
|  | ||||
|         $csv = Reader::createFromStream($file); | ||||
|         $csv = Reader::from($file); | ||||
|         $csv->setDelimiter(','); | ||||
|         $stmt = new Statement(); | ||||
|         $stmt = $stmt->process($csv, [ | ||||
|   | ||||
Some files were not shown because too many files have changed in this diff Show More
		Reference in New Issue
	
	Block a user