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/BlogSerializable.java

Rule Violation Ligne
FieldNamingConventions The field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' 8
FieldNamingConventions The field name 'contentLabel' doesn't match '_[a-z][a-zA-Z0-9]*' 9

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

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

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]*' 317
LocalVariableNamingConventions The local variable name '_nDocContentInSession' doesn't match '[a-z][a-zA-Z0-9]*' 1426

Priority 2

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

Rule Violation Ligne
GuardLogStatement Logger calls should be surrounded by log level guards. 758
GuardLogStatement Logger calls should be surrounded by log level guards. 770
GuardLogStatement Logger calls should be surrounded by log level guards. 781

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

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

Priority 3

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

Rule Violation Ligne
ExcessivePublicCount This class has a bunch of public methods and attributes 34661
TooManyFields Too many fields 54661
GodClass Possible God Class (WMC=47, ATFD=20, TCC=3.243%) 54661
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. 52951
CyclomaticComplexity The method 'getDaoFromFilter(String, BlogFilter)' has a cyclomatic complexity of 31. 651787
ExcessiveMethodLength Avoid really long methods. 651787
NPathComplexity The method 'getDaoFromFilter(String, BlogFilter)' has an NPath complexity of 1064448 651787
NcssCount The method 'getDaoFromFilter(String, BlogFilter)' has a NCSS line count of 65. 651787
ShortVariable Avoid variables with short names like i 660
ConfusingTernary Avoid if (x != y) ..; else ..; 682
ConfusingTernary Avoid if (x != y) ..; else ..; 700
ConfusingTernary Avoid if (x != y) ..; else ..; 705
ConfusingTernary Avoid if (x != y) ..; else ..; 710
ConfusingTernary Avoid if (x != y) ..; else ..; 714
ConfusingTernary Avoid if (x != y) ..; else ..; 718
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 724
ConfusingTernary Avoid if (x != y) ..; else ..; 724
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 725
ConfusingTernary Avoid if (x != y) ..; else ..; 731
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 731
ConfusingTernary Avoid if (x != y) ..; else ..; 737
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 737
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 745
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 751
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 755

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

Rule Violation Ligne
MethodReturnsInternalArray Returning '_arrayTagsId' may expose an internal array. 60
ArrayIsStoredDirectly The user-supplied array 'arrayTagsId' is stored directly. 67
MethodReturnsInternalArray Returning '_arrayId' may expose an internal array. 97
ArrayIsStoredDirectly The user-supplied array 'arrayId' is stored directly. 104

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

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 47348

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

Rule Violation Ligne
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 Ligne
MethodReturnsInternalArray Returning '_strTag' may expose an internal array. 102
ArrayIsStoredDirectly The user-supplied array 'strTag' is stored directly. 111

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

Rule Violation Ligne
DataClass The class 'BlogSerializable' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=6) 656
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 656
CommentRequired Header comments are required 656
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the id field if you want a default access modifier 8
DefaultPackage Use explicit scoping instead of the default package private level 8
ShortVariable Avoid variables with short names like id 8
DefaultPackage Use explicit scoping instead of the default package private level 9
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the contentLabel field if you want a default access modifier 9
CommentRequired Public method and constructor comments are required 1113
UncommentedEmptyConstructor Document empty constructor 1113
CommentRequired Public method and constructor comments are required 1519
ShortVariable Avoid variables with short names like id 15
ShortVariable Avoid variables with short names like id 34

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. 46293

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

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 45239
CommentRequired Public method and constructor comments are required 155
CommentRequired Public method and constructor comments are required 234
CommentRequired Public method and constructor comments are required 236
CommentRequired Public method and constructor comments are required 238

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
CommentRequired Public method and constructor comments are required 5659
UncommentedEmptyConstructor Document empty constructor 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/insertservice/BlogLinkPOJO.java

