Fork me on GitHub

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 1

fr/paris/lutece/plugins/pluginwizard/business/model/PluginModel.java

Rule Violation Line
MethodNamingConventions The instance method name 'BusinessClass' doesn't match '[a-z][a-zA-Z0-9]*' 626629

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminJspGenerator.java

Rule Violation Line
FieldNamingConventions The static field name '_jsp_prefix' doesn't match '_[a-z][a-zA-Z0-9]*' 53

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminTemplateGenerator.java

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]*' 120

fr/paris/lutece/plugins/pluginwizard/service/generator/GenerationScheme.java

Rule Violation Line
FormalParameterNamingConventions The method parameter name '_bIsDefault' doesn't match '[a-z][a-zA-Z0-9]*' 129

fr/paris/lutece/plugins/pluginwizard/service/generator/Markers.java

Rule Violation Line
ClassNamingConventions The utility class name 'Markers' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 3989

fr/paris/lutece/plugins/pluginwizard/service/generator/PluginXmlGenerator.java

Rule Violation Line
LocalVariableNamingConventions The local variable name 'BusinessNames' doesn't match '[a-z][a-zA-Z0-9]*' 87

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletJspBeanGenerator.java

Rule Violation Line
LocalVariableNamingConventions The local variable name '_path' doesn't match '[a-z][a-zA-Z0-9]*' 60

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletJspGenerator.java

Rule Violation Line
LocalVariableNamingConventions The local variable name '_path' doesn't match '[a-z][a-zA-Z0-9]*' 64

fr/paris/lutece/plugins/pluginwizard/service/generator/XPageTemplateGenerator.java

Rule Violation Line
FieldNamingConventions The static field name '_template_prefix' doesn't match '_[a-z][a-zA-Z0-9]*' 57

fr/paris/lutece/plugins/pluginwizard/util/Utils.java

Rule Violation Line
ClassNamingConventions The utility class name 'Utils' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 3997

fr/paris/lutece/plugins/pluginwizard/web/Constants.java

Rule Violation Line
ClassNamingConventions The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 4176

fr/paris/lutece/plugins/pluginwizard/web/formbean/DescriptionFormBean.java

Rule Violation Line
FormalParameterNamingConventions The method parameter name '_strType' doesn't match '[a-z][a-zA-Z0-9]*' 304
FormalParameterNamingConventions The method parameter name '_bIsModule' doesn't match '[a-z][a-zA-Z0-9]*' 335

Priority 3

fr/paris/lutece/plugins/pluginwizard/business/ConfigurationKeyDAO.java

Rule Violation Line
MissingOverride The method 'insert(ConfigurationKey, Plugin)' is missing an @Override annotation. 91105
MissingOverride The method 'load(int, Plugin)' is missing an @Override annotation. 116138
MissingOverride The method 'delete(int, Plugin)' is missing an @Override annotation. 148156
MissingOverride The method 'store(ConfigurationKey, Plugin)' is missing an @Override annotation. 166179
MissingOverride The method 'selectConfigurationKeysList(Plugin)' is missing an @Override annotation. 188210

fr/paris/lutece/plugins/pluginwizard/business/model/Application.java

Rule Violation Line
ConfusingTernary Avoid if (x != y) ..; else ..; 132139
ConfusingTernary Avoid if (x != y) ..; else ..; 150157
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 156

fr/paris/lutece/plugins/pluginwizard/business/model/Attribute.java

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

fr/paris/lutece/plugins/pluginwizard/business/model/BusinessClass.java

Rule Violation Line
ConfusingTernary Avoid if (x != y) ..; else ..; 142149
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 148
ConfusingTernary Avoid if (x != y) ..; else ..; 159166

fr/paris/lutece/plugins/pluginwizard/business/model/Feature.java

Rule Violation Line
ConfusingTernary Avoid if (x != y) ..; else ..; 229236
ConfusingTernary Avoid if (x != y) ..; else ..; 247254
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 253

fr/paris/lutece/plugins/pluginwizard/business/model/PluginModel.java

Rule Violation Line
ExcessivePublicCount This class has a bunch of public methods and attributes 34684
TooManyFields Too many fields 51684
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 195
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 199
ConfusingTernary Avoid if (x != y) ..; else ..; 514521
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 520
ConfusingTernary Avoid if (x != y) ..; else ..; 563570
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 569
ConfusingTernary Avoid if (x != y) ..; else ..; 591598
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 597
CommentRequired Public method and constructor comments are required 621624
CommentRequired Public method and constructor comments are required 626629

fr/paris/lutece/plugins/pluginwizard/business/model/PluginModelDAO.java

