Fork me on GitHub

Résultats PMD

Le document suivant contient les résultats de PMD 6.13.0.

Violations By Priority

Priority 1

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

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

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

Rule Violation Ligne
FieldNamingConventions The final field name '_blogServiceSession' doesn't match '[a-z][a-zA-Z0-9]*' 261
LocalVariableNamingConventions The local variable name 'BlogList' doesn't match '[a-z][a-zA-Z0-9]*' 1356

Priority 2

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

Rule Violation Ligne
GuardLogStatement Logger calls should be surrounded by log level guards. 680
GuardLogStatement Logger calls should be surrounded by log level guards. 692
GuardLogStatement Logger calls should be surrounded by log level guards. 703

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

Rule Violation Ligne
AvoidUsingVolatile Use of modifier volatile is not recommended. 94

Priority 3

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

Rule Violation Ligne
ExcessivePublicCount This class has a bunch of public methods and attributes 34643
TooManyFields Too many fields 56643
CommentRequired Public method and constructor comments are required 291294
CommentRequired Public method and constructor comments are required 296299

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

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 51789
UnusedPrivateField Avoid unused private fields such as 'SQL_ORDER_BY_ORDER_DOCUMENT'. 91
NPathComplexity The method 'getDaoFromFilter(String, BlogFilter)' has an NPath complexity of 133056 589709
CyclomaticComplexity The method 'getDaoFromFilter(String, BlogFilter)' has a cyclomatic complexity of 27. 589709
ExcessiveMethodLength Avoid really long methods. 589709
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
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 661
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. 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 678

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

Rule Violation Ligne
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 Ligne
TooManyMethods This class has too many methods, consider refactoring it. 47289

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

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

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

Rule Violation Ligne
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 Ligne
CommentRequired Header comments are required 3684

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

Rule Violation Ligne
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 Ligne
TooManyMethods This class has too many methods, consider refactoring it. 46295

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

Rule Violation Ligne
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 Ligne
CommentRequired Header comments are required 40145

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

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

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

Rule Violation Ligne
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 Ligne
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 Ligne
TooManyMethods This class has too many methods, consider refactoring it. 47326

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

Rule Violation Ligne
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 Ligne
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 Ligne
RedundantFieldInitializer Avoid using redundant field initializer for '_singleton' 56
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 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 Ligne
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 Ligne
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 Ligne
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 Ligne
CommentRequired Header comments are required 41171

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

Rule Violation Ligne
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 Ligne
CommentRequired Header comments are required 45240

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

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

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

Rule Violation Ligne
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 Ligne
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 Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34316
UnusedPrivateField Avoid unused private fields such as 'TEMPLATE_PUSH_RSS_XML_BLOG'. 77
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 249
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 298

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

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

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

Rule Violation Ligne
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 Ligne
GodClass Possible God Class (WMC=53, ATFD=98, TCC=0.000%) 58459
TooManyMethods This class has too many methods, consider refactoring it. 59459
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
CyclomaticComplexity The method 'updateBlog(Blog, List)' has a cyclomatic complexity of 11. 198246
CollapsibleIfStatements These nested if statements could be combined 216220
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 239
CyclomaticComplexity The method 'updateBlogWithoutVersion(Blog, List)' has a cyclomatic complexity of 11. 256303
CollapsibleIfStatements These nested if statements could be combined 274278
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 296
CommentRequired Public method and constructor comments are required 433440
CommentRequired Public method and constructor comments are required 442449
CommentRequired Public method and constructor comments are required 451458

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

Rule Violation Ligne
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 Ligne
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 309

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

Rule Violation Ligne
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 Ligne
CommentRequired Header comments are required 4168

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

Rule Violation Ligne
CommentRequired Header comments are required 4057

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

Rule Violation Ligne
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 Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34494
ImmutableField Private field '_indexer' could be made final; it is only initialized in the declaration or constructor. 105
ImmutableField Private field '_nWriterMergeFactor' could be made final; it is only initialized in the declaration or constructor. 106
NonThreadSafeSingleton Singleton is not thread safe 147151
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 147 characters appended. 201
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 207
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 248
NPathComplexity The method 'getSearchResults(BlogSearchFilter, Plugin, List)' has an NPath complexity of 866 286418
CyclomaticComplexity The method 'getSearchResults(BlogSearchFilter, Plugin, List)' has a cyclomatic complexity of 14. 286418
UnusedFormalParameter Avoid unused method parameters such as 'plugin'. 286
ExcessiveMethodLength Avoid really long methods. 286418
NcssCount The method 'getSearchResults(BlogSearchFilter, Plugin, List)' has a NCSS line count of 81. 286418
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 385
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 385
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 386
ShortVariable Avoid variables with short names like si 405
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 410

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

