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/blog/business/portlet/BlogPublicationHome.java

Rule Violation Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 45240

fr/paris/lutece/plugins/blog/web/BlogJspBean.java

Rule Violation Line
FieldNamingConventions The final field name '_blogServiceSession' doesn't match '[a-z][a-zA-Z0-9]*' 266

Priority 2

fr/paris/lutece/plugins/blog/business/BlogDAO.java

Rule Violation Line
GuardLogStatement Logger calls should be surrounded by log level guards. 688
GuardLogStatement Logger calls should be surrounded by log level guards. 700
GuardLogStatement Logger calls should be surrounded by log level guards. 711

fr/paris/lutece/plugins/blog/service/docsearch/BlogSearchService.java

Rule Violation Line
AvoidUsingVolatile Use of modifier volatile is not recommended. 92

Priority 3

fr/paris/lutece/plugins/blog/business/Blog.java

Rule Violation Line
ExcessivePublicCount This class has a bunch of public methods and attributes 34641
TooManyFields Too many fields 54641
CommentRequired Public method and constructor comments are required 289292
CommentRequired Public method and constructor comments are required 294297

fr/paris/lutece/plugins/blog/business/BlogDAO.java

Rule Violation Line
TooManyMethods This class has too many methods, consider refactoring it. 51797
NPathComplexity The method 'getDaoFromFilter(String, BlogFilter)' has an NPath complexity of 266112 589717
CyclomaticComplexity The method 'getDaoFromFilter(String, BlogFilter)' has a cyclomatic complexity of 28. 589717
ExcessiveMethodLength Avoid really long methods. 589717
NcssCount The method 'getDaoFromFilter(String, BlogFilter)' has a NCSS line count of 61. 589717
ShortVariable Avoid variables with short names like i 598
ConfusingTernary Avoid if (x != y) ..; else ..; 620
ConfusingTernary Avoid if (x != y) ..; else ..; 638
ConfusingTernary Avoid if (x != y) ..; else ..; 643
ConfusingTernary Avoid if (x != y) ..; else ..; 648
ConfusingTernary Avoid if (x != y) ..; else ..; 654
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 654
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 655
ConfusingTernary Avoid if (x != y) ..; else ..; 661
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 661
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 667
ConfusingTernary Avoid if (x != y) ..; else ..; 667
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 675
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 681
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 685

fr/paris/lutece/plugins/blog/business/BlogFilter.java

Rule Violation Line
MethodReturnsInternalArray Returning '_arrayTagsId' may expose an internal array. 59
ArrayIsStoredDirectly The user-supplied array 'arrayTagsId' is stored directly. 66
MethodReturnsInternalArray Returning '_arrayId' may expose an internal array. 96
ArrayIsStoredDirectly The user-supplied array 'arrayId' is stored directly. 103

fr/paris/lutece/plugins/blog/business/BlogHome.java

Rule Violation Line
TooManyMethods This class has too many methods, consider refactoring it. 47289

fr/paris/lutece/plugins/blog/business/BlogPageTemplateDAO.java

Rule Violation Line
DefaultPackage Use explicit scoping instead of the default package private level 6375
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the newPrimaryKey method if you want a default access modifier 6375
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 8497

fr/paris/lutece/plugins/blog/business/BlogSearchFilter.java

Rule Violation Line
MethodReturnsInternalArray Returning '_strTag' may expose an internal array. 101
ArrayIsStoredDirectly The user-supplied array 'strTag' is stored directly. 110

fr/paris/lutece/plugins/blog/business/ContentType.java

Rule Violation Line
CommentRequired Header comments are required 3684

fr/paris/lutece/plugins/blog/business/DocContent.java

Rule Violation Line
ArrayIsStoredDirectly The user-supplied array 'bytes' is stored directly. 106
MethodReturnsInternalArray Returning '_bytes' may expose an internal array. 118

fr/paris/lutece/plugins/blog/business/DocContentDAO.java

Rule Violation Line
TooManyMethods This class has too many methods, consider refactoring it. 46293

fr/paris/lutece/plugins/blog/business/IBlogDAO.java

Rule Violation Line
TooManyMethods This class has too many methods, consider refactoring it. 45211
CommentRequired Public method and constructor comments are required 209

fr/paris/lutece/plugins/blog/business/IDocContentDAO.java

Rule Violation Line
CommentRequired Header comments are required 40145

fr/paris/lutece/plugins/blog/business/ITagDAO.java

Rule Violation Line
CommentRequired Header comments are required 41172
TooManyMethods This class has too many methods, consider refactoring it. 42172

fr/paris/lutece/plugins/blog/business/IndexerActionDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 8092

