Fork me on GitHub

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 1

fr/paris/lutece/plugins/jasper/service/CompiledJasperTemplateCacheService.java

Rule Violation Line
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 108

fr/paris/lutece/plugins/jasper/service/FileTypeContext.java

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

fr/paris/lutece/plugins/jasper/service/JasperFileLinkService.java

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

fr/paris/lutece/plugins/jasper/service/purge/ImagePurgeService.java

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

fr/paris/lutece/plugins/jasper/web/JasperJspBean.java

Rule Violation Line
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 353

Priority 3

fr/paris/lutece/plugins/jasper/business/JasperReport.java

Rule Violation Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 50
ImmutableField Private field '_listFileFormats' could be made final; it is only initialized in the declaration or constructor. 50
LooseCoupling Avoid using implementation types like 'ArrayList'; use the interface instead 50
CommentRequired Public method and constructor comments are required 9497
CommentRequired Public method and constructor comments are required 99102
CommentRequired Public method and constructor comments are required 125128
LooseCoupling Avoid using implementation types like 'ArrayList'; use the interface instead 179
LooseCoupling Avoid using implementation types like 'ArrayList'; use the interface instead 187
CommentRequired Public method and constructor comments are required 187190

fr/paris/lutece/plugins/jasper/business/JasperReportDAO.java

Rule Violation Line
MissingOverride The method 'insert(JasperReport, Plugin)' is missing an @Override annotation. 92105
MissingOverride The method 'load(int, Plugin)' is missing an @Override annotation. 116139
LooseCoupling Avoid using implementation types like 'ArrayList'; use the interface instead 150
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 152
MissingOverride The method 'delete(int, Plugin)' is missing an @Override annotation. 176183
MissingOverride The method 'store(JasperReport, Plugin)' is missing an @Override annotation. 193203
MissingOverride The method 'selectJasperReportsList(Plugin)' is missing an @Override annotation. 212234
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 214
CommentRequired Public method and constructor comments are required 236256
MissingOverride The method 'loadByCode(String, Plugin)' is missing an @Override annotation. 236256

fr/paris/lutece/plugins/jasper/business/portlet/IJasperPortletDAO.java

Rule Violation Line
CommentRequired Header comments are required 3982
MissingOverride The method 'delete(int)' is missing an @Override annotation. 47
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 55
MissingOverride The method 'load(int)' is missing an @Override annotation. 64
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 72

fr/paris/lutece/plugins/jasper/business/portlet/JasperPortlet.java

Rule Violation Line
CallSuperInConstructor It is a good practice to call super() in a constructor 5659
MissingOverride The method 'getXmlDocument(HttpServletRequest)' is missing an @Override annotation. 8992
MissingOverride The method 'getXml(HttpServletRequest)' is missing an @Override annotation. 101110
MissingOverride The method 'remove()' is missing an @Override annotation. 123126

fr/paris/lutece/plugins/jasper/business/portlet/JasperPortletDAO.java

Rule Violation Line
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 6171
ShortVariable Avoid variables with short names like p 65
MissingOverride The method 'delete(int)' is missing an @Override annotation. 7986
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 94106
ShortVariable Avoid variables with short names like p 98
MissingOverride The method 'load(int)' is missing an @Override annotation. 115132
MissingOverride The method 'checkNoPortletLinked(int)' is missing an @Override annotation. 141157

fr/paris/lutece/plugins/jasper/business/portlet/JasperPortletHome.java

Rule Violation Line
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 50
CallSuperInConstructor It is a good practice to call super() in a constructor 5561
MissingOverride The method 'getPortletTypeId()' is missing an @Override annotation. 6874
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strPortletTypeId' 73
NonThreadSafeSingleton Singleton is not thread safe 8386
MissingOverride The method 'getDAO()' is missing an @Override annotation. 9699

fr/paris/lutece/plugins/jasper/service/CompiledJasperTemplateCacheService.java

Rule Violation Line
CallSuperInConstructor It is a good practice to call super() in a constructor 5760
MissingOverride The method 'getName()' is missing an @Override annotation. 6770
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 73
NonThreadSafeSingleton Singleton is not thread safe 8285
ShortVariable Avoid variables with short names like r 105
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 124