Rule Violation Ligne
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 Ligne
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 101

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

Rule Violation Ligne
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 Ligne
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 92
AssignmentInOperand Avoid assignments in operands 92

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

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

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

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

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

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 341454
ExcessiveClassLength Avoid really long classes. 1141454
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 1141454
TooManyMethods This class has too many methods, consider refactoring it. 1151454
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 245
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsChecked' 247
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsUnpulished' 249
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 255
CyclomaticComplexity The method 'getManageBlogs(HttpServletRequest)' has a cyclomatic complexity of 25. 271402
NcssCount The method 'getManageBlogs(HttpServletRequest)' has a NCSS line count of 85. 271402
NPathComplexity The method 'getManageBlogs(HttpServletRequest)' has an NPath complexity of 9270 271402
ExcessiveMethodLength Avoid really long methods. 271402
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 273
ConfusingTernary Avoid if (x != y) ..; else ..; 287308
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 302
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 303
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 305
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 306
ControlStatementBraces This statement should have braces 315
ControlStatementBraces This statement should have braces 317
ControlStatementBraces This statement should have braces 319
ControlStatementBraces This statement should have braces 321
ControlStatementBraces This statement should have braces 323
ControlStatementBraces This statement should have braces 325
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 382
AddEmptyString Do not add empty strings 390
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 411
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 414
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 478
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 517
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 558
CyclomaticComplexity The method 'doUpdatePriorityTag(HttpServletRequest)' has a cyclomatic complexity of 12. 601662
ShortVariable Avoid variables with short names like tg 603
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 603
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 604
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 605
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 606
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 608
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 609
PositionLiteralsFirstInComparisons Position literals first in String comparisons 636
PositionLiteralsFirstInComparisons Position literals first in String comparisons 644
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 677
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 727
CommentRequired Public method and constructor comments are required 745769
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 790
ConfusingTernary Avoid if (x != y) ..; else ..; 799810
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 880
ConfusingTernary Avoid if (x != y) ..; else ..; 937944
ConfusingTernary Avoid if (x != y) ..; else ..; 977984
ShortVariable Avoid variables with short names like tf 1009
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 1030
ShortVariable Avoid variables with short names like is 1102
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1150
CyclomaticComplexity The method 'doUpdatePriorityContent(HttpServletRequest)' has a cyclomatic complexity of 12. 11921253
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1194
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1195
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1196
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1197
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1199
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1200
PositionLiteralsFirstInComparisons Position literals first in String comparisons 1227
PositionLiteralsFirstInComparisons Position literals first in String comparisons 1235
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1264
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1265
LinguisticNaming Linguistics Antipattern - The setter 'setContent' should not return any type except void linguistically 13071333
SimplifyBooleanReturns Avoid unnecessary if..then..else statements when returning booleans 13781383

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

Rule Violation Ligne
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 Ligne
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 Ligne
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 124
ConfusingTernary Avoid if (x != y) ..; else ..; 157
ConfusingTernary Avoid if (x != y) ..; else ..; 178
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 226
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 280
ConfusingTernary Avoid if (x != y) ..; else ..; 311

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

Rule Violation Ligne
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
CyclomaticComplexity The method 'getPaginatedListModel(HttpServletRequest)' has a cyclomatic complexity of 16. 167256
NPathComplexity The method 'getPaginatedListModel(HttpServletRequest)' has an NPath complexity of 420 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 Ligne
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 Ligne
ShortClassName Avoid short class names like Blog 56643
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 419422
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 439442

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

Rule Violation Ligne
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 Ligne
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 Ligne
ShortClassName Avoid short class names like Tag 40145

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

Rule Violation Ligne
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 Ligne
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 Ligne
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 Ligne
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 Ligne
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 1446
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 1449
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 1449

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

Rule Violation Ligne
UnusedImports Avoid unused imports such as 'fr.paris.lutece.api.user.User' 73
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/TagJspBean.java