fr/paris/lutece/plugins/blog/business/IndexerActionHome.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 65113
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 121124

fr/paris/lutece/plugins/blog/business/Tag.java

Rule Violation Line
CommentRequired Header comments are required 40145
UncommentedEmptyConstructor Document empty constructor 5659
CommentRequired Public method and constructor comments are required 5659
CommentRequired Public method and constructor comments are required 6166

fr/paris/lutece/plugins/blog/business/TagDAO.java

Rule Violation Line
TooManyMethods This class has too many methods, consider refactoring it. 47326

fr/paris/lutece/plugins/blog/business/portlet/BlogListPortlet.java

Rule Violation Line
ImmutableField Private field '_removedBlogsId' could be made final; it is only initialized in the declaration or constructor. 76
CallSuperInConstructor It is a good practice to call super() in a constructor 8184
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 95

fr/paris/lutece/plugins/blog/business/portlet/BlogListPortletDAO.java

Rule Violation Line
TooManyMethods This class has too many methods, consider refactoring it. 52395
ShortVariable Avoid variables with short names like p 82
ShortVariable Avoid variables with short names like p 103
ShortVariable Avoid variables with short names like p 131
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 212
ShortVariable Avoid variables with short names like p 262
CyclomaticComplexity The method 'selectPortletByType(int, PortletOrder, PortletFilter)' has a cyclomatic complexity of 10. 320374
ConfusingTernary Avoid if (x != y) ..; else ..; 325

fr/paris/lutece/plugins/blog/business/portlet/BlogListPortletHome.java

Rule Violation Line
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 56
RedundantFieldInitializer Avoid using redundant field initializer for '_singleton' 56
CallSuperInConstructor It is a good practice to call super() in a constructor 6167
NonThreadSafeSingleton Singleton is not thread safe 7679

fr/paris/lutece/plugins/blog/business/portlet/BlogPortlet.java

Rule Violation Line
CallSuperInConstructor It is a good practice to call super() in a constructor 6669
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 71
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 73
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 75

fr/paris/lutece/plugins/blog/business/portlet/BlogPortletDAO.java

Rule Violation Line
ShortVariable Avoid variables with short names like p 71
ShortVariable Avoid variables with short names like p 109
ShortVariable Avoid variables with short names like p 160
CyclomaticComplexity The method 'selectPortletByType(int, PortletOrder, PortletFilter)' has a cyclomatic complexity of 10. 195245
ConfusingTernary Avoid if (x != y) ..; else ..; 200

fr/paris/lutece/plugins/blog/business/portlet/BlogPortletHome.java

Rule Violation Line
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 53
CallSuperInConstructor It is a good practice to call super() in a constructor 5864
NonThreadSafeSingleton Singleton is not thread safe 8588

fr/paris/lutece/plugins/blog/business/portlet/BlogPublication.java

Rule Violation Line
CommentRequired Header comments are required 41171

fr/paris/lutece/plugins/blog/business/portlet/BlogPublicationDAO.java

Rule Violation Line
CommentRequired Header comments are required 45434
TooManyMethods This class has too many methods, consider refactoring it. 46434

fr/paris/lutece/plugins/blog/business/portlet/BlogPublicationHome.java

Rule Violation Line
CommentRequired Header comments are required 45240

fr/paris/lutece/plugins/blog/business/portlet/IBlogPortletDAO.java

Rule Violation Line
MissingOverride The method 'load(int)' is missing an @Override annotation. 74

fr/paris/lutece/plugins/blog/business/portlet/IBlogPublicationDAO.java

Rule Violation Line
CommentRequired Header comments are required 42204
TooManyMethods This class has too many methods, consider refactoring it. 43204

fr/paris/lutece/plugins/blog/business/portlet/PortletFilter.java

Rule Violation Line
ArrayIsStoredDirectly The user-supplied array 'aPageTitle' is stored directly. 68
MethodReturnsInternalArray Returning '_aPageName' may expose an internal array. 80
ArrayIsStoredDirectly The user-supplied array 'aPortletTitle' is stored directly. 89
MethodReturnsInternalArray Returning '_aPortletName' may expose an internal array. 101
CyclomaticComplexity The method 'getSQLFilter()' has a cyclomatic complexity of 13. 172221

fr/paris/lutece/plugins/blog/business/rss/BlogResourceRss.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34315
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 248
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 297

fr/paris/lutece/plugins/blog/business/rss/BlogResourceRssConfigDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 5970

fr/paris/lutece/plugins/blog/service/BlogIdService.java

Rule Violation Line
CommentRequired Header comments are required 47116
ShortVariable Avoid variables with short names like rt 61
ShortVariable Avoid variables with short names like p 67

