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/rss/service/AutoUpdatingRss.java

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

fr/paris/lutece/plugins/rss/service/RssAdvanceParamService.java

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

fr/paris/lutece/plugins/rss/service/RssContentLoader.java

Rule Violation Ligne
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 170
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 226

fr/paris/lutece/plugins/rss/service/RssGeneratorService.java

Rule Violation Ligne
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 259

fr/paris/lutece/plugins/rss/service/type/AbstractFeedTypeProvider.java

Rule Violation Ligne
AbstractClassWithoutAnyMethod No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 4245

fr/paris/lutece/plugins/rss/web/RssFeedsJspBean.java

Rule Violation Ligne
LocalVariableNamingConventions The local variable name 'paginator_active' doesn't match '[a-z][a-zA-Z0-9]*' 181
LocalVariableNamingConventions The local variable name 'paginator_inactive' doesn't match '[a-z][a-zA-Z0-9]*' 188

fr/paris/lutece/plugins/rss/web/RssJspBean.java

Rule Violation Ligne
FormalParameterNamingConventions The method parameter name 'rss_type' doesn't match '[a-z][a-zA-Z0-9]*' 945
FormalParameterNamingConventions The method parameter name 'id_rss' doesn't match '[a-z][a-zA-Z0-9]*' 964
FormalParameterNamingConventions The method parameter name 'rss_type' doesn't match '[a-z][a-zA-Z0-9]*' 965

Priority 2

fr/paris/lutece/plugins/rss/business/RssFeedHome.java

Rule Violation Ligne
AvoidReassigningParameters Avoid reassigning parameters such as 'rssFeed' 102

fr/paris/lutece/plugins/rss/web/RssJspBean.java

Rule Violation Ligne
StringInstantiation Avoid instantiating String objects; this is usually unnecessary. 947
StringInstantiation Avoid instantiating String objects; this is usually unnecessary. 967

Priority 3

fr/paris/lutece/plugins/rss/business/IRssGeneratedFileDAO.java

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

fr/paris/lutece/plugins/rss/business/RssFeed.java

Rule Violation Ligne
MissingOverride The method 'getWorkgroup()' is missing an @Override annotation. 193196

fr/paris/lutece/plugins/rss/business/RssFeedDAO.java

Rule Violation Ligne
MissingOverride The method 'newPrimaryKey(boolean)' is missing an @Override annotation. 7491
MissingOverride The method 'insert(RssFeed)' is missing an @Override annotation. 97114
MissingOverride The method 'load(int, boolean)' is missing an @Override annotation. 122147
MissingOverride The method 'delete(RssFeed)' is missing an @Override annotation. 153161
MissingOverride The method 'store(RssFeed)' is missing an @Override annotation. 167181
MissingOverride The method 'storeLastFetchInfos(RssFeed)' is missing an @Override annotation. 187199
MissingOverride The method 'selectRssFeeds(boolean)' is missing an @Override annotation. 206233
MissingOverride The method 'checkUrlNotUsed(String)' is missing an @Override annotation. 240257
MissingOverride The method 'selectRssFeedReferenceList(boolean)' is missing an @Override annotation. 264282

fr/paris/lutece/plugins/rss/business/RssFeedHome.java

Rule Violation Ligne
NPathComplexity The method 'updateOrder(RssFeed, int)' has an NPath complexity of 480 184253
CyclomaticComplexity The method 'updateOrder(RssFeed, int)' has a cyclomatic complexity of 17. 184253
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 185
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 193
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 205
LinguisticNaming Linguistics Antipattern - The setter 'setActive' should not return any type except void linguistically 261267

fr/paris/lutece/plugins/rss/business/RssGeneratedFile.java

Rule Violation Ligne
NonThreadSafeSingleton Singleton is not thread safe 7074
MissingOverride The method 'getWorkgroup()' is missing an @Override annotation. 207210

fr/paris/lutece/plugins/rss/business/RssGeneratedFileDAO.java

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 51462
MissingOverride The method 'insert(RssGeneratedFile)' is missing an @Override annotation. 116136
MissingOverride The method 'delete(int)' is missing an @Override annotation. 142149
MissingOverride The method 'store(RssGeneratedFile)' is missing an @Override annotation. 155173
MissingOverride The method 'updateState(RssGeneratedFile)' is missing an @Override annotation. 179189
MissingOverride The method 'load(int)' is missing an @Override annotation. 196224
MissingOverride The method 'checkExistPushrssByPortlet(int)' is missing an @Override annotation. 231248
MissingOverride The method 'checkRssFileFileName(String)' is missing an @Override annotation. 255271
MissingOverride The method 'selectRssFileList()' is missing an @Override annotation. 277304
MissingOverride The method 'selectRssPortlets()' is missing an @Override annotation. 310329
MissingOverride The method 'selectAllRssPortlets()' is missing an @Override annotation. 335356
MissingOverride The method 'selectRssFilePortletName(int)' is missing an @Override annotation. 363379
MissingOverride The method 'checkRssFilePortlet(int)' is missing an @Override annotation. 386403
MissingOverride The method 'selectDocumentsByPortlet(int)' is missing an @Override annotation. 411435
MissingOverride The method 'selectXslFile(int)' is missing an @Override annotation. 442461

fr/paris/lutece/plugins/rss/business/RssGeneratedFileHome.java

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

fr/paris/lutece/plugins/rss/business/RssGeneratedFilePortletRemovalListener.java

