PMD Results
The following document contains the results of PMD 4.2.5.
Files
info/magnolia/cms/beans/config/ConfigLoader.java
Violation | Line |
---|
Avoid unused constructor parameters such as 'context'. | 65 - 82 |
info/magnolia/cms/beans/config/ObservedManager.java
Violation | Line |
---|
Avoid unnecessary return statements | 115 |
info/magnolia/cms/core/AbstractContent.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 67 |
An empty statement (semicolon) not part of a loop | 289 |
Avoid empty catch blocks | 362 - 364 |
info/magnolia/cms/core/AbstractNodeData.java
Violation | Line |
---|
Avoid empty catch blocks | 129 - 131 |
info/magnolia/cms/core/BinaryNodeData.java
Violation | Line |
---|
These nested if statements could be combined | 75 - 82 |
info/magnolia/cms/core/DefaultContent.java
Violation | Line |
---|
Avoid empty catch blocks | 303 - 305 |
Avoid empty catch blocks | 345 - 347 |
Avoid empty catch blocks | 385 - 387 |
info/magnolia/cms/core/DefaultHierarchyManager.java
Violation | Line |
---|
Avoid empty catch blocks | 382 - 384 |
Avoid empty catch blocks | 461 - 463 |
info/magnolia/cms/core/MetaData.java
Violation | Line |
---|
Avoid empty catch blocks | 108 - 110 |
info/magnolia/cms/core/Path.java
Violation | Line |
---|
Avoid unused private methods such as 'checkAppRootDir()'. | 128 |
Avoid empty catch blocks | 137 - 139 |
These nested if statements could be combined | 315 - 317 |
info/magnolia/cms/filters/FilterDecorator.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 74 |
info/magnolia/cms/filters/RepositoryMappingFilter.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 60 |
info/magnolia/cms/i18n/AbstractMessagesImpl.java
Violation | Line |
---|
Ensure you override both equals() and hashCode() | 132 |
info/magnolia/cms/link/LinkResolverImpl.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 50 |
info/magnolia/cms/link/UUIDLink.java
Violation | Line |
---|
These nested if statements could be combined | 205 - 208 |
These nested if statements could be combined | 219 - 222 |
info/magnolia/cms/security/AbstractUser.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 52 |
info/magnolia/cms/security/DelegatingUserManager.java
Violation | Line |
---|
Avoid empty catch blocks | 117 - 119 |
info/magnolia/cms/security/ExternalUser.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 57 |
info/magnolia/cms/security/LogoutFilter.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 59 |
Avoid unused private fields such as 'servletContext'. | 63 |
info/magnolia/cms/security/MgnlRole.java
Violation | Line |
---|
These nested if statements could be combined | 97 - 100 |
These nested if statements could be combined | 132 - 135 |
info/magnolia/cms/security/auth/callback/RealmCallback.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 54 |
info/magnolia/cms/security/auth/callback/UserCallback.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 55 |
info/magnolia/cms/security/auth/login/BasicLogin.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 50 |
info/magnolia/cms/security/auth/login/FormLogin.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 52 |
info/magnolia/cms/security/auth/login/LoginFilter.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 58 |
info/magnolia/cms/servlets/ClasspathSpool.java
Violation | Line |
---|
Avoid empty catch blocks | 104 - 106 |
info/magnolia/cms/util/ClasspathResourcesUtil.java
Violation | Line |
---|
Avoid empty catch blocks | 145 - 147 |
These nested if statements could be combined | 255 - 257 |
info/magnolia/cms/util/ContentUtil.java
Violation | Line |
---|
Avoid empty catch blocks | 433 - 435 |
Avoid empty catch blocks | 448 - 450 |
These nested if statements could be combined | 523 - 526 |
info/magnolia/cms/util/InheritanceContentWrapper.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 75 |
info/magnolia/cms/util/NodeDataUtil.java
Violation | Line |
---|
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 426 |
Avoid empty catch blocks | 542 - 544 |
info/magnolia/cms/util/RequestFormUtil.java
Violation | Line |
---|
Avoid empty catch blocks | 159 - 161 |
info/magnolia/cms/util/SystemContentWrapper.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 51 |
info/magnolia/content2bean/impl/CollectionPropertyHidingTransformer.java
Violation | Line |
---|
These nested if statements could be combined | 81 - 85 |
info/magnolia/content2bean/impl/Content2BeanTransformerImpl.java
Violation | Line |
---|
These nested if statements could be combined | 317 - 322 |
info/magnolia/content2bean/impl/TypeMappingImpl.java
Violation | Line |
---|
These nested if statements could be combined | 136 - 149 |
These nested if statements could be combined | 209 - 211 |
info/magnolia/context/AbstractContext.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 63 |
info/magnolia/context/SingleJCRSessionSystemContext.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 47 |
info/magnolia/context/SystemRepositoryStrategy.java
Violation | Line |
---|
Avoid unused constructor parameters such as 'context'. | 52 - 53 |
info/magnolia/freemarker/models/BinaryNodeDataModel.java
Violation | Line |
---|
Avoid empty catch blocks | 72 - 74 |
Avoid empty catch blocks | 83 - 85 |
Avoid empty catch blocks | 96 - 98 |
Avoid empty catch blocks | 123 - 125 |
info/magnolia/freemarker/models/MagnoliaObjectWrapper.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 57 |
info/magnolia/importexport/PropertiesImportExport.java
Violation | Line |
---|
Avoid empty if statements | 144 - 146 |
info/magnolia/jackrabbit/ProviderImpl.java
Violation | Line |
---|
Avoid empty catch blocks | 183 - 185 |
Avoid empty catch blocks | 479 - 481 |
info/magnolia/link/Link.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 55 |
info/magnolia/link/LinkFactory.java
Violation | Line |
---|
Avoid empty if statements | 125 - 127 |
info/magnolia/link/LinkTransformerManager.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 51 |
info/magnolia/module/delta/QueryTask.java
Violation | Line |
---|
Avoid unused local variables such as 'it'. | 68 |
info/magnolia/module/files/MD5CheckingFileExtractorOperation.java
Violation | Line |
---|
Avoid empty if statements | 103 - 105 |
Avoid empty while statements | 159 - 160 |
info/magnolia/setup/for3_5/CheckAndUpdateExistingFilters.java
Violation | Line |
---|
Avoid unused private fields such as 'clazz'. | 200 |
Avoid unused private fields such as 'priority'. | 201 |
info/magnolia/setup/for3_5/ReconfigureCommands.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 63 |
Avoid empty if statements | 85 - 87 |
info/magnolia/setup/for3_5/UpdateURI2RepositoryMappings.java
Violation | Line |
---|
Avoid unused private fields such as 'mappingsNode'. | 57 |
Avoid unused local variables such as 'configHM'. | 69 |
info/magnolia/stats/JCRStats.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 53 |
info/magnolia/voting/voters/AbstractBoolVoter.java
Violation | Line |
---|
Avoid unused private fields such as 'log'. | 53 |
info/magnolia/voting/voters/URIStartsWithVoter.java
Violation | Line |
---|
An empty statement (semicolon) not part of a loop | 53 |