fr/paris/lutece/plugins/blog/service/BlogService.java

Rule Violation Line
GodClass Possible God Class (WMC=48, ATFD=85, TCC=0.000%) 58442
TooManyMethods This class has too many methods, consider refactoring it. 59442
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 117
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 158
ConfusingTernary Avoid if (x != y) ..; else ..; 175186
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 218
CollapsibleIfStatements These nested if statements could be combined 237241
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 279
CommentRequired Public method and constructor comments are required 416423
CommentRequired Public method and constructor comments are required 425432
CommentRequired Public method and constructor comments are required 434441

fr/paris/lutece/plugins/blog/service/BlogSessionListner.java

Rule Violation Line
ShortVariable Avoid variables with short names like se 57
ShortVariable Avoid variables with short names like se 66
CommentRequired Public method and constructor comments are required 7377
CommentRequired Public method and constructor comments are required 7984

fr/paris/lutece/plugins/blog/service/PublishingService.java

Rule Violation Line
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 309

fr/paris/lutece/plugins/blog/service/TagIdService.java

Rule Violation Line
CommentRequired Header comments are required 47110
ShortVariable Avoid variables with short names like rt 60
ShortVariable Avoid variables with short names like p 66

fr/paris/lutece/plugins/blog/service/TagService.java

Rule Violation Line
CommentRequired Header comments are required 4168

fr/paris/lutece/plugins/blog/service/UnlockedBlogDaemon.java

Rule Violation Line
CommentRequired Header comments are required 4057

fr/paris/lutece/plugins/blog/service/docsearch/BlogSearchItem.java

Rule Violation Line
DataClass The class 'BlogSearchItem' is suspected to be a Data Class (WOC=0.000%, NOPA=7, NOAM=0, WMC=1) 4386
CommentRequired Public method and constructor comments are required 4548
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 53
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 58
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 63
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 69
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 75
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 80
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 84

fr/paris/lutece/plugins/blog/service/docsearch/BlogSearchService.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34486
ImmutableField Private field '_indexer' could be made final; it is only initialized in the declaration or constructor. 102
ImmutableField Private field '_nWriterMergeFactor' could be made final; it is only initialized in the declaration or constructor. 103
NonThreadSafeSingleton Singleton is not thread safe 144147
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 147 characters appended. 196
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 202
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 243
ShortVariable Avoid variables with short names like si 307
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 312
NPathComplexity The method 'prepareQueryForFilter(BlogSearchFilter)' has an NPath complexity of 432 321410
CyclomaticComplexity The method 'prepareQueryForFilter(BlogSearchFilter)' has a cyclomatic complexity of 12. 321410
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 408
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 408
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 409

fr/paris/lutece/plugins/blog/service/docsearch/DefaultBlogIndexer.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34407
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 123194
ConfusingTernary Avoid if (x != y) ..; else ..; 128191
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 181
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 212
ShortVariable Avoid variables with short names like ft 217
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 275
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 276
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 277
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 278
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 279
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 280

fr/paris/lutece/plugins/blog/service/search/BlogSearchIndexer.java

Rule Violation Line
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 101

fr/paris/lutece/plugins/blog/utils/BlogLock.java

Rule Violation Line
CommentRequired Header comments are required 3875
CommentRequired Public method and constructor comments are required 4449
CommentRequired Public method and constructor comments are required 5155
CommentRequired Public method and constructor comments are required 5761
CommentRequired Public method and constructor comments are required 6367
CommentRequired Public method and constructor comments are required 6973

fr/paris/lutece/plugins/blog/utils/BlogUtils.java

Rule Violation Line
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 92
AssignmentInOperand Avoid assignments in operands 92

fr/paris/lutece/plugins/blog/web/BlogDashboardComponent.java

Rule Violation Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 108
ShortVariable Avoid variables with short names like t 119

fr/paris/lutece/plugins/blog/web/BlogFileServlet.java

Rule Violation Line
ShortVariable Avoid variables with short names like os 104

