Fork me on GitHub

PMD Results

The following document contains the results of PMD 7.17.0.

Violations By Priority

Priority 2

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/util/AntPathMatcher.java

Rule Violation Line
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 242
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 390

Priority 3

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/ContentMigrationTask.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getLogger': the method is declared in an interface type 33
UnnecessaryModifier Unnecessary modifier 'public' on method 'setLogger': the method is declared in an interface type 39
UnnecessaryModifier Unnecessary modifier 'public' on method 'start': the method is declared in an interface type 45
UnnecessaryModifier Unnecessary modifier 'public' on method 'stop': the method is declared in an interface type 51
UnnecessaryModifier Unnecessary modifier 'public' on method 'getStartedTimeMillis': the method is declared in an interface type 57
UnnecessaryModifier Unnecessary modifier 'public' on method 'getStoppedTimeMillis': the method is declared in an interface type 63
UnnecessaryModifier Unnecessary modifier 'public' on method 'beginRecord': the method is declared in an interface type 72
UnnecessaryModifier Unnecessary modifier 'public' on method 'endRecord': the method is declared in an interface type 78
UnnecessaryModifier Unnecessary modifier 'public' on method 'getContentMigrationRecords': the method is declared in an interface type 84
UnnecessaryModifier Unnecessary modifier 'public' on method 'getSummary': the method is declared in an interface type 90
UnnecessaryModifier Unnecessary modifier 'public' on method 'logSummary': the method is declared in an interface type 95
UnnecessaryModifier Unnecessary modifier 'public' on method 'readContentNodeFromJsonFile': the method is declared in an interface type 104
UnnecessaryModifier Unnecessary modifier 'public' on method 'writeContentNodeToJsonFile': the method is declared in an interface type 112
UnnecessaryModifier Unnecessary modifier 'public' on method 'readContentNodeFromXmlFile': the method is declared in an interface type 122
UnnecessaryModifier Unnecessary modifier 'public' on method 'writeContentNodeToXmlFile': the method is declared in an interface type 130

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/DocumentManager.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getDocumentWorkflow': the method is declared in an interface type 243

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/util/AntPathMatcher.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 301–304
CollapsibleIfStatements This if statement could be combined with its parent 317–320
CollapsibleIfStatements This if statement could be combined with its parent 339–342
CollapsibleIfStatements This if statement could be combined with its parent 383–385

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/util/ContentNodeUtils.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 116–117
EmptyCatchBlock Avoid empty catch blocks 151–152

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/util/HippoNodeUtils.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 325–331
CollapsibleIfStatements This if statement could be combined with its parent 528–534
CollapsibleIfStatements This if statement could be combined with its parent 571–575

Hippo Content Export / Import Repository JAX-RS Services - org/onehippo/forge/content/exim/repository/jaxrs/AbstractContentEximService.java

Rule Violation Line
CollapsibleIfStatements This if statement could be combined with its parent 406–410

Hippo Content Export / Import Repository JAX-RS Services - org/onehippo/forge/content/exim/repository/jaxrs/ContentEximImportService.java

Rule Violation Line
UnusedFormalParameter Avoid unused method parameters such as 'processStatus'. 509
UnusedFormalParameter Avoid unused method parameters such as 'processStatus'. 569

Priority 4

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/util/HippoBinaryNodeUtils.java

Rule Violation Line
UselessParentheses Useless parentheses around `resourceNode.hasProperty("jcr:mimeType")`. 169

Hippo Content Export / Import Repository JAX-RS Services - org/onehippo/forge/content/exim/repository/jaxrs/ContentEximImportService.java

Rule Violation Line
UselessParentheses Useless parentheses around `contentNode.hasProperty("hippotranslation:locale")`. 449

Hippo Content Export / Import Repository JAX-RS Services - org/onehippo/forge/content/exim/repository/jaxrs/util/ResultItemSetCollector.java

Rule Violation Line
UselessParentheses Useless parentheses around `StringUtils.startsWithIgnoreCase(query, "select")`. 147

Hippo Content Export / Import Repository JAX-RS Services - org/onehippo/forge/content/exim/repository/jaxrs/util/ZipCompressUtils.java

