Last Published: 2015-04-16
PMD Results
The following document contains the results of PMD 5.1.2.
Files
info/magnolia/channel/ChannelResolver.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 46 |
| Avoid modifiers which are implied by the context | 51 |
info/magnolia/cms/beans/config/ConfigLoader.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'context'. | 82 |
info/magnolia/cms/beans/config/ConfigurationException.java
| Violation | Line |
|---|---|
| Useless parentheses. | 46 |
info/magnolia/cms/beans/config/HostBasedVirtualURIMapping.java
| Violation | Line |
|---|---|
| Useless parentheses. | 171 |
info/magnolia/cms/beans/config/MIMEMapping.java
| Violation | Line |
|---|---|
| Useless parentheses. | 219 |
info/magnolia/cms/beans/config/ObservedManager.java
| Violation | Line |
|---|---|
| Avoid unnecessary return statements | 130 |
info/magnolia/cms/beans/config/RotatingVirtualURIMapping.java
| Violation | Line |
|---|---|
| Useless parentheses. | 112 |
info/magnolia/cms/beans/config/URI2RepositoryMapping.java
| Violation | Line |
|---|---|
| Useless parentheses. | 111 |
info/magnolia/cms/beans/config/VirtualURIMapping.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 59–83 |
info/magnolia/cms/beans/runtime/Document.java
| Violation | Line |
|---|---|
| Useless parentheses. | 195 |
info/magnolia/cms/core/AbstractContent.java
| Violation | Line |
|---|---|
| Useless parentheses. | 341 |
| Useless parentheses. | 346 |
| Avoid empty catch blocks | 407–409 |
info/magnolia/cms/core/AbstractNodeData.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 129–131 |
| Avoid empty catch blocks | 200–202 |
info/magnolia/cms/core/BinaryNodeData.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 75–82 |
info/magnolia/cms/core/Content.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 725 |
| Avoid modifiers which are implied by the context | 733–742 |
| Avoid modifiers which are implied by the context | 740 |
info/magnolia/cms/core/DefaultACLBasedPermissions.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'configuration'. | 107 |
| Useless parentheses. | 117 |
| Useless parentheses. | 117 |
info/magnolia/cms/core/DefaultContent.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 310–312 |
| Avoid empty catch blocks | 355–357 |
| Avoid unused local variables such as 'nodeType'. | 518 |
| Avoid empty catch blocks | 544–546 |
| Avoid empty catch blocks | 546–548 |
| Ensure you override both equals() and hashCode() | 674 |
| Useless parentheses. | 688 |
info/magnolia/cms/core/DefaultHierarchyManager.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'ignoredAccessManager'. | 88 |
| Avoid unused constructor parameters such as 'ignoredUserId'. | 92 |
| Avoid unused constructor parameters such as 'ignored'. | 99 |
| Useless parentheses. | 174 |
| Avoid unused local variables such as 'type'. | 316 |
| Useless parentheses. | 340 |
| Avoid empty catch blocks | 392–394 |
info/magnolia/cms/core/MagnoliaAccessProvider.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'compiledPermissionsClass'. | 79 |
info/magnolia/cms/core/MetaData.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'ignoredAccessManager'. | 153 |
| Avoid unused private methods such as 'setJCRProperty(String,Object)'. | 399 |
info/magnolia/cms/core/NodeTypeBasedPermissions.java
| Violation | Line |
|---|---|
| Useless parentheses. | 111 |
| Useless parentheses. | 111 |
info/magnolia/cms/core/Path.java
| Violation | Line |
|---|---|
| Useless parentheses. | 210 |
| Useless parentheses. | 312 |
| Useless parentheses. | 312 |
| Useless parentheses. | 312 |
| Useless parentheses. | 313 |
| Useless parentheses. | 313 |
| Useless parentheses. | 313 |
| Useless parentheses. | 314 |
| Useless parentheses. | 314 |
| Useless parentheses. | 314 |
| These nested if statements could be combined | 338–340 |
| Useless parentheses. | 356 |
| Useless parentheses. | 360 |
info/magnolia/cms/core/SystemProperty.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getMagnoliaConfigurationProperties()'. | 158 |
info/magnolia/cms/core/search/QueryManagerImpl.java
| Violation | Line |
|---|---|
| Useless parentheses. | 94 |
| Useless parentheses. | 100 |
info/magnolia/cms/core/search/QueryResultImpl.java
| Violation | Line |
|---|---|
| Useless parentheses. | 137 |
info/magnolia/cms/core/version/ContentVersion.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'wrap(Collection)'. | 400 |
| Useless parentheses. | 413 |
| Useless parentheses. | 422 |
info/magnolia/cms/core/version/MgnlVersioningNodeWrapper.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 139–141 |
| Avoid empty catch blocks | 141–143 |
info/magnolia/cms/core/version/VersionManager.java
| Violation | Line |
|---|---|
| Useless parentheses. | 91 |
info/magnolia/cms/exchange/ActivationManager.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 44 |
| Avoid modifiers which are implied by the context | 50 |
| Avoid modifiers which are implied by the context | 56 |
| Avoid modifiers which are implied by the context | 64 |
| Avoid modifiers which are implied by the context | 66 |
| Avoid modifiers which are implied by the context | 71 |
info/magnolia/cms/exchange/ExchangeException.java
| Violation | Line |
|---|---|
| Useless parentheses. | 55 |
info/magnolia/cms/exchange/Subscriber.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 43 |
| Avoid modifiers which are implied by the context | 45 |
| Avoid modifiers which are implied by the context | 47 |
| Avoid modifiers which are implied by the context | 49 |
| Avoid modifiers which are implied by the context | 51 |
| Avoid modifiers which are implied by the context | 53 |
| Avoid modifiers which are implied by the context | 55 |
| Avoid modifiers which are implied by the context | 57 |
| Avoid modifiers which are implied by the context | 59 |
| Avoid modifiers which are implied by the context | 61 |
| Avoid modifiers which are implied by the context | 63 |
| Avoid modifiers which are implied by the context | 65 |
| Avoid modifiers which are implied by the context | 67 |
| Avoid modifiers which are implied by the context | 69 |
| Avoid modifiers which are implied by the context | 71 |
info/magnolia/cms/exchange/Subscription.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 43 |
| Avoid modifiers which are implied by the context | 45 |
| Avoid modifiers which are implied by the context | 47 |
| Avoid modifiers which are implied by the context | 49 |
| Avoid modifiers which are implied by the context | 51 |
| Avoid modifiers which are implied by the context | 53 |
| Avoid modifiers which are implied by the context | 55 |
| Avoid modifiers which are implied by the context | 57 |
info/magnolia/cms/exchange/Syndicator.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 60 |
| Avoid modifiers which are implied by the context | 68 |
| Avoid modifiers which are implied by the context | 77 |
| Avoid modifiers which are implied by the context | 85 |
| Avoid modifiers which are implied by the context | 94 |
| Avoid modifiers which are implied by the context | 99 |
| Avoid modifiers which are implied by the context | 104 |
info/magnolia/cms/filters/AggregatorFilter.java
| Violation | Line |
|---|---|
| Useless parentheses. | 193 |
| Useless parentheses. | 193 |
info/magnolia/cms/filters/CosMultipartRequestFilter.java
| Violation | Line |
|---|---|
| Useless parentheses. | 80 |
info/magnolia/cms/filters/FilterManager.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 52 |
info/magnolia/cms/filters/RepositoryMappingFilter.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 61 |
info/magnolia/cms/filters/WebContainerResources.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 50 |
info/magnolia/cms/i18n/AbstractMessagesImpl.java
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 136 |
info/magnolia/cms/i18n/DefaultI18nContentSupport.java
| Violation | Line |
|---|---|
| Useless parentheses. | 71 |
info/magnolia/cms/i18n/I18nContentSupport.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 55 |
| Avoid modifiers which are implied by the context | 57 |
| Avoid modifiers which are implied by the context | 62 |
| Avoid modifiers which are implied by the context | 64 |
| Avoid modifiers which are implied by the context | 69 |
| Avoid modifiers which are implied by the context | 74 |
| Avoid modifiers which are implied by the context | 79 |
| Avoid modifiers which are implied by the context | 87 |
| Avoid modifiers which are implied by the context | 95 |
| Avoid modifiers which are implied by the context | 100 |
| Avoid modifiers which are implied by the context | 105 |
| Avoid modifiers which are implied by the context | 110 |
| Avoid modifiers which are implied by the context | 115 |
| Avoid modifiers which are implied by the context | 120 |
| Avoid modifiers which are implied by the context | 122 |
| Avoid modifiers which are implied by the context | 124 |
info/magnolia/cms/i18n/Messages.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 48 |
| Avoid modifiers which are implied by the context | 54 |
| Avoid modifiers which are implied by the context | 61 |
| Avoid modifiers which are implied by the context | 67 |
| Avoid modifiers which are implied by the context | 75 |
| Avoid modifiers which are implied by the context | 84 |
| Avoid modifiers which are implied by the context | 89 |
| Avoid modifiers which are implied by the context | 94 |
info/magnolia/cms/security/AccessDeniedException.java
| Violation | Line |
|---|---|
| Useless parentheses. | 50 |
info/magnolia/cms/security/AccessManagerImpl.java
| Violation | Line |
|---|---|
| Useless parentheses. | 85 |
info/magnolia/cms/security/DelegatingUserManager.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 144–146 |
| Avoid empty catch blocks | 167–169 |
| Avoid empty catch blocks | 191–193 |
info/magnolia/cms/security/Digester.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 76 |
info/magnolia/cms/security/ExternalUser.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'userDetails'. | 62 |
info/magnolia/cms/security/Group.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 44 |
| Avoid modifiers which are implied by the context | 50 |
| Avoid modifiers which are implied by the context | 56 |
| Avoid modifiers which are implied by the context | 62 |
| Avoid modifiers which are implied by the context | 68 |
| Avoid modifiers which are implied by the context | 70 |
| Avoid modifiers which are implied by the context | 85 |
| Avoid modifiers which are implied by the context | 90 |
| Avoid modifiers which are implied by the context | 95 |
| Avoid modifiers which are implied by the context | 100 |
info/magnolia/cms/security/GroupManager.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 55 |
| Avoid modifiers which are implied by the context | 62 |
| Avoid modifiers which are implied by the context | 67 |
| Avoid modifiers which are implied by the context | 72 |
| Avoid modifiers which are implied by the context | 74 |
| Avoid modifiers which are implied by the context | 81 |
| Avoid modifiers which are implied by the context | 88 |
info/magnolia/cms/security/IPSecurityManager.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 51–55 |
info/magnolia/cms/security/IPSecurityManagerImpl.java
| Violation | Line |
|---|---|
| Useless parentheses. | 84 |
info/magnolia/cms/security/Lock.java
| Violation | Line |
|---|---|
| Useless parentheses. | 86 |
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/MgnlGroup.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 47 |
info/magnolia/cms/security/MgnlRole.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'acls'. | 56 |
info/magnolia/cms/security/MgnlRoleManager.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 154–156 |
| These nested if statements could be combined | 193–196 |
info/magnolia/cms/security/MgnlUserManager.java
| Violation | Line |
|---|---|
| Useless parentheses. | 320 |
| Avoid unnecessary return statements | 573 |
info/magnolia/cms/security/Realm.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 47 |
| Avoid modifiers which are implied by the context | 52 |
| Avoid modifiers which are implied by the context | 57 |
| Avoid modifiers which are implied by the context | 62 |
| Avoid modifiers which are implied by the context | 97–104 |
info/magnolia/cms/security/RepositoryBackedSecurityManager.java
| Violation | Line |
|---|---|
| Useless parentheses. | 83 |
| Useless parentheses. | 139 |
info/magnolia/cms/security/Role.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 43 |
| Avoid modifiers which are implied by the context | 49 |
| Avoid modifiers which are implied by the context | 55 |
| Avoid modifiers which are implied by the context | 61 |
| Avoid modifiers which are implied by the context | 67 |
info/magnolia/cms/security/RoleManager.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 54 |
| Avoid modifiers which are implied by the context | 56 |
| Avoid modifiers which are implied by the context | 63 |
| Avoid modifiers which are implied by the context | 70 |
| Avoid modifiers which are implied by the context | 75 |
| Avoid modifiers which are implied by the context | 80 |
| Avoid modifiers which are implied by the context | 85 |
info/magnolia/cms/security/SecurityCallbackFilter.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 68 |
| Unnecessary use of fully qualified name 'HttpServletResponse.SC_UNAUTHORIZED' due to existing import 'javax.servlet.http.HttpServletResponse' | 90 |
info/magnolia/cms/security/SecuritySupport.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 68–72 |
info/magnolia/cms/security/SystemUserManager.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'password'. | 166 |
info/magnolia/cms/security/UserManager.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 53 |
| Avoid modifiers which are implied by the context | 58 |
| Avoid modifiers which are implied by the context | 63 |
| Avoid modifiers which are implied by the context | 70 |
| Avoid modifiers which are implied by the context | 77 |
| Avoid modifiers which are implied by the context | 85 |
| Avoid modifiers which are implied by the context | 91 |
| Avoid modifiers which are implied by the context | 97 |
| Avoid modifiers which are implied by the context | 104 |
| Avoid modifiers which are implied by the context | 110 |
| Avoid modifiers which are implied by the context | 116 |
| Avoid modifiers which are implied by the context | 123 |
| Avoid modifiers which are implied by the context | 136 |
| Avoid modifiers which are implied by the context | 141 |
| Avoid modifiers which are implied by the context | 149 |
| Avoid modifiers which are implied by the context | 155 |
| Avoid modifiers which are implied by the context | 161 |
| Avoid modifiers which are implied by the context | 167 |
| Avoid modifiers which are implied by the context | 173 |
| Avoid modifiers which are implied by the context | 180 |
| Avoid modifiers which are implied by the context | 187 |
| Avoid modifiers which are implied by the context | 196 |
| Avoid modifiers which are implied by the context | 201 |
| Avoid modifiers which are implied by the context | 208 |
| Avoid modifiers which are implied by the context | 215 |
info/magnolia/cms/security/auth/Entity.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 46 |
| Avoid modifiers which are implied by the context | 48 |
| Avoid modifiers which are implied by the context | 50 |
| Avoid modifiers which are implied by the context | 52 |
| Avoid modifiers which are implied by the context | 54 |
| Avoid modifiers which are implied by the context | 56 |
| Avoid modifiers which are implied by the context | 58 |
| Avoid modifiers which are implied by the context | 64 |
| Avoid modifiers which are implied by the context | 70 |
| Avoid modifiers which are implied by the context | 72 |
| Avoid modifiers which are implied by the context | 77 |
info/magnolia/cms/security/auth/PrincipalCollection.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 48 |
| Avoid modifiers which are implied by the context | 50 |
| Avoid modifiers which are implied by the context | 52 |
| Avoid modifiers which are implied by the context | 54 |
| Avoid modifiers which are implied by the context | 56 |
| Avoid modifiers which are implied by the context | 58 |
| Avoid modifiers which are implied by the context | 60 |
| Avoid modifiers which are implied by the context | 62 |
| Avoid modifiers which are implied by the context | 65 |
| Avoid modifiers which are implied by the context | 70 |
| Avoid modifiers which are implied by the context | 75 |
| Avoid modifiers which are implied by the context | 77 |
info/magnolia/cms/security/auth/callback/Base64CallbackHandler.java
| Violation | Line |
|---|---|
| Useless parentheses. | 63 |
info/magnolia/cms/security/auth/login/BasicLogin.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 49 |
info/magnolia/cms/security/auth/login/LoginHandler.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 48 |
info/magnolia/cms/security/operations/AccessDefinition.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 45 |
| Avoid modifiers which are implied by the context | 47 |
| Avoid modifiers which are implied by the context | 49 |
| Avoid modifiers which are implied by the context | 51 |
info/magnolia/cms/security/operations/OperationPermissionDefinition.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 43 |
| Avoid modifiers which are implied by the context | 44 |
| Avoid modifiers which are implied by the context | 45 |
| Avoid modifiers which are implied by the context | 47 |
| Avoid modifiers which are implied by the context | 48 |
| Avoid modifiers which are implied by the context | 49 |
| Avoid modifiers which are implied by the context | 51 |
| Avoid modifiers which are implied by the context | 53 |
| Avoid modifiers which are implied by the context | 55 |
| Avoid modifiers which are implied by the context | 57 |
| Avoid modifiers which are implied by the context | 59 |
| Avoid modifiers which are implied by the context | 61 |
| Avoid modifiers which are implied by the context | 63 |
| Avoid modifiers which are implied by the context | 65 |
| Avoid modifiers which are implied by the context | 67 |
| Avoid modifiers which are implied by the context | 69 |
| Avoid modifiers which are implied by the context | 71 |
| Avoid modifiers which are implied by the context | 73 |
info/magnolia/cms/servlets/MgnlServletContextListener.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 49 |
info/magnolia/cms/util/ClasspathResourcesUtil.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 73–75 |
| Avoid modifiers which are implied by the context | 74 |
| Avoid empty catch blocks | 145–147 |
| These nested if statements could be combined | 267–269 |
info/magnolia/cms/util/ContentUtil.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 134–136 |
| Avoid modifiers which are implied by the context | 141–143 |
| Avoid empty catch blocks | 461–463 |
| Avoid empty catch blocks | 476–478 |
| These nested if statements could be combined | 524–527 |
| Useless parentheses. | 598 |
info/magnolia/cms/util/DateUtil.java
| Violation | Line |
|---|---|
| Useless parentheses. | 209 |
info/magnolia/cms/util/DelayedExecutor.java
| Violation | Line |
|---|---|
| Useless parentheses. | 82 |
info/magnolia/cms/util/DeprecationUtil.java
| Violation | Line |
|---|---|
| Unnecessary use of fully qualified name 'org.slf4j.LoggerFactory.getLogger' due to existing import 'org.slf4j.LoggerFactory' | 45 |
| Unnecessary use of fully qualified name 'org.slf4j.Logger' due to existing import 'org.slf4j.Logger' | 45 |
| Useless parentheses. | 64 |
info/magnolia/cms/util/InheritanceContentWrapper.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 73 |
info/magnolia/cms/util/LazyNodeDataWrapper.java
| Violation | Line |
|---|---|
| Useless parentheses. | 74 |
info/magnolia/cms/util/NodeDataUtil.java
| Violation | Line |
|---|---|
| Useless parentheses. | 110 |
| Useless parentheses. | 112 |
| Useless parentheses. | 114 |
| Useless parentheses. | 116 |
| Useless parentheses. | 118 |
| Useless parentheses. | 121 |
| Useless parentheses. | 195 |
| Useless parentheses. | 197 |
| Useless parentheses. | 199 |
| Useless parentheses. | 201 |
| Useless parentheses. | 203 |
| Useless parentheses. | 205 |
| Useless parentheses. | 237 |
| Useless parentheses. | 242 |
info/magnolia/cms/util/NodeMapWrapper.java
| Violation | Line |
|---|---|
| Useless parentheses. | 250 |
info/magnolia/cms/util/PathUtil.java
| Violation | Line |
|---|---|
| Useless parentheses. | 46 |
info/magnolia/cms/util/QueryUtil.java
| Violation | Line |
|---|---|
| Unnecessary use of fully qualified name 'javax.jcr.query.Query.JCR_SQL2' due to existing import 'javax.jcr.query.Query' | 231 |
info/magnolia/cms/util/RequestDispatchUtil.java
| Violation | Line |
|---|---|
| Useless parentheses. | 131 |
| Useless parentheses. | 131 |
info/magnolia/cms/util/SearchReplace.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'group'. | 87 |
info/magnolia/cms/util/ServletUtil.java
| Violation | Line |
|---|---|
| Useless parentheses. | 97 |
| Avoid empty if statements | 176–178 |
| Useless parentheses. | 176 |
| Useless parentheses. | 176 |
info/magnolia/cms/util/UnicodeNormalizer.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 53 |
info/magnolia/cms/util/UrlPattern.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 71–103 |
info/magnolia/commands/MgnlCatalog.java
| Violation | Line |
|---|---|
| Useless parentheses. | 75 |
info/magnolia/commands/chain/ChainBase.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 51 |
| Avoid empty catch blocks | 123–125 |
| Useless parentheses. | 129 |
| Useless parentheses. | 132 |
info/magnolia/commands/chain/Command.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 45 |
| Avoid modifiers which are implied by the context | 46 |
| Avoid modifiers which are implied by the context | 58 |
info/magnolia/commands/impl/ExportCommand.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 126–128 |
info/magnolia/content2bean/Bean2ContentProcessor.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 46 |
| Avoid modifiers which are implied by the context | 48 |
info/magnolia/content2bean/Bean2ContentTransformer.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 49 |
| Avoid modifiers which are implied by the context | 54 |
| Avoid modifiers which are implied by the context | 59 |
| Avoid modifiers which are implied by the context | 64 |
| Avoid modifiers which are implied by the context | 69 |
| Avoid modifiers which are implied by the context | 74 |
| Avoid modifiers which are implied by the context | 79 |
| Avoid modifiers which are implied by the context | 84 |
info/magnolia/content2bean/Content2BeanProcessor.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 51–52 |
| Avoid modifiers which are implied by the context | 57 |
info/magnolia/content2bean/Content2BeanTransformer.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 53 |
| Avoid modifiers which are implied by the context | 58 |
| Avoid modifiers which are implied by the context | 63 |
| Avoid modifiers which are implied by the context | 68 |
| Avoid modifiers which are implied by the context | 73 |
| Avoid modifiers which are implied by the context | 78 |
| Avoid modifiers which are implied by the context | 83 |
| Avoid modifiers which are implied by the context | 88 |
| Avoid modifiers which are implied by the context | 93 |
| Avoid modifiers which are implied by the context | 100 |
info/magnolia/content2bean/TransformationState.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 48 |
| Avoid modifiers which are implied by the context | 50 |
| Avoid modifiers which are implied by the context | 52 |
| Avoid modifiers which are implied by the context | 54 |
| Avoid modifiers which are implied by the context | 56 |
| Avoid modifiers which are implied by the context | 58 |
| Avoid modifiers which are implied by the context | 60 |
| Avoid modifiers which are implied by the context | 62 |
| Avoid modifiers which are implied by the context | 64 |
| Avoid modifiers which are implied by the context | 66 |
| Avoid modifiers which are implied by the context | 68 |
| Avoid modifiers which are implied by the context | 70 |
| Avoid modifiers which are implied by the context | 72 |
| Avoid modifiers which are implied by the context | 77 |
info/magnolia/content2bean/TypeMapping.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 47 |
| Avoid modifiers which are implied by the context | 49 |
| Avoid modifiers which are implied by the context | 51 |
| Avoid modifiers which are implied by the context | 56 |
| Avoid modifiers which are implied by the context | 61 |
info/magnolia/content2bean/impl/CollectionPropertyHidingTransformer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 97–101 |
info/magnolia/content2bean/impl/Content2BeanTransformerImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 352–357 |
info/magnolia/content2bean/impl/TypeMappingImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 141–154 |
| These nested if statements could be combined | 254–256 |
info/magnolia/context/AbstractContext.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 68 |
info/magnolia/context/Context.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 61 |
| Avoid modifiers which are implied by the context | 66 |
| Avoid modifiers which are implied by the context | 71 |
| Avoid modifiers which are implied by the context | 73 |
| Avoid modifiers which are implied by the context | 75 |
| Avoid modifiers which are implied by the context | 77 |
| Avoid modifiers which are implied by the context | 79 |
| Avoid modifiers which are implied by the context | 81 |
| Avoid modifiers which are implied by the context | 83 |
| Avoid modifiers which are implied by the context | 85 |
| Avoid modifiers which are implied by the context | 87 |
| Avoid modifiers which are implied by the context | 89 |
| Avoid modifiers which are implied by the context | 94 |
| Avoid modifiers which are implied by the context | 96 |
| Avoid modifiers which are implied by the context | 98 |
| Avoid modifiers which are implied by the context | 103 |
| Avoid modifiers which are implied by the context | 105 |
| Avoid modifiers which are implied by the context | 115 |
| Avoid modifiers which are implied by the context | 120 |
| Avoid modifiers which are implied by the context | 127 |
| Avoid modifiers which are implied by the context | 134 |
| Avoid modifiers which are implied by the context | 142 |
| Avoid modifiers which are implied by the context | 150 |
| Avoid modifiers which are implied by the context | 157 |
| Avoid modifiers which are implied by the context | 162 |
| Avoid modifiers which are implied by the context | 168 |
| Avoid modifiers which are implied by the context | 174 |
| Avoid modifiers which are implied by the context | 181 |
| Avoid modifiers which are implied by the context | 186 |
info/magnolia/context/MgnlContext.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 404–406 |
info/magnolia/context/SingleJCRSessionSystemContext.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 49 |
info/magnolia/context/ThreadDependentSystemContext.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 46 |
info/magnolia/context/WebContext.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 59 |
| Avoid modifiers which are implied by the context | 65 |
| Avoid modifiers which are implied by the context | 70 |
| Avoid modifiers which are implied by the context | 75 |
| Avoid modifiers which are implied by the context | 86 |
| Avoid modifiers which are implied by the context | 92 |
| Avoid modifiers which are implied by the context | 98 |
| Avoid modifiers which are implied by the context | 103 |
| Avoid modifiers which are implied by the context | 109 |
| Avoid modifiers which are implied by the context | 114 |
| Avoid modifiers which are implied by the context | 143 |
| Avoid modifiers which are implied by the context | 148 |
| Avoid modifiers which are implied by the context | 154 |
info/magnolia/debug/TrackingStatus.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'trace'. | 50 |
info/magnolia/event/SystemEventBus.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 41 |
info/magnolia/freemarker/FreemarkerServletContextWrapper.java
| Violation | Line |
|---|---|
| Useless parentheses. | 91 |
| Useless parentheses. | 91 |
| Useless parentheses. | 108 |
| Useless parentheses. | 108 |
| Avoid empty catch blocks | 112–114 |
| Avoid empty catch blocks | 177–179 |
info/magnolia/freemarker/models/BinaryNodeDataModel.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 73–75 |
| Avoid empty catch blocks | 85–87 |
| Avoid empty catch blocks | 99–101 |
| Useless parentheses. | 118 |
| Avoid empty catch blocks | 127–129 |
| Useless parentheses. | 138 |
| Useless parentheses. | 147 |
info/magnolia/freemarker/models/ContentMapModel.java
| Violation | Line |
|---|---|
| Useless parentheses. | 98 |
info/magnolia/freemarker/models/ContentModel.java
| Violation | Line |
|---|---|
| Useless parentheses. | 125 |
info/magnolia/freemarker/models/MagnoliaObjectWrapper.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 50 |
| Useless parentheses. | 77 |
info/magnolia/importexport/BootstrapFilesComparator.java
| Violation | Line |
|---|---|
| Useless parentheses. | 73 |
| Useless parentheses. | 73 |
info/magnolia/importexport/DataTransporter.java
| Violation | Line |
|---|---|
| Useless parentheses. | 449 |
| Useless parentheses. | 451 |
info/magnolia/importexport/PropertiesImportExport.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 141–143 |
| Avoid unused method parameters such as 'dumpMetaData'. | 249 |
| Useless parentheses. | 287 |
| Useless parentheses. | 290 |
| Useless parentheses. | 293 |
| Useless parentheses. | 296 |
| Useless parentheses. | 299 |
| Useless parentheses. | 302 |
| Useless parentheses. | 305 |
info/magnolia/importexport/filters/MetadataUuidFilter.java
| Violation | Line |
|---|---|
| Useless parentheses. | 133 |
info/magnolia/importexport/filters/SkipNodePropertyFilter.java
| Violation | Line |
|---|---|
| Useless parentheses. | 132 |
info/magnolia/importexport/postprocessors/MetaDataAsMixinConversionHelper.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'moveProperties(Node,Node,Map)'. | 211 |
info/magnolia/jackrabbit/ProviderImpl.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 182–184 |
| Avoid empty catch blocks | 477–479 |
info/magnolia/jcr/decoration/ContentDecoratorNodeWrapper.java
| Violation | Line |
|---|---|
| Useless parentheses. | 302 |
| Useless parentheses. | 303 |
info/magnolia/jcr/node2bean/Node2BeanProcessor.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 51 |
| Avoid modifiers which are implied by the context | 59 |
| Avoid modifiers which are implied by the context | 69 |
| Avoid modifiers which are implied by the context | 75 |
info/magnolia/jcr/node2bean/Node2BeanTransformer.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 51 |
| Avoid modifiers which are implied by the context | 56 |
| Avoid modifiers which are implied by the context | 61 |
| Avoid modifiers which are implied by the context | 66 |
| Avoid modifiers which are implied by the context | 71 |
| Avoid modifiers which are implied by the context | 77 |
| Avoid modifiers which are implied by the context | 82 |
info/magnolia/jcr/node2bean/TransformationState.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 46 |
| Avoid modifiers which are implied by the context | 48 |
| Avoid modifiers which are implied by the context | 50 |
| Avoid modifiers which are implied by the context | 52 |
| Avoid modifiers which are implied by the context | 54 |
| Avoid modifiers which are implied by the context | 56 |
| Avoid modifiers which are implied by the context | 58 |
| Avoid modifiers which are implied by the context | 60 |
| Avoid modifiers which are implied by the context | 62 |
| Avoid modifiers which are implied by the context | 64 |
| Avoid modifiers which are implied by the context | 66 |
| Avoid modifiers which are implied by the context | 68 |
| Avoid modifiers which are implied by the context | 70 |
| Avoid modifiers which are implied by the context | 75 |
info/magnolia/jcr/node2bean/TypeMapping.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 45 |
| Avoid modifiers which are implied by the context | 47 |
| Avoid modifiers which are implied by the context | 49 |
info/magnolia/jcr/node2bean/impl/CollectionPropertyHidingTransformer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 102–106 |
info/magnolia/jcr/node2bean/impl/Node2BeanTransformerImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 291–296 |
info/magnolia/jcr/node2bean/impl/TypeMappingImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 264–266 |
info/magnolia/jcr/util/ContentMap.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 116–118 |
| Avoid empty if statements | 200–203 |
| Unnecessary use of fully qualified name 'info.magnolia.link.LinkUtil.convertLinksFromUUIDPattern' due to existing import 'info.magnolia.link.LinkUtil' | 226 |
| Avoid empty catch blocks | 240–242 |
| Avoid empty catch blocks | 253–255 |
| Avoid empty catch blocks | 267–269 |
| Unnecessary use of fully qualified name 'java.util.Map.Entry' due to existing import 'java.util.Map' | 277 |
info/magnolia/jcr/util/MetaDataUtil.java
| Violation | Line |
|---|---|
| Useless parentheses. | 80 |
info/magnolia/jcr/util/PropertiesImportExport.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 153–155 |
| Useless parentheses. | 260 |
| Useless parentheses. | 263 |
| Useless parentheses. | 266 |
| Useless parentheses. | 269 |
| Useless parentheses. | 272 |
| Useless parentheses. | 275 |
| Useless parentheses. | 278 |
info/magnolia/jcr/util/PropertyUtil.java
| Violation | Line |
|---|---|
| Useless parentheses. | 91 |
| Avoid empty catch blocks | 184–186 |
| Useless parentheses. | 285 |
| Useless parentheses. | 287 |
| Useless parentheses. | 289 |
| Useless parentheses. | 291 |
| Useless parentheses. | 293 |
| Useless parentheses. | 296 |
| Useless parentheses. | 477 |
| Useless parentheses. | 479 |
| Useless parentheses. | 481 |
| Useless parentheses. | 483 |
| Useless parentheses. | 485 |
| Useless parentheses. | 487 |
| Useless parentheses. | 489 |
| Avoid unused private methods such as 'createBinaryFromInputStream(InputStream)'. | 505 |
info/magnolia/jcr/wrapper/ChannelVisibilityContentDecorator.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 76–90 |
info/magnolia/jcr/wrapper/DelegateNodeWrapper.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 582–585 |
info/magnolia/jcr/wrapper/DelegateSessionWrapper.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 354–357 |
info/magnolia/jcr/wrapper/HTMLEscapingPropertyWrapper.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'transformLineBreaks'. | 49 |
info/magnolia/jcr/wrapper/MgnlPropertySettingContentDecorator.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 192–202 |
| Avoid unnecessary return statements | 220 |
| Useless parentheses. | 626 |
info/magnolia/jcr/wrapper/MgnlPropertySettingNodeWrapper.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 211–213 |
info/magnolia/link/Link.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 140–142 |
| Avoid empty catch blocks | 156–158 |
| Avoid empty catch blocks | 158–159 |
| Avoid empty catch blocks | 194–196 |
info/magnolia/link/LinkFactory.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 48 |
info/magnolia/link/LinkTransformerManager.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 52 |
info/magnolia/link/LinkUtil.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 543–545 |
| Avoid unused private methods such as 'getURI2RepositoryManager()'. | 671 |
info/magnolia/module/InstallContext.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 101–137 |
info/magnolia/module/ModuleLifecycleContext.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 49 |
| Avoid modifiers which are implied by the context | 54 |
| Avoid modifiers which are implied by the context | 59 |
info/magnolia/module/ModuleManager.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 88–95 |
| Avoid modifiers which are implied by the context | 100–163 |
| These nested if statements could be combined | 144–146 |
| Avoid modifiers which are implied by the context | 168–208 |
info/magnolia/module/ModuleRegistry.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 95–102 |
info/magnolia/module/delta/ChangeAllPropertiesWithCertainValueTask.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 92–94 |
info/magnolia/module/delta/FilterOrderingTask.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getLastChild(Node,Collection)'. | 116 |
| Avoid unused private methods such as 'hasNodes(Node,Collection)'. | 128 |
info/magnolia/module/delta/MoveAndRenamePropertyTask.java
| Violation | Line |
|---|---|
| Useless parentheses. | 69 |
info/magnolia/module/delta/OrderFilterBeforeTask.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getFirstChild(Node,Collection)'. | 110 |
| Avoid unused private methods such as 'hasNodes(Node,Collection)'. | 121 |
info/magnolia/module/delta/RemoveDuplicatePermissionTask.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 91–95 |
info/magnolia/module/delta/RenamePropertyTask.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'name'. | 43 |
info/magnolia/module/files/MD5CheckingFileExtractorOperation.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 121–123 |
| Avoid empty while statements | 187–188 |
info/magnolia/module/model/reader/DependencyLevelComparator.java
| Violation | Line |
|---|---|
| Useless parentheses. | 111 |
info/magnolia/monitoring/SystemMonitor.java
| Violation | Line |
|---|---|
| Useless parentheses. | 77 |
info/magnolia/objectfactory/CandidateParameterResolver.java
| Violation | Line |
|---|---|
| Unnecessary use of fully qualified name 'javax.inject.Provider' due to existing import 'javax.inject.Provider' | 61 |
info/magnolia/objectfactory/ObjectManufacturer.java
| Violation | Line |
|---|---|
| Useless parentheses. | 139 |
info/magnolia/objectfactory/ParameterResolver.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 41 |
info/magnolia/objectfactory/guice/GuiceComponentConfigurationModule.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'bindProvider(ProviderConfiguration)'. | 119 |
| Avoid unused private methods such as 'bindInstance(InstanceConfiguration)'. | 134 |
| Avoid unused private methods such as 'bindImplementation(ImplementationConfiguration)'. | 147 |
| Avoid unused private methods such as 'bindInScope(ScopedBindingBuilder,ComponentConfiguration)'. | 168 |
info/magnolia/objectfactory/guice/GuiceComponentProvider.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'concat(ParameterResolver,ParameterResolver)'. | 126 |
info/magnolia/objectfactory/guice/GuicePropertyConfigurer.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 58–61 |
info/magnolia/repository/DefaultRepositoryManager.java
| Violation | Line |
|---|---|
| Useless parentheses. | 137 |
info/magnolia/repository/RepositoryConstants.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 45 |
| Avoid modifiers which are implied by the context | 46 |
| Avoid modifiers which are implied by the context | 47 |
| Avoid modifiers which are implied by the context | 48 |
| Avoid modifiers which are implied by the context | 49 |
| Avoid modifiers which are implied by the context | 50 |
| Avoid modifiers which are implied by the context | 54 |
| Avoid modifiers which are implied by the context | 55 |
info/magnolia/repository/RepositoryNotInitializedException.java
| Violation | Line |
|---|---|
| Useless parentheses. | 49 |
info/magnolia/setup/CoreModuleVersionHandler.java
| Violation | Line |
|---|---|
| Useless parentheses. | 150 |
| Useless parentheses. | 161 |
| Useless parentheses. | 164 |
| Useless parentheses. | 166 |
| Useless parentheses. | 169 |
| Useless parentheses. | 182 |
| Useless parentheses. | 186 |
| Useless parentheses. | 189 |
| Useless parentheses. | 192 |
| Useless parentheses. | 197 |
info/magnolia/setup/for4_5/UpdateSecurityFilterClientCallbacksConfiguration.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'ctx'. | 189 |
| Avoid unused method parameters such as 'ctx'. | 205 |
| Unnecessary use of fully qualified name 'ArrayUtils.contains' due to existing import 'org.apache.commons.lang.ArrayUtils' | 208 |
info/magnolia/setup/for5_2/IsNotAProblematicEnvironmentCondition.java
| Violation | Line |
|---|---|
| Useless parentheses. | 84 |
| Useless parentheses. | 84 |
info/magnolia/setup/initial/AddURIPermissionsToAllRoles.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'addPermission(Content,String,String,long)'. | 94 |
info/magnolia/setup/initial/LoginAuthTypePropertyMovedToFilter.java
| Violation | Line |
|---|---|
| Useless parentheses. | 48 |
info/magnolia/stats/JCRStatsMBean.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 40 |