Skip to content

Action and code for syncing tasks to zaaksysteem #5

Action and code for syncing tasks to zaaksysteem

Action and code for syncing tasks to zaaksysteem #5

Triggered via push October 4, 2024 14:33
Status Failure
Total duration 17s
Artifacts

phpmd.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

1 error and 7 warnings
build
Process completed with exit code 2.
build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
build: src/Service/KTBToXxllncService.php#L43
Avoid excessively long variable names like $oldSynchronizationService. Keep variable name length under 20.
build: src/Service/KTBToXxllncService.php#L44
Avoid excessively long variable names like $synchronizationService. Keep variable name length under 20.
build: src/Service/KTBToXxllncService.php#L71
Avoid using static access to class '\Ramsay\Uuid' in method 'getZaakSourceId'.
build: src/Service/KTBToXxllncService.php#L99
The method synchronizeTaak() has a Cyclomatic Complexity of 13. The configured cyclomatic complexity threshold is 10.
build: src/Service/KTBToXxllncService.php#L126
Avoid using static access to class '\Ramsay\Uuid' in method 'synchronizeTaak'.
build: src/Service/KTBToXxllncService.php#L169
The method synchronizeTaak uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.