Rule Violation Line
MissingOverride The method 'insert(PluginModel, Plugin)' is missing an @Override annotation. 93118
MissingOverride The method 'load(int, Plugin)' is missing an @Override annotation. 129164
MissingOverride The method 'delete(int, Plugin)' is missing an @Override annotation. 174182
MissingOverride The method 'store(PluginModel, Plugin)' is missing an @Override annotation. 192216
MissingOverride The method 'selectPluginModelsList(Plugin)' is missing an @Override annotation. 225258
MissingOverride The method 'selectPluginModelByName(Plugin, String)' is missing an @Override annotation. 269287
MissingOverride The method 'pluginExists(String, Plugin)' is missing an @Override annotation. 298321

fr/paris/lutece/plugins/pluginwizard/business/model/Rest.java

Rule Violation Line
ConfusingTernary Avoid if (x != y) ..; else ..; 8188
ConfusingTernary Avoid if (x != y) ..; else ..; 99106
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 105

fr/paris/lutece/plugins/pluginwizard/service/AttributeService.java

Rule Violation Line
ConfusingTernary Avoid if (x != y) ..; else ..; 6067
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 66

fr/paris/lutece/plugins/pluginwizard/service/MapperService.java

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

fr/paris/lutece/plugins/pluginwizard/service/ModelService.java

Rule Violation Line
ExcessiveClassLength Avoid really long classes. 631269
GodClass Possible God Class (WMC=136, ATFD=143, TCC=1.714%) 631269
CyclomaticComplexity The class 'ModelService' has a total cyclomatic complexity of 136 (highest 7). 631269
TooManyMethods This class has too many methods, consider refactoring it. 641269
ShortVariable Avoid variables with short names like ID 67
ShortVariable Avoid variables with short names like pm 87
ShortVariable Avoid variables with short names like pm 105
ConfusingTernary Avoid if (x != y) ..; else ..; 108116
ShortVariable Avoid variables with short names like pm 127
ConfusingTernary Avoid if (x != y) ..; else ..; 136143
ShortVariable Avoid variables with short names like pm 153
ShortVariable Avoid variables with short names like pm 191
ShortVariable Avoid variables with short names like pm 211
ShortVariable Avoid variables with short names like pm 231
ShortVariable Avoid variables with short names like pm 256
ShortVariable Avoid variables with short names like f 261
ShortVariable Avoid variables with short names like pm 285
ShortVariable Avoid variables with short names like f 290
ShortVariable Avoid variables with short names like pm 318
ShortVariable Avoid variables with short names like pm 332
ShortVariable Avoid variables with short names like pm 355
ShortVariable Avoid variables with short names like pm 371
ShortVariable Avoid variables with short names like pm 396
ShortVariable Avoid variables with short names like pm 425
ShortVariable Avoid variables with short names like f 430
ShortVariable Avoid variables with short names like pm 458
ShortVariable Avoid variables with short names like pm 481
ShortVariable Avoid variables with short names like pm 496
ShortVariable Avoid variables with short names like pm 521
ShortVariable Avoid variables with short names like p 526
ShortVariable Avoid variables with short names like pm 550
ShortVariable Avoid variables with short names like p 555
ShortVariable Avoid variables with short names like pm 581
ShortVariable Avoid variables with short names like pm 595
ShortVariable Avoid variables with short names like bc 617
ShortVariable Avoid variables with short names like pm 619
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 619
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 637644
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 641
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 643
ShortVariable Avoid variables with short names like pm 662
ShortVariable Avoid variables with short names like pm 687
ShortVariable Avoid variables with short names like bc 692
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 698
ShortVariable Avoid variables with short names like pm 735
ShortVariable Avoid variables with short names like bc 740
ShortVariable Avoid variables with short names like pm 761
ShortVariable Avoid variables with short names like pm 784
ShortVariable Avoid variables with short names like bc 785
ConfusingTernary Avoid if (x != y) ..; else ..; 786803
ShortVariable Avoid variables with short names like pm 818
ShortVariable Avoid variables with short names like bc 819
ShortVariable Avoid variables with short names like pm 867
ShortVariable Avoid variables with short names like bc 868
ShortVariable Avoid variables with short names like pm 901
ShortVariable Avoid variables with short names like bc 902
ShortVariable Avoid variables with short names like pm 936
ShortVariable Avoid variables with short names like pm 950
ShortVariable Avoid variables with short names like pm 964
ShortVariable Avoid variables with short names like pm 976
ShortVariable Avoid variables with short names like pm 1004
ShortVariable Avoid variables with short names like pm 1038
ShortVariable Avoid variables with short names like pm 1072
ShortVariable Avoid variables with short names like pm 1107
CommentRequired Public method and constructor comments are required 12251239
ShortVariable Avoid variables with short names like pm 1227
CommentRequired Public method and constructor comments are required 12411254
ShortVariable Avoid variables with short names like pm 1243
CommentRequired Public method and constructor comments are required 12561268
ShortVariable Avoid variables with short names like pm 1258