Rule Violation Line
UselessParentheses Useless parentheses around `StringUtils.isEmpty(prefix)`. 109

Files

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/ContentMigrationTask.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getLogger': the method is declared in an interface type 3 33
UnnecessaryModifier Unnecessary modifier 'public' on method 'setLogger': the method is declared in an interface type 3 39
UnnecessaryModifier Unnecessary modifier 'public' on method 'start': the method is declared in an interface type 3 45
UnnecessaryModifier Unnecessary modifier 'public' on method 'stop': the method is declared in an interface type 3 51
UnnecessaryModifier Unnecessary modifier 'public' on method 'getStartedTimeMillis': the method is declared in an interface type 3 57
UnnecessaryModifier Unnecessary modifier 'public' on method 'getStoppedTimeMillis': the method is declared in an interface type 3 63
UnnecessaryModifier Unnecessary modifier 'public' on method 'beginRecord': the method is declared in an interface type 3 72
UnnecessaryModifier Unnecessary modifier 'public' on method 'endRecord': the method is declared in an interface type 3 78
UnnecessaryModifier Unnecessary modifier 'public' on method 'getContentMigrationRecords': the method is declared in an interface type 3 84
UnnecessaryModifier Unnecessary modifier 'public' on method 'getSummary': the method is declared in an interface type 3 90
UnnecessaryModifier Unnecessary modifier 'public' on method 'logSummary': the method is declared in an interface type 3 95
UnnecessaryModifier Unnecessary modifier 'public' on method 'readContentNodeFromJsonFile': the method is declared in an interface type 3 104
UnnecessaryModifier Unnecessary modifier 'public' on method 'writeContentNodeToJsonFile': the method is declared in an interface type 3 112
UnnecessaryModifier Unnecessary modifier 'public' on method 'readContentNodeFromXmlFile': the method is declared in an interface type 3 122
UnnecessaryModifier Unnecessary modifier 'public' on method 'writeContentNodeToXmlFile': the method is declared in an interface type 3 130

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/DocumentManager.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getDocumentWorkflow': the method is declared in an interface type 3 243

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/util/AntPathMatcher.java

Rule Violation Priority Line
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 2 242
CollapsibleIfStatements This if statement could be combined with its parent 3 301–304
CollapsibleIfStatements This if statement could be combined with its parent 3 317–320
CollapsibleIfStatements This if statement could be combined with its parent 3 339–342
CollapsibleIfStatements This if statement could be combined with its parent 3 383–385
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 2 390

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/util/ContentNodeUtils.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 116–117
EmptyCatchBlock Avoid empty catch blocks 3 151–152

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/util/HippoBinaryNodeUtils.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `resourceNode.hasProperty("jcr:mimeType")`. 4 169

Hippo Content Export / Import Core - org/onehippo/forge/content/exim/core/util/HippoNodeUtils.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 325–331
CollapsibleIfStatements This if statement could be combined with its parent 3 528–534
CollapsibleIfStatements This if statement could be combined with its parent 3 571–575

Hippo Content Export / Import Repository JAX-RS Services - org/onehippo/forge/content/exim/repository/jaxrs/AbstractContentEximService.java

Rule Violation Priority Line
CollapsibleIfStatements This if statement could be combined with its parent 3 406–410

Hippo Content Export / Import Repository JAX-RS Services - org/onehippo/forge/content/exim/repository/jaxrs/ContentEximImportService.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `contentNode.hasProperty("hippotranslation:locale")`. 4 449
UnusedFormalParameter Avoid unused method parameters such as 'processStatus'. 3 509
UnusedFormalParameter Avoid unused method parameters such as 'processStatus'. 3 569

Hippo Content Export / Import Repository JAX-RS Services - org/onehippo/forge/content/exim/repository/jaxrs/util/ResultItemSetCollector.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `StringUtils.startsWithIgnoreCase(query, "select")`. 4 147

Hippo Content Export / Import Repository JAX-RS Services - org/onehippo/forge/content/exim/repository/jaxrs/util/ZipCompressUtils.java

Rule Violation Priority Line
UselessParentheses Useless parentheses around `StringUtils.isEmpty(prefix)`. 4 109