fr/paris/lutece/plugins/blog/web/BlogJspBean.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 341470
ExcessiveClassLength Avoid really long classes. 1171470
TooManyMethods This class has too many methods, consider refactoring it. 1181470
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 250
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsChecked' 252
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsUnpulished' 254
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 260
NcssCount The method 'getManageBlogs(HttpServletRequest)' has a NCSS line count of 85. 276423
ExcessiveMethodLength Avoid really long methods. 276423
CyclomaticComplexity The method 'getManageBlogs(HttpServletRequest)' has a cyclomatic complexity of 24. 276423
NPathComplexity The method 'getManageBlogs(HttpServletRequest)' has an NPath complexity of 6390 276423
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 278
ConfusingTernary Avoid if (x != y) ..; else ..; 292313
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 307
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 308
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 310
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 311
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 403
AddEmptyString Do not add empty strings 411
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 432
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 435
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 500
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 539
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 580
CyclomaticComplexity The method 'doUpdatePriorityTag(HttpServletRequest)' has a cyclomatic complexity of 12. 623684
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 625
ShortVariable Avoid variables with short names like tg 625
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 626
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 627
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 628
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 630
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 631
PositionLiteralsFirstInComparisons Position literals first in String comparisons 658
PositionLiteralsFirstInComparisons Position literals first in String comparisons 666
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 699
CommentRequired Public method and constructor comments are required 767791
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 812
ConfusingTernary Avoid if (x != y) ..; else ..; 821832
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 903
ConfusingTernary Avoid if (x != y) ..; else ..; 960967
ConfusingTernary Avoid if (x != y) ..; else ..; 10001007
ShortVariable Avoid variables with short names like tf 1032
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 1053
ShortVariable Avoid variables with short names like is 1125
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1173
CyclomaticComplexity The method 'doUpdatePriorityContent(HttpServletRequest)' has a cyclomatic complexity of 12. 12151276
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1217
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1218
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1219
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1220
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1222
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1223
PositionLiteralsFirstInComparisons Position literals first in String comparisons 1250
PositionLiteralsFirstInComparisons Position literals first in String comparisons 1258
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1287
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1288
LinguisticNaming Linguistics Antipattern - The setter 'setContent' should not return any type except void linguistically 13301356

fr/paris/lutece/plugins/blog/web/BlogPublicationJspBean.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34480
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 146
NPathComplexity The method 'getManageBlogPublication(HttpServletRequest)' has an NPath complexity of 360 147215
CyclomaticComplexity The method 'getManageBlogPublication(HttpServletRequest)' has a cyclomatic complexity of 12. 147215
ConfusingTernary Avoid if (x != y) ..; else ..; 154
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 236
ConfusingTernary Avoid if (x != y) ..; else ..; 285
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 292
LinguisticNaming Linguistics Antipattern - The setter 'setFillFilter' should not return any type except void linguistically 323362
ConfusingTernary Avoid if (x != y) ..; else ..; 328359
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 423

fr/paris/lutece/plugins/blog/web/ManageBlogJspBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 52141
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 79
AddEmptyString Do not add empty strings 135

fr/paris/lutece/plugins/blog/web/TagJspBean.java

Rule Violation Line
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 124
ConfusingTernary Avoid if (x != y) ..; else ..; 160
ConfusingTernary Avoid if (x != y) ..; else ..; 181
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 229
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 283
ConfusingTernary Avoid if (x != y) ..; else ..; 314

fr/paris/lutece/plugins/blog/web/portlet/BlogListPortletJspBean.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34539
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsChecked' 140
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsUnpulished' 142
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 145
NPathComplexity The method 'getPaginatedListModel(HttpServletRequest)' has an NPath complexity of 420 167256
CyclomaticComplexity The method 'getPaginatedListModel(HttpServletRequest)' has a cyclomatic complexity of 16. 167256
AddEmptyString Do not add empty strings 241
ConfusingTernary Avoid if (x != y) ..; else ..; 265287
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 281
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 282
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 284
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 285
ConfusingTernary Avoid if (x != y) ..; else ..; 302
ConfusingTernary Avoid if (x != y) ..; else ..; 322323
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 347
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 357
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 360
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 389
CyclomaticComplexity The method 'doModify(HttpServletRequest)' has a cyclomatic complexity of 11. 407468
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 422
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 425
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 460
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 490

fr/paris/lutece/plugins/blog/web/portlet/BlogPortletJspBean.java

Rule Violation Line
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 145
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 149
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 168
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 189
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 211
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 214

Priority 4

fr/paris/lutece/plugins/blog/business/Blog.java

Rule Violation Line
ShortClassName Avoid short class names like Blog 54641
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 417420
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 437440

fr/paris/lutece/plugins/blog/business/BlogFilter.java

Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 66
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 103
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 113116
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 185188

fr/paris/lutece/plugins/blog/business/BlogSearchFilter.java

Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 110
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 162165

fr/paris/lutece/plugins/blog/business/Tag.java

Rule Violation Line
ShortClassName Avoid short class names like Tag 40145

fr/paris/lutece/plugins/blog/business/portlet/BlogListPortlet.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.GregorianCalendar' due to existing import 'java.util.GregorianCalendar' 89

fr/paris/lutece/plugins/blog/business/portlet/BlogPortlet.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.GregorianCalendar' due to existing import 'java.util.GregorianCalendar' 87

