The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| MethodNamingConventions | The instance method name 'BusinessClass' doesn't match '[a-z][a-zA-Z0-9]*' | 672–675 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The static field name 'bValidUniqueBusinessClassName' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| FieldNamingConventions | The static field name 'bValidUniquePluralBusinessClassName' doesn't match '_[a-z][a-zA-Z0-9]*' | 54 |
| FieldNamingConventions | The static field name 'bValidUniqueTableName' doesn't match '_[a-z][a-zA-Z0-9]*' | 55 |
| FieldNamingConventions | The static field name 'bValidUniqueAttributeName' doesn't match '_[a-z][a-zA-Z0-9]*' | 58 |
| FieldNamingConventions | The static field name 'bValideUniqueFeatureRight' doesn't match '_[a-z][a-zA-Z0-9]*' | 61 |
| FieldNamingConventions | The static field name 'bValideUniqueFeatureTitle' doesn't match '_[a-z][a-zA-Z0-9]*' | 62 |
| FieldNamingConventions | The static field name 'bValideUniqueFeatureTechName' doesn't match '_[a-z][a-zA-Z0-9]*' | 63 |
| FieldNamingConventions | The static field name 'bValideUniqueAdministrationName' doesn't match '_[a-z][a-zA-Z0-9]*' | 66 |
| FieldNamingConventions | The static field name 'bValideUniqueAdministrationClass' doesn't match '_[a-z][a-zA-Z0-9]*' | 67 |
| FieldNamingConventions | The static field name 'bValideUniquePortletClassName' doesn't match '_[a-z][a-zA-Z0-9]*' | 70 |
| FieldNamingConventions | The static field name 'bValideUniquePortletType' doesn't match '_[a-z][a-zA-Z0-9]*' | 71 |
| FieldNamingConventions | The static field name 'bValideUniquePortletJspName' doesn't match '_[a-z][a-zA-Z0-9]*' | 72 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The static field name '_jsp_prefix' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The static field name '_template_prefix' doesn't match '_[a-z][a-zA-Z0-9]*' | 57 |
| LocalVariableNamingConventions | The local variable name 'admin_feature' doesn't match '[a-z][a-zA-Z0-9]*' | 84 |
| FormalParameterNamingConventions | The method parameter name 'admin_feature' doesn't match '[a-z][a-zA-Z0-9]*' | 130 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Markers' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 39–95 |
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name 'BusinessNames' doesn't match '[a-z][a-zA-Z0-9]*' | 88 |
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name '_path' doesn't match '[a-z][a-zA-Z0-9]*' | 60 |
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name '_path' doesn't match '[a-z][a-zA-Z0-9]*' | 64 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The static field name '_template_prefix' doesn't match '_[a-z][a-zA-Z0-9]*' | 57 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Utils' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 39–97 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 41–76 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_strType' doesn't match '[a-z][a-zA-Z0-9]*' | 305 |
| FormalParameterNamingConventions | The method parameter name '_bIsModule' doesn't match '[a-z][a-zA-Z0-9]*' | 336 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'insert(ConfigurationKey, Plugin)' is missing an @Override annotation. | 91–105 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 116–138 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 148–156 |
| MissingOverride | The method 'store(ConfigurationKey, Plugin)' is missing an @Override annotation. | 166–179 |
| MissingOverride | The method 'selectConfigurationKeysList(Plugin)' is missing an @Override annotation. | 188–210 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 132–139 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 150–157 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 156 |
| Rule | Violation | Line |
|---|---|---|
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 157 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 190 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 201 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 166–173 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 172 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 183–190 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 229–236 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 247–254 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 253 |
| Rule | Violation | Line |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–739 |
| TooManyFields | Too many fields | 53–739 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 209 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 213 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 226 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 539–546 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 545 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 609–616 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 615 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 637–644 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 643 |
| CommentRequired | Public method and constructor comments are required | 667–670 |
| CommentRequired | Public method and constructor comments are required | 672–675 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'insert(PluginModel, Plugin)' is missing an @Override annotation. | 93–118 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 129–164 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 174–182 |
| MissingOverride | The method 'store(PluginModel, Plugin)' is missing an @Override annotation. | 192–216 |
| MissingOverride | The method 'selectPluginModelsList(Plugin)' is missing an @Override annotation. | 225–258 |
| MissingOverride | The method 'selectPluginModelByName(Plugin, String)' is missing an @Override annotation. | 269–287 |
| MissingOverride | The method 'pluginExists(String, Plugin)' is missing an @Override annotation. | 298–321 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 81–88 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 99–106 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 105 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 60–67 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 66 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like sw | 67 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'sw' | 68 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 91 |
| Rule | Violation | Line |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–1320 |
| ExcessiveClassLength | Avoid really long classes. | 65–1320 |
| CyclomaticComplexity | The class 'ModelService' has a total cyclomatic complexity of 139 (highest 7). | 65–1320 |
| GodClass | Possible God Class (WMC=139, ATFD=144, TCC=1.647%) | 65–1320 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 66–1320 |
| ShortVariable | Avoid variables with short names like ID | 69 |
| ShortVariable | Avoid variables with short names like pm | 89 |
| ShortVariable | Avoid variables with short names like pm | 108 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 111–119 |
| ShortVariable | Avoid variables with short names like pm | 130 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 139–146 |
| ShortVariable | Avoid variables with short names like pm | 156 |
| ShortVariable | Avoid variables with short names like pm | 194 |
| ShortVariable | Avoid variables with short names like pm | 214 |
| ShortVariable | Avoid variables with short names like pm | 234 |
| ShortVariable | Avoid variables with short names like pm | 259 |
| ShortVariable | Avoid variables with short names like f | 264 |
| ShortVariable | Avoid variables with short names like pm | 288 |
| ShortVariable | Avoid variables with short names like f | 293 |
| ShortVariable | Avoid variables with short names like pm | 321 |
| ShortVariable | Avoid variables with short names like pm | 335 |
| ShortVariable | Avoid variables with short names like pm | 358 |
| ShortVariable | Avoid variables with short names like pm | 374 |
| ShortVariable | Avoid variables with short names like pm | 399 |
| ShortVariable | Avoid variables with short names like pm | 428 |
| ShortVariable | Avoid variables with short names like f | 433 |
| ShortVariable | Avoid variables with short names like pm | 461 |
| ShortVariable | Avoid variables with short names like pm | 484 |
| ShortVariable | Avoid variables with short names like pm | 499 |
| ShortVariable | Avoid variables with short names like pm | 524 |
| ShortVariable | Avoid variables with short names like p | 529 |
| ShortVariable | Avoid variables with short names like pm | 553 |
| ShortVariable | Avoid variables with short names like p | 558 |
| ShortVariable | Avoid variables with short names like pm | 584 |
| ShortVariable | Avoid variables with short names like pm | 598 |
| ShortVariable | Avoid variables with short names like bc | 620 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 622 |
| ShortVariable | Avoid variables with short names like pm | 622 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 640–647 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 644 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 646 |
| ShortVariable | Avoid variables with short names like pm | 665 |
| ShortVariable | Avoid variables with short names like pm | 690 |
| ShortVariable | Avoid variables with short names like bc | 695 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 701 |
| ShortVariable | Avoid variables with short names like pm | 738 |
| ShortVariable | Avoid variables with short names like bc | 743 |
| ShortVariable | Avoid variables with short names like pm | 764 |
| ShortVariable | Avoid variables with short names like pm | 787 |
| ShortVariable | Avoid variables with short names like bc | 788 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 789–806 |
| ShortVariable | Avoid variables with short names like pm | 821 |
| ShortVariable | Avoid variables with short names like bc | 822 |
| ShortVariable | Avoid variables with short names like pm | 870 |
| ShortVariable | Avoid variables with short names like bc | 871 |
| ShortVariable | Avoid variables with short names like pm | 904 |
| ShortVariable | Avoid variables with short names like bc | 905 |
| ShortVariable | Avoid variables with short names like pm | 939 |
| ShortVariable | Avoid variables with short names like pm | 953 |
| ShortVariable | Avoid variables with short names like pm | 967 |
| ShortVariable | Avoid variables with short names like pm | 979 |
| ShortVariable | Avoid variables with short names like pm | 1007 |
| ShortVariable | Avoid variables with short names like pm | 1041 |
| ShortVariable | Avoid variables with short names like pm | 1075 |
| ShortVariable | Avoid variables with short names like pm | 1110 |
| ShortVariable | Avoid variables with short names like pm | 1238 |
| ShortVariable | Avoid variables with short names like pm | 1262 |
| ShortVariable | Avoid variables with short names like pm | 1285 |
| ShortVariable | Avoid variables with short names like pm | 1306 |
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as '_plugin'. | 50 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 57–65 |
| CommentRequired | Public method and constructor comments are required | 83–89 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 66–74 |
| ShortVariable | Avoid variables with short names like sb | 104 |
| AssignmentInOperand | Avoid assignments in operands | 110 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 122 |
| Rule | Violation | Line |
|---|---|---|
| GodClass | Possible God Class (WMC=56, ATFD=42, TCC=0.000%) | 50–448 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 51–448 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 51–448 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 59 |
| ShortVariable | Avoid variables with short names like pm | 89 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 91 |
| ShortVariable | Avoid variables with short names like pm | 108 |
| ShortVariable | Avoid variables with short names like pm | 120 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 108 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 140 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 140 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'strValueCap' | 142 |
| ShortVariable | Avoid variables with short names like pm | 152 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 154 |
| ShortVariable | Avoid variables with short names like pm | 167 |
| ShortVariable | Avoid variables with short names like pm | 181 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 66–76 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 68–75 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 74 |
| MissingOverride | The method 'generate(PluginModel, String)' is missing an @Override annotation. | 92–139 |
| CyclomaticComplexity | The method 'generate(PluginModel, String)' has a cyclomatic complexity of 15. | 92–139 |
| ShortVariable | Avoid variables with short names like pm | 92 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 154–155 |
| ShortVariable | Avoid variables with short names like pm | 154 |
| ShortVariable | Avoid variables with short names like pm | 182 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 182–183 |
| ShortVariable | Avoid variables with short names like pm | 207 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 81 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 122 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 123 |
| UnusedPrivateMethod | Avoid unused private methods such as 'getFeatureJspFile(String,String,boolean)'. | 141 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 143 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 85 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 103 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 148 |
| AddEmptyString | Do not add empty strings | 152 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 155 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 173 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 68 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 92 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 94 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 98 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 107 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 133 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 133 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 169 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 169 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 65–72 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 71 |
| ShortVariable | Avoid variables with short names like pm | 79 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 150 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 152 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 173 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 175 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 62–69 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 68 |
| ShortVariable | Avoid variables with short names like pm | 76 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 116 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 118 |
| Rule | Violation | Line |
|---|---|---|
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsDefault' | 47 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsWorkflow' | 48 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 89–96 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 105–112 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 111 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 68 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 80 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 61 |
| ShortVariable | Avoid variables with short names like pm | 70 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 72 |
| CommentRequired | Public method and constructor comments are required | 80–117 |
| ShortVariable | Avoid variables with short names like pm | 80 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 86 |
| ShortVariable | Avoid variables with short names like i | 89 |
| ShortVariable | Avoid variables with short names like id | 90 |
| ShortVariable | Avoid variables with short names like pm | 123 |
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'KEY_PREFIX'. | 51 |
| ShortVariable | Avoid variables with short names like pm | 65 |
| ShortVariable | Avoid variables with short names like pm | 77 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 79 |
| ShortVariable | Avoid variables with short names like pm | 107 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 62–69 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 68 |
| ShortVariable | Avoid variables with short names like pm | 76 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 114 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 55 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 84 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 59 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 95 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 57 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 66 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 108 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 56 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 84 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 84 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'strValueCap' | 86 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 100 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 52 |
| ShortVariable | Avoid variables with short names like pm | 61 |
| Rule | Violation | Line |
|---|---|---|
| GodClass | Possible God Class (WMC=47, ATFD=77, TCC=0.000%) | 55–482 |
| ShortVariable | Avoid variables with short names like pm | 77 |
| ShortVariable | Avoid variables with short names like pm | 105 |
| ShortVariable | Avoid variables with short names like sb | 107 |
| ShortVariable | Avoid variables with short names like sb | 138 |
| ShortVariable | Avoid variables with short names like pm | 138 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 143 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 144 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 144 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 145 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 150 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 151 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 151 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 152 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 152 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 153 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 153 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 153 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 157 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 158 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 158 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 159 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 162 |
| ShortVariable | Avoid variables with short names like sb | 173 |
| ShortVariable | Avoid variables with short names like pm | 173 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 181 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 181 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 183 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 186 |
| ShortVariable | Avoid variables with short names like sb | 190 |
| ShortVariable | Avoid variables with short names like pm | 190 |
| AvoidDuplicateLiterals | The String literal "xpage." appears 4 times in this file; the first occurrence is on line 201 | 201 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 202 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 202 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 204 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 214 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 215 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 215 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 215 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 216 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 217 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 217 |
| CyclomaticComplexity | The method 'generateBusinessClassKeys(StringBuilder, PluginModel, String)' has a cyclomatic complexity of 13. | 235–310 |
| ShortVariable | Avoid variables with short names like sb | 235 |
| ShortVariable | Avoid variables with short names like pm | 235 |
| NPathComplexity | The method 'generateBusinessClassKeys(StringBuilder, PluginModel, String)' has an NPath complexity of 482 | 235–310 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 244 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 246 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 247 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 248 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 249 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 253 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 253 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 258 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 259 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 260 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 265 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 266 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 270 |
| AvoidDuplicateLiterals | The String literal "label" appears 4 times in this file; the first occurrence is on line 270 | 270 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 270 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 276 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 277 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 277 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 282 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 291 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 297 |
| AddEmptyString | Do not add empty strings | 297 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 303 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 307 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 307 |
| ShortVariable | Avoid variables with short names like sb | 322 |
| ShortVariable | Avoid variables with short names like pm | 322 |
| ShortVariable | Avoid variables with short names like bc | 327 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 334 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 335 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 340 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 340 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 356 |
| AddEmptyString | Do not add empty strings | 362 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 362 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 373 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 373 |
| ShortVariable | Avoid variables with short names like pm | 389 |
| UnusedFormalParameter | Avoid unused method parameters such as 'strLanguage'. | 389 |
| ShortVariable | Avoid variables with short names like sb | 389 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 391 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 391 |
| UnusedLocalVariable | Avoid unused local variables such as 'strPrefix'. | 393 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 394 |
| ShortVariable | Avoid variables with short names like pm | 407 |
| UnusedFormalParameter | Avoid unused method parameters such as 'pm'. | 407 |
| ShortVariable | Avoid variables with short names like sb | 407 |
| UnusedFormalParameter | Avoid unused method parameters such as 'strLanguage'. | 407 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 409 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 409 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 6 consecutive times with literals. Use a single append with a single combined String. | 409 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 410 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 410 |
| ShortVariable | Avoid variables with short names like sb | 421 |
| ShortVariable | Avoid variables with short names like pm | 421 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 429 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 430 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 433 |
| ShortVariable | Avoid variables with short names like sb | 445 |
| ShortVariable | Avoid variables with short names like pm | 445 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 455 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 455 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 455 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 456 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 456 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 460 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 60 |
| ShortVariable | Avoid variables with short names like pm | 85 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 60 |
| ShortVariable | Avoid variables with short names like pm | 89 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 60 |
| ShortVariable | Avoid variables with short names like pm | 85 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 60 |
| ShortVariable | Avoid variables with short names like pm | 85 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 52 |
| ShortVariable | Avoid variables with short names like pm | 76 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 78 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 56 |
| ShortVariable | Avoid variables with short names like pm | 68 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 61 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 69 |
| ShortVariable | Avoid variables with short names like pm | 112 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 114 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 59 |
| ShortVariable | Avoid variables with short names like pm | 83 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 85 |
| Rule | Violation | Line |
|---|---|---|
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 87 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 88 |
| SimplifyStartsWith | This call to String.startsWith can be rewritten using String.charAt(0) | 97 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 126 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 58 |
| ShortVariable | Avoid variables with short names like pm | 80 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 82 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 57 |
| ShortVariable | Avoid variables with short names like pm | 80 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 82 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 73–80 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 79 |
| CyclomaticComplexity | The method 'generate(PluginModel, String)' has a cyclomatic complexity of 11. | 89–128 |
| ShortVariable | Avoid variables with short names like pm | 89 |
| UnusedLocalVariable | Avoid unused local variables such as 'strFilesPath'. | 92 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 92 |
| ShortVariable | Avoid variables with short names like pm | 145 |
| ShortVariable | Avoid variables with short names like pm | 172 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 68 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 96 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 98 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 102 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 111 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 139 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 173 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 202 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like c | 63 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 65 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 75 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 95 |
| Rule | Violation | Line |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–2087 |
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–2087 |
| ExcessiveClassLength | Avoid really long classes. | 89–2087 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 90–2087 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 321–324 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 399 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 460 |
| CommentRequired | Public method and constructor comments are required | 466–472 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 513 |
| ShortVariable | Avoid variables with short names like pm | 670 |
| ShortVariable | Avoid variables with short names like pm | 691 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 692 |
| ShortVariable | Avoid variables with short names like pm | 713 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 753–774 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 770 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 800–820 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 816 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 917 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 945 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 971–980 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 983 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1005–1020 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1010 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 1023 |
| CommentRequired | Public method and constructor comments are required | 1127–1136 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1153 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 1177 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1205–1213 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 1216 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1238–1246 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 1249 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1367–1377 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 1380 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1406–1416 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 1419 |
| CommentRequired | Public method and constructor comments are required | 1481–1497 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1489 |
| ShortVariable | Avoid variables with short names like pm | 1530 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1531 |
| ShortVariable | Avoid variables with short names like pm | 1552 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1593–1603 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 1606 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1634–1643 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 1646 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1741 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1786–1796 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 1799 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1817–1827 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 1830 |
| ShortVariable | Avoid variables with short names like pm | 1908 |
| ShortVariable | Avoid variables with short names like e | 1986 |
| Rule | Violation | Line |
|---|---|---|
| SingularField | Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. | 109 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 134 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 134 |
| AddEmptyString | Do not add empty strings | 136 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 156 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 156 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 172 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 174 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 176 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 181 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 239 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 239 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 259 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 44–207 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 44–169 |
| UncommentedEmptyConstructor | Document empty constructor | 61–63 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 61–63 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 47–340 |
| UncommentedEmptyConstructor | Document empty constructor | 76–78 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 76–78 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–43 |
| CommentRequired | Public method and constructor comments are required | 38 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getName': the method is declared in an interface type | 38 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'setName': the method is declared in an interface type | 39 |
| CommentRequired | Public method and constructor comments are required | 39 |
| CommentRequired | Public method and constructor comments are required | 40 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getType': the method is declared in an interface type | 40 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'setType': the method is declared in an interface type | 41 |
| CommentRequired | Public method and constructor comments are required | 41 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 44–87 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 57–60 |
| MissingOverride | The method 'setName(String)' is missing an @Override annotation. | 66–69 |
| MissingOverride | The method 'getType()' is missing an @Override annotation. | 74–77 |
| MissingOverride | The method 'setType(String)' is missing an @Override annotation. | 83–86 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 44–85 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 55–58 |
| MissingOverride | The method 'setName(String)' is missing an @Override annotation. | 64–67 |
| MissingOverride | The method 'getType()' is missing an @Override annotation. | 72–75 |
| MissingOverride | The method 'setType(String)' is missing an @Override annotation. | 81–84 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 44–86 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 56–59 |
| MissingOverride | The method 'setName(String)' is missing an @Override annotation. | 65–68 |
| MissingOverride | The method 'getType()' is missing an @Override annotation. | 73–76 |
| MissingOverride | The method 'setType(String)' is missing an @Override annotation. | 82–85 |
| Rule | Violation | Line |
|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 226–229 |
| Rule | Violation | Line |
|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 714–717 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like Rest | 43–108 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.pluginwizard.util.Utils' | 40 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.pluginwizard.util.Utils' | 40 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.pluginwizard.util.Utils' | 40 |
| Rule | Violation | Line |
|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 325–328 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'insert(ConfigurationKey, Plugin)' is missing an @Override annotation. | 3 | 91–105 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 116–138 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 148–156 |
| MissingOverride | The method 'store(ConfigurationKey, Plugin)' is missing an @Override annotation. | 3 | 166–179 |
| MissingOverride | The method 'selectConfigurationKeysList(Plugin)' is missing an @Override annotation. | 3 | 188–210 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 132–139 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 150–157 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 156 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 157 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 190 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 201 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 226–229 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 166–173 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 172 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 183–190 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 229–236 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 247–254 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 253 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–739 |
| TooManyFields | Too many fields | 3 | 53–739 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 209 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 213 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 226 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 539–546 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 545 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 609–616 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 615 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 637–644 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 643 |
| CommentRequired | Public method and constructor comments are required | 3 | 667–670 |
| MethodNamingConventions | The instance method name 'BusinessClass' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 672–675 |
| CommentRequired | Public method and constructor comments are required | 3 | 672–675 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 714–717 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'insert(PluginModel, Plugin)' is missing an @Override annotation. | 3 | 93–118 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 129–164 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 174–182 |
| MissingOverride | The method 'store(PluginModel, Plugin)' is missing an @Override annotation. | 3 | 192–216 |
| MissingOverride | The method 'selectPluginModelsList(Plugin)' is missing an @Override annotation. | 3 | 225–258 |
| MissingOverride | The method 'selectPluginModelByName(Plugin, String)' is missing an @Override annotation. | 3 | 269–287 |
| MissingOverride | The method 'pluginExists(String, Plugin)' is missing an @Override annotation. | 3 | 298–321 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortClassName | Avoid short class names like Rest | 4 | 43–108 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 81–88 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 99–106 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 105 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 60–67 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 66 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like sw | 3 | 67 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'sw' | 3 | 68 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 91 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–1320 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 65–1320 |
| CyclomaticComplexity | The class 'ModelService' has a total cyclomatic complexity of 139 (highest 7). | 3 | 65–1320 |
| GodClass | Possible God Class (WMC=139, ATFD=144, TCC=1.647%) | 3 | 65–1320 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 66–1320 |
| ShortVariable | Avoid variables with short names like ID | 3 | 69 |
| ShortVariable | Avoid variables with short names like pm | 3 | 89 |
| ShortVariable | Avoid variables with short names like pm | 3 | 108 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 111–119 |
| ShortVariable | Avoid variables with short names like pm | 3 | 130 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 139–146 |
| ShortVariable | Avoid variables with short names like pm | 3 | 156 |
| ShortVariable | Avoid variables with short names like pm | 3 | 194 |
| ShortVariable | Avoid variables with short names like pm | 3 | 214 |
| ShortVariable | Avoid variables with short names like pm | 3 | 234 |
| ShortVariable | Avoid variables with short names like pm | 3 | 259 |
| ShortVariable | Avoid variables with short names like f | 3 | 264 |
| ShortVariable | Avoid variables with short names like pm | 3 | 288 |
| ShortVariable | Avoid variables with short names like f | 3 | 293 |
| ShortVariable | Avoid variables with short names like pm | 3 | 321 |
| ShortVariable | Avoid variables with short names like pm | 3 | 335 |
| ShortVariable | Avoid variables with short names like pm | 3 | 358 |
| ShortVariable | Avoid variables with short names like pm | 3 | 374 |
| ShortVariable | Avoid variables with short names like pm | 3 | 399 |
| ShortVariable | Avoid variables with short names like pm | 3 | 428 |
| ShortVariable | Avoid variables with short names like f | 3 | 433 |
| ShortVariable | Avoid variables with short names like pm | 3 | 461 |
| ShortVariable | Avoid variables with short names like pm | 3 | 484 |
| ShortVariable | Avoid variables with short names like pm | 3 | 499 |
| ShortVariable | Avoid variables with short names like pm | 3 | 524 |
| ShortVariable | Avoid variables with short names like p | 3 | 529 |
| ShortVariable | Avoid variables with short names like pm | 3 | 553 |
| ShortVariable | Avoid variables with short names like p | 3 | 558 |
| ShortVariable | Avoid variables with short names like pm | 3 | 584 |
| ShortVariable | Avoid variables with short names like pm | 3 | 598 |
| ShortVariable | Avoid variables with short names like bc | 3 | 620 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 622 |
| ShortVariable | Avoid variables with short names like pm | 3 | 622 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 640–647 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 644 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 646 |
| ShortVariable | Avoid variables with short names like pm | 3 | 665 |
| ShortVariable | Avoid variables with short names like pm | 3 | 690 |
| ShortVariable | Avoid variables with short names like bc | 3 | 695 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 701 |
| ShortVariable | Avoid variables with short names like pm | 3 | 738 |
| ShortVariable | Avoid variables with short names like bc | 3 | 743 |
| ShortVariable | Avoid variables with short names like pm | 3 | 764 |
| ShortVariable | Avoid variables with short names like pm | 3 | 787 |
| ShortVariable | Avoid variables with short names like bc | 3 | 788 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 789–806 |
| ShortVariable | Avoid variables with short names like pm | 3 | 821 |
| ShortVariable | Avoid variables with short names like bc | 3 | 822 |
| ShortVariable | Avoid variables with short names like pm | 3 | 870 |
| ShortVariable | Avoid variables with short names like bc | 3 | 871 |
| ShortVariable | Avoid variables with short names like pm | 3 | 904 |
| ShortVariable | Avoid variables with short names like bc | 3 | 905 |
| ShortVariable | Avoid variables with short names like pm | 3 | 939 |
| ShortVariable | Avoid variables with short names like pm | 3 | 953 |
| ShortVariable | Avoid variables with short names like pm | 3 | 967 |
| ShortVariable | Avoid variables with short names like pm | 3 | 979 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1007 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1041 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1075 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1110 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1238 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1262 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1285 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1306 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as '_plugin'. | 3 | 50 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 57–65 |
| CommentRequired | Public method and constructor comments are required | 3 | 83–89 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 66–74 |
| ShortVariable | Avoid variables with short names like sb | 3 | 104 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 110 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 122 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| GodClass | Possible God Class (WMC=56, ATFD=42, TCC=0.000%) | 3 | 50–448 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 51–448 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 51–448 |
| FieldNamingConventions | The static field name 'bValidUniqueBusinessClassName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| FieldNamingConventions | The static field name 'bValidUniquePluralBusinessClassName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 54 |
| FieldNamingConventions | The static field name 'bValidUniqueTableName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 55 |
| FieldNamingConventions | The static field name 'bValidUniqueAttributeName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 58 |
| FieldNamingConventions | The static field name 'bValideUniqueFeatureRight' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 61 |
| FieldNamingConventions | The static field name 'bValideUniqueFeatureTitle' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 62 |
| FieldNamingConventions | The static field name 'bValideUniqueFeatureTechName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 63 |
| FieldNamingConventions | The static field name 'bValideUniqueAdministrationName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 66 |
| FieldNamingConventions | The static field name 'bValideUniqueAdministrationClass' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 67 |
| FieldNamingConventions | The static field name 'bValideUniquePortletClassName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 70 |
| FieldNamingConventions | The static field name 'bValideUniquePortletType' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 71 |
| FieldNamingConventions | The static field name 'bValideUniquePortletJspName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 72 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 59 |
| ShortVariable | Avoid variables with short names like pm | 3 | 89 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 91 |
| ShortVariable | Avoid variables with short names like pm | 3 | 108 |
| ShortVariable | Avoid variables with short names like pm | 3 | 120 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 108 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 140 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 140 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'strValueCap' | 3 | 142 |
| ShortVariable | Avoid variables with short names like pm | 3 | 152 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 154 |
| ShortVariable | Avoid variables with short names like pm | 3 | 167 |
| ShortVariable | Avoid variables with short names like pm | 3 | 181 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 66–76 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 68–75 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 74 |
| MissingOverride | The method 'generate(PluginModel, String)' is missing an @Override annotation. | 3 | 92–139 |
| CyclomaticComplexity | The method 'generate(PluginModel, String)' has a cyclomatic complexity of 15. | 3 | 92–139 |
| ShortVariable | Avoid variables with short names like pm | 3 | 92 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 154–155 |
| ShortVariable | Avoid variables with short names like pm | 3 | 154 |
| ShortVariable | Avoid variables with short names like pm | 3 | 182 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 182–183 |
| ShortVariable | Avoid variables with short names like pm | 3 | 207 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.pluginwizard.util.Utils' | 4 | 40 |
| ShortVariable | Avoid variables with short names like pm | 3 | 81 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 122 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 123 |
| UnusedPrivateMethod | Avoid unused private methods such as 'getFeatureJspFile(String,String,boolean)'. | 3 | 141 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 143 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The static field name '_jsp_prefix' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| ShortVariable | Avoid variables with short names like pm | 3 | 85 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 103 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 148 |
| AddEmptyString | Do not add empty strings | 3 | 152 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 155 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 173 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.pluginwizard.util.Utils' | 4 | 40 |
| FieldNamingConventions | The static field name '_template_prefix' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 57 |
| ShortVariable | Avoid variables with short names like pm | 3 | 68 |
| LocalVariableNamingConventions | The local variable name 'admin_feature' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 84 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 92 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 94 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 98 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 107 |
| FormalParameterNamingConventions | The method parameter name 'admin_feature' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 130 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 133 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 133 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 169 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 169 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 65–72 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 71 |
| ShortVariable | Avoid variables with short names like pm | 3 | 79 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 150 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 152 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 173 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 175 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 62–69 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 68 |
| ShortVariable | Avoid variables with short names like pm | 3 | 76 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 116 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 118 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsDefault' | 3 | 47 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsWorkflow' | 3 | 48 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 89–96 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 105–112 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 111 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 68 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Markers' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 39–95 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 61 |
| ShortVariable | Avoid variables with short names like pm | 3 | 70 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 72 |
| CommentRequired | Public method and constructor comments are required | 3 | 80–117 |
| ShortVariable | Avoid variables with short names like pm | 3 | 80 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 86 |
| LocalVariableNamingConventions | The local variable name 'BusinessNames' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 88 |
| ShortVariable | Avoid variables with short names like i | 3 | 89 |
| ShortVariable | Avoid variables with short names like id | 3 | 90 |
| ShortVariable | Avoid variables with short names like pm | 3 | 123 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'KEY_PREFIX'. | 3 | 51 |
| ShortVariable | Avoid variables with short names like pm | 3 | 65 |
| ShortVariable | Avoid variables with short names like pm | 3 | 77 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 79 |
| ShortVariable | Avoid variables with short names like pm | 3 | 107 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 62–69 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 68 |
| ShortVariable | Avoid variables with short names like pm | 3 | 76 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 114 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 55 |
| LocalVariableNamingConventions | The local variable name '_path' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 60 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 84 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 59 |
| LocalVariableNamingConventions | The local variable name '_path' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 64 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 95 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 57 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 66 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 108 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 56 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 84 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 84 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'strValueCap' | 3 | 86 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 100 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 52 |
| ShortVariable | Avoid variables with short names like pm | 3 | 61 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| GodClass | Possible God Class (WMC=47, ATFD=77, TCC=0.000%) | 3 | 55–482 |
| ShortVariable | Avoid variables with short names like pm | 3 | 77 |
| ShortVariable | Avoid variables with short names like pm | 3 | 105 |
| ShortVariable | Avoid variables with short names like sb | 3 | 107 |
| ShortVariable | Avoid variables with short names like sb | 3 | 138 |
| ShortVariable | Avoid variables with short names like pm | 3 | 138 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 143 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 144 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 144 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 145 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 150 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 151 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 151 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 152 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 152 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 153 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 153 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 153 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 157 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 158 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 158 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 159 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 162 |
| ShortVariable | Avoid variables with short names like sb | 3 | 173 |
| ShortVariable | Avoid variables with short names like pm | 3 | 173 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 181 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 181 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 183 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 186 |
| ShortVariable | Avoid variables with short names like sb | 3 | 190 |
| ShortVariable | Avoid variables with short names like pm | 3 | 190 |
| AvoidDuplicateLiterals | The String literal "xpage." appears 4 times in this file; the first occurrence is on line 201 | 3 | 201 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 202 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 202 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 204 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 214 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 215 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 215 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 215 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 216 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 217 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 217 |
| CyclomaticComplexity | The method 'generateBusinessClassKeys(StringBuilder, PluginModel, String)' has a cyclomatic complexity of 13. | 3 | 235–310 |
| ShortVariable | Avoid variables with short names like sb | 3 | 235 |
| ShortVariable | Avoid variables with short names like pm | 3 | 235 |
| NPathComplexity | The method 'generateBusinessClassKeys(StringBuilder, PluginModel, String)' has an NPath complexity of 482 | 3 | 235–310 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 244 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 246 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 247 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 248 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 249 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 253 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 253 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 258 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 259 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 260 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 265 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 266 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 270 |
| AvoidDuplicateLiterals | The String literal "label" appears 4 times in this file; the first occurrence is on line 270 | 3 | 270 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 270 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 276 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 277 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 277 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 282 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 291 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 297 |
| AddEmptyString | Do not add empty strings | 3 | 297 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 303 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 307 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 307 |
| ShortVariable | Avoid variables with short names like sb | 3 | 322 |
| ShortVariable | Avoid variables with short names like pm | 3 | 322 |
| ShortVariable | Avoid variables with short names like bc | 3 | 327 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 3 | 334 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 335 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 340 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 340 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 356 |
| AddEmptyString | Do not add empty strings | 3 | 362 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 362 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 373 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 373 |
| ShortVariable | Avoid variables with short names like pm | 3 | 389 |
| UnusedFormalParameter | Avoid unused method parameters such as 'strLanguage'. | 3 | 389 |
| ShortVariable | Avoid variables with short names like sb | 3 | 389 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 391 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 3 | 391 |
| UnusedLocalVariable | Avoid unused local variables such as 'strPrefix'. | 3 | 393 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 394 |
| ShortVariable | Avoid variables with short names like pm | 3 | 407 |
| UnusedFormalParameter | Avoid unused method parameters such as 'pm'. | 3 | 407 |
| ShortVariable | Avoid variables with short names like sb | 3 | 407 |
| UnusedFormalParameter | Avoid unused method parameters such as 'strLanguage'. | 3 | 407 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 409 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 409 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 6 consecutive times with literals. Use a single append with a single combined String. | 3 | 409 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 410 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 410 |
| ShortVariable | Avoid variables with short names like sb | 3 | 421 |
| ShortVariable | Avoid variables with short names like pm | 3 | 421 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 429 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 430 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 433 |
| ShortVariable | Avoid variables with short names like sb | 3 | 445 |
| ShortVariable | Avoid variables with short names like pm | 3 | 445 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 455 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 455 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 455 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 456 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 456 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 460 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 60 |
| ShortVariable | Avoid variables with short names like pm | 3 | 85 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 60 |
| ShortVariable | Avoid variables with short names like pm | 3 | 89 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 60 |
| ShortVariable | Avoid variables with short names like pm | 3 | 85 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 60 |
| ShortVariable | Avoid variables with short names like pm | 3 | 85 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 52 |
| ShortVariable | Avoid variables with short names like pm | 3 | 76 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 78 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 56 |
| ShortVariable | Avoid variables with short names like pm | 3 | 68 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 61 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 69 |
| ShortVariable | Avoid variables with short names like pm | 3 | 112 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 114 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 59 |
| ShortVariable | Avoid variables with short names like pm | 3 | 83 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 85 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 87 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 88 |
| SimplifyStartsWith | This call to String.startsWith can be rewritten using String.charAt(0) | 3 | 97 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 126 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 58 |
| ShortVariable | Avoid variables with short names like pm | 3 | 80 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 82 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like pm | 3 | 57 |
| ShortVariable | Avoid variables with short names like pm | 3 | 80 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 82 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 73–80 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 79 |
| CyclomaticComplexity | The method 'generate(PluginModel, String)' has a cyclomatic complexity of 11. | 3 | 89–128 |
| ShortVariable | Avoid variables with short names like pm | 3 | 89 |
| UnusedLocalVariable | Avoid unused local variables such as 'strFilesPath'. | 3 | 92 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 92 |
| ShortVariable | Avoid variables with short names like pm | 3 | 145 |
| ShortVariable | Avoid variables with short names like pm | 3 | 172 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.pluginwizard.util.Utils' | 4 | 40 |
| FieldNamingConventions | The static field name '_template_prefix' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 57 |
| ShortVariable | Avoid variables with short names like pm | 3 | 68 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 96 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 98 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 102 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 111 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 139 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 173 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 202 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Utils' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 39–97 |
| ShortVariable | Avoid variables with short names like c | 3 | 63 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 65 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 75 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 95 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 41–76 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–2087 |
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–2087 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 89–2087 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 90–2087 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 321–324 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 399 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 460 |
| CommentRequired | Public method and constructor comments are required | 3 | 466–472 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 513 |
| ShortVariable | Avoid variables with short names like pm | 3 | 670 |
| ShortVariable | Avoid variables with short names like pm | 3 | 691 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 692 |
| ShortVariable | Avoid variables with short names like pm | 3 | 713 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 753–774 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 770 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 800–820 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 816 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 917 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 945 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 971–980 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 983 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1005–1020 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1010 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 1023 |
| CommentRequired | Public method and constructor comments are required | 3 | 1127–1136 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1153 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 1177 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1205–1213 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 1216 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1238–1246 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 1249 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1367–1377 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 1380 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1406–1416 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 1419 |
| CommentRequired | Public method and constructor comments are required | 3 | 1481–1497 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1489 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1530 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1531 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1552 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1593–1603 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 1606 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1634–1643 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 1646 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1741 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1786–1796 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 1799 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1817–1827 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 1830 |
| ShortVariable | Avoid variables with short names like pm | 3 | 1908 |
| ShortVariable | Avoid variables with short names like e | 3 | 1986 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SingularField | Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. | 3 | 109 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 134 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 134 |
| AddEmptyString | Do not add empty strings | 3 | 136 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 156 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 156 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 172 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 174 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 176 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 181 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 239 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 239 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 259 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 44–207 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 44–169 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 61–63 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 61–63 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 47–340 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 76–78 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 76–78 |
| FormalParameterNamingConventions | The method parameter name '_strType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 305 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 325–328 |
| FormalParameterNamingConventions | The method parameter name '_bIsModule' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 336 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–43 |
| CommentRequired | Public method and constructor comments are required | 3 | 38 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getName': the method is declared in an interface type | 3 | 38 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'setName': the method is declared in an interface type | 3 | 39 |
| CommentRequired | Public method and constructor comments are required | 3 | 39 |
| CommentRequired | Public method and constructor comments are required | 3 | 40 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getType': the method is declared in an interface type | 3 | 40 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'setType': the method is declared in an interface type | 3 | 41 |
| CommentRequired | Public method and constructor comments are required | 3 | 41 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 44–87 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 3 | 57–60 |
| MissingOverride | The method 'setName(String)' is missing an @Override annotation. | 3 | 66–69 |
| MissingOverride | The method 'getType()' is missing an @Override annotation. | 3 | 74–77 |
| MissingOverride | The method 'setType(String)' is missing an @Override annotation. | 3 | 83–86 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 44–85 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 3 | 55–58 |
| MissingOverride | The method 'setName(String)' is missing an @Override annotation. | 3 | 64–67 |
| MissingOverride | The method 'getType()' is missing an @Override annotation. | 3 | 72–75 |
| MissingOverride | The method 'setType(String)' is missing an @Override annotation. | 3 | 81–84 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 44–86 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 3 | 56–59 |
| MissingOverride | The method 'setName(String)' is missing an @Override annotation. | 3 | 65–68 |
| MissingOverride | The method 'getType()' is missing an @Override annotation. | 3 | 73–76 |
| MissingOverride | The method 'setType(String)' is missing an @Override annotation. | 3 | 82–85 |