Rule Violation Ligne
UnusedImports Avoid unused imports such as 'fr.paris.lutece.api.user.User' 36

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

Rule Violation Ligne
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

Fichiers

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

Rule Violation Priority Ligne
ExcessivePublicCount This class has a bunch of public methods and attributes 3 34643
ShortClassName Avoid short class names like Blog 4 56643
TooManyFields Too many fields 3 56643
CommentRequired Public method and constructor comments are required 3 291294
CommentRequired Public method and constructor comments are required 3 296299
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 419422
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 439442

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

Rule Violation Priority Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 51789
UnusedPrivateField Avoid unused private fields such as 'SQL_ORDER_BY_ORDER_DOCUMENT'. 3 91
NPathComplexity The method 'getDaoFromFilter(String, BlogFilter)' has an NPath complexity of 133056 3 589709
CyclomaticComplexity The method 'getDaoFromFilter(String, BlogFilter)' has a cyclomatic complexity of 27. 3 589709
ExcessiveMethodLength Avoid really long methods. 3 589709
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
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 661
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 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 678
GuardLogStatement Logger calls should be surrounded by log level guards. 2 680
GuardLogStatement Logger calls should be surrounded by log level guards. 2 692
GuardLogStatement Logger calls should be surrounded by log level guards. 2 703

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

Rule Violation Priority Ligne
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 Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 47289

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

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

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

Rule Violation Priority Ligne
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 Ligne
CommentRequired Header comments are required 3 3684

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

Rule Violation Priority Ligne
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 Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 46295

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

Rule Violation Priority Ligne
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 Ligne
CommentRequired Header comments are required 3 40145

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

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

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

Rule Violation Priority Ligne
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 Ligne
CommentRequired Header comments are required 3 40145
ShortClassName Avoid short class names like Tag 4 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 Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 47326

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

Rule Violation Priority Ligne
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 Ligne
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 Ligne
RedundantFieldInitializer Avoid using redundant field initializer for '_singleton' 3 56
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 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 Ligne
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 Ligne
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 Ligne
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 Ligne
CommentRequired Header comments are required 3 41171

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

Rule Violation Priority Ligne
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 Ligne
CommentRequired Header comments are required 3 45240
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 45240

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

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

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

Rule Violation Priority Ligne
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 Ligne
ArrayIsStoredDirectly The user-supplied array 'aPageTitle' is stored directly. 3 68
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 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 Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34316
UnusedPrivateField Avoid unused private fields such as 'TEMPLATE_PUSH_RSS_XML_BLOG'. 3 77
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 249
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 298

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

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

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

Rule Violation Priority Ligne
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 Ligne
GodClass Possible God Class (WMC=53, ATFD=98, TCC=0.000%) 3 58459
TooManyMethods This class has too many methods, consider refactoring it. 3 59459
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
CyclomaticComplexity The method 'updateBlog(Blog, List)' has a cyclomatic complexity of 11. 3 198246
CollapsibleIfStatements These nested if statements could be combined 3 216220
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 239
CyclomaticComplexity The method 'updateBlogWithoutVersion(Blog, List)' has a cyclomatic complexity of 11. 3 256303
CollapsibleIfStatements These nested if statements could be combined 3 274278
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 296
CommentRequired Public method and constructor comments are required 3 433440
CommentRequired Public method and constructor comments are required 3 442449
CommentRequired Public method and constructor comments are required 3 451458

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

Rule Violation Priority Ligne
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 Ligne
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 309

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

Rule Violation Priority Ligne
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 Ligne
CommentRequired Header comments are required 3 4168

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

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 4057

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

Rule Violation Priority Ligne
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 Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34494
AvoidUsingVolatile Use of modifier volatile is not recommended. 2 94
ImmutableField Private field '_indexer' could be made final; it is only initialized in the declaration or constructor. 3 105
ImmutableField Private field '_nWriterMergeFactor' could be made final; it is only initialized in the declaration or constructor. 3 106
NonThreadSafeSingleton Singleton is not thread safe 3 147151
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 147 characters appended. 3 201
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 207
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 248
NPathComplexity The method 'getSearchResults(BlogSearchFilter, Plugin, List)' has an NPath complexity of 866 3 286418
CyclomaticComplexity The method 'getSearchResults(BlogSearchFilter, Plugin, List)' has a cyclomatic complexity of 14. 3 286418
UnusedFormalParameter Avoid unused method parameters such as 'plugin'. 3 286
ExcessiveMethodLength Avoid really long methods. 3 286418
NcssCount The method 'getSearchResults(BlogSearchFilter, Plugin, List)' has a NCSS line count of 81. 3 286418
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 385
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 385
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 386
ShortVariable Avoid variables with short names like si 3 405
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 410

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