fr/paris/lutece/plugins/blog/business/portlet/PortletFilter.java

Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 68
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 89

fr/paris/lutece/plugins/blog/service/docsearch/DefaultBlogIndexer.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.blog.business.IndexerAction' due to existing import 'fr.paris.lutece.plugins.blog.business.IndexerAction' 132
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 205
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 208
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 208

fr/paris/lutece/plugins/blog/web/BlogJspBean.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 1462
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 1465
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 1465

fr/paris/lutece/plugins/blog/web/BlogPublicationJspBean.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 289
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 290
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 298
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 303

fr/paris/lutece/plugins/blog/web/portlet/BlogPortletJspBean.java

Rule Violation Line
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' 247
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' 247

Files

fr/paris/lutece/plugins/blog/business/Blog.java

Rule Violation Priority Line
ExcessivePublicCount This class has a bunch of public methods and attributes 3 34641
ShortClassName Avoid short class names like Blog 4 54641
TooManyFields Too many fields 3 54641
CommentRequired Public method and constructor comments are required 3 289292
CommentRequired Public method and constructor comments are required 3 294297
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 417420
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 437440

fr/paris/lutece/plugins/blog/business/BlogDAO.java

Rule Violation Priority Line
TooManyMethods This class has too many methods, consider refactoring it. 3 51797
NPathComplexity The method 'getDaoFromFilter(String, BlogFilter)' has an NPath complexity of 266112 3 589717
CyclomaticComplexity The method 'getDaoFromFilter(String, BlogFilter)' has a cyclomatic complexity of 28. 3 589717
ExcessiveMethodLength Avoid really long methods. 3 589717
NcssCount The method 'getDaoFromFilter(String, BlogFilter)' has a NCSS line count of 61. 3 589717
ShortVariable Avoid variables with short names like i 3 598
ConfusingTernary Avoid if (x != y) ..; else ..; 3 620
ConfusingTernary Avoid if (x != y) ..; else ..; 3 638
ConfusingTernary Avoid if (x != y) ..; else ..; 3 643
ConfusingTernary Avoid if (x != y) ..; else ..; 3 648
ConfusingTernary Avoid if (x != y) ..; else ..; 3 654
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 654
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 655
ConfusingTernary Avoid if (x != y) ..; else ..; 3 661
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 661
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 667
ConfusingTernary Avoid if (x != y) ..; else ..; 3 667
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 675
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 681
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 685
GuardLogStatement Logger calls should be surrounded by log level guards. 2 688
GuardLogStatement Logger calls should be surrounded by log level guards. 2 700
GuardLogStatement Logger calls should be surrounded by log level guards. 2 711

fr/paris/lutece/plugins/blog/business/BlogFilter.java

Rule Violation Priority Line
MethodReturnsInternalArray Returning '_arrayTagsId' may expose an internal array. 3 59
ArrayIsStoredDirectly The user-supplied array 'arrayTagsId' is stored directly. 3 66
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 66
MethodReturnsInternalArray Returning '_arrayId' may expose an internal array. 3 96
ArrayIsStoredDirectly The user-supplied array 'arrayId' is stored directly. 3 103
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 103
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 113116
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 185188

fr/paris/lutece/plugins/blog/business/BlogHome.java

Rule Violation Priority Line
TooManyMethods This class has too many methods, consider refactoring it. 3 47289

fr/paris/lutece/plugins/blog/business/BlogPageTemplateDAO.java

Rule Violation Priority Line
DefaultPackage Use explicit scoping instead of the default package private level 3 6375
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the newPrimaryKey method if you want a default access modifier 3 6375
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 8497

fr/paris/lutece/plugins/blog/business/BlogSearchFilter.java

Rule Violation Priority Line
MethodReturnsInternalArray Returning '_strTag' may expose an internal array. 3 101
ArrayIsStoredDirectly The user-supplied array 'strTag' is stored directly. 3 110
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 110
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 162165

fr/paris/lutece/plugins/blog/business/ContentType.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 3684

fr/paris/lutece/plugins/blog/business/DocContent.java

Rule Violation Priority Line
ArrayIsStoredDirectly The user-supplied array 'bytes' is stored directly. 3 106
MethodReturnsInternalArray Returning '_bytes' may expose an internal array. 3 118

fr/paris/lutece/plugins/blog/business/DocContentDAO.java

Rule Violation Priority Line
TooManyMethods This class has too many methods, consider refactoring it. 3 46293

fr/paris/lutece/plugins/blog/business/IBlogDAO.java

Rule Violation Priority Line
TooManyMethods This class has too many methods, consider refactoring it. 3 45211
CommentRequired Public method and constructor comments are required 3 209

