The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 42–81 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 42–81 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'first' doesn't match '_[a-z][a-zA-Z0-9]*' | 693 |
| FieldNamingConventions | The field name 'second' doesn't match '_[a-z][a-zA-Z0-9]*' | 694 |
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name '_listAttributesAssociation' doesn't match '[a-z][a-zA-Z0-9]*' | 466 |
| LocalVariableNamingConventions | The local variable name 'docType_code' doesn't match '[a-z][a-zA-Z0-9]*' | 485 |
| LocalVariableNamingConventions | The local variable name '_docAttr' doesn't match '[a-z][a-zA-Z0-9]*' | 486 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–54 |
| CommentRequired | Public method and constructor comments are required | 43 |
| CommentRequired | Public method and constructor comments are required | 45 |
| CommentRequired | Public method and constructor comments are required | 47 |
| CommentRequired | Public method and constructor comments are required | 49 |
| CommentRequired | Public method and constructor comments are required | 51 |
| CommentRequired | Public method and constructor comments are required | 53 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–52 |
| CommentRequired | Public method and constructor comments are required | 43 |
| CommentRequired | Public method and constructor comments are required | 45 |
| CommentRequired | Public method and constructor comments are required | 47 |
| CommentRequired | Public method and constructor comments are required | 49 |
| CommentRequired | Public method and constructor comments are required | 51 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–54 |
| CommentRequired | Public method and constructor comments are required | 43 |
| CommentRequired | Public method and constructor comments are required | 45 |
| CommentRequired | Public method and constructor comments are required | 47 |
| CommentRequired | Public method and constructor comments are required | 49 |
| CommentRequired | Public method and constructor comments are required | 51 |
| CommentRequired | Public method and constructor comments are required | 53 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 40–137 |
| CommentRequired | Public method and constructor comments are required | 47–50 |
| CommentRequired | Public method and constructor comments are required | 52–55 |
| CommentRequired | Public method and constructor comments are required | 57–60 |
| CommentRequired | Public method and constructor comments are required | 62–65 |
| CommentRequired | Public method and constructor comments are required | 67–70 |
| CommentRequired | Public method and constructor comments are required | 72–75 |
| CommentRequired | Public method and constructor comments are required | 77–80 |
| CommentRequired | Public method and constructor comments are required | 82–85 |
| CommentRequired | Public method and constructor comments are required | 87–99 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 95 |
| CommentRequired | Header comments are required | 101–136 |
| CommentRequired | Public method and constructor comments are required | 107–110 |
| CommentRequired | Public method and constructor comments are required | 112–115 |
| CommentRequired | Public method and constructor comments are required | 117–120 |
| CommentRequired | Public method and constructor comments are required | 122–125 |
| CommentRequired | Public method and constructor comments are required | 127–130 |
| CommentRequired | Public method and constructor comments are required | 132–135 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 44–229 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 57–70 |
| CommentRequired | Public method and constructor comments are required | 57–70 |
| CommentRequired | Public method and constructor comments are required | 72–98 |
| MissingOverride | The method 'insert(LibraryMapping, Plugin)' is missing an @Override annotation. | 72–98 |
| CommentRequired | Public method and constructor comments are required | 100–127 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 100–127 |
| MissingOverride | The method 'selectByMedia(int, Plugin)' is missing an @Override annotation. | 129–162 |
| CommentRequired | Public method and constructor comments are required | 129–162 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 131 |
| MissingOverride | The method 'store(LibraryMapping, Plugin)' is missing an @Override annotation. | 164–196 |
| CommentRequired | Public method and constructor comments are required | 164–196 |
| CommentRequired | Public method and constructor comments are required | 221–228 |
| MissingOverride | The method 'deleteAttributeAssociation(int, Plugin)' is missing an @Override annotation. | 221–228 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–81 |
| CommentRequired | Public method and constructor comments are required | 52–55 |
| CommentRequired | Public method and constructor comments are required | 57–60 |
| CommentRequired | Public method and constructor comments are required | 62–65 |
| CommentRequired | Public method and constructor comments are required | 67–70 |
| CommentRequired | Public method and constructor comments are required | 72–75 |
| CommentRequired | Public method and constructor comments are required | 77–80 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–115 |
| CommentRequired | Public method and constructor comments are required | 50–53 |
| CommentRequired | Public method and constructor comments are required | 55–58 |
| CommentRequired | Public method and constructor comments are required | 60–63 |
| CommentRequired | Public method and constructor comments are required | 65–68 |
| CommentRequired | Public method and constructor comments are required | 70–73 |
| CommentRequired | Public method and constructor comments are required | 75–78 |
| CommentRequired | Public method and constructor comments are required | 80–83 |
| CommentRequired | Public method and constructor comments are required | 85–88 |
| CommentRequired | Public method and constructor comments are required | 100–104 |
| CommentRequired | Public method and constructor comments are required | 106–109 |
| CommentRequired | Public method and constructor comments are required | 111–114 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 43–153 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 52–58 |
| CommentRequired | Public method and constructor comments are required | 52–58 |
| CommentRequired | Public method and constructor comments are required | 60–73 |
| MissingOverride | The method 'insert(LibraryMedia, Plugin)' is missing an @Override annotation. | 60–73 |
| CommentRequired | Public method and constructor comments are required | 75–95 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 75–95 |
| MissingOverride | The method 'selectAll(Plugin)' is missing an @Override annotation. | 97–116 |
| CommentRequired | Public method and constructor comments are required | 97–116 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 99 |
| CommentRequired | Public method and constructor comments are required | 118–128 |
| MissingOverride | The method 'store(LibraryMedia, Plugin)' is missing an @Override annotation. | 118–128 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–76 |
| CommentRequired | Public method and constructor comments are required | 52–55 |
| CommentRequired | Public method and constructor comments are required | 57–60 |
| CommentRequired | Public method and constructor comments are required | 62–65 |
| CommentRequired | Public method and constructor comments are required | 67–70 |
| CommentRequired | Public method and constructor comments are required | 72–75 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–177 |
| CommentRequired | Public method and constructor comments are required | 62–65 |
| CommentRequired | Public method and constructor comments are required | 67–70 |
| CommentRequired | Public method and constructor comments are required | 72–75 |
| CommentRequired | Public method and constructor comments are required | 77–80 |
| CommentRequired | Public method and constructor comments are required | 82–85 |
| CommentRequired | Public method and constructor comments are required | 87–90 |
| CommentRequired | Public method and constructor comments are required | 92–95 |
| CommentRequired | Public method and constructor comments are required | 97–100 |
| CommentRequired | Public method and constructor comments are required | 102–106 |
| CommentRequired | Public method and constructor comments are required | 108–111 |
| CommentRequired | Public method and constructor comments are required | 113–116 |
| CommentRequired | Public method and constructor comments are required | 118–151 |
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 153–156 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 43–167 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 53–59 |
| CommentRequired | Public method and constructor comments are required | 53–59 |
| CommentRequired | Public method and constructor comments are required | 61–73 |
| MissingOverride | The method 'insert(MediaAttribute, Plugin)' is missing an @Override annotation. | 61–73 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 75–96 |
| CommentRequired | Public method and constructor comments are required | 75–96 |
| CommentRequired | Public method and constructor comments are required | 98–120 |
| MissingOverride | The method 'selectAllAttributesForMedia(int, Plugin)' is missing an @Override annotation. | 98–120 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 100 |
| CommentRequired | Public method and constructor comments are required | 122–135 |
| MissingOverride | The method 'store(MediaAttribute, Plugin)' is missing an @Override annotation. | 122–135 |
| CommentRequired | Public method and constructor comments are required | 160–166 |
| MissingOverride | The method 'deleteAllForMedia(int, Plugin)' is missing an @Override annotation. | 160–166 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–81 |
| CommentRequired | Public method and constructor comments are required | 52–55 |
| CommentRequired | Public method and constructor comments are required | 57–60 |
| CommentRequired | Public method and constructor comments are required | 62–65 |
| CommentRequired | Public method and constructor comments are required | 67–70 |
| CommentRequired | Public method and constructor comments are required | 72–75 |
| CommentRequired | Public method and constructor comments are required | 77–80 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–184 |
| ShortVariable | Avoid variables with short names like o | 175 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–712 |
| UnusedPrivateField | Avoid unused private fields such as 'PARAMETER_PAGE_INDEX'. | 104 |
| UnusedPrivateField | Avoid unused private fields such as 'DEFAULT_RESULTS_PER_PAGE'. | 136 |
| SingularField | Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. | 150 |
| CommentRequired | Public method and constructor comments are required | 152–164 |
| MissingOverride | The method 'getInsertServiceSelectorUI(HttpServletRequest)' is missing an @Override annotation. | 152–164 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 155 |
| CyclomaticComplexity | The method 'getSelectMedia(HttpServletRequest)' has a cyclomatic complexity of 11. | 166–254 |
| CommentRequired | Public method and constructor comments are required | 166–254 |
| NPathComplexity | The method 'getSelectMedia(HttpServletRequest)' has an NPath complexity of 256 | 166–254 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 197 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 198 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 199 |
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 249 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 249 |
| CommentRequired | Public method and constructor comments are required | 256–296 |
| CyclomaticComplexity | The method 'doSelectMedia(HttpServletRequest)' has a cyclomatic complexity of 10. | 298–368 |
| CommentRequired | Public method and constructor comments are required | 298–368 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 304 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 319 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 331 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 344 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 355 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 357 |
| CommentRequired | Public method and constructor comments are required | 370–425 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 383 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 385 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'document' | 399 |
| CommentRequired | Public method and constructor comments are required | 427–488 |
| CommentRequired | Public method and constructor comments are required | 490–570 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 557 |
| UseProperClassLoader | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | 560 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 567 |
| ShortVariable | Avoid variables with short names like m | 572 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nSpaceId'. | 595 |
| ShortVariable | Avoid variables with short names like m | 595 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 599 |
| ShortVariable | Avoid variables with short names like m | 614 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 616 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 645 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 647 |
| IdempotentOperations | Avoid idempotent operations (like assigning a variable to itself). | 664 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 686–687 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'paginator' | 688 |
| CommentRequired | Header comments are required | 691–711 |
| ImmutableField | Private field 'first' could be made final; it is only initialized in the declaration or constructor. | 693 |
| ImmutableField | Private field 'second' could be made final; it is only initialized in the declaration or constructor. | 694 |
| CommentRequired | Public method and constructor comments are required | 696–700 |
| ShortVariable | Avoid variables with short names like x | 696 |
| ShortVariable | Avoid variables with short names like y | 696 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–724 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 146 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 146 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 258 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 258 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 278 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 315 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 315 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 335 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 372 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 394 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 524 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 524 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 534 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 534 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 563 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 563 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 636 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 636 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–565 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 181 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 183 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 305 |
| AddEmptyString | Do not add empty strings | 306 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 343 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 408–415 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 428–435 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 477–490 |
| ControlStatementBraces | This statement should have braces | 487 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 514 |
| UseProperClassLoader | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | 517 |
| ShortVariable | Avoid variables with short names like m | 553 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 555 |
| Rule | Violation | Line |
|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 106–109 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like Pair | 691–711 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–54 |
| CommentRequired | Public method and constructor comments are required | 3 | 43 |
| CommentRequired | Public method and constructor comments are required | 3 | 45 |
| CommentRequired | Public method and constructor comments are required | 3 | 47 |
| CommentRequired | Public method and constructor comments are required | 3 | 49 |
| CommentRequired | Public method and constructor comments are required | 3 | 51 |
| CommentRequired | Public method and constructor comments are required | 3 | 53 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–52 |
| CommentRequired | Public method and constructor comments are required | 3 | 43 |
| CommentRequired | Public method and constructor comments are required | 3 | 45 |
| CommentRequired | Public method and constructor comments are required | 3 | 47 |
| CommentRequired | Public method and constructor comments are required | 3 | 49 |
| CommentRequired | Public method and constructor comments are required | 3 | 51 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–54 |
| CommentRequired | Public method and constructor comments are required | 3 | 43 |
| CommentRequired | Public method and constructor comments are required | 3 | 45 |
| CommentRequired | Public method and constructor comments are required | 3 | 47 |
| CommentRequired | Public method and constructor comments are required | 3 | 49 |
| CommentRequired | Public method and constructor comments are required | 3 | 51 |
| CommentRequired | Public method and constructor comments are required | 3 | 53 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 40–137 |
| CommentRequired | Public method and constructor comments are required | 3 | 47–50 |
| CommentRequired | Public method and constructor comments are required | 3 | 52–55 |
| CommentRequired | Public method and constructor comments are required | 3 | 57–60 |
| CommentRequired | Public method and constructor comments are required | 3 | 62–65 |
| CommentRequired | Public method and constructor comments are required | 3 | 67–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 72–75 |
| CommentRequired | Public method and constructor comments are required | 3 | 77–80 |
| CommentRequired | Public method and constructor comments are required | 3 | 82–85 |
| CommentRequired | Public method and constructor comments are required | 3 | 87–99 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 95 |
| CommentRequired | Header comments are required | 3 | 101–136 |
| CommentRequired | Public method and constructor comments are required | 3 | 107–110 |
| CommentRequired | Public method and constructor comments are required | 3 | 112–115 |
| CommentRequired | Public method and constructor comments are required | 3 | 117–120 |
| CommentRequired | Public method and constructor comments are required | 3 | 122–125 |
| CommentRequired | Public method and constructor comments are required | 3 | 127–130 |
| CommentRequired | Public method and constructor comments are required | 3 | 132–135 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 44–229 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 57–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 57–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 72–98 |
| MissingOverride | The method 'insert(LibraryMapping, Plugin)' is missing an @Override annotation. | 3 | 72–98 |
| CommentRequired | Public method and constructor comments are required | 3 | 100–127 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 100–127 |
| MissingOverride | The method 'selectByMedia(int, Plugin)' is missing an @Override annotation. | 3 | 129–162 |
| CommentRequired | Public method and constructor comments are required | 3 | 129–162 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 131 |
| MissingOverride | The method 'store(LibraryMapping, Plugin)' is missing an @Override annotation. | 3 | 164–196 |
| CommentRequired | Public method and constructor comments are required | 3 | 164–196 |
| CommentRequired | Public method and constructor comments are required | 3 | 221–228 |
| MissingOverride | The method 'deleteAttributeAssociation(int, Plugin)' is missing an @Override annotation. | 3 | 221–228 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–81 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 42–81 |
| CommentRequired | Public method and constructor comments are required | 3 | 52–55 |
| CommentRequired | Public method and constructor comments are required | 3 | 57–60 |
| CommentRequired | Public method and constructor comments are required | 3 | 62–65 |
| CommentRequired | Public method and constructor comments are required | 3 | 67–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 72–75 |
| CommentRequired | Public method and constructor comments are required | 3 | 77–80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–115 |
| CommentRequired | Public method and constructor comments are required | 3 | 50–53 |
| CommentRequired | Public method and constructor comments are required | 3 | 55–58 |
| CommentRequired | Public method and constructor comments are required | 3 | 60–63 |
| CommentRequired | Public method and constructor comments are required | 3 | 65–68 |
| CommentRequired | Public method and constructor comments are required | 3 | 70–73 |
| CommentRequired | Public method and constructor comments are required | 3 | 75–78 |
| CommentRequired | Public method and constructor comments are required | 3 | 80–83 |
| CommentRequired | Public method and constructor comments are required | 3 | 85–88 |
| CommentRequired | Public method and constructor comments are required | 3 | 100–104 |
| CommentRequired | Public method and constructor comments are required | 3 | 106–109 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 106–109 |
| CommentRequired | Public method and constructor comments are required | 3 | 111–114 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 43–153 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 52–58 |
| CommentRequired | Public method and constructor comments are required | 3 | 52–58 |
| CommentRequired | Public method and constructor comments are required | 3 | 60–73 |
| MissingOverride | The method 'insert(LibraryMedia, Plugin)' is missing an @Override annotation. | 3 | 60–73 |
| CommentRequired | Public method and constructor comments are required | 3 | 75–95 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 75–95 |
| MissingOverride | The method 'selectAll(Plugin)' is missing an @Override annotation. | 3 | 97–116 |
| CommentRequired | Public method and constructor comments are required | 3 | 97–116 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 99 |
| CommentRequired | Public method and constructor comments are required | 3 | 118–128 |
| MissingOverride | The method 'store(LibraryMedia, Plugin)' is missing an @Override annotation. | 3 | 118–128 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–76 |
| CommentRequired | Public method and constructor comments are required | 3 | 52–55 |
| CommentRequired | Public method and constructor comments are required | 3 | 57–60 |
| CommentRequired | Public method and constructor comments are required | 3 | 62–65 |
| CommentRequired | Public method and constructor comments are required | 3 | 67–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 72–75 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–177 |
| CommentRequired | Public method and constructor comments are required | 3 | 62–65 |
| CommentRequired | Public method and constructor comments are required | 3 | 67–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 72–75 |
| CommentRequired | Public method and constructor comments are required | 3 | 77–80 |
| CommentRequired | Public method and constructor comments are required | 3 | 82–85 |
| CommentRequired | Public method and constructor comments are required | 3 | 87–90 |
| CommentRequired | Public method and constructor comments are required | 3 | 92–95 |
| CommentRequired | Public method and constructor comments are required | 3 | 97–100 |
| CommentRequired | Public method and constructor comments are required | 3 | 102–106 |
| CommentRequired | Public method and constructor comments are required | 3 | 108–111 |
| CommentRequired | Public method and constructor comments are required | 3 | 113–116 |
| CommentRequired | Public method and constructor comments are required | 3 | 118–151 |
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 3 | 153–156 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 43–167 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 53–59 |
| CommentRequired | Public method and constructor comments are required | 3 | 53–59 |
| CommentRequired | Public method and constructor comments are required | 3 | 61–73 |
| MissingOverride | The method 'insert(MediaAttribute, Plugin)' is missing an @Override annotation. | 3 | 61–73 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 75–96 |
| CommentRequired | Public method and constructor comments are required | 3 | 75–96 |
| CommentRequired | Public method and constructor comments are required | 3 | 98–120 |
| MissingOverride | The method 'selectAllAttributesForMedia(int, Plugin)' is missing an @Override annotation. | 3 | 98–120 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 100 |
| CommentRequired | Public method and constructor comments are required | 3 | 122–135 |
| MissingOverride | The method 'store(MediaAttribute, Plugin)' is missing an @Override annotation. | 3 | 122–135 |
| CommentRequired | Public method and constructor comments are required | 3 | 160–166 |
| MissingOverride | The method 'deleteAllForMedia(int, Plugin)' is missing an @Override annotation. | 3 | 160–166 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–81 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 42–81 |
| CommentRequired | Public method and constructor comments are required | 3 | 52–55 |
| CommentRequired | Public method and constructor comments are required | 3 | 57–60 |
| CommentRequired | Public method and constructor comments are required | 3 | 62–65 |
| CommentRequired | Public method and constructor comments are required | 3 | 67–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 72–75 |
| CommentRequired | Public method and constructor comments are required | 3 | 77–80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–184 |
| ShortVariable | Avoid variables with short names like o | 3 | 175 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–712 |
| UnusedPrivateField | Avoid unused private fields such as 'PARAMETER_PAGE_INDEX'. | 3 | 104 |
| UnusedPrivateField | Avoid unused private fields such as 'DEFAULT_RESULTS_PER_PAGE'. | 3 | 136 |
| SingularField | Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. | 3 | 150 |
| CommentRequired | Public method and constructor comments are required | 3 | 152–164 |
| MissingOverride | The method 'getInsertServiceSelectorUI(HttpServletRequest)' is missing an @Override annotation. | 3 | 152–164 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 155 |
| CyclomaticComplexity | The method 'getSelectMedia(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 166–254 |
| CommentRequired | Public method and constructor comments are required | 3 | 166–254 |
| NPathComplexity | The method 'getSelectMedia(HttpServletRequest)' has an NPath complexity of 256 | 3 | 166–254 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 197 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 198 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 199 |
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 249 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 249 |
| CommentRequired | Public method and constructor comments are required | 3 | 256–296 |
| CyclomaticComplexity | The method 'doSelectMedia(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 298–368 |
| CommentRequired | Public method and constructor comments are required | 3 | 298–368 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 304 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 319 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 331 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 344 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 355 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 357 |
| CommentRequired | Public method and constructor comments are required | 3 | 370–425 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 383 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 385 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'document' | 3 | 399 |
| CommentRequired | Public method and constructor comments are required | 3 | 427–488 |
| CommentRequired | Public method and constructor comments are required | 3 | 490–570 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 557 |
| UseProperClassLoader | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | 3 | 560 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 567 |
| ShortVariable | Avoid variables with short names like m | 3 | 572 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nSpaceId'. | 3 | 595 |
| ShortVariable | Avoid variables with short names like m | 3 | 595 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 599 |
| ShortVariable | Avoid variables with short names like m | 3 | 614 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 616 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 645 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 647 |
| IdempotentOperations | Avoid idempotent operations (like assigning a variable to itself). | 3 | 664 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 686–687 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'paginator' | 3 | 688 |
| CommentRequired | Header comments are required | 3 | 691–711 |
| ShortClassName | Avoid short class names like Pair | 4 | 691–711 |
| ImmutableField | Private field 'first' could be made final; it is only initialized in the declaration or constructor. | 3 | 693 |
| FieldNamingConventions | The field name 'first' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 693 |
| FieldNamingConventions | The field name 'second' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 694 |
| ImmutableField | Private field 'second' could be made final; it is only initialized in the declaration or constructor. | 3 | 694 |
| CommentRequired | Public method and constructor comments are required | 3 | 696–700 |
| ShortVariable | Avoid variables with short names like x | 3 | 696 |
| ShortVariable | Avoid variables with short names like y | 3 | 696 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–724 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 146 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 146 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 258 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 258 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 278 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 315 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 315 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 335 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 372 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 394 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 524 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 524 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 534 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 534 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 563 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 563 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 636 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 636 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–565 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 181 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 183 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 305 |
| AddEmptyString | Do not add empty strings | 3 | 306 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 343 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 408–415 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 428–435 |
| LocalVariableNamingConventions | The local variable name '_listAttributesAssociation' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 466 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 477–490 |
| LocalVariableNamingConventions | The local variable name 'docType_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 485 |
| LocalVariableNamingConventions | The local variable name '_docAttr' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 486 |
| ControlStatementBraces | This statement should have braces | 3 | 487 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 514 |
| UseProperClassLoader | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | 3 | 517 |
| ShortVariable | Avoid variables with short names like m | 3 | 553 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 555 |