Rule Violation Ligne
CommentRequired Public method and constructor comments are required 4245
CommentRequired Public method and constructor comments are required 4750
CommentRequired Public method and constructor comments are required 5255
CommentRequired Public method and constructor comments are required 5760
CommentRequired Public method and constructor comments are required 6265
CommentRequired Public method and constructor comments are required 6770
CommentRequired Public method and constructor comments are required 7275
CommentRequired Public method and constructor comments are required 7780

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. 52396
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 132
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 213
ShortVariable Avoid variables with short names like p 263
CyclomaticComplexity The method 'selectPortletByType(int, PortletOrder, PortletFilter)' has a cyclomatic complexity of 10. 321375
ConfusingTernary Avoid if (x != y) ..; else ..; 326

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 6770
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 72
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 74
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 76

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. 196246
ConfusingTernary Avoid if (x != y) ..; else ..; 201

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 45253

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. 66
MethodReturnsInternalArray Returning '_aPageName' may expose an internal array. 78
ArrayIsStoredDirectly The user-supplied array 'aPortletTitle' is stored directly. 87
MethodReturnsInternalArray Returning '_aPortletName' may expose an internal array. 99
CyclomaticComplexity The method 'getSQLFilter()' has a cyclomatic complexity of 13. 170219

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. 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 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 47122
ShortVariable Avoid variables with short names like rt 62
ShortVariable Avoid variables with short names like p 68

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

Rule Violation Ligne
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 94
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 97
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 116
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 138

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

Rule Violation Ligne
GodClass Possible God Class (WMC=57, ATFD=99, TCC=0.000%) 60532
TooManyMethods This class has too many methods, consider refactoring it. 61532
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 119
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 169
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 224
ConfusingTernary Avoid if (x != y) ..; else ..; 241252
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 284
CollapsibleIfStatements These nested if statements could be combined 303307
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 345
CommentRequired Public method and constructor comments are required 506513
CommentRequired Public method and constructor comments are required 515522
CommentRequired Public method and constructor comments are required 524531

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

Rule Violation Ligne
ShortVariable Avoid variables with short names like e 216
CollapsibleIfStatements These nested if statements could be combined 222230
ShortVariable Avoid variables with short names like e 258

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=8, NOAM=0, WMC=1) 4391
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. 74
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 79
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 84
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 89

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. 34512
ImmutableField Private field '_indexer' could be made final; it is only initialized in the declaration or constructor. 103
ImmutableField Private field '_nWriterMergeFactor' could be made final; it is only initialized in the declaration or constructor. 104
NonThreadSafeSingleton Singleton is not thread safe 145148
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 147 characters appended. 197
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 203
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 244
CommentRequired Public method and constructor comments are required 270285
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 280
ShortVariable Avoid variables with short names like si 324
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 329
CyclomaticComplexity The method 'prepareQueryForFilter(BlogSearchFilter)' has a cyclomatic complexity of 14. 338436
NPathComplexity The method 'prepareQueryForFilter(BlogSearchFilter)' has an NPath complexity of 1296 338436
NcssCount The method 'prepareQueryForFilter(BlogSearchFilter)' has a NCSS line count of 64. 338436
CollapsibleIfStatements These nested if statements could be combined 418425
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 434
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 434
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 435

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. 34425
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 140211
ConfusingTernary Avoid if (x != y) ..; else ..; 145208
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 198
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 229
ShortVariable Avoid variables with short names like ft 234
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 293
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 294
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 295
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 296
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 297
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 298

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