fr/paris/lutece/plugins/pluginwizard/service/PluginWizardJsonService.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as '_plugin'. 50
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 5765

fr/paris/lutece/plugins/pluginwizard/service/PluginWizardZipService.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 6674
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

fr/paris/lutece/plugins/pluginwizard/service/QualityService.java

Rule Violation Line
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. 4466

fr/paris/lutece/plugins/pluginwizard/service/generator/AbstractFileGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/AbstractGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 108
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 136
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 136
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strValueCap' 138
ShortVariable Avoid variables with short names like pm 148
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 150
ShortVariable Avoid variables with short names like pm 163
ShortVariable Avoid variables with short names like pm 177

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminJspBeanGenerator.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 6575
ConfusingTernary Avoid if (x != y) ..; else ..; 6774
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 73
ShortVariable Avoid variables with short names like pm 91
CyclomaticComplexity The method 'generate(PluginModel, String)' has a cyclomatic complexity of 12. 91137
MissingOverride The method 'generate(PluginModel, String)' is missing an @Override annotation. 91137
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 152153
ShortVariable Avoid variables with short names like pm 152
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 180181
ShortVariable Avoid variables with short names like pm 180
ShortVariable Avoid variables with short names like pm 205
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 205

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminJspControllerGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminJspGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 85
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 137
AddEmptyString Do not add empty strings 141
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 144
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 162

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminTemplateGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 68
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 89
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 97
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 123
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 123
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 159
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 159

fr/paris/lutece/plugins/pluginwizard/service/generator/BusinessClassGenerator.java

Rule Violation Line
ConfusingTernary Avoid if (x != y) ..; else ..; 6370
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 69
ShortVariable Avoid variables with short names like pm 77
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 117
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 119

fr/paris/lutece/plugins/pluginwizard/service/generator/BusinessKotlinClassGenerator.java

Rule Violation Line
ConfusingTernary Avoid if (x != y) ..; else ..; 6269
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

fr/paris/lutece/plugins/pluginwizard/service/generator/GenerationScheme.java

Rule Violation Line
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsDefault' 47
ConfusingTernary Avoid if (x != y) ..; else ..; 8895
ConfusingTernary Avoid if (x != y) ..; else ..; 104111
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 110

fr/paris/lutece/plugins/pluginwizard/service/generator/GeneratorService.java

Rule Violation Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 67
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 79

fr/paris/lutece/plugins/pluginwizard/service/generator/PluginXmlGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 60
ShortVariable Avoid variables with short names like pm 69
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 71
ShortVariable Avoid variables with short names like pm 79
CommentRequired Public method and constructor comments are required 79116
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 85
ShortVariable Avoid variables with short names like i 88
ShortVariable Avoid variables with short names like id 89
ShortVariable Avoid variables with short names like pm 122
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 124

fr/paris/lutece/plugins/pluginwizard/service/generator/PomGenerator.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'KEY_PREFIX'. 53
ShortVariable Avoid variables with short names like pm 64
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
ShortVariable Avoid variables with short names like pm 101

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletGenerator.java

Rule Violation Line
ConfusingTernary Avoid if (x != y) ..; else ..; 6269
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

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletJspBeanGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletJspGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletTemplateGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletXslGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/PropertiesGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 51
ShortVariable Avoid variables with short names like pm 60
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 62

fr/paris/lutece/plugins/pluginwizard/service/generator/ResourcesGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 73
ShortVariable Avoid variables with short names like pm 101
ShortVariable Avoid variables with short names like sb 103
ShortVariable Avoid variables with short names like pm 122
ShortVariable Avoid variables with short names like sb 122
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 127
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 127
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 128
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 132
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 132
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 133
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 136
ShortVariable Avoid variables with short names like pm 147
ShortVariable Avoid variables with short names like sb 147
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 155
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 155
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 157
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 160
ShortVariable Avoid variables with short names like pm 164
ShortVariable Avoid variables with short names like sb 164
AvoidDuplicateLiterals The String literal "xpage." appears 4 times in this file; the first occurrence is on line 175 175
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 176
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 176
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 178
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 188
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 189
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 189
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 189
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 190
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 191
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 191
ShortVariable Avoid variables with short names like sb 209
ShortVariable Avoid variables with short names like pm 209
CyclomaticComplexity The method 'generateBusinessClassKeys(StringBuilder, PluginModel, String)' has a cyclomatic complexity of 11. 209278
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 218
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 220
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 221
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 222
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 223
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 227
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 227
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 232
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 233
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 234
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 238
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 238
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 244
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 245
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 245
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 250
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 259
AddEmptyString Do not add empty strings 265
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 265
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 271
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 275
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 275
ShortVariable Avoid variables with short names like sb 288
ShortVariable Avoid variables with short names like pm 288
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 296
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 297
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 300
ShortVariable Avoid variables with short names like sb 312
ShortVariable Avoid variables with short names like pm 312
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 322
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 322
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 322
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 323
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 323
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 327