fr/paris/lutece/plugins/blog/business/IDocContentDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 40145

fr/paris/lutece/plugins/blog/business/ITagDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 41172
TooManyMethods This class has too many methods, consider refactoring it. 3 42172

fr/paris/lutece/plugins/blog/business/IndexerActionDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 8092

fr/paris/lutece/plugins/blog/business/IndexerActionHome.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 65113
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 121124

fr/paris/lutece/plugins/blog/business/Tag.java

Rule Violation Priority Line
ShortClassName Avoid short class names like Tag 4 40145
CommentRequired Header comments are required 3 40145
UncommentedEmptyConstructor Document empty constructor 3 5659
CommentRequired Public method and constructor comments are required 3 5659
CommentRequired Public method and constructor comments are required 3 6166

fr/paris/lutece/plugins/blog/business/TagDAO.java

Rule Violation Priority Line
TooManyMethods This class has too many methods, consider refactoring it. 3 47326

fr/paris/lutece/plugins/blog/business/portlet/BlogListPortlet.java

Rule Violation Priority Line
ImmutableField Private field '_removedBlogsId' could be made final; it is only initialized in the declaration or constructor. 3 76
CallSuperInConstructor It is a good practice to call super() in a constructor 3 8184
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.GregorianCalendar' due to existing import 'java.util.GregorianCalendar' 4 89
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 95

fr/paris/lutece/plugins/blog/business/portlet/BlogListPortletDAO.java

Rule Violation Priority Line
TooManyMethods This class has too many methods, consider refactoring it. 3 52395
ShortVariable Avoid variables with short names like p 3 82
ShortVariable Avoid variables with short names like p 3 103
ShortVariable Avoid variables with short names like p 3 131
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 212
ShortVariable Avoid variables with short names like p 3 262
CyclomaticComplexity The method 'selectPortletByType(int, PortletOrder, PortletFilter)' has a cyclomatic complexity of 10. 3 320374
ConfusingTernary Avoid if (x != y) ..; else ..; 3 325

fr/paris/lutece/plugins/blog/business/portlet/BlogListPortletHome.java

Rule Violation Priority Line
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 3 56
RedundantFieldInitializer Avoid using redundant field initializer for '_singleton' 3 56
CallSuperInConstructor It is a good practice to call super() in a constructor 3 6167
NonThreadSafeSingleton Singleton is not thread safe 3 7679

fr/paris/lutece/plugins/blog/business/portlet/BlogPortlet.java

Rule Violation Priority Line
CallSuperInConstructor It is a good practice to call super() in a constructor 3 6669
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 71
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 73
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 75
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.GregorianCalendar' due to existing import 'java.util.GregorianCalendar' 4 87

fr/paris/lutece/plugins/blog/business/portlet/BlogPortletDAO.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like p 3 71
ShortVariable Avoid variables with short names like p 3 109
ShortVariable Avoid variables with short names like p 3 160
CyclomaticComplexity The method 'selectPortletByType(int, PortletOrder, PortletFilter)' has a cyclomatic complexity of 10. 3 195245
ConfusingTernary Avoid if (x != y) ..; else ..; 3 200

fr/paris/lutece/plugins/blog/business/portlet/BlogPortletHome.java

Rule Violation Priority Line
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 3 53
CallSuperInConstructor It is a good practice to call super() in a constructor 3 5864
NonThreadSafeSingleton Singleton is not thread safe 3 8588

fr/paris/lutece/plugins/blog/business/portlet/BlogPublication.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 41171

fr/paris/lutece/plugins/blog/business/portlet/BlogPublicationDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 45434
TooManyMethods This class has too many methods, consider refactoring it. 3 46434

fr/paris/lutece/plugins/blog/business/portlet/BlogPublicationHome.java

Rule Violation Priority Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 45240
CommentRequired Header comments are required 3 45240

fr/paris/lutece/plugins/blog/business/portlet/IBlogPortletDAO.java

Rule Violation Priority Line
MissingOverride The method 'load(int)' is missing an @Override annotation. 3 74

fr/paris/lutece/plugins/blog/business/portlet/IBlogPublicationDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 42204
TooManyMethods This class has too many methods, consider refactoring it. 3 43204

fr/paris/lutece/plugins/blog/business/portlet/PortletFilter.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 68
ArrayIsStoredDirectly The user-supplied array 'aPageTitle' is stored directly. 3 68
MethodReturnsInternalArray Returning '_aPageName' may expose an internal array. 3 80
ArrayIsStoredDirectly The user-supplied array 'aPortletTitle' is stored directly. 3 89
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 89
MethodReturnsInternalArray Returning '_aPortletName' may expose an internal array. 3 101
CyclomaticComplexity The method 'getSQLFilter()' has a cyclomatic complexity of 13. 3 172221

