PMD Results

The following document contains the results of PMD 6.29.0.

Violations By Priority

Priority 1

com/varmateo/yawg/core/DirBakeOptions.java

Rule Violation Line
FieldNamingConventions The final field name 'templateName' doesn't match '_[a-z][a-zA-Z0-9]*' 31
FieldNamingConventions The final field name 'filesToExclude' doesn't match '_[a-z][a-zA-Z0-9]*' 37
FieldNamingConventions The final field name 'filesToExcludeHere' doesn't match '_[a-z][a-zA-Z0-9]*' 42
FieldNamingConventions The final field name 'filesToIncludeHere' doesn't match '_[a-z][a-zA-Z0-9]*' 48
FieldNamingConventions The final field name 'bakerTypes' doesn't match '_[a-z][a-zA-Z0-9]*' 53
FieldNamingConventions The final field name 'pageVars' doesn't match '_[a-z][a-zA-Z0-9]*' 59
FieldNamingConventions The final field name 'pageVarsHere' doesn't match '_[a-z][a-zA-Z0-9]*' 66
FieldNamingConventions The final field name 'templatesHere' doesn't match '_[a-z][a-zA-Z0-9]*' 72
FieldNamingConventions The final field name 'extraDirsHere' doesn't match '_[a-z][a-zA-Z0-9]*' 81

Priority 3

com/varmateo/yawg/breadcrumbs/Breadcrumbs.java

Rule Violation Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 98

com/varmateo/yawg/commonmark/CommonMarkPageBaker.java

Rule Violation Line
TooManyMethods This class has too many methods, consider refactoring it. 39–274

com/varmateo/yawg/core/BakerMatcher.java

Rule Violation Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 158
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 207
AccessorClassGeneration Avoid instantiation through private constructors from outside of the constructors class. 218

com/varmateo/yawg/core/CoreSiteBaker.java

Rule Violation Line
ProperLogger Logger should be defined private static final and have the correct class 35

com/varmateo/yawg/core/DirBakeOptions.java

Rule Violation Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 89
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 90
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 91
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 92
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 93
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 94
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 95
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 96
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 97
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 140–141
TooManyMethods This class has too many methods, consider refactoring it. 154–400

com/varmateo/yawg/core/DirBakeOptionsDao.java

Rule Violation Line
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 70
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 159
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 184
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 195
LocalVariableCouldBeFinal Local variable 'bakerType' could be declared final 204
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 223
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 238
LocalVariableCouldBeFinal Local variable 'templateName' could be declared final 247
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 268
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 288

com/varmateo/yawg/core/DirBaker.java

Rule Violation Line
ProperLogger Logger should be defined private static final and have the correct class 31

com/varmateo/yawg/core/FileBaker.java

Rule Violation Line
ProperLogger Logger should be defined private static final and have the correct class 31
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 63
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 66

com/varmateo/yawg/freemarker/FreemarkerTemplateService.java

Rule Violation Line
UnusedAssignment The initializer for variable 'fmTemplate' is never used (overwritten on line 108) 105

com/varmateo/yawg/spi/PageContextBuilder.java

Rule Violation Line
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 33
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 35
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 37
DataClass The class 'PageContextImpl' is suspected to be a Data Class (WOC=20.000%, NOPA=0, NOAM=4, WMC=6) 188–256
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 204
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 205
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 206
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 207
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 208

com/varmateo/yawg/spi/TemplateContextBuilder.java

Rule Violation Line
ImmutableField Private field '_authors' could be made final; it is only initialized in the declaration or constructor. 23
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 38
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 39
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 40
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 42
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 43
DataClass The class 'TemplateContextImpl' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=7, WMC=8) 145–235
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 163
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 164
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 165
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 166
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 167
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 168
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 169

com/varmateo/yawg/util/FileUtils.java

Rule Violation Line
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 169
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 184
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 212
UnusedLocalVariable Avoid unused local variables such as 'result'. 212
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 215

com/varmateo/yawg/util/GlobMatcher.java

Rule Violation Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 149
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 150
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 162
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 176
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 177
AccessorClassGeneration Avoid instantiation through private constructors from outside of the constructors class. 188

com/varmateo/yawg/util/Trys.java

Rule Violation Line
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 44
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 67
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 69
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 97
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 99

com/varmateo/yawg/util/YamlParser.java

Rule Violation Line
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. 20–54

Priority 4

com/varmateo/yawg/asciidoctor/AsciidoctorTemplateContext.java

Rule Violation Line
UselessParentheses Useless parentheses. 103
UselessParentheses Useless parentheses. 103

com/varmateo/yawg/breadcrumbs/BreadcrumbsBakeListener.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 11

com/varmateo/yawg/core/CollectiveTemplateService.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 15

com/varmateo/yawg/core/CoreSiteBaker.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 17

com/varmateo/yawg/core/DirBaker.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 16

com/varmateo/yawg/logging/Log.java

Rule Violation Line
ShortClassName Avoid short class names like Log 14–95

com/varmateo/yawg/logging/LoggerLog.java

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

com/varmateo/yawg/logging/Logs.java

Rule Violation Line
ShortClassName Avoid short class names like Logs 20–135

com/varmateo/yawg/spi/PageContextBuilder.java

Rule Violation Line
UselessParentheses Useless parentheses. 34

com/varmateo/yawg/util/FileUtils.java

Rule Violation Line
UselessParentheses Useless parentheses. 55

com/varmateo/yawg/util/SimpleList.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 15
UselessParentheses Useless parentheses. 97

com/varmateo/yawg/util/SimpleMap.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 16
UselessParentheses Useless parentheses. 115

com/varmateo/yawg/util/Trys.java