fr/paris/lutece/plugins/pluginwizard/service/generator/RestConstantsGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 60
ShortVariable Avoid variables with short names like pm 85

fr/paris/lutece/plugins/pluginwizard/service/generator/RestGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 60
ShortVariable Avoid variables with short names like pm 89

fr/paris/lutece/plugins/pluginwizard/service/generator/RestSwaggerGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 60
ShortVariable Avoid variables with short names like pm 85

fr/paris/lutece/plugins/pluginwizard/service/generator/RestSwaggerJsonGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 60
ShortVariable Avoid variables with short names like pm 85

fr/paris/lutece/plugins/pluginwizard/service/generator/SiteDocumentationGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/SpringContextXmlGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 56
ShortVariable Avoid variables with short names like pm 68
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 70

fr/paris/lutece/plugins/pluginwizard/service/generator/SqlCodeGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 60
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 68
ShortVariable Avoid variables with short names like pm 105
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 107

fr/paris/lutece/plugins/pluginwizard/service/generator/XPageGenerator.java

Rule Violation Line
ConfusingTernary Avoid if (x != y) ..; else ..; 7380
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 79
ShortVariable Avoid variables with short names like pm 89
CyclomaticComplexity The method 'generate(PluginModel, String)' has a cyclomatic complexity of 11. 89128
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 92
UnusedLocalVariable Avoid unused local variables such as 'strFilesPath'. 92
ShortVariable Avoid variables with short names like pm 145
ShortVariable Avoid variables with short names like pm 172

fr/paris/lutece/plugins/pluginwizard/service/generator/XPageTemplateGenerator.java

Rule Violation Line
ShortVariable Avoid variables with short names like pm 68
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 93
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 101
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 129
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 163
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 192

fr/paris/lutece/plugins/pluginwizard/util/Utils.java

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

fr/paris/lutece/plugins/pluginwizard/web/PluginWizardApp.java

Rule Violation Line
ExcessivePublicCount This class has a bunch of public methods and attributes 341507
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 341507
ExcessiveClassLength Avoid really long classes. 831507
TooManyMethods This class has too many methods, consider refactoring it. 841507
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 271274
CommentRequired Public method and constructor comments are required 385391
ConfusingTernary Avoid if (x != y) ..; else ..; 431
ShortVariable Avoid variables with short names like pm 520
ShortVariable Avoid variables with short names like pm 540
ConfusingTernary Avoid if (x != y) ..; else ..; 541
ShortVariable Avoid variables with short names like pm 561
ConfusingTernary Avoid if (x != y) ..; else ..; 592610
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 606
ConfusingTernary Avoid if (x != y) ..; else ..; 628646
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 642
ConfusingTernary Avoid if (x != y) ..; else ..; 715
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 742
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 773
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 801
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 942
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 969
CommentRequired Public method and constructor comments are required 10141017
ShortVariable Avoid variables with short names like pm 1049
ConfusingTernary Avoid if (x != y) ..; else ..; 1050
ShortVariable Avoid variables with short names like pm 1070
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 1108
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 1135
ConfusingTernary Avoid if (x != y) ..; else ..; 1207
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 1256
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 1280
ShortVariable Avoid variables with short names like pm 1332
ShortVariable Avoid variables with short names like e 1408

fr/paris/lutece/plugins/pluginwizard/web/PluginwizardJspBean.java

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
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 156
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 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
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 239
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 239
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 259

fr/paris/lutece/plugins/pluginwizard/web/formbean/BusinessClassFormBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 44183

fr/paris/lutece/plugins/pluginwizard/web/formbean/DescriptionFormBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 47339
UncommentedEmptyConstructor Document empty constructor 7577
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 7577

fr/paris/lutece/plugins/pluginwizard/web/formbean/PluginNameFormBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 4466

Priority 4

fr/paris/lutece/plugins/pluginwizard/business/model/Attribute.java

Rule Violation Line
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 226229

fr/paris/lutece/plugins/pluginwizard/business/model/PluginModel.java

Rule Violation Line
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 668671

fr/paris/lutece/plugins/pluginwizard/business/model/Rest.java

Rule Violation Line
ShortClassName Avoid short class names like Rest 43108

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminJspControllerGenerator.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.pluginwizard.util.Utils' 40

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminTemplateGenerator.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.pluginwizard.util.Utils' 40