fr/paris/lutece/plugins/blog/business/rss/BlogResourceRss.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34315
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 248
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 297

fr/paris/lutece/plugins/blog/business/rss/BlogResourceRssConfigDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 5970

fr/paris/lutece/plugins/blog/service/BlogIdService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 47116
ShortVariable Avoid variables with short names like rt 3 61
ShortVariable Avoid variables with short names like p 3 67

fr/paris/lutece/plugins/blog/service/BlogService.java

Rule Violation Priority Line
GodClass Possible God Class (WMC=48, ATFD=85, TCC=0.000%) 3 58442
TooManyMethods This class has too many methods, consider refactoring it. 3 59442
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 117
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 158
ConfusingTernary Avoid if (x != y) ..; else ..; 3 175186
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 218
CollapsibleIfStatements These nested if statements could be combined 3 237241
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 279
CommentRequired Public method and constructor comments are required 3 416423
CommentRequired Public method and constructor comments are required 3 425432
CommentRequired Public method and constructor comments are required 3 434441

fr/paris/lutece/plugins/blog/service/BlogSessionListner.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like se 3 57
ShortVariable Avoid variables with short names like se 3 66
CommentRequired Public method and constructor comments are required 3 7377
CommentRequired Public method and constructor comments are required 3 7984

fr/paris/lutece/plugins/blog/service/PublishingService.java

Rule Violation Priority Line
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 309

fr/paris/lutece/plugins/blog/service/TagIdService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 47110
ShortVariable Avoid variables with short names like rt 3 60
ShortVariable Avoid variables with short names like p 3 66

fr/paris/lutece/plugins/blog/service/TagService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 4168

fr/paris/lutece/plugins/blog/service/UnlockedBlogDaemon.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 4057

fr/paris/lutece/plugins/blog/service/docsearch/BlogSearchItem.java

Rule Violation Priority Line
DataClass The class 'BlogSearchItem' is suspected to be a Data Class (WOC=0.000%, NOPA=7, NOAM=0, WMC=1) 3 4386
CommentRequired Public method and constructor comments are required 3 4548
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 53
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 58
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 63
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 69
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 75
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 80
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 84

fr/paris/lutece/plugins/blog/service/docsearch/BlogSearchService.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34486
AvoidUsingVolatile Use of modifier volatile is not recommended. 2 92
ImmutableField Private field '_indexer' could be made final; it is only initialized in the declaration or constructor. 3 102
ImmutableField Private field '_nWriterMergeFactor' could be made final; it is only initialized in the declaration or constructor. 3 103
NonThreadSafeSingleton Singleton is not thread safe 3 144147
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 147 characters appended. 3 196
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 202
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 243
ShortVariable Avoid variables with short names like si 3 307
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 312
NPathComplexity The method 'prepareQueryForFilter(BlogSearchFilter)' has an NPath complexity of 432 3 321410
CyclomaticComplexity The method 'prepareQueryForFilter(BlogSearchFilter)' has a cyclomatic complexity of 12. 3 321410
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 408
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 408
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 409

fr/paris/lutece/plugins/blog/service/docsearch/DefaultBlogIndexer.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34407
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 123194
ConfusingTernary Avoid if (x != y) ..; else ..; 3 128191
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.blog.business.IndexerAction' due to existing import 'fr.paris.lutece.plugins.blog.business.IndexerAction' 4 132
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 181
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 4 205
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 4 208
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 4 208
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 212
ShortVariable Avoid variables with short names like ft 3 217
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 275
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 276
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 277
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 278
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 279
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 280

fr/paris/lutece/plugins/blog/service/search/BlogSearchIndexer.java

Rule Violation Priority Line
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 101

fr/paris/lutece/plugins/blog/utils/BlogLock.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 3875
CommentRequired Public method and constructor comments are required 3 4449
CommentRequired Public method and constructor comments are required 3 5155
CommentRequired Public method and constructor comments are required 3 5761
CommentRequired Public method and constructor comments are required 3 6367
CommentRequired Public method and constructor comments are required 3 6973

fr/paris/lutece/plugins/blog/utils/BlogUtils.java

Rule Violation Priority Line
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 92
AssignmentInOperand Avoid assignments in operands 3 92

fr/paris/lutece/plugins/blog/web/BlogDashboardComponent.java

Rule Violation Priority Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 108
ShortVariable Avoid variables with short names like t 3 119

fr/paris/lutece/plugins/blog/web/BlogFileServlet.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like os 3 104