Rule Violation Ligne
MissingOverride The method 'canBeRemoved(String)' is missing an @Override annotation. 5573
MissingOverride The method 'getRemovalRefusedMessage(String, Locale)' is missing an @Override annotation. 8185

fr/paris/lutece/plugins/rss/business/parameter/RssFeedParameterDAO.java

Rule Violation Ligne
MissingOverride The method 'load(String, Plugin)' is missing an @Override annotation. 5774
MissingOverride The method 'store(ReferenceItem, Plugin)' is missing an @Override annotation. 7988
MissingOverride The method 'selectAll(Plugin)' is missing an @Override annotation. 93111

fr/paris/lutece/plugins/rss/business/parameter/RssFeedParameterHome.java

Rule Violation Ligne
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 4884

fr/paris/lutece/plugins/rss/business/portlet/IRssListPortletDAO.java

Rule Violation Ligne
MissingOverride The method 'delete(int)' is missing an @Override annotation. 52
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 60
MissingOverride The method 'load(int)' is missing an @Override annotation. 68
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 76

fr/paris/lutece/plugins/rss/business/portlet/IRssPortletDAO.java

Rule Violation Ligne
MissingOverride The method 'delete(int)' is missing an @Override annotation. 54
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 62
MissingOverride The method 'load(int)' is missing an @Override annotation. 70
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 78

fr/paris/lutece/plugins/rss/business/portlet/RssListPortlet.java

Rule Violation Ligne
CallSuperInConstructor It is a good practice to call super() in a constructor 5861
MissingOverride The method 'getXmlDocument(HttpServletRequest)' is missing an @Override annotation. 6972
MissingOverride The method 'getXml(HttpServletRequest)' is missing an @Override annotation. 8097
MissingOverride The method 'remove()' is missing an @Override annotation. 110113

fr/paris/lutece/plugins/rss/business/portlet/RssListPortletDAO.java

Rule Violation Ligne
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 6067
ShortVariable Avoid variables with short names like p 62
MissingOverride The method 'delete(int)' is missing an @Override annotation. 7480
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 8796
ShortVariable Avoid variables with short names like p 89
MissingOverride The method 'load(int)' is missing an @Override annotation. 103118

fr/paris/lutece/plugins/rss/business/portlet/RssListPortletHome.java

Rule Violation Ligne
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 52
CallSuperInConstructor It is a good practice to call super() in a constructor 5763
MissingOverride The method 'getPortletTypeId()' is missing an @Override annotation. 7076
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strPortletTypeId' 75
NonThreadSafeSingleton Singleton is not thread safe 8588
MissingOverride The method 'getDAO()' is missing an @Override annotation. 98101

fr/paris/lutece/plugins/rss/business/portlet/RssPortlet.java

Rule Violation Ligne
CallSuperInConstructor It is a good practice to call super() in a constructor 5659
MissingOverride The method 'getXmlDocument(HttpServletRequest)' is missing an @Override annotation. 8790
MissingOverride The method 'getXml(HttpServletRequest)' is missing an @Override annotation. 98116
MissingOverride The method 'remove()' is missing an @Override annotation. 129132

fr/paris/lutece/plugins/rss/business/portlet/RssPortletDAO.java

Rule Violation Ligne
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 6472
ShortVariable Avoid variables with short names like p 66
MissingOverride The method 'delete(int)' is missing an @Override annotation. 7985
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 92102
ShortVariable Avoid variables with short names like p 94
MissingOverride The method 'load(int)' is missing an @Override annotation. 109125
MissingOverride The method 'checkLinkedPortlet(int)' is missing an @Override annotation. 132151
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 138

fr/paris/lutece/plugins/rss/business/portlet/RssPortletHome.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
MissingOverride The method 'getPortletTypeId()' is missing an @Override annotation. 7177
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strPortletTypeId' 76
NonThreadSafeSingleton Singleton is not thread safe 8689
MissingOverride The method 'getDAO()' is missing an @Override annotation. 99102

fr/paris/lutece/plugins/rss/business/resourceRss/IResourceRssFactory.java

Rule Violation Ligne
PackageCase Package name contains upper case characters 34

fr/paris/lutece/plugins/rss/business/resourceRss/ResourceRssFactory.java

Rule Violation Ligne
PackageCase Package name contains upper case characters 34
MissingOverride The method 'getResourceRss(String)' is missing an @Override annotation. 53101
IdenticalCatchBranches 'catch' branch identical to 'ClassNotFoundException' branch 8792
IdenticalCatchBranches 'catch' branch identical to 'ClassNotFoundException' branch 9397
MissingOverride The method 'getAllResourceRssType()' is missing an @Override annotation. 107110

fr/paris/lutece/plugins/rss/service/AutoUpdatingRss.java

Rule Violation Ligne
CyclomaticComplexity The method 'processUpdatingRssFileFile(Plugin)' has a cyclomatic complexity of 11. 73160
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 73160
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 77
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 103
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 153

fr/paris/lutece/plugins/rss/service/RssAdvanceParamService.java

Rule Violation Ligne
CommentRequired Header comments are required 4697
UnusedPrivateField Avoid unused private fields such as 'PROPERTY_LOADERS'. 51
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 86
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 86

fr/paris/lutece/plugins/rss/service/RssContentLoader.java

Rule Violation Ligne
MissingOverride The method 'getResources()' is missing an @Override annotation. 7589
ShortVariable Avoid variables with short names like i 79
AddEmptyString Do not add empty strings 84
MissingOverride The method 'getResource(String)' is missing an @Override annotation. 96118
AssignmentInOperand Avoid assignments in operands 174
IdenticalCatchBranches 'catch' branch identical to 'FileNotFoundException' branch 185188
AddEmptyString Do not add empty strings 215
ShortVariable Avoid variables with short names like i 245
InefficientStringBuffering Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 260

