Skip to content

Merge pull request #113 from CommonGateway/feature/GW-1596/Symfony-up… #80

Merge pull request #113 from CommonGateway/feature/GW-1596/Symfony-up…

Merge pull request #113 from CommonGateway/feature/GW-1596/Symfony-up… #80

Triggered via push March 15, 2024 10:33
Status Failure
Total duration 46s
Artifacts

phpmd.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

1 error and 11 warnings
build
Process completed with exit code 2.
build
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
build: src/Command/UpdateChecklistCommand.php#L57
Avoid unused parameters such as '$input'.
build: src/Command/UpdateChecklistCommand.php#L57
Avoid unused parameters such as '$output'.
build: src/DependencyInjection/HuwelijksplannerExtension.php#L14
Avoid unused parameters such as '$configs'.
build: src/Service/AssentService.php#L83
The method createPerson() has a Cyclomatic Complexity of 38. The configured cyclomatic complexity threshold is 10.
build: src/Service/AssentService.php#L83
The method createPerson() has an NPath complexity of 1572864. The configured NPath complexity threshold is 200.
build: src/Service/AssentService.php#L83
Avoid unused parameters such as '$huwelijk'.
build: src/Service/AssentService.php#L94
Avoid excessively long variable names like $landVanwaarIngeschreven. Keep variable name length under 20.
build: src/Service/AssentService.php#L155
Avoid unused parameters such as '$config'.
build: src/Service/AssentService.php#L186
The method updateAssentHandler uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
build: src/Service/CreateAvailabilityService.php#L96
The method createAvailabilityHandler uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.