fr/paris/lutece/plugins/blog/web/BlogJspBean.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 341470
ExcessiveClassLength Avoid really long classes. 3 1171470
TooManyMethods This class has too many methods, consider refactoring it. 3 1181470
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 250
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsChecked' 3 252
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsUnpulished' 3 254
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 3 260
FieldNamingConventions The final field name '_blogServiceSession' doesn't match '[a-z][a-zA-Z0-9]*' 1 266
NcssCount The method 'getManageBlogs(HttpServletRequest)' has a NCSS line count of 85. 3 276423
ExcessiveMethodLength Avoid really long methods. 3 276423
CyclomaticComplexity The method 'getManageBlogs(HttpServletRequest)' has a cyclomatic complexity of 24. 3 276423
NPathComplexity The method 'getManageBlogs(HttpServletRequest)' has an NPath complexity of 6390 3 276423
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 278
ConfusingTernary Avoid if (x != y) ..; else ..; 3 292313
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 307
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 308
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 310
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 311
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 403
AddEmptyString Do not add empty strings 3 411
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 432
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 435
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 500
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 539
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 580
CyclomaticComplexity The method 'doUpdatePriorityTag(HttpServletRequest)' has a cyclomatic complexity of 12. 3 623684
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 625
ShortVariable Avoid variables with short names like tg 3 625
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 626
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 627
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 628
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 630
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 631
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 658
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 666
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 699
CommentRequired Public method and constructor comments are required 3 767791
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 812
ConfusingTernary Avoid if (x != y) ..; else ..; 3 821832
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 903
ConfusingTernary Avoid if (x != y) ..; else ..; 3 960967
ConfusingTernary Avoid if (x != y) ..; else ..; 3 10001007
ShortVariable Avoid variables with short names like tf 3 1032
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 1053
ShortVariable Avoid variables with short names like is 3 1125
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1173
CyclomaticComplexity The method 'doUpdatePriorityContent(HttpServletRequest)' has a cyclomatic complexity of 12. 3 12151276
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1217
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1218
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1219
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1220
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1222
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1223
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 1250
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 1258
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1287
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1288
LinguisticNaming Linguistics Antipattern - The setter 'setContent' should not return any type except void linguistically 3 13301356
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 4 1462
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 4 1465
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 4 1465

fr/paris/lutece/plugins/blog/web/BlogPublicationJspBean.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34480
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 146
NPathComplexity The method 'getManageBlogPublication(HttpServletRequest)' has an NPath complexity of 360 3 147215
CyclomaticComplexity The method 'getManageBlogPublication(HttpServletRequest)' has a cyclomatic complexity of 12. 3 147215
ConfusingTernary Avoid if (x != y) ..; else ..; 3 154
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 3 236
ConfusingTernary Avoid if (x != y) ..; else ..; 3 285
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 4 289
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 4 290
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 3 292
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 4 298
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 4 303
LinguisticNaming Linguistics Antipattern - The setter 'setFillFilter' should not return any type except void linguistically 3 323362
ConfusingTernary Avoid if (x != y) ..; else ..; 3 328359
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 423

fr/paris/lutece/plugins/blog/web/ManageBlogJspBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 52141
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 3 79
AddEmptyString Do not add empty strings 3 135

fr/paris/lutece/plugins/blog/web/TagJspBean.java

Rule Violation Priority Line
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 124
ConfusingTernary Avoid if (x != y) ..; else ..; 3 160
ConfusingTernary Avoid if (x != y) ..; else ..; 3 181
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 229
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 283
ConfusingTernary Avoid if (x != y) ..; else ..; 3 314

fr/paris/lutece/plugins/blog/web/portlet/BlogListPortletJspBean.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34539
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsChecked' 3 140
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsUnpulished' 3 142
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 3 145
NPathComplexity The method 'getPaginatedListModel(HttpServletRequest)' has an NPath complexity of 420 3 167256
CyclomaticComplexity The method 'getPaginatedListModel(HttpServletRequest)' has a cyclomatic complexity of 16. 3 167256
AddEmptyString Do not add empty strings 3 241
ConfusingTernary Avoid if (x != y) ..; else ..; 3 265287
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 281
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 282
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 284
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 285
ConfusingTernary Avoid if (x != y) ..; else ..; 3 302
ConfusingTernary Avoid if (x != y) ..; else ..; 3 322323
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 347
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 357
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 360
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 389
CyclomaticComplexity The method 'doModify(HttpServletRequest)' has a cyclomatic complexity of 11. 3 407468
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 422
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 425
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 460
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 3 490

fr/paris/lutece/plugins/blog/web/portlet/BlogPortletJspBean.java

Rule Violation Priority Line
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 145
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 149
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 168
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 189
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 211
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 214
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' 4 247
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' 4 247