fr/paris/lutece/plugins/pluginwizard/service/generator/PomGenerator.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.portal.service.datastore.DatastoreService' 39
UnusedImports Avoid unused imports such as 'fr.paris.lutece.util.ReferenceList' 40

fr/paris/lutece/plugins/pluginwizard/service/generator/XPageTemplateGenerator.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.pluginwizard.util.Utils' 40

fr/paris/lutece/plugins/pluginwizard/web/formbean/DescriptionFormBean.java

Rule Violation Line
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 324327

Files

fr/paris/lutece/plugins/pluginwizard/business/ConfigurationKeyDAO.java

Rule Violation Priority Line
MissingOverride The method 'insert(ConfigurationKey, Plugin)' is missing an @Override annotation. 3 91105
MissingOverride The method 'load(int, Plugin)' is missing an @Override annotation. 3 116138
MissingOverride The method 'delete(int, Plugin)' is missing an @Override annotation. 3 148156
MissingOverride The method 'store(ConfigurationKey, Plugin)' is missing an @Override annotation. 3 166179
MissingOverride The method 'selectConfigurationKeysList(Plugin)' is missing an @Override annotation. 3 188210

fr/paris/lutece/plugins/pluginwizard/business/model/Application.java

Rule Violation Priority Line
ConfusingTernary Avoid if (x != y) ..; else ..; 3 132139
ConfusingTernary Avoid if (x != y) ..; else ..; 3 150157
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 156

fr/paris/lutece/plugins/pluginwizard/business/model/Attribute.java

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 226229

fr/paris/lutece/plugins/pluginwizard/business/model/BusinessClass.java

Rule Violation Priority Line
ConfusingTernary Avoid if (x != y) ..; else ..; 3 142149
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 148
ConfusingTernary Avoid if (x != y) ..; else ..; 3 159166

fr/paris/lutece/plugins/pluginwizard/business/model/Feature.java

Rule Violation Priority Line
ConfusingTernary Avoid if (x != y) ..; else ..; 3 229236
ConfusingTernary Avoid if (x != y) ..; else ..; 3 247254
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 253

fr/paris/lutece/plugins/pluginwizard/business/model/PluginModel.java

Rule Violation Priority Line
ExcessivePublicCount This class has a bunch of public methods and attributes 3 34684
TooManyFields Too many fields 3 51684
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 195
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 199
ConfusingTernary Avoid if (x != y) ..; else ..; 3 514521
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 520
ConfusingTernary Avoid if (x != y) ..; else ..; 3 563570
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 569
ConfusingTernary Avoid if (x != y) ..; else ..; 3 591598
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 597
CommentRequired Public method and constructor comments are required 3 621624
CommentRequired Public method and constructor comments are required 3 626629
MethodNamingConventions The instance method name 'BusinessClass' doesn't match '[a-z][a-zA-Z0-9]*' 1 626629
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 668671

fr/paris/lutece/plugins/pluginwizard/business/model/PluginModelDAO.java

Rule Violation Priority Line
MissingOverride The method 'insert(PluginModel, Plugin)' is missing an @Override annotation. 3 93118
MissingOverride The method 'load(int, Plugin)' is missing an @Override annotation. 3 129164
MissingOverride The method 'delete(int, Plugin)' is missing an @Override annotation. 3 174182
MissingOverride The method 'store(PluginModel, Plugin)' is missing an @Override annotation. 3 192216
MissingOverride The method 'selectPluginModelsList(Plugin)' is missing an @Override annotation. 3 225258
MissingOverride The method 'selectPluginModelByName(Plugin, String)' is missing an @Override annotation. 3 269287
MissingOverride The method 'pluginExists(String, Plugin)' is missing an @Override annotation. 3 298321

fr/paris/lutece/plugins/pluginwizard/business/model/Rest.java

Rule Violation Priority Line
ShortClassName Avoid short class names like Rest 4 43108
ConfusingTernary Avoid if (x != y) ..; else ..; 3 8188
ConfusingTernary Avoid if (x != y) ..; else ..; 3 99106
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 105

fr/paris/lutece/plugins/pluginwizard/service/AttributeService.java

Rule Violation Priority Line
ConfusingTernary Avoid if (x != y) ..; else ..; 3 6067
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 66

fr/paris/lutece/plugins/pluginwizard/service/MapperService.java

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

fr/paris/lutece/plugins/pluginwizard/service/ModelService.java