Rule Violation Priority Ligne
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 Ligne
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 101

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

Rule Violation Priority Ligne
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 Ligne
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 Ligne
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 107
ShortVariable Avoid variables with short names like t 3 118

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

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

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

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 341454
ExcessiveClassLength Avoid really long classes. 3 1141454
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 1141454
TooManyMethods This class has too many methods, consider refactoring it. 3 1151454
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 245
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsChecked' 3 247
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsUnpulished' 3 249
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 3 255
FieldNamingConventions The final field name '_blogServiceSession' doesn't match '[a-z][a-zA-Z0-9]*' 1 261
CyclomaticComplexity The method 'getManageBlogs(HttpServletRequest)' has a cyclomatic complexity of 25. 3 271402
NcssCount The method 'getManageBlogs(HttpServletRequest)' has a NCSS line count of 85. 3 271402
NPathComplexity The method 'getManageBlogs(HttpServletRequest)' has an NPath complexity of 9270 3 271402
ExcessiveMethodLength Avoid really long methods. 3 271402
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 273
ConfusingTernary Avoid if (x != y) ..; else ..; 3 287308
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 302
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 303
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 305
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 306
ControlStatementBraces This statement should have braces 3 315
ControlStatementBraces This statement should have braces 3 317
ControlStatementBraces This statement should have braces 3 319
ControlStatementBraces This statement should have braces 3 321
ControlStatementBraces This statement should have braces 3 323
ControlStatementBraces This statement should have braces 3 325
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 382
AddEmptyString Do not add empty strings 3 390
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 411
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 414
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 478
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 517
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 558
CyclomaticComplexity The method 'doUpdatePriorityTag(HttpServletRequest)' has a cyclomatic complexity of 12. 3 601662
ShortVariable Avoid variables with short names like tg 3 603
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 603
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 604
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 605
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 606
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 608
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 609
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 636
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 644
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 677
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 3 727
CommentRequired Public method and constructor comments are required 3 745769
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 790
ConfusingTernary Avoid if (x != y) ..; else ..; 3 799810
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 880
ConfusingTernary Avoid if (x != y) ..; else ..; 3 937944
ConfusingTernary Avoid if (x != y) ..; else ..; 3 977984
ShortVariable Avoid variables with short names like tf 3 1009
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 1030
ShortVariable Avoid variables with short names like is 3 1102
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1150
CyclomaticComplexity The method 'doUpdatePriorityContent(HttpServletRequest)' has a cyclomatic complexity of 12. 3 11921253
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1194
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1195
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1196
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1197
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1199
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1200
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 1227
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 1235
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1264
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1265
LinguisticNaming Linguistics Antipattern - The setter 'setContent' should not return any type except void linguistically 3 13071333
LocalVariableNamingConventions The local variable name 'BlogList' doesn't match '[a-z][a-zA-Z0-9]*' 1 1356
SimplifyBooleanReturns Avoid unnecessary if..then..else statements when returning booleans 3 13781383
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 4 1446
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 4 1449
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 4 1449

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

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34480
UnusedImports Avoid unused imports such as 'fr.paris.lutece.api.user.User' 4 73
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 Ligne
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 Ligne
UnusedImports Avoid unused imports such as 'fr.paris.lutece.api.user.User' 4 36
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 124
ConfusingTernary Avoid if (x != y) ..; else ..; 3 157
ConfusingTernary Avoid if (x != y) ..; else ..; 3 178
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 226
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 280
ConfusingTernary Avoid if (x != y) ..; else ..; 3 311

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

Rule Violation Priority Ligne
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
CyclomaticComplexity The method 'getPaginatedListModel(HttpServletRequest)' has a cyclomatic complexity of 16. 3 167256
NPathComplexity The method 'getPaginatedListModel(HttpServletRequest)' has an NPath complexity of 420 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 Ligne
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