Rule Violation Ligne
CommentRequired Public method and constructor comments are required 49

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 110
AssignmentInOperand Avoid assignments in operands 110
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 171172
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 220
CommentRequired Public method and constructor comments are required 244247

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 108
ShortVariable Avoid variables with short names like t 119

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. 342212
ExcessiveClassLength Avoid really long classes. 1242212
TooManyFields Too many fields 1242212
TooManyMethods This class has too many methods, consider refactoring it. 1252212
UnusedPrivateField Avoid unused private fields such as 'MARK_NUMBER_MANDATORY_TAGS'. 206
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 299
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsChecked' 301
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 309
NPathComplexity The method 'getManageBlogs(HttpServletRequest)' has an NPath complexity of 36240 327516
CyclomaticComplexity The method 'getManageBlogs(HttpServletRequest)' has a cyclomatic complexity of 28. 327516
NcssCount The method 'getManageBlogs(HttpServletRequest)' has a NCSS line count of 113. 327516
ExcessiveMethodLength Avoid really long methods. 327516
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 329
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 336
ConfusingTernary Avoid if (x != y) ..; else ..; 345373
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 367
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 368
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 369
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 370
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 390
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 482
ConfusingTernary Avoid if (x != y) ..; else ..; 483491
AddEmptyString Do not add empty strings 501
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 525
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 528
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 555
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 556
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 591
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 716
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 765
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 806
CyclomaticComplexity The method 'doUpdatePriorityTag(HttpServletRequest)' has a cyclomatic complexity of 12. 849910
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 851
ShortVariable Avoid variables with short names like tg 851
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 852
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 853
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 854
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 856
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 857
PositionLiteralsFirstInComparisons Position literals first in String comparisons 884
PositionLiteralsFirstInComparisons Position literals first in String comparisons 892
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 925
CommentRequired Public method and constructor comments are required 9951019
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1040
ConfusingTernary Avoid if (x != y) ..; else ..; 10451067
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1148
ConfusingTernary Avoid if (x != y) ..; else ..; 12171224
ConfusingTernary Avoid if (x != y) ..; else ..; 12571264
ShortVariable Avoid variables with short names like tf 1289
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 1310
CyclomaticComplexity The method 'addContent(HttpServletRequest)' has a cyclomatic complexity of 14. 13361443
ExcessiveMethodLength Avoid really long methods. 13361443
NPathComplexity The method 'addContent(HttpServletRequest)' has an NPath complexity of 720 13361443
ShortVariable Avoid variables with short names like is 1387
ConfusingTernary Avoid if (x != y) ..; else ..; 14141439
ConfusingTernary Avoid if (x != y) ..; else ..; 14631500
NPathComplexity The method 'doUpdatePriorityContent(HttpServletRequest)' has an NPath complexity of 1080 15121587
CyclomaticComplexity The method 'doUpdatePriorityContent(HttpServletRequest)' has a cyclomatic complexity of 16. 15121587
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1514
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1515
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1516
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1517
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1519
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1520
ConfusingTernary Avoid if (x != y) ..; else ..; 15271546
PositionLiteralsFirstInComparisons Position literals first in String comparisons 1558
PositionLiteralsFirstInComparisons Position literals first in String comparisons 1566
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1598
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1599
LinguisticNaming Linguistics Antipattern - The setter 'setContent' should not return any type except void linguistically 16411667
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 1727
UnusedLocalVariable Avoid unused local variables such as 'adminUser'. 1727
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 1747
CyclomaticComplexity The method 'getConfirmArchiveBlogs(HttpServletRequest)' has a cyclomatic complexity of 10. 18291876
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 1862
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 1909
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 1968
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 1979
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 1996
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 2039
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 2043
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 21012117
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 21252142

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. 34529
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 149
CyclomaticComplexity The method 'getManageBlogPublication(HttpServletRequest)' has a cyclomatic complexity of 12. 150212
NPathComplexity The method 'getManageBlogPublication(HttpServletRequest)' has an NPath complexity of 360 150212
ConfusingTernary Avoid if (x != y) ..; else ..; 154
ConfusingTernary Avoid if (x != y) ..; else ..; 281
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 288
LinguisticNaming Linguistics Antipattern - The setter 'setFillFilter' should not return any type except void linguistically 319358
ConfusingTernary Avoid if (x != y) ..; else ..; 324355
UnusedFormalParameter Avoid unused method parameters such as 'pFilter'. 406
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 419

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 ..; 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/admindashboard/BlogAdminDashboardComponent.java

Rule Violation Ligne
CommentRequired Header comments are required 2355
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 41

fr/paris/lutece/plugins/blog/web/admindashboard/BlogAdminDashboardJspBean.java

Rule Violation Ligne
CommentRequired Public method and constructor comments are required 2442

fr/paris/lutece/plugins/blog/web/insertservice/AbstractBlogInsertServiceJspBean.java

Rule Violation Ligne
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 114

fr/paris/lutece/plugins/blog/web/insertservice/BlogPortletInsertServiceJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 50105

fr/paris/lutece/plugins/blog/web/insertservice/BlogResourceInsertServiceJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 49169