Rule Violation Priority Line
ExcessiveClassLength Avoid really long classes. 3 631269
GodClass Possible God Class (WMC=136, ATFD=143, TCC=1.714%) 3 631269
CyclomaticComplexity The class 'ModelService' has a total cyclomatic complexity of 136 (highest 7). 3 631269
TooManyMethods This class has too many methods, consider refactoring it. 3 641269
ShortVariable Avoid variables with short names like ID 3 67
ShortVariable Avoid variables with short names like pm 3 87
ShortVariable Avoid variables with short names like pm 3 105
ConfusingTernary Avoid if (x != y) ..; else ..; 3 108116
ShortVariable Avoid variables with short names like pm 3 127
ConfusingTernary Avoid if (x != y) ..; else ..; 3 136143
ShortVariable Avoid variables with short names like pm 3 153
ShortVariable Avoid variables with short names like pm 3 191
ShortVariable Avoid variables with short names like pm 3 211
ShortVariable Avoid variables with short names like pm 3 231
ShortVariable Avoid variables with short names like pm 3 256
ShortVariable Avoid variables with short names like f 3 261
ShortVariable Avoid variables with short names like pm 3 285
ShortVariable Avoid variables with short names like f 3 290
ShortVariable Avoid variables with short names like pm 3 318
ShortVariable Avoid variables with short names like pm 3 332
ShortVariable Avoid variables with short names like pm 3 355
ShortVariable Avoid variables with short names like pm 3 371
ShortVariable Avoid variables with short names like pm 3 396
ShortVariable Avoid variables with short names like pm 3 425
ShortVariable Avoid variables with short names like f 3 430
ShortVariable Avoid variables with short names like pm 3 458
ShortVariable Avoid variables with short names like pm 3 481
ShortVariable Avoid variables with short names like pm 3 496
ShortVariable Avoid variables with short names like pm 3 521
ShortVariable Avoid variables with short names like p 3 526
ShortVariable Avoid variables with short names like pm 3 550
ShortVariable Avoid variables with short names like p 3 555
ShortVariable Avoid variables with short names like pm 3 581
ShortVariable Avoid variables with short names like pm 3 595
ShortVariable Avoid variables with short names like bc 3 617
ShortVariable Avoid variables with short names like pm 3 619
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 619
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 637644
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 641
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 643
ShortVariable Avoid variables with short names like pm 3 662
ShortVariable Avoid variables with short names like pm 3 687
ShortVariable Avoid variables with short names like bc 3 692
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 698
ShortVariable Avoid variables with short names like pm 3 735
ShortVariable Avoid variables with short names like bc 3 740
ShortVariable Avoid variables with short names like pm 3 761
ShortVariable Avoid variables with short names like pm 3 784
ShortVariable Avoid variables with short names like bc 3 785
ConfusingTernary Avoid if (x != y) ..; else ..; 3 786803
ShortVariable Avoid variables with short names like pm 3 818
ShortVariable Avoid variables with short names like bc 3 819
ShortVariable Avoid variables with short names like pm 3 867
ShortVariable Avoid variables with short names like bc 3 868
ShortVariable Avoid variables with short names like pm 3 901
ShortVariable Avoid variables with short names like bc 3 902
ShortVariable Avoid variables with short names like pm 3 936
ShortVariable Avoid variables with short names like pm 3 950
ShortVariable Avoid variables with short names like pm 3 964
ShortVariable Avoid variables with short names like pm 3 976
ShortVariable Avoid variables with short names like pm 3 1004
ShortVariable Avoid variables with short names like pm 3 1038
ShortVariable Avoid variables with short names like pm 3 1072
ShortVariable Avoid variables with short names like pm 3 1107
CommentRequired Public method and constructor comments are required 3 12251239
ShortVariable Avoid variables with short names like pm 3 1227
CommentRequired Public method and constructor comments are required 3 12411254
ShortVariable Avoid variables with short names like pm 3 1243
CommentRequired Public method and constructor comments are required 3 12561268
ShortVariable Avoid variables with short names like pm 3 1258

fr/paris/lutece/plugins/pluginwizard/service/PluginWizardJsonService.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as '_plugin'. 3 50
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 5765

fr/paris/lutece/plugins/pluginwizard/service/PluginWizardZipService.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 6674
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

fr/paris/lutece/plugins/pluginwizard/service/QualityService.java

Rule Violation Priority Line
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 4466

fr/paris/lutece/plugins/pluginwizard/service/generator/AbstractFileGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/AbstractGenerator.java

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 136
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 136
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strValueCap' 3 138
ShortVariable Avoid variables with short names like pm 3 148
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 150
ShortVariable Avoid variables with short names like pm 3 163
ShortVariable Avoid variables with short names like pm 3 177

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminJspBeanGenerator.java

Rule Violation Priority Line
CommentRequired Public method and constructor comments are required 3 6575
ConfusingTernary Avoid if (x != y) ..; else ..; 3 6774
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 73
ShortVariable Avoid variables with short names like pm 3 91
CyclomaticComplexity The method 'generate(PluginModel, String)' has a cyclomatic complexity of 12. 3 91137
MissingOverride The method 'generate(PluginModel, String)' is missing an @Override annotation. 3 91137
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 152153
ShortVariable Avoid variables with short names like pm 3 152
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 180181
ShortVariable Avoid variables with short names like pm 3 180
ShortVariable Avoid variables with short names like pm 3 205
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 205

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminJspControllerGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminJspGenerator.java

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
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 137
AddEmptyString Do not add empty strings 3 141
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 144
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 162