Rule Violation Line
ShortClassName Avoid short class names like Trys 18–106

com/varmateo/yawg/util/YamlParser.java

Rule Violation Line
UselessParentheses Useless parentheses. 41
UselessParentheses Useless parentheses. 41

Files

com/varmateo/yawg/asciidoctor/AsciidoctorTemplateContext.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 103
UselessParentheses Useless parentheses. 4 103

com/varmateo/yawg/breadcrumbs/Breadcrumbs.java

Rule Violation Priority Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 98

com/varmateo/yawg/breadcrumbs/BreadcrumbsBakeListener.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 4 11

com/varmateo/yawg/commonmark/CommonMarkPageBaker.java

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

com/varmateo/yawg/core/BakerMatcher.java

Rule Violation Priority Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 158
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 207
AccessorClassGeneration Avoid instantiation through private constructors from outside of the constructors class. 3 218

com/varmateo/yawg/core/CollectiveTemplateService.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 4 15

com/varmateo/yawg/core/CoreSiteBaker.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 4 17
ProperLogger Logger should be defined private static final and have the correct class 3 35

com/varmateo/yawg/core/DirBakeOptions.java

Rule Violation Priority Line
FieldNamingConventions The final field name 'templateName' doesn't match '_[a-z][a-zA-Z0-9]*' 1 31
FieldNamingConventions The final field name 'filesToExclude' doesn't match '_[a-z][a-zA-Z0-9]*' 1 37
FieldNamingConventions The final field name 'filesToExcludeHere' doesn't match '_[a-z][a-zA-Z0-9]*' 1 42
FieldNamingConventions The final field name 'filesToIncludeHere' doesn't match '_[a-z][a-zA-Z0-9]*' 1 48
FieldNamingConventions The final field name 'bakerTypes' doesn't match '_[a-z][a-zA-Z0-9]*' 1 53
FieldNamingConventions The final field name 'pageVars' doesn't match '_[a-z][a-zA-Z0-9]*' 1 59
FieldNamingConventions The final field name 'pageVarsHere' doesn't match '_[a-z][a-zA-Z0-9]*' 1 66
FieldNamingConventions The final field name 'templatesHere' doesn't match '_[a-z][a-zA-Z0-9]*' 1 72
FieldNamingConventions The final field name 'extraDirsHere' doesn't match '_[a-z][a-zA-Z0-9]*' 1 81
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 89
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 90
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 91
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 92
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 93
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 94
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 95
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 96
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 97
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 140–141
TooManyMethods This class has too many methods, consider refactoring it. 3 154–400

com/varmateo/yawg/core/DirBakeOptionsDao.java

Rule Violation Priority Line
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 70
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 159
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 184
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 195
LocalVariableCouldBeFinal Local variable 'bakerType' could be declared final 3 204
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 223
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 238
LocalVariableCouldBeFinal Local variable 'templateName' could be declared final 3 247
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 268
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 288

com/varmateo/yawg/core/DirBaker.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 4 16
ProperLogger Logger should be defined private static final and have the correct class 3 31

com/varmateo/yawg/core/FileBaker.java

Rule Violation Priority Line
ProperLogger Logger should be defined private static final and have the correct class 3 31
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 63
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 66

com/varmateo/yawg/freemarker/FreemarkerTemplateService.java

Rule Violation Priority Line
UnusedAssignment The initializer for variable 'fmTemplate' is never used (overwritten on line 108) 3 105

com/varmateo/yawg/logging/Log.java

Rule Violation Priority Line
ShortClassName Avoid short class names like Log 4 14–95

com/varmateo/yawg/logging/LoggerLog.java

Rule Violation Priority Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 170
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 185

com/varmateo/yawg/logging/Logs.java

Rule Violation Priority Line
ShortClassName Avoid short class names like Logs 4 20–135

com/varmateo/yawg/spi/PageContextBuilder.java

Rule Violation Priority Line
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 33
UselessParentheses Useless parentheses. 4 34
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 35
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 37
DataClass The class 'PageContextImpl' is suspected to be a Data Class (WOC=20.000%, NOPA=0, NOAM=4, WMC=6) 3 188–256
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 204
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 205
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 206
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 207
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 208

com/varmateo/yawg/spi/TemplateContextBuilder.java

Rule Violation Priority Line
ImmutableField Private field '_authors' could be made final; it is only initialized in the declaration or constructor. 3 23
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 38
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 39
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 40
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 42
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 43
DataClass The class 'TemplateContextImpl' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=7, WMC=8) 3 145–235
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 163
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 164
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 165
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 166
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 167
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 168
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 169

com/varmateo/yawg/util/FileUtils.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 55
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 169
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 184
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 212
UnusedLocalVariable Avoid unused local variables such as 'result'. 3 212
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 215

com/varmateo/yawg/util/GlobMatcher.java

Rule Violation Priority Line
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 149
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 150
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 162
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 176
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 177
AccessorClassGeneration Avoid instantiation through private constructors from outside of the constructors class. 3 188

com/varmateo/yawg/util/SimpleList.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 4 15
UselessParentheses Useless parentheses. 4 97

com/varmateo/yawg/util/SimpleMap.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'com.varmateo.yawg.api.YawgException' 4 16
UselessParentheses Useless parentheses. 4 115

com/varmateo/yawg/util/Trys.java

Rule Violation Priority Line
ShortClassName Avoid short class names like Trys 4 18–106
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 44
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 67
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 69
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 97
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 99

com/varmateo/yawg/util/YamlParser.java

Rule Violation Priority Line
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 20–54
UselessParentheses Useless parentheses. 4 41
UselessParentheses Useless parentheses. 4 41