fr/paris/lutece/plugins/blog/web/insertservice/BlogUrlInsertServiceJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 4798

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. 34596
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsChecked' 143
RedundantFieldInitializer Avoid using redundant field initializer for '_nIsUnpublished' 145
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 148
AddEmptyString Do not add empty strings 204
CyclomaticComplexity The method 'filterBlogIds(HttpServletRequest)' has a cyclomatic complexity of 12. 227271
ConfusingTernary Avoid if (x != y) ..; else ..; 281312
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 306
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. 309
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 310
CommentRequired Public method and constructor comments are required 315346
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 317
ConfusingTernary Avoid if (x != y) ..; else ..; 360
ConfusingTernary Avoid if (x != y) ..; else ..; 380381
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 405
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 415
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 418
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 447
CyclomaticComplexity The method 'doModify(HttpServletRequest)' has a cyclomatic complexity of 11. 465526
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 480
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 483
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 518

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. 147
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 151
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 170
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 195
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 217
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 220

Priority 4

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

Rule Violation Ligne
ShortClassName Avoid short class names like Blog 54661
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. 67
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 104
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 114117
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 186189

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. 111

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

Rule Violation Ligne
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'fr.paris.lutece.portal.service.plugin.Plugin' due to existing import 'fr.paris.lutece.portal.service.plugin.Plugin' 236

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' 88

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. 66
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 87

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

Rule Violation Ligne
UnusedImports Avoid unused imports such as 'org.apache.lucene.document.IntPoint' 48
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' 149
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 222
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 225
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 225

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

Rule Violation Ligne
DuplicateImports Avoid duplicate imports such as 'fr.paris.lutece.portal.service.i18n.I18nService' 100
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 2204
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 2207
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 2207

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

Rule Violation Ligne
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 285
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 286
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 294
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 299

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

Rule Violation Ligne
UnusedImports Avoid unused imports such as 'java.sql.Date' 36
UnusedImports Avoid unused imports such as 'java.text.SimpleDateFormat' 38
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.blog.web.BlogPublicationJspBean' 62

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' 253
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' 253

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 34661
TooManyFields Too many fields 3 54661
ShortClassName Avoid short class names like Blog 4 54661
GodClass Possible God Class (WMC=47, ATFD=20, TCC=3.243%) 3 54661
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 52951
CyclomaticComplexity The method 'getDaoFromFilter(String, BlogFilter)' has a cyclomatic complexity of 31. 3 651787
ExcessiveMethodLength Avoid really long methods. 3 651787
NPathComplexity The method 'getDaoFromFilter(String, BlogFilter)' has an NPath complexity of 1064448 3 651787
NcssCount The method 'getDaoFromFilter(String, BlogFilter)' has a NCSS line count of 65. 3 651787
ShortVariable Avoid variables with short names like i 3 660
ConfusingTernary Avoid if (x != y) ..; else ..; 3 682
ConfusingTernary Avoid if (x != y) ..; else ..; 3 700
ConfusingTernary Avoid if (x != y) ..; else ..; 3 705
ConfusingTernary Avoid if (x != y) ..; else ..; 3 710
ConfusingTernary Avoid if (x != y) ..; else ..; 3 714
ConfusingTernary Avoid if (x != y) ..; else ..; 3 718
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 724
ConfusingTernary Avoid if (x != y) ..; else ..; 3 724
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 725
ConfusingTernary Avoid if (x != y) ..; else ..; 3 731
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 731
ConfusingTernary Avoid if (x != y) ..; else ..; 3 737
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 737
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 745
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 751
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 755
GuardLogStatement Logger calls should be surrounded by log level guards. 2 758
GuardLogStatement Logger calls should be surrounded by log level guards. 2 770
GuardLogStatement Logger calls should be surrounded by log level guards. 2 781

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

Rule Violation Priority Ligne
MethodReturnsInternalArray Returning '_arrayTagsId' may expose an internal array. 3 60
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 67
ArrayIsStoredDirectly The user-supplied array 'arrayTagsId' is stored directly. 3 67
MethodReturnsInternalArray Returning '_arrayId' may expose an internal array. 3 97
ArrayIsStoredDirectly The user-supplied array 'arrayId' is stored directly. 3 104
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 104
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 114117
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 186189

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

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

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

Rule Violation Priority Ligne
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 Ligne
MethodReturnsInternalArray Returning '_strTag' may expose an internal array. 3 102
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 111
ArrayIsStoredDirectly The user-supplied array 'strTag' is stored directly. 3 111

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