fr/paris/lutece/plugins/pluginwizard/service/generator/AdminTemplateGenerator.java

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
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 89
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 97
FormalParameterNamingConventions The method parameter name 'admin_feature' doesn't match '[a-z][a-zA-Z0-9]*' 1 120
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 123
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 123
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 159
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 159

fr/paris/lutece/plugins/pluginwizard/service/generator/BusinessClassGenerator.java

Rule Violation Priority Line
ConfusingTernary Avoid if (x != y) ..; else ..; 3 6370
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 69
ShortVariable Avoid variables with short names like pm 3 77
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 117
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 119

fr/paris/lutece/plugins/pluginwizard/service/generator/BusinessKotlinClassGenerator.java

Rule Violation Priority Line
ConfusingTernary Avoid if (x != y) ..; else ..; 3 6269
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

fr/paris/lutece/plugins/pluginwizard/service/generator/GenerationScheme.java

Rule Violation Priority Line
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsDefault' 3 47
ConfusingTernary Avoid if (x != y) ..; else ..; 3 8895
ConfusingTernary Avoid if (x != y) ..; else ..; 3 104111
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 110
FormalParameterNamingConventions The method parameter name '_bIsDefault' doesn't match '[a-z][a-zA-Z0-9]*' 1 129

fr/paris/lutece/plugins/pluginwizard/service/generator/GeneratorService.java

Rule Violation Priority Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 67
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 79

fr/paris/lutece/plugins/pluginwizard/service/generator/Markers.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'Markers' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 3989

fr/paris/lutece/plugins/pluginwizard/service/generator/PluginXmlGenerator.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like pm 3 60
ShortVariable Avoid variables with short names like pm 3 69
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 71
ShortVariable Avoid variables with short names like pm 3 79
CommentRequired Public method and constructor comments are required 3 79116
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 3 85
LocalVariableNamingConventions The local variable name 'BusinessNames' doesn't match '[a-z][a-zA-Z0-9]*' 1 87
ShortVariable Avoid variables with short names like i 3 88
ShortVariable Avoid variables with short names like id 3 89
ShortVariable Avoid variables with short names like pm 3 122
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 124

fr/paris/lutece/plugins/pluginwizard/service/generator/PomGenerator.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.portal.service.datastore.DatastoreService' 4 39
UnusedImports Avoid unused imports such as 'fr.paris.lutece.util.ReferenceList' 4 40
UnusedPrivateField Avoid unused private fields such as 'KEY_PREFIX'. 3 53
ShortVariable Avoid variables with short names like pm 3 64
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
ShortVariable Avoid variables with short names like pm 3 101

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletGenerator.java

Rule Violation Priority Line
ConfusingTernary Avoid if (x != y) ..; else ..; 3 6269
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

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletJspBeanGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletJspGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletTemplateGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/PortletXslGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/PropertiesGenerator.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like pm 3 51
ShortVariable Avoid variables with short names like pm 3 60
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 62

fr/paris/lutece/plugins/pluginwizard/service/generator/ResourcesGenerator.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like pm 3 73
ShortVariable Avoid variables with short names like pm 3 101
ShortVariable Avoid variables with short names like sb 3 103
ShortVariable Avoid variables with short names like pm 3 122
ShortVariable Avoid variables with short names like sb 3 122
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 127
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 127
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 128
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 132
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 132
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 133
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 136
ShortVariable Avoid variables with short names like pm 3 147
ShortVariable Avoid variables with short names like sb 3 147
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 155
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 155
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 157
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 160
ShortVariable Avoid variables with short names like pm 3 164
ShortVariable Avoid variables with short names like sb 3 164
AvoidDuplicateLiterals The String literal "xpage." appears 4 times in this file; the first occurrence is on line 175 3 175
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 176
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 176
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 178
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 188
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 189
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 189
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 189
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 190
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 191
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 191
ShortVariable Avoid variables with short names like sb 3 209
ShortVariable Avoid variables with short names like pm 3 209
CyclomaticComplexity The method 'generateBusinessClassKeys(StringBuilder, PluginModel, String)' has a cyclomatic complexity of 11. 3 209278
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 218
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 220
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 221
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 222
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 223
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 227
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 227
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 232
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 233
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 234
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 238
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 238
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 244
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 245
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 245
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 250
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 259
AddEmptyString Do not add empty strings 3 265
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 265
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 271
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 275
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 275
ShortVariable Avoid variables with short names like sb 3 288
ShortVariable Avoid variables with short names like pm 3 288
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 296
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 297
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 300
ShortVariable Avoid variables with short names like sb 3 312
ShortVariable Avoid variables with short names like pm 3 312
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 322
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 322
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 322
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 323
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 323
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 327