fr/paris/lutece/plugins/jasper/service/DownloadFileServlet.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 5457
CommentRequired Public method and constructor comments are required 5457
ConfusingTernary Avoid if (x != y) ..; else ..; 84
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 89
MissingOverride The method 'doGet(HttpServletRequest, HttpServletResponse)' is missing an @Override annotation. 111114
MissingOverride The method 'doPost(HttpServletRequest, HttpServletResponse)' is missing an @Override annotation. 128131
MissingOverride The method 'getServletInfo()' is missing an @Override annotation. 138141

fr/paris/lutece/plugins/jasper/service/ExportFormatService.java

Rule Violation Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 59
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 59
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 83
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 96
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 102111
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 106110
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 119
CommentRequired Public method and constructor comments are required 134138

fr/paris/lutece/plugins/jasper/service/FileTypeContext.java

Rule Violation Line
CommentRequired Header comments are required 4391
ImmutableField Private field 'strategy' could be made final; it is only initialized in the declaration or constructor. 45
CommentRequired Public method and constructor comments are required 4851
CommentRequired Public method and constructor comments are required 5356
CommentRequired Public method and constructor comments are required 5861
CommentRequired Public method and constructor comments are required 6367
CommentRequired Public method and constructor comments are required 6972
CommentRequired Public method and constructor comments are required 7479
CommentRequired Public method and constructor comments are required 8185
CommentRequired Public method and constructor comments are required 8790

fr/paris/lutece/plugins/jasper/service/JasperConnectionService.java

Rule Violation Line
ReplaceHashtableWithMap Consider replacing this Hashtable with the newer java.util.Map 48
LooseCoupling Avoid using implementation types like 'Hashtable'; use the interface instead 48
ConfusingTernary Avoid if (x != y) ..; else ..; 6883

fr/paris/lutece/plugins/jasper/service/JasperFileLinkService.java

Rule Violation Line
CommentRequired Enum comments are required 52251
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 68
CommentRequired Public method and constructor comments are required 7079
CommentRequired Public method and constructor comments are required 110114
CommentRequired Public method and constructor comments are required 116125
CommentRequired Public method and constructor comments are required 127131
CommentRequired Public method and constructor comments are required 133142
CommentRequired Public method and constructor comments are required 144149
CommentRequired Public method and constructor comments are required 151161
EmptyStatementNotInLoop An empty statement (semicolon) not part of a loop 172
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 192
ShortVariable Avoid variables with short names like i 193
CommentRequired Public method and constructor comments are required 207222
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 209
ShortVariable Avoid variables with short names like i 210
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 246

fr/paris/lutece/plugins/jasper/service/export/AbstractDefaultJasperRender.java

Rule Violation Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 121
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 121
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 214
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 228

fr/paris/lutece/plugins/jasper/service/export/HtmlJasperRender.java

Rule Violation Line
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 8494
ShortVariable Avoid variables with short names like sb 116

fr/paris/lutece/plugins/jasper/service/purge/DaemonPurgeJasperImage.java

Rule Violation Line
CommentRequired Header comments are required 3844
MissingOverride The method 'run()' is missing an @Override annotation. 4043
CommentRequired Public method and constructor comments are required 4043

fr/paris/lutece/plugins/jasper/service/purge/ImagePurgeService.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 5463
CommentRequired Public method and constructor comments are required 6583
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 7179

fr/paris/lutece/plugins/jasper/web/JasperApp.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 51124
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 58
CallSuperInConstructor It is a good practice to call super() in a constructor 6370
MissingOverride The method 'getPage(HttpServletRequest, int, Plugin)' is missing an @Override annotation. 84117
MissingOverride The method 'getName()' is missing an @Override annotation. 119122
CommentRequired Public method and constructor comments are required 119122

