The following document contains the results of PMD 6.46.0.
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 21–1137 |
| AvoidUsingHardCodedIP | Do not hard code the IP address | 697 |
| AvoidUsingHardCodedIP | Do not hard code the IP address | 708 |
| AvoidUsingHardCodedIP | Do not hard code the IP address | 718 |
| UnusedPrivateField | Avoid unused private fields such as 'help'. | 1132 |
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 21–112 |
| Rule | Violation | Line |
|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 50–52 |
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'handler'. | 54 |
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 21–177 |
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 268–270 |
| UnusedFormalParameter | Avoid unused method parameters such as 'confs'. | 277 |
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 21–479 |
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 187–189 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 60 |
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 69–71 |
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'identityManager'. | 58 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PipelinedHandler.pipe' due to existing static import 'org.restheart.handlers.PipelinedHandler.pipe' | 1100 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Policy.ON_REQUIRES_CONTENT_AFTER_AUTH' due to existing static import 'org.restheart.handlers.injectors.RequestContentInjector.Policy.ON_REQUIRES_CONTENT_AFTER_AUTH' | 110 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Policy.ON_REQUIRES_CONTENT_AFTER_AUTH' due to existing static import 'org.restheart.handlers.injectors.RequestContentInjector.Policy.ON_REQUIRES_CONTENT_AFTER_AUTH' | 119 |
| UselessParentheses | Useless parentheses. | 128 |
| UselessParentheses | Useless parentheses. | 129 |
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 360 |
| UselessParentheses | Useless parentheses. | 432 |
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 297–299 |
| UselessParentheses | Useless parentheses. | 300 |
| UselessParentheses | Useless parentheses. | 301–303 |
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 206 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 21–1137 |
| AvoidUsingHardCodedIP | Do not hard code the IP address | 3 | 697 |
| AvoidUsingHardCodedIP | Do not hard code the IP address | 3 | 708 |
| AvoidUsingHardCodedIP | Do not hard code the IP address | 3 | 718 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PipelinedHandler.pipe' due to existing static import 'org.restheart.handlers.PipelinedHandler.pipe' | 4 | 1100 |
| UnusedPrivateField | Avoid unused private fields such as 'help'. | 3 | 1132 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 21–112 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 3 | 50–52 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'handler'. | 3 | 54 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 21–177 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Policy.ON_REQUIRES_CONTENT_AFTER_AUTH' due to existing static import 'org.restheart.handlers.injectors.RequestContentInjector.Policy.ON_REQUIRES_CONTENT_AFTER_AUTH' | 4 | 110 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Policy.ON_REQUIRES_CONTENT_AFTER_AUTH' due to existing static import 'org.restheart.handlers.injectors.RequestContentInjector.Policy.ON_REQUIRES_CONTENT_AFTER_AUTH' | 4 | 119 |
| UselessParentheses | Useless parentheses. | 4 | 128 |
| UselessParentheses | Useless parentheses. | 4 | 129 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 268–270 |
| UnusedFormalParameter | Avoid unused method parameters such as 'confs'. | 3 | 277 |
| UselessParentheses | Useless parentheses. | 4 | 360 |
| UselessParentheses | Useless parentheses. | 4 | 432 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 21–479 |
| UselessParentheses | Useless parentheses. | 4 | 297–299 |
| UselessParentheses | Useless parentheses. | 4 | 300 |
| UselessParentheses | Useless parentheses. | 4 | 301–303 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 187–189 |
| UselessParentheses | Useless parentheses. | 4 | 206 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 60 |
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 69–71 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'identityManager'. | 3 | 58 |