The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getWorkgroup()' is missing an @Override annotation. | 107–110 |
| Rule | Violation | Line |
|---|---|---|
| DefaultPackage | Use explicit scoping instead of the default package private level | 62–65 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getInstance method if you want a default access modifier | 62–65 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the selectPageListbyName method if you want a default access modifier | 75–105 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 75–105 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 70–244 |
| ImmutableField | Private field '_pageService' could be made final; it is only initialized in the declaration or constructor. | 106 |
| MissingOverride | The method 'getInsertServiceSelectorUI(HttpServletRequest)' is missing an @Override annotation. | 117–158 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 129 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 202 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 203 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 204 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 205 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 205 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 207 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 236 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'getWorkgroup()' is missing an @Override annotation. | 3 | 107–110 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 62–65 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getInstance method if you want a default access modifier | 3 | 62–65 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the selectPageListbyName method if you want a default access modifier | 3 | 75–105 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 75–105 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 70–244 |
| ImmutableField | Private field '_pageService' could be made final; it is only initialized in the declaration or constructor. | 3 | 106 |
| MissingOverride | The method 'getInsertServiceSelectorUI(HttpServletRequest)' is missing an @Override annotation. | 3 | 117–158 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 129 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 202 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 203 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 204 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 205 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 205 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 207 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 236 |