fr/paris/lutece/plugins/jasper/web/JasperJspBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 71394
SingularField Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. 116
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 140
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 140
AddEmptyString Do not add empty strings 142
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 162
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 162
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 183
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 184
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 267
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 267
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 285
CyclomaticComplexity The method 'localTemplateFile(JasperReport, FileItem, String, boolean)' has a cyclomatic complexity of 11. 314373
ConfusingTernary Avoid if (x != y) ..; else ..; 319372
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 334
AvoidRethrowingException A catch statement that catches an exception only to rethrow it should be avoided. 358361
ControlStatementBraces This statement should have braces 365
CommentRequired Public method and constructor comments are required 375393
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 381389

fr/paris/lutece/plugins/jasper/web/JasperLinkServiceJspBean.java

Rule Violation Line
CommentRequired Header comments are required 57124
CommentRequired Public method and constructor comments are required 71101
MissingOverride The method 'getInsertServiceSelectorUI(HttpServletRequest)' is missing an @Override annotation. 71101
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 75
UnusedLocalVariable Avoid unused local variables such as 'locale'. 117
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 117

fr/paris/lutece/plugins/jasper/web/portlet/JasperPortletJspBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 51222
MissingOverride The method 'getCreate(HttpServletRequest)' is missing an @Override annotation. 91101
MissingOverride The method 'getModify(HttpServletRequest)' is missing an @Override annotation. 110123
MissingOverride The method 'doCreate(HttpServletRequest)' is missing an @Override annotation. 132157
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 138
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 139
MissingOverride The method 'doModify(HttpServletRequest)' is missing an @Override annotation. 166190
DefaultPackage Use explicit scoping instead of the default package private level 201210
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getFeedIndexCombo method if you want a default access modifier 201210
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 203
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 203
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getDummyList method if you want a default access modifier 213221
DefaultPackage Use explicit scoping instead of the default package private level 213221

Priority 4

fr/paris/lutece/plugins/jasper/service/JasperFileLinkService.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppLogService' 48

fr/paris/lutece/plugins/jasper/web/JasperLinkServiceJspBean.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'java.util.Map' 53

Files

fr/paris/lutece/plugins/jasper/business/JasperReport.java

Rule Violation Priority Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 50
ImmutableField Private field '_listFileFormats' could be made final; it is only initialized in the declaration or constructor. 3 50
LooseCoupling Avoid using implementation types like 'ArrayList'; use the interface instead 3 50
CommentRequired Public method and constructor comments are required 3 9497
CommentRequired Public method and constructor comments are required 3 99102
CommentRequired Public method and constructor comments are required 3 125128
LooseCoupling Avoid using implementation types like 'ArrayList'; use the interface instead 3 179
LooseCoupling Avoid using implementation types like 'ArrayList'; use the interface instead 3 187
CommentRequired Public method and constructor comments are required 3 187190

fr/paris/lutece/plugins/jasper/business/JasperReportDAO.java

Rule Violation Priority Line
MissingOverride The method 'insert(JasperReport, Plugin)' is missing an @Override annotation. 3 92105
MissingOverride The method 'load(int, Plugin)' is missing an @Override annotation. 3 116139
LooseCoupling Avoid using implementation types like 'ArrayList'; use the interface instead 3 150
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 152
MissingOverride The method 'delete(int, Plugin)' is missing an @Override annotation. 3 176183
MissingOverride The method 'store(JasperReport, Plugin)' is missing an @Override annotation. 3 193203
MissingOverride The method 'selectJasperReportsList(Plugin)' is missing an @Override annotation. 3 212234
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 214
CommentRequired Public method and constructor comments are required 3 236256
MissingOverride The method 'loadByCode(String, Plugin)' is missing an @Override annotation. 3 236256

fr/paris/lutece/plugins/jasper/business/portlet/IJasperPortletDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 3982
MissingOverride The method 'delete(int)' is missing an @Override annotation. 3 47
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 3 55
MissingOverride The method 'load(int)' is missing an @Override annotation. 3 64
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 3 72

fr/paris/lutece/plugins/jasper/business/portlet/JasperPortlet.java