Rule Violation Priority Ligne
DataClass The class 'BlogSerializable' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=6) 3 656
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 656
CommentRequired Header comments are required 3 656
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the id field if you want a default access modifier 3 8
DefaultPackage Use explicit scoping instead of the default package private level 3 8
FieldNamingConventions The field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' 1 8
ShortVariable Avoid variables with short names like id 3 8
DefaultPackage Use explicit scoping instead of the default package private level 3 9
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the contentLabel field if you want a default access modifier 3 9
FieldNamingConventions The field name 'contentLabel' doesn't match '_[a-z][a-zA-Z0-9]*' 1 9
CommentRequired Public method and constructor comments are required 3 1113
UncommentedEmptyConstructor Document empty constructor 3 1113
CommentRequired Public method and constructor comments are required 3 1519
ShortVariable Avoid variables with short names like id 3 15
ShortVariable Avoid variables with short names like id 3 34

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 46293

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

Rule Violation Priority Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 45239
CommentRequired Public method and constructor comments are required 3 155
CommentRequired Public method and constructor comments are required 3 234
CommentRequired Public method and constructor comments are required 3 236
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'fr.paris.lutece.portal.service.plugin.Plugin' due to existing import 'fr.paris.lutece.portal.service.plugin.Plugin' 4 236
CommentRequired Public method and constructor comments are required 3 238

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
CommentRequired Public method and constructor comments are required 3 5659
UncommentedEmptyConstructor Document empty constructor 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/insertservice/BlogLinkPOJO.java

Rule Violation Priority Ligne
CommentRequired Public method and constructor comments are required 3 4245
CommentRequired Public method and constructor comments are required 3 4750
CommentRequired Public method and constructor comments are required 3 5255
CommentRequired Public method and constructor comments are required 3 5760
CommentRequired Public method and constructor comments are required 3 6265
CommentRequired Public method and constructor comments are required 3 6770
CommentRequired Public method and constructor comments are required 3 7275
CommentRequired Public method and constructor comments are required 3 7780

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 52396
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 132
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 213
ShortVariable Avoid variables with short names like p 3 263
CyclomaticComplexity The method 'selectPortletByType(int, PortletOrder, PortletFilter)' has a cyclomatic complexity of 10. 3 321375
ConfusingTernary Avoid if (x != y) ..; else ..; 3 326

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 6770
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 72
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 74
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 76
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.GregorianCalendar' due to existing import 'java.util.GregorianCalendar' 4 88

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 196246
ConfusingTernary Avoid if (x != y) ..; else ..; 3 201

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

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 66
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 66
MethodReturnsInternalArray Returning '_aPageName' may expose an internal array. 3 78
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 87
ArrayIsStoredDirectly The user-supplied array 'aPortletTitle' is stored directly. 3 87
MethodReturnsInternalArray Returning '_aPortletName' may expose an internal array. 3 99
CyclomaticComplexity The method 'getSQLFilter()' has a cyclomatic complexity of 13. 3 170219

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 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 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 47122
ShortVariable Avoid variables with short names like rt 3 62
ShortVariable Avoid variables with short names like p 3 68

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

Rule Violation Priority Ligne
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 3 94
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 3 97
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 3 116
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 138

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

Rule Violation Priority Ligne
GodClass Possible God Class (WMC=57, ATFD=99, TCC=0.000%) 3 60532
TooManyMethods This class has too many methods, consider refactoring it. 3 61532
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 119
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 169
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 224
ConfusingTernary Avoid if (x != y) ..; else ..; 3 241252
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 284
CollapsibleIfStatements These nested if statements could be combined 3 303307
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 345
CommentRequired Public method and constructor comments are required 3 506513
CommentRequired Public method and constructor comments are required 3 515522
CommentRequired Public method and constructor comments are required 3 524531

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