fr/paris/lutece/plugins/rss/service/RssContentService.java

Rule Violation Ligne
CallSuperInConstructor It is a good practice to call super() in a constructor 5357

fr/paris/lutece/plugins/rss/service/RssFeedsService.java

Rule Violation Ligne
CommentRequired Header comments are required 4358
CommentRequired Public method and constructor comments are required 4952
CommentRequired Public method and constructor comments are required 5457

fr/paris/lutece/plugins/rss/service/RssGeneratorService.java

Rule Violation Ligne
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 168
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 178
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 178
AvoidCatchingNPE Avoid catching NullPointerException; consider removing the cause of the NPE. 277
IdenticalCatchBranches 'catch' branch identical to 'IOException' branch 277280
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 277
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 304
AvoidCatchingNPE Avoid catching NullPointerException; consider removing the cause of the NPE. 304
ShortVariable Avoid variables with short names like sb 316
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 79, but has at least 86 characters appended. 316

fr/paris/lutece/plugins/rss/service/RssParsingException.java

Rule Violation Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 4051
ShortVariable Avoid variables with short names like e 47

fr/paris/lutece/plugins/rss/service/RssPlugin.java

Rule Violation Ligne
MissingOverride The method 'init()' is missing an @Override annotation. 4953

fr/paris/lutece/plugins/rss/service/RssService.java

Rule Violation Ligne
NonThreadSafeSingleton Singleton is not thread safe 7881
MissingOverride The method 'getRssContent(int)' is missing an @Override annotation. 91105
CommentRequired Public method and constructor comments are required 107110
MissingOverride The method 'getRefListRssFeed()' is missing an @Override annotation. 107110
CommentRequired Public method and constructor comments are required 112115

fr/paris/lutece/plugins/rss/service/daemon/FetchRssFeedsDaemon.java

Rule Violation Ligne
MissingOverride The method 'run()' is missing an @Override annotation. 5255

fr/paris/lutece/plugins/rss/service/daemon/GenerateRssFilesDaemon.java

Rule Violation Ligne
MissingOverride The method 'run()' is missing an @Override annotation. 5358

fr/paris/lutece/plugins/rss/service/daemon/ResetRssFeedsMemoryCacheDaemon.java

Rule Violation Ligne
MissingOverride The method 'run()' is missing an @Override annotation. 5257

fr/paris/lutece/plugins/rss/service/type/atom/AbstractAtomFeedTypeProvider.java

Rule Violation Ligne
MissingOverride The method 'getWireFeed(String, IFeedResource, String, int)' is missing an @Override annotation. 7284

fr/paris/lutece/plugins/rss/service/type/atom/DefaultAtomFeedTypeProvider.java

Rule Violation Ligne
MissingOverride The method 'isInvoked(String)' is missing an @Override annotation. 5962
MissingOverride The method 'getATOMEntries(List, int)' is missing an @Override annotation. 6792
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 69
MissingOverride The method 'getATOMEntry(IFeedResourceItem)' is missing an @Override annotation. 97114

fr/paris/lutece/plugins/rss/service/type/rss/AbstractRssFeedTypeProvider.java

Rule Violation Ligne
MissingOverride The method 'getWireFeed(String, IFeedResource, String, int)' is missing an @Override annotation. 7397

fr/paris/lutece/plugins/rss/service/type/rss/DefaultRssFeedTypeProvider.java

Rule Violation Ligne
MissingOverride The method 'isInvoked(String)' is missing an @Override annotation. 6063
MissingOverride The method 'getRSSItems(List, int)' is missing an @Override annotation. 6893
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 70
MissingOverride The method 'getRSSItem(IFeedResourceItem)' is missing an @Override annotation. 98119

fr/paris/lutece/plugins/rss/service/type/rss/Rss09FeedTypeProvider.java

Rule Violation Ligne
MissingOverride The method 'isInvoked(String)' is missing an @Override annotation. 6265
MissingOverride The method 'getRSSItems(List, int)' is missing an @Override annotation. 7095
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 72
MissingOverride The method 'getRSSItem(IFeedResourceItem)' is missing an @Override annotation. 100130

fr/paris/lutece/plugins/rss/web/RssAdminDashboardComponent.java

Rule Violation Ligne
CommentRequired Header comments are required 5070

fr/paris/lutece/plugins/rss/web/RssFeedInclude.java

Rule Violation Ligne
MissingOverride The method 'fillTemplate(Map, PageData, int, HttpServletRequest)' is missing an @Override annotation. 110128
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 115
UselessStringValueOf No need to call String.valueOf to append to a string. 124
CommentRequired Public method and constructor comments are required 130133
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 144
ConfusingTernary Avoid if (x != y) ..; else ..; 150185
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 177
ConfusingTernary Avoid if (x != y) ..; else ..; 206213

fr/paris/lutece/plugins/rss/web/RssFeedsJspBean.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34595
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 81595
UnusedPrivateField Avoid unused private fields such as 'MARK_RSS_INCLUDE_TAG'. 123
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 166
UselessStringValueOf No need to call String.valueOf to append to a string. 173
AddEmptyString Do not add empty strings 193
AddEmptyString Do not add empty strings 194
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 271
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 374
ConfusingTernary Avoid if (x != y) ..; else ..; 475489
ConfusingTernary Avoid if (x != y) ..; else ..; 481488
ConfusingTernary Avoid if (x != y) ..; else ..; 503504
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 558