Rule Violation Priority Line
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 8992
MissingOverride The method 'getXml(HttpServletRequest)' is missing an @Override annotation. 3 101110
MissingOverride The method 'remove()' is missing an @Override annotation. 3 123126

fr/paris/lutece/plugins/jasper/business/portlet/JasperPortletDAO.java

Rule Violation Priority Line
MissingOverride The method 'insert(Portlet)' is missing an @Override annotation. 3 6171
ShortVariable Avoid variables with short names like p 3 65
MissingOverride The method 'delete(int)' is missing an @Override annotation. 3 7986
MissingOverride The method 'store(Portlet)' is missing an @Override annotation. 3 94106
ShortVariable Avoid variables with short names like p 3 98
MissingOverride The method 'load(int)' is missing an @Override annotation. 3 115132
MissingOverride The method 'checkNoPortletLinked(int)' is missing an @Override annotation. 3 141157

fr/paris/lutece/plugins/jasper/business/portlet/JasperPortletHome.java

Rule Violation Priority Line
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 3 50
CallSuperInConstructor It is a good practice to call super() in a constructor 3 5561
MissingOverride The method 'getPortletTypeId()' is missing an @Override annotation. 3 6874
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strPortletTypeId' 3 73
NonThreadSafeSingleton Singleton is not thread safe 3 8386
MissingOverride The method 'getDAO()' is missing an @Override annotation. 3 9699

fr/paris/lutece/plugins/jasper/service/CompiledJasperTemplateCacheService.java

Rule Violation Priority Line
CallSuperInConstructor It is a good practice to call super() in a constructor 3 5760
MissingOverride The method 'getName()' is missing an @Override annotation. 3 6770
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 73
NonThreadSafeSingleton Singleton is not thread safe 3 8285
ShortVariable Avoid variables with short names like r 3 105
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 108
AppendCharacterWithChar Avoid appending characters as strings in StringBuffer.append. 3 124

fr/paris/lutece/plugins/jasper/service/DownloadFileServlet.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 5457
CommentRequired Public method and constructor comments are required 3 5457
ConfusingTernary Avoid if (x != y) ..; else ..; 3 84
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 89
MissingOverride The method 'doGet(HttpServletRequest, HttpServletResponse)' is missing an @Override annotation. 3 111114
MissingOverride The method 'doPost(HttpServletRequest, HttpServletResponse)' is missing an @Override annotation. 3 128131
MissingOverride The method 'getServletInfo()' is missing an @Override annotation. 3 138141

fr/paris/lutece/plugins/jasper/service/ExportFormatService.java

Rule Violation Priority Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 59
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 59
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 83
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 96
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 102111
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 106110
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 119
CommentRequired Public method and constructor comments are required 3 134138

fr/paris/lutece/plugins/jasper/service/FileTypeContext.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 4391
ImmutableField Private field 'strategy' could be made final; it is only initialized in the declaration or constructor. 3 45
FieldNamingConventions The field name 'strategy' doesn't match '_[a-z][a-zA-Z0-9]*' 1 45
CommentRequired Public method and constructor comments are required 3 4851
CommentRequired Public method and constructor comments are required 3 5356
CommentRequired Public method and constructor comments are required 3 5861
CommentRequired Public method and constructor comments are required 3 6367
CommentRequired Public method and constructor comments are required 3 6972
CommentRequired Public method and constructor comments are required 3 7479
CommentRequired Public method and constructor comments are required 3 8185
CommentRequired Public method and constructor comments are required 3 8790

fr/paris/lutece/plugins/jasper/service/JasperConnectionService.java

Rule Violation Priority Line
ReplaceHashtableWithMap Consider replacing this Hashtable with the newer java.util.Map 3 48
LooseCoupling Avoid using implementation types like 'Hashtable'; use the interface instead 3 48
ConfusingTernary Avoid if (x != y) ..; else ..; 3 6883