Rule Violation Priority Ligne
ShortVariable Avoid variables with short names like e 3 216
CollapsibleIfStatements These nested if statements could be combined 3 222230
ShortVariable Avoid variables with short names like e 3 258

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=8, NOAM=0, WMC=1) 3 4391
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 74
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 79
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 84
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 89

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 34512
AvoidUsingVolatile Use of modifier volatile is not recommended. 2 93
ImmutableField Private field '_indexer' could be made final; it is only initialized in the declaration or constructor. 3 103
ImmutableField Private field '_nWriterMergeFactor' could be made final; it is only initialized in the declaration or constructor. 3 104
NonThreadSafeSingleton Singleton is not thread safe 3 145148
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 147 characters appended. 3 197
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 203
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 244
CommentRequired Public method and constructor comments are required 3 270285
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 280
ShortVariable Avoid variables with short names like si 3 324
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 329
CyclomaticComplexity The method 'prepareQueryForFilter(BlogSearchFilter)' has a cyclomatic complexity of 14. 3 338436
NPathComplexity The method 'prepareQueryForFilter(BlogSearchFilter)' has an NPath complexity of 1296 3 338436
NcssCount The method 'prepareQueryForFilter(BlogSearchFilter)' has a NCSS line count of 64. 3 338436
CollapsibleIfStatements These nested if statements could be combined 3 418425
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 434
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 434
OptimizableToArrayCall This call to Collection.toArray() may be optimizable 3 435

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 34425
UnusedImports Avoid unused imports such as 'org.apache.lucene.document.IntPoint' 4 48
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 140211
ConfusingTernary Avoid if (x != y) ..; else ..; 3 145208
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 149
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 198
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 4 222
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 4 225
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' 4 225
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 229
ShortVariable Avoid variables with short names like ft 3 234
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 293
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 294
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 295
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 296
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 297
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 298

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

Rule Violation Priority Ligne
CommentRequired Public method and constructor comments are required 3 49

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 110
AssignmentInOperand Avoid assignments in operands 3 110
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 171172
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 220
CommentRequired Public method and constructor comments are required 3 244247

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 108
ShortVariable Avoid variables with short names like t 3 119

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 342212
DuplicateImports Avoid duplicate imports such as 'fr.paris.lutece.portal.service.i18n.I18nService' 4 100
ExcessiveClassLength Avoid really long classes. 3 1242212
TooManyFields Too many fields 3 1242212
TooManyMethods This class has too many methods, consider refactoring it. 3 1252212
UnusedPrivateField Avoid unused private fields such as 'MARK_NUMBER_MANDATORY_TAGS'. 3 206
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 299
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsChecked' 3 301
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 3 309
FieldNamingConventions The final field name '_blogServiceSession' doesn't match '[a-z][a-zA-Z0-9]*' 1 317
NPathComplexity The method 'getManageBlogs(HttpServletRequest)' has an NPath complexity of 36240 3 327516
CyclomaticComplexity The method 'getManageBlogs(HttpServletRequest)' has a cyclomatic complexity of 28. 3 327516
NcssCount The method 'getManageBlogs(HttpServletRequest)' has a NCSS line count of 113. 3 327516
ExcessiveMethodLength Avoid really long methods. 3 327516
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 329
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 3 336
ConfusingTernary Avoid if (x != y) ..; else ..; 3 345373
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 367
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 368
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 369
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 370
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 390
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 482
ConfusingTernary Avoid if (x != y) ..; else ..; 3 483491
AddEmptyString Do not add empty strings 3 501
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 525
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 528
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 555
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 556
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 591
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 716
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 765
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 806
CyclomaticComplexity The method 'doUpdatePriorityTag(HttpServletRequest)' has a cyclomatic complexity of 12. 3 849910
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 851
ShortVariable Avoid variables with short names like tg 3 851
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 852
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 853
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 854
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 856
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 857
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 884
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 892
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 925
CommentRequired Public method and constructor comments are required 3 9951019
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1040
ConfusingTernary Avoid if (x != y) ..; else ..; 3 10451067
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1148
ConfusingTernary Avoid if (x != y) ..; else ..; 3 12171224
ConfusingTernary Avoid if (x != y) ..; else ..; 3 12571264
ShortVariable Avoid variables with short names like tf 3 1289
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 1310
CyclomaticComplexity The method 'addContent(HttpServletRequest)' has a cyclomatic complexity of 14. 3 13361443
ExcessiveMethodLength Avoid really long methods. 3 13361443
NPathComplexity The method 'addContent(HttpServletRequest)' has an NPath complexity of 720 3 13361443
ShortVariable Avoid variables with short names like is 3 1387
ConfusingTernary Avoid if (x != y) ..; else ..; 3 14141439
LocalVariableNamingConventions The local variable name '_nDocContentInSession' doesn't match '[a-z][a-zA-Z0-9]*' 1 1426
ConfusingTernary Avoid if (x != y) ..; else ..; 3 14631500
NPathComplexity The method 'doUpdatePriorityContent(HttpServletRequest)' has an NPath complexity of 1080 3 15121587
CyclomaticComplexity The method 'doUpdatePriorityContent(HttpServletRequest)' has a cyclomatic complexity of 16. 3 15121587
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1514
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1515
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1516
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1517
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1519
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1520
ConfusingTernary Avoid if (x != y) ..; else ..; 3 15271546
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 1558
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 1566
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1598
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1599
LinguisticNaming Linguistics Antipattern - The setter 'setContent' should not return any type except void linguistically 3 16411667
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 1727
UnusedLocalVariable Avoid unused local variables such as 'adminUser'. 3 1727
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 1747
CyclomaticComplexity The method 'getConfirmArchiveBlogs(HttpServletRequest)' has a cyclomatic complexity of 10. 3 18291876
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 1862
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 1909
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 1968
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 1979
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 1996
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 2039
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 2043
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 21012117
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 21252142
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 4 2204
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 4 2207
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' 4 2207

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 34529
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 149
CyclomaticComplexity The method 'getManageBlogPublication(HttpServletRequest)' has a cyclomatic complexity of 12. 3 150212
NPathComplexity The method 'getManageBlogPublication(HttpServletRequest)' has an NPath complexity of 360 3 150212
ConfusingTernary Avoid if (x != y) ..; else ..; 3 154
ConfusingTernary Avoid if (x != y) ..; else ..; 3 281
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 4 285
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 4 286
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 3 288
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 4 294
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' 4 299
LinguisticNaming Linguistics Antipattern - The setter 'setFillFilter' should not return any type except void linguistically 3 319358
ConfusingTernary Avoid if (x != y) ..; else ..; 3 324355
UnusedFormalParameter Avoid unused method parameters such as 'pFilter'. 3 406
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 419

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
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/admindashboard/BlogAdminDashboardComponent.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 2355
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 41