fr/paris/lutece/plugins/rss/web/RssInclude.java

Rule Violation Ligne
MissingOverride The method 'fillTemplate(Map, PageData, int, HttpServletRequest)' is missing an @Override annotation. 83104
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 85
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 87

fr/paris/lutece/plugins/rss/web/RssJspBean.java

Rule Violation Ligne
CyclomaticComplexity The class 'RssJspBean' has a total cyclomatic complexity of 102 (highest 15). 741181
ExcessiveClassLength Avoid really long classes. 741181
GodClass Possible God Class (WMC=102, ATFD=241, TCC=18.615%) 741181
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 741181
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 171
AddEmptyString Do not add empty strings 173
NPathComplexity The method 'doModifyRssFilePortlet(HttpServletRequest)' has an NPath complexity of 288 206302
CyclomaticComplexity The method 'doModifyRssFilePortlet(HttpServletRequest)' has a cyclomatic complexity of 10. 206302
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 215
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 216
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. 218
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 220
NPathComplexity The method 'doModifyRssFileResource(HttpServletRequest)' has an NPath complexity of 3072 310440
CyclomaticComplexity The method 'doModifyRssFileResource(HttpServletRequest)' has a cyclomatic complexity of 14. 310440
ExcessiveMethodLength Avoid really long methods. 310440
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 321
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 323
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 324
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 325
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 326
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 327
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 352
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 354
NPathComplexity The method 'doCreateRssFile(HttpServletRequest)' has an NPath complexity of 400 448572
CyclomaticComplexity The method 'doCreateRssFile(HttpServletRequest)' has a cyclomatic complexity of 14. 448572
ExcessiveMethodLength Avoid really long methods. 448572
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 456
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 480
ConfusingTernary Avoid if (x != y) ..; else ..; 482489
ConfusingTernary Avoid if (x != y) ..; else ..; 491498
ConfusingTernary Avoid if (x != y) ..; else ..; 500507
ConfusingTernary Avoid if (x != y) ..; else ..; 509516
ConfusingTernary Avoid if (x != y) ..; else ..; 518525
ConfusingTernary Avoid if (x != y) ..; else ..; 542549
CyclomaticComplexity The method 'doCreateRssFilePortlet(HttpServletRequest)' has a cyclomatic complexity of 14. 615726
ExcessiveMethodLength Avoid really long methods. 615726
NPathComplexity The method 'doCreateRssFilePortlet(HttpServletRequest)' has an NPath complexity of 3456 615726
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 623
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 624
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 625
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 626
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 627
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 628
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 635
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 658
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 660
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 740
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 772
CyclomaticComplexity The method 'doCreateRssFileResource(HttpServletRequest)' has a cyclomatic complexity of 15. 805927
ExcessiveMethodLength Avoid really long methods. 805927
NPathComplexity The method 'doCreateRssFileResource(HttpServletRequest)' has an NPath complexity of 6144 805927
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 812
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 813
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 814
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 815
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 816
PositionLiteralsFirstInComparisons Position literals first in String comparisons 828
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 846
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 848
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 944945
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 951
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 964965
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 971
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 994
SimplifyStartsWith This call to String.startsWith can be rewritten using String.charAt(0) 1067
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 1073
ConfusingTernary Avoid if (x != y) ..; else ..; 11351149
ConfusingTernary Avoid if (x != y) ..; else ..; 11411148
ConfusingTernary Avoid if (x != y) ..; else ..; 1162

fr/paris/lutece/plugins/rss/web/RssListApp.java

Rule Violation Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 5688
MissingOverride The method 'getPage(HttpServletRequest, int, Plugin)' is missing an @Override annotation. 6687
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 72
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 72

fr/paris/lutece/plugins/rss/web/RssListInclude.java

Rule Violation Ligne
MissingOverride The method 'fillTemplate(Map, PageData, int, HttpServletRequest)' is missing an @Override annotation. 7382
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 75

fr/paris/lutece/plugins/rss/web/filter/RssGeneratorFilter.java

Rule Violation Ligne
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 82

fr/paris/lutece/plugins/rss/web/portlet/RssListPortletJspBean.java

Rule Violation Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 49181
MissingOverride The method 'getCreate(HttpServletRequest)' is missing an @Override annotation. 8087
MissingOverride The method 'getModify(HttpServletRequest)' is missing an @Override annotation. 94102
MissingOverride The method 'doCreate(HttpServletRequest)' is missing an @Override annotation. 110142
MissingOverride The method 'doModify(HttpServletRequest)' is missing an @Override annotation. 150180

fr/paris/lutece/plugins/rss/web/portlet/RssPortletJspBean.java

Rule Violation Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 56243
MissingOverride The method 'getCreate(HttpServletRequest)' is missing an @Override annotation. 96106
MissingOverride The method 'getModify(HttpServletRequest)' is missing an @Override annotation. 113126
MissingOverride The method 'doCreate(HttpServletRequest)' is missing an @Override annotation. 134176
MissingOverride The method 'doModify(HttpServletRequest)' is missing an @Override annotation. 184225
DefaultPackage Use explicit scoping instead of the default package private level 233242
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getFeedIndexCombo method if you want a default access modifier 233242
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 235

Priority 4

fr/paris/lutece/plugins/rss/business/RssFeed.java

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

Fichiers

fr/paris/lutece/plugins/rss/business/IRssGeneratedFileDAO.java

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

fr/paris/lutece/plugins/rss/business/RssFeed.java