fr/paris/lutece/plugins/jasper/service/JasperFileLinkService.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppLogService' 4 48
CommentRequired Enum comments are required 3 52251
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 3 68
FieldNamingConventions The field name 'mapStrategies' doesn't match '_[a-z][a-zA-Z0-9]*' 1 68
CommentRequired Public method and constructor comments are required 3 7079
CommentRequired Public method and constructor comments are required 3 110114
CommentRequired Public method and constructor comments are required 3 116125
CommentRequired Public method and constructor comments are required 3 127131
CommentRequired Public method and constructor comments are required 3 133142
CommentRequired Public method and constructor comments are required 3 144149
CommentRequired Public method and constructor comments are required 3 151161
EmptyStatementNotInLoop An empty statement (semicolon) not part of a loop 3 172
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 192
ShortVariable Avoid variables with short names like i 3 193
CommentRequired Public method and constructor comments are required 3 207222
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 209
ShortVariable Avoid variables with short names like i 3 210
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 246

fr/paris/lutece/plugins/jasper/service/export/AbstractDefaultJasperRender.java

Rule Violation Priority Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 121
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 121
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 214
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 228

fr/paris/lutece/plugins/jasper/service/export/HtmlJasperRender.java

Rule Violation Priority Line
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 8494
ShortVariable Avoid variables with short names like sb 3 116

fr/paris/lutece/plugins/jasper/service/purge/DaemonPurgeJasperImage.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 3844
MissingOverride The method 'run()' is missing an @Override annotation. 3 4043
CommentRequired Public method and constructor comments are required 3 4043

fr/paris/lutece/plugins/jasper/service/purge/ImagePurgeService.java

Rule Violation Priority Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 4684
CommentRequired Public method and constructor comments are required 3 5463
CommentRequired Public method and constructor comments are required 3 6583
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 7179

fr/paris/lutece/plugins/jasper/web/JasperApp.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 51124
AssignmentToNonFinalStatic Possible unsafe assignment to a non-final static field in a constructor. 3 58
CallSuperInConstructor It is a good practice to call super() in a constructor 3 6370
MissingOverride The method 'getPage(HttpServletRequest, int, Plugin)' is missing an @Override annotation. 3 84117
MissingOverride The method 'getName()' is missing an @Override annotation. 3 119122
CommentRequired Public method and constructor comments are required 3 119122

fr/paris/lutece/plugins/jasper/web/JasperJspBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 71394
SingularField Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. 3 116
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 140
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 140
AddEmptyString Do not add empty strings 3 142
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 162
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 162
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 183
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 184
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 267
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 267
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 285
CyclomaticComplexity The method 'localTemplateFile(JasperReport, FileItem, String, boolean)' has a cyclomatic complexity of 11. 3 314373
ConfusingTernary Avoid if (x != y) ..; else ..; 3 319372
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 334
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 353
AvoidRethrowingException A catch statement that catches an exception only to rethrow it should be avoided. 3 358361
ControlStatementBraces This statement should have braces 3 365
CommentRequired Public method and constructor comments are required 3 375393
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 381389

fr/paris/lutece/plugins/jasper/web/JasperLinkServiceJspBean.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'java.util.Map' 4 53
CommentRequired Header comments are required 3 57124
CommentRequired Public method and constructor comments are required 3 71101
MissingOverride The method 'getInsertServiceSelectorUI(HttpServletRequest)' is missing an @Override annotation. 3 71101
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 75
UnusedLocalVariable Avoid unused local variables such as 'locale'. 3 117
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 117

fr/paris/lutece/plugins/jasper/web/portlet/JasperPortletJspBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 51222
MissingOverride The method 'getCreate(HttpServletRequest)' is missing an @Override annotation. 3 91101
MissingOverride The method 'getModify(HttpServletRequest)' is missing an @Override annotation. 3 110123
MissingOverride The method 'doCreate(HttpServletRequest)' is missing an @Override annotation. 3 132157
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 138
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 139
MissingOverride The method 'doModify(HttpServletRequest)' is missing an @Override annotation. 3 166190
DefaultPackage Use explicit scoping instead of the default package private level 3 201210
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getFeedIndexCombo method if you want a default access modifier 3 201210
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 203
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 203
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getDummyList method if you want a default access modifier 3 213221
DefaultPackage Use explicit scoping instead of the default package private level 3 213221