Small fixes / changes for TilburgCases sync #730
WilcoLouwerseAnnotations
1 error and 11 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/ActionHandler/SyncNotubizHandler.php#L47
The method getConfiguration() has 103 lines of code. Current threshold is set to 100. Avoid really long methods.
|
|
build:
src/ActionHandler/SyncXxllncCasesHandler.php#L47
The method getConfiguration() has 100 lines of code. Current threshold is set to 100. Avoid really long methods.
|
|
build:
src/ActionHandler/WooNotificationHandler.php#L47
The method getConfiguration() has 103 lines of code. Current threshold is set to 100. Avoid really long methods.
|
|
build:
src/Command/SyncWooCommand.php#L31
The class SyncWooCommand has a coupling between objects value of 14. Consider to reduce the number of dependencies under 13.
|
|
build:
src/Command/SyncWooCommand.php#L46
Avoid excessively long variable names like $syncXxllncCasesService. Keep variable name length under 20.
|
|
build:
src/Command/SyncWooCommand.php#L83
Avoid excessively long variable names like $syncXxllncCasesService. Keep variable name length under 20.
|
|
build:
src/Command/SyncWooCommand.php#L126
The method execute() has a Cyclomatic Complexity of 13. The configured cyclomatic complexity threshold is 10.
|
|
build:
src/Command/SyncWooCommand.php#L126
Avoid assigning values to variables in if clauses and the like (line '131', column '14').
|
|
build:
src/Command/SyncWooCommand.php#L146
Avoid using static access to class '\Ramsey\Uuid\Uuid' in method 'execute'.
|
|
build:
src/Command/SyncWooCommand.php#L179
The method execute uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
|