Rule Violation Priority Ligne
MissingOverride The method 'getWorkgroup()' is missing an @Override annotation. 3 193196
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 247250

fr/paris/lutece/plugins/rss/business/RssFeedDAO.java

Rule Violation Priority Ligne
MissingOverride The method 'newPrimaryKey(boolean)' is missing an @Override annotation. 3 7491
MissingOverride The method 'insert(RssFeed)' is missing an @Override annotation. 3 97114
MissingOverride The method 'load(int, boolean)' is missing an @Override annotation. 3 122147
MissingOverride The method 'delete(RssFeed)' is missing an @Override annotation. 3 153161
MissingOverride The method 'store(RssFeed)' is missing an @Override annotation. 3 167181
MissingOverride The method 'storeLastFetchInfos(RssFeed)' is missing an @Override annotation. 3 187199
MissingOverride The method 'selectRssFeeds(boolean)' is missing an @Override annotation. 3 206233
MissingOverride The method 'checkUrlNotUsed(String)' is missing an @Override annotation. 3 240257
MissingOverride The method 'selectRssFeedReferenceList(boolean)' is missing an @Override annotation. 3 264282

fr/paris/lutece/plugins/rss/business/RssFeedHome.java

Rule Violation Priority Ligne
AvoidReassigningParameters Avoid reassigning parameters such as 'rssFeed' 2 102
NPathComplexity The method 'updateOrder(RssFeed, int)' has an NPath complexity of 480 3 184253
CyclomaticComplexity The method 'updateOrder(RssFeed, int)' has a cyclomatic complexity of 17. 3 184253
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 185
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 193
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 205
LinguisticNaming Linguistics Antipattern - The setter 'setActive' should not return any type except void linguistically 3 261267

fr/paris/lutece/plugins/rss/business/RssGeneratedFile.java

Rule Violation Priority Ligne
NonThreadSafeSingleton Singleton is not thread safe 3 7074
MissingOverride The method 'getWorkgroup()' is missing an @Override annotation. 3 207210

fr/paris/lutece/plugins/rss/business/RssGeneratedFileDAO.java

Rule Violation Priority Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 51462
MissingOverride The method 'insert(RssGeneratedFile)' is missing an @Override annotation. 3 116136
MissingOverride The method 'delete(int)' is missing an @Override annotation. 3 142149
MissingOverride The method 'store(RssGeneratedFile)' is missing an @Override annotation. 3 155173
MissingOverride The method 'updateState(RssGeneratedFile)' is missing an @Override annotation. 3 179189
MissingOverride The method 'load(int)' is missing an @Override annotation. 3 196224
MissingOverride The method 'checkExistPushrssByPortlet(int)' is missing an @Override annotation. 3 231248
MissingOverride The method 'checkRssFileFileName(String)' is missing an @Override annotation. 3 255271
MissingOverride The method 'selectRssFileList()' is missing an @Override annotation. 3 277304
MissingOverride The method 'selectRssPortlets()' is missing an @Override annotation. 3 310329
MissingOverride The method 'selectAllRssPortlets()' is missing an @Override annotation. 3 335356
MissingOverride The method 'selectRssFilePortletName(int)' is missing an @Override annotation. 3 363379
MissingOverride The method 'checkRssFilePortlet(int)' is missing an @Override annotation. 3 386403
MissingOverride The method 'selectDocumentsByPortlet(int)' is missing an @Override annotation. 3 411435
MissingOverride The method 'selectXslFile(int)' is missing an @Override annotation. 3 442461

fr/paris/lutece/plugins/rss/business/RssGeneratedFileHome.java

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

fr/paris/lutece/plugins/rss/business/RssGeneratedFilePortletRemovalListener.java

Rule Violation Priority Ligne
MissingOverride The method 'canBeRemoved(String)' is missing an @Override annotation. 3 5573
MissingOverride The method 'getRemovalRefusedMessage(String, Locale)' is missing an @Override annotation. 3 8185

fr/paris/lutece/plugins/rss/business/parameter/RssFeedParameterDAO.java

Rule Violation Priority Ligne
MissingOverride The method 'load(String, Plugin)' is missing an @Override annotation. 3 5774
MissingOverride The method 'store(ReferenceItem, Plugin)' is missing an @Override annotation. 3 7988
MissingOverride The method 'selectAll(Plugin)' is missing an @Override annotation. 3 93111

fr/paris/lutece/plugins/rss/business/parameter/RssFeedParameterHome.java

Rule Violation Priority Ligne
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 3 4884

fr/paris/lutece/plugins/rss/business/portlet/IRssListPortletDAO.java

Rule Violation Priority Ligne
MissingOverride The method 'delete(int)' is missing an @Override annotation. 3 52
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 3 60
MissingOverride The method 'load(int)' is missing an @Override annotation. 3 68
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 3 76

fr/paris/lutece/plugins/rss/business/portlet/IRssPortletDAO.java

Rule Violation Priority Ligne
MissingOverride The method 'delete(int)' is missing an @Override annotation. 3 54
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 3 62
MissingOverride The method 'load(int)' is missing an @Override annotation. 3 70
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 3 78

fr/paris/lutece/plugins/rss/business/portlet/RssListPortlet.java

Rule Violation Priority Ligne
CallSuperInConstructor It is a good practice to call super() in a constructor 3 5861
MissingOverride The method 'getXmlDocument(HttpServletRequest)' is missing an @Override annotation. 3 6972
MissingOverride The method 'getXml(HttpServletRequest)' is missing an @Override annotation. 3 8097
MissingOverride The method 'remove()' is missing an @Override annotation. 3 110113

