Merge pull request #18 from CommonGateway/feature/OP-163/publiccode #232
WilcoLouwerseAnnotations
1 error and 10 warnings
|
build
Process completed with exit code 2.
|
|
build:
src/Service/SimTaxService.php#L27
The class SimTaxService has an overall complexity of 95 which is very high. The configured complexity threshold is 50.
|
|
build:
src/Service/SimTaxService.php#L27
The class SimTaxService has a coupling between objects value of 14. Consider to reduce the number of dependencies under 13.
|
|
build:
src/Service/SimTaxService.php#L62
Avoid excessively long variable names like $synchronizationService. Keep variable name length under 20.
|
|
build:
src/Service/SimTaxService.php#L124
Avoid excessively long variable names like $synchronizationService. Keep variable name length under 20.
|
|
build:
src/Service/SimTaxService.php#L264
The method getAanslagenFilter() has a Cyclomatic Complexity of 12. The configured cyclomatic complexity threshold is 10.
|
|
build:
src/Service/SimTaxService.php#L264
The method getAanslagenFilter() has an NPath complexity of 232. The configured NPath complexity threshold is 200.
|
|
build:
src/Service/SimTaxService.php#L294
The method getAanslagenFilter uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
|
|
build:
src/Service/SimTaxService.php#L303
Avoid using static access to class '\DateInterval' in method 'getAanslagenFilter'.
|
|
build:
src/Service/SimTaxService.php#L443
The method mapXMLToBezwaar() has a Cyclomatic Complexity of 12. The configured cyclomatic complexity threshold is 10.
|
|
build:
src/Service/SimTaxService.php#L443
The method mapXMLToBezwaar() has an NPath complexity of 450. The configured NPath complexity threshold is 200.
|