fr/paris/lutece/plugins/blog/web/admindashboard/BlogAdminDashboardJspBean.java

Rule Violation Priority Ligne
CommentRequired Public method and constructor comments are required 3 2442

fr/paris/lutece/plugins/blog/web/insertservice/AbstractBlogInsertServiceJspBean.java

Rule Violation Priority Ligne
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 114

fr/paris/lutece/plugins/blog/web/insertservice/BlogPortletInsertServiceJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 50105

fr/paris/lutece/plugins/blog/web/insertservice/BlogResourceInsertServiceJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 49169

fr/paris/lutece/plugins/blog/web/insertservice/BlogUrlInsertServiceJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 4798

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 34596
UnusedImports Avoid unused imports such as 'java.sql.Date' 4 36
UnusedImports Avoid unused imports such as 'java.text.SimpleDateFormat' 4 38
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.blog.web.BlogPublicationJspBean' 4 62
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsChecked' 3 143
RedundantFieldInitializer Avoid using redundant field initializer for '_nIsUnpublished' 3 145
RedundantFieldInitializer Avoid using redundant field initializer for '_bIsSorted' 3 148
AddEmptyString Do not add empty strings 3 204
CyclomaticComplexity The method 'filterBlogIds(HttpServletRequest)' has a cyclomatic complexity of 12. 3 227271
ConfusingTernary Avoid if (x != y) ..; else ..; 3 281312
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 306
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 309
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 310
CommentRequired Public method and constructor comments are required 3 315346
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 317
ConfusingTernary Avoid if (x != y) ..; else ..; 3 360
ConfusingTernary Avoid if (x != y) ..; else ..; 3 380381
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 405
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 415
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 418
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 447
CyclomaticComplexity The method 'doModify(HttpServletRequest)' has a cyclomatic complexity of 11. 3 465526
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 480
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 483
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 518

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 147
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 151
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 170
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 195
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 217
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 220
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' 4 253
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' 4 253