fr/paris/lutece/plugins/rss/business/portlet/RssListPortletDAO.java

Rule Violation Priority Ligne
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 3 6067
ShortVariable Avoid variables with short names like p 3 62
MissingOverride The method 'delete(int)' is missing an @Override annotation. 3 7480
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 3 8796
ShortVariable Avoid variables with short names like p 3 89
MissingOverride The method 'load(int)' is missing an @Override annotation. 3 103118

fr/paris/lutece/plugins/rss/business/portlet/RssListPortletHome.java

Rule Violation Priority Ligne
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 3 52
CallSuperInConstructor It is a good practice to call super() in a constructor 3 5763
MissingOverride The method 'getPortletTypeId()' is missing an @Override annotation. 3 7076
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strPortletTypeId' 3 75
NonThreadSafeSingleton Singleton is not thread safe 3 8588
MissingOverride The method 'getDAO()' is missing an @Override annotation. 3 98101

fr/paris/lutece/plugins/rss/business/portlet/RssPortlet.java

Rule Violation Priority Ligne
CallSuperInConstructor It is a good practice to call super() in a constructor 3 5659
MissingOverride The method 'getXmlDocument(HttpServletRequest)' is missing an @Override annotation. 3 8790
MissingOverride The method 'getXml(HttpServletRequest)' is missing an @Override annotation. 3 98116
MissingOverride The method 'remove()' is missing an @Override annotation. 3 129132

fr/paris/lutece/plugins/rss/business/portlet/RssPortletDAO.java

Rule Violation Priority Ligne
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 3 6472
ShortVariable Avoid variables with short names like p 3 66
MissingOverride The method 'delete(int)' is missing an @Override annotation. 3 7985
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 3 92102
ShortVariable Avoid variables with short names like p 3 94
MissingOverride The method 'load(int)' is missing an @Override annotation. 3 109125
MissingOverride The method 'checkLinkedPortlet(int)' is missing an @Override annotation. 3 132151
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 138

fr/paris/lutece/plugins/rss/business/portlet/RssPortletHome.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
MissingOverride The method 'getPortletTypeId()' is missing an @Override annotation. 3 7177
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strPortletTypeId' 3 76
NonThreadSafeSingleton Singleton is not thread safe 3 8689
MissingOverride The method 'getDAO()' is missing an @Override annotation. 3 99102

fr/paris/lutece/plugins/rss/business/resourceRss/IResourceRssFactory.java

Rule Violation Priority Ligne
PackageCase Package name contains upper case characters 3 34

fr/paris/lutece/plugins/rss/business/resourceRss/ResourceRssFactory.java

Rule Violation Priority Ligne
PackageCase Package name contains upper case characters 3 34
MissingOverride The method 'getResourceRss(String)' is missing an @Override annotation. 3 53101
IdenticalCatchBranches 'catch' branch identical to 'ClassNotFoundException' branch 3 8792
IdenticalCatchBranches 'catch' branch identical to 'ClassNotFoundException' branch 3 9397
MissingOverride The method 'getAllResourceRssType()' is missing an @Override annotation. 3 107110

fr/paris/lutece/plugins/rss/service/AutoUpdatingRss.java

Rule Violation Priority Ligne
ClassNamingConventions The utility class name 'AutoUpdatingRss' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 55161
CyclomaticComplexity The method 'processUpdatingRssFileFile(Plugin)' has a cyclomatic complexity of 11. 3 73160
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 73160
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 77
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 3 103
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 153

fr/paris/lutece/plugins/rss/service/RssAdvanceParamService.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 4697
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 4697
UnusedPrivateField Avoid unused private fields such as 'PROPERTY_LOADERS'. 3 51
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 86
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 86

fr/paris/lutece/plugins/rss/service/RssContentLoader.java

Rule Violation Priority Ligne
MissingOverride The method 'getResources()' is missing an @Override annotation. 3 7589
ShortVariable Avoid variables with short names like i 3 79
AddEmptyString Do not add empty strings 3 84
MissingOverride The method 'getResource(String)' is missing an @Override annotation. 3 96118
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 170
AssignmentInOperand Avoid assignments in operands 3 174
IdenticalCatchBranches 'catch' branch identical to 'FileNotFoundException' branch 3 185188
AddEmptyString Do not add empty strings 3 215
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 226
ShortVariable Avoid variables with short names like i 3 245
InefficientStringBuffering Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 3 260

fr/paris/lutece/plugins/rss/service/RssContentService.java

Rule Violation Priority Ligne
CallSuperInConstructor It is a good practice to call super() in a constructor 3 5357

fr/paris/lutece/plugins/rss/service/RssFeedsService.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 4358
CommentRequired Public method and constructor comments are required 3 4952
CommentRequired Public method and constructor comments are required 3 5457

fr/paris/lutece/plugins/rss/service/RssGeneratorService.java

Rule Violation Priority Ligne
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 168
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 178
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 178
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 259
AvoidCatchingNPE Avoid catching NullPointerException; consider removing the cause of the NPE. 3 277
IdenticalCatchBranches 'catch' branch identical to 'IOException' branch 3 277280
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 277
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 304
AvoidCatchingNPE Avoid catching NullPointerException; consider removing the cause of the NPE. 3 304
ShortVariable Avoid variables with short names like sb 3 316
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 79, but has at least 86 characters appended. 3 316

fr/paris/lutece/plugins/rss/service/RssParsingException.java