fr/paris/lutece/plugins/pluginwizard/service/generator/RestConstantsGenerator.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like pm 3 60
ShortVariable Avoid variables with short names like pm 3 85

fr/paris/lutece/plugins/pluginwizard/service/generator/RestGenerator.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like pm 3 60
ShortVariable Avoid variables with short names like pm 3 89

fr/paris/lutece/plugins/pluginwizard/service/generator/RestSwaggerGenerator.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like pm 3 60
ShortVariable Avoid variables with short names like pm 3 85

fr/paris/lutece/plugins/pluginwizard/service/generator/RestSwaggerJsonGenerator.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like pm 3 60
ShortVariable Avoid variables with short names like pm 3 85

fr/paris/lutece/plugins/pluginwizard/service/generator/SiteDocumentationGenerator.java

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

fr/paris/lutece/plugins/pluginwizard/service/generator/SpringContextXmlGenerator.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like pm 3 56
ShortVariable Avoid variables with short names like pm 3 68
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 70

fr/paris/lutece/plugins/pluginwizard/service/generator/SqlCodeGenerator.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like pm 3 60
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 68
ShortVariable Avoid variables with short names like pm 3 105
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 107

fr/paris/lutece/plugins/pluginwizard/service/generator/XPageGenerator.java

Rule Violation Priority Line
ConfusingTernary Avoid if (x != y) ..; else ..; 3 7380
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 79
ShortVariable Avoid variables with short names like pm 3 89
CyclomaticComplexity The method 'generate(PluginModel, String)' has a cyclomatic complexity of 11. 3 89128
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 92
UnusedLocalVariable Avoid unused local variables such as 'strFilesPath'. 3 92
ShortVariable Avoid variables with short names like pm 3 145
ShortVariable Avoid variables with short names like pm 3 172

fr/paris/lutece/plugins/pluginwizard/service/generator/XPageTemplateGenerator.java

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
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 93
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 101
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 129
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 163
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 192

fr/paris/lutece/plugins/pluginwizard/util/Utils.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'Utils' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 3997
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

fr/paris/lutece/plugins/pluginwizard/web/Constants.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 4176

fr/paris/lutece/plugins/pluginwizard/web/PluginWizardApp.java

Rule Violation Priority Line
ExcessivePublicCount This class has a bunch of public methods and attributes 3 341507
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 341507
ExcessiveClassLength Avoid really long classes. 3 831507
TooManyMethods This class has too many methods, consider refactoring it. 3 841507
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 271274
CommentRequired Public method and constructor comments are required 3 385391
ConfusingTernary Avoid if (x != y) ..; else ..; 3 431
ShortVariable Avoid variables with short names like pm 3 520
ShortVariable Avoid variables with short names like pm 3 540
ConfusingTernary Avoid if (x != y) ..; else ..; 3 541
ShortVariable Avoid variables with short names like pm 3 561
ConfusingTernary Avoid if (x != y) ..; else ..; 3 592610
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 606
ConfusingTernary Avoid if (x != y) ..; else ..; 3 628646
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 642
ConfusingTernary Avoid if (x != y) ..; else ..; 3 715
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 742
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 773
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 801
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 942
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 969
CommentRequired Public method and constructor comments are required 3 10141017
ShortVariable Avoid variables with short names like pm 3 1049
ConfusingTernary Avoid if (x != y) ..; else ..; 3 1050
ShortVariable Avoid variables with short names like pm 3 1070
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 1108
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 1135
ConfusingTernary Avoid if (x != y) ..; else ..; 3 1207
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 1256
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 1280
ShortVariable Avoid variables with short names like pm 3 1332
ShortVariable Avoid variables with short names like e 3 1408

fr/paris/lutece/plugins/pluginwizard/web/PluginwizardJspBean.java

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
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 156
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 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
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 239
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 239
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 259

fr/paris/lutece/plugins/pluginwizard/web/formbean/BusinessClassFormBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 44183

fr/paris/lutece/plugins/pluginwizard/web/formbean/DescriptionFormBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 47339
UncommentedEmptyConstructor Document empty constructor 3 7577
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 7577
FormalParameterNamingConventions The method parameter name '_strType' doesn't match '[a-z][a-zA-Z0-9]*' 1 304
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 324327
FormalParameterNamingConventions The method parameter name '_bIsModule' doesn't match '[a-z][a-zA-Z0-9]*' 1 335

fr/paris/lutece/plugins/pluginwizard/web/formbean/PluginNameFormBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 4466