Rule Violation Priority Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 4051
ShortVariable Avoid variables with short names like e 3 47

fr/paris/lutece/plugins/rss/service/RssPlugin.java

Rule Violation Priority Ligne
MissingOverride The method 'init()' is missing an @Override annotation. 3 4953

fr/paris/lutece/plugins/rss/service/RssService.java

Rule Violation Priority Ligne
NonThreadSafeSingleton Singleton is not thread safe 3 7881
MissingOverride The method 'getRssContent(int)' is missing an @Override annotation. 3 91105
CommentRequired Public method and constructor comments are required 3 107110
MissingOverride The method 'getRefListRssFeed()' is missing an @Override annotation. 3 107110
CommentRequired Public method and constructor comments are required 3 112115

fr/paris/lutece/plugins/rss/service/daemon/FetchRssFeedsDaemon.java

Rule Violation Priority Ligne
MissingOverride The method 'run()' is missing an @Override annotation. 3 5255

fr/paris/lutece/plugins/rss/service/daemon/GenerateRssFilesDaemon.java

Rule Violation Priority Ligne
MissingOverride The method 'run()' is missing an @Override annotation. 3 5358

fr/paris/lutece/plugins/rss/service/daemon/ResetRssFeedsMemoryCacheDaemon.java

Rule Violation Priority Ligne
MissingOverride The method 'run()' is missing an @Override annotation. 3 5257

fr/paris/lutece/plugins/rss/service/type/AbstractFeedTypeProvider.java

Rule Violation Priority Ligne
AbstractClassWithoutAnyMethod No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 1 4245

fr/paris/lutece/plugins/rss/service/type/atom/AbstractAtomFeedTypeProvider.java

Rule Violation Priority Ligne
MissingOverride The method 'getWireFeed(String, IFeedResource, String, int)' is missing an @Override annotation. 3 7284

fr/paris/lutece/plugins/rss/service/type/atom/DefaultAtomFeedTypeProvider.java

Rule Violation Priority Ligne
MissingOverride The method 'isInvoked(String)' is missing an @Override annotation. 3 5962
MissingOverride The method 'getATOMEntries(List, int)' is missing an @Override annotation. 3 6792
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 69
MissingOverride The method 'getATOMEntry(IFeedResourceItem)' is missing an @Override annotation. 3 97114

fr/paris/lutece/plugins/rss/service/type/rss/AbstractRssFeedTypeProvider.java

Rule Violation Priority Ligne
MissingOverride The method 'getWireFeed(String, IFeedResource, String, int)' is missing an @Override annotation. 3 7397

fr/paris/lutece/plugins/rss/service/type/rss/DefaultRssFeedTypeProvider.java

Rule Violation Priority Ligne
MissingOverride The method 'isInvoked(String)' is missing an @Override annotation. 3 6063
MissingOverride The method 'getRSSItems(List, int)' is missing an @Override annotation. 3 6893
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 70
MissingOverride The method 'getRSSItem(IFeedResourceItem)' is missing an @Override annotation. 3 98119

fr/paris/lutece/plugins/rss/service/type/rss/Rss09FeedTypeProvider.java

Rule Violation Priority Ligne
MissingOverride The method 'isInvoked(String)' is missing an @Override annotation. 3 6265
MissingOverride The method 'getRSSItems(List, int)' is missing an @Override annotation. 3 7095
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 72
MissingOverride The method 'getRSSItem(IFeedResourceItem)' is missing an @Override annotation. 3 100130

fr/paris/lutece/plugins/rss/web/RssAdminDashboardComponent.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 5070

fr/paris/lutece/plugins/rss/web/RssFeedInclude.java

Rule Violation Priority Ligne
MissingOverride The method 'fillTemplate(Map, PageData, int, HttpServletRequest)' is missing an @Override annotation. 3 110128
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 115
UselessStringValueOf No need to call String.valueOf to append to a string. 3 124
CommentRequired Public method and constructor comments are required 3 130133
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 144
ConfusingTernary Avoid if (x != y) ..; else ..; 3 150185
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 177
ConfusingTernary Avoid if (x != y) ..; else ..; 3 206213

fr/paris/lutece/plugins/rss/web/RssFeedsJspBean.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34595
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 81595
UnusedPrivateField Avoid unused private fields such as 'MARK_RSS_INCLUDE_TAG'. 3 123
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 166
UselessStringValueOf No need to call String.valueOf to append to a string. 3 173
LocalVariableNamingConventions The local variable name 'paginator_active' doesn't match '[a-z][a-zA-Z0-9]*' 1 181
LocalVariableNamingConventions The local variable name 'paginator_inactive' doesn't match '[a-z][a-zA-Z0-9]*' 1 188
AddEmptyString Do not add empty strings 3 193
AddEmptyString Do not add empty strings 3 194
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 271
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 374
ConfusingTernary Avoid if (x != y) ..; else ..; 3 475489
ConfusingTernary Avoid if (x != y) ..; else ..; 3 481488
ConfusingTernary Avoid if (x != y) ..; else ..; 3 503504
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 558

fr/paris/lutece/plugins/rss/web/RssInclude.java

Rule Violation Priority Ligne
MissingOverride The method 'fillTemplate(Map, PageData, int, HttpServletRequest)' is missing an @Override annotation. 3 83104
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 85
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 87

fr/paris/lutece/plugins/rss/web/RssJspBean.java

Rule Violation Priority Ligne
CyclomaticComplexity The class 'RssJspBean' has a total cyclomatic complexity of 102 (highest 15). 3 741181
ExcessiveClassLength Avoid really long classes. 3 741181
GodClass Possible God Class (WMC=102, ATFD=241, TCC=18.615%) 3 741181
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 741181
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 171
AddEmptyString Do not add empty strings 3 173
NPathComplexity The method 'doModifyRssFilePortlet(HttpServletRequest)' has an NPath complexity of 288 3 206302
CyclomaticComplexity The method 'doModifyRssFilePortlet(HttpServletRequest)' has a cyclomatic complexity of 10. 3 206302
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 215
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 216
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 218
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 220
NPathComplexity The method 'doModifyRssFileResource(HttpServletRequest)' has an NPath complexity of 3072 3 310440
CyclomaticComplexity The method 'doModifyRssFileResource(HttpServletRequest)' has a cyclomatic complexity of 14. 3 310440
ExcessiveMethodLength Avoid really long methods. 3 310440
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 321
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 323
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 324
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 325
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 326
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 327
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 352
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 354
NPathComplexity The method 'doCreateRssFile(HttpServletRequest)' has an NPath complexity of 400 3 448572
CyclomaticComplexity The method 'doCreateRssFile(HttpServletRequest)' has a cyclomatic complexity of 14. 3 448572
ExcessiveMethodLength Avoid really long methods. 3 448572
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 456
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 480
ConfusingTernary Avoid if (x != y) ..; else ..; 3 482489
ConfusingTernary Avoid if (x != y) ..; else ..; 3 491498
ConfusingTernary Avoid if (x != y) ..; else ..; 3 500507
ConfusingTernary Avoid if (x != y) ..; else ..; 3 509516
ConfusingTernary Avoid if (x != y) ..; else ..; 3 518525
ConfusingTernary Avoid if (x != y) ..; else ..; 3 542549
CyclomaticComplexity The method 'doCreateRssFilePortlet(HttpServletRequest)' has a cyclomatic complexity of 14. 3 615726
ExcessiveMethodLength Avoid really long methods. 3 615726
NPathComplexity The method 'doCreateRssFilePortlet(HttpServletRequest)' has an NPath complexity of 3456 3 615726
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 623
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 624
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 625
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 626
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 627
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 628
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 635
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 658
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 660
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 740
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 772
CyclomaticComplexity The method 'doCreateRssFileResource(HttpServletRequest)' has a cyclomatic complexity of 15. 3 805927
ExcessiveMethodLength Avoid really long methods. 3 805927
NPathComplexity The method 'doCreateRssFileResource(HttpServletRequest)' has an NPath complexity of 6144 3 805927
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 812
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 813
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 814
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 815
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 816
PositionLiteralsFirstInComparisons Position literals first in String comparisons 3 828
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 846
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 848
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 944945
FormalParameterNamingConventions The method parameter name 'rss_type' doesn't match '[a-z][a-zA-Z0-9]*' 1 945
StringInstantiation Avoid instantiating String objects; this is usually unnecessary. 2 947
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 951
FormalParameterNamingConventions The method parameter name 'id_rss' doesn't match '[a-z][a-zA-Z0-9]*' 1 964
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 964965
FormalParameterNamingConventions The method parameter name 'rss_type' doesn't match '[a-z][a-zA-Z0-9]*' 1 965
StringInstantiation Avoid instantiating String objects; this is usually unnecessary. 2 967
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 971
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 994
SimplifyStartsWith This call to String.startsWith can be rewritten using String.charAt(0) 3 1067
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 1073
ConfusingTernary Avoid if (x != y) ..; else ..; 3 11351149
ConfusingTernary Avoid if (x != y) ..; else ..; 3 11411148
ConfusingTernary Avoid if (x != y) ..; else ..; 3 1162

fr/paris/lutece/plugins/rss/web/RssListApp.java

Rule Violation Priority Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 5688
MissingOverride The method 'getPage(HttpServletRequest, int, Plugin)' is missing an @Override annotation. 3 6687
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 72
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 72

fr/paris/lutece/plugins/rss/web/RssListInclude.java

Rule Violation Priority Ligne
MissingOverride The method 'fillTemplate(Map, PageData, int, HttpServletRequest)' is missing an @Override annotation. 3 7382
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 75

fr/paris/lutece/plugins/rss/web/filter/RssGeneratorFilter.java

Rule Violation Priority Ligne
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 82

fr/paris/lutece/plugins/rss/web/portlet/RssListPortletJspBean.java

Rule Violation Priority Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 49181
MissingOverride The method 'getCreate(HttpServletRequest)' is missing an @Override annotation. 3 8087
MissingOverride The method 'getModify(HttpServletRequest)' is missing an @Override annotation. 3 94102
MissingOverride The method 'doCreate(HttpServletRequest)' is missing an @Override annotation. 3 110142
MissingOverride The method 'doModify(HttpServletRequest)' is missing an @Override annotation. 3 150180

fr/paris/lutece/plugins/rss/web/portlet/RssPortletJspBean.java

Rule Violation Priority Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 56243
MissingOverride The method 'getCreate(HttpServletRequest)' is missing an @Override annotation. 3 96106
MissingOverride The method 'getModify(HttpServletRequest)' is missing an @Override annotation. 3 113126
MissingOverride The method 'doCreate(HttpServletRequest)' is missing an @Override annotation. 3 134176
MissingOverride The method 'doModify(HttpServletRequest)' is missing an @Override annotation. 3 184225
DefaultPackage Use explicit scoping instead of the default package private level 3 233242
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getFeedIndexCombo method if you want a default access modifier 3 233242
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 235