PMD Results
The following document contains the results of PMD 5.1.2.
Files
info/magnolia/module/mail/MailConstants.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 |
| Do not hard code the IP address |
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/module/mail/MailModule.java
| Violation |
Line |
| Avoid unused private fields such as 'log'. |
56 |
info/magnolia/module/mail/handlers/ThreadedMailHandler.java
| Violation |
Line |
| Avoid empty if statements |
128–130 |
info/magnolia/module/mail/pages/SendMailPage.java
| Violation |
Line |
| Avoid unused private fields such as 'log'. |
58 |
| Avoid empty if statements |
72–74 |
info/magnolia/module/mail/templates/MailAttachment.java
| Violation |
Line |
| Unnecessary use of fully qualified name 'java.net.URL' due to existing import 'java.net.URL' |
131 |
| Unnecessary use of fully qualified name 'java.io.File' due to existing import 'java.io.File' |
145 |
| Unnecessary use of fully qualified name 'java.net.URL' due to existing import 'java.net.URL' |
171 |
info/magnolia/module/mail/templates/MgnlMultipartEmail.java
| Violation |
Line |
| Useless parentheses. |
77 |
info/magnolia/module/mail/templates/impl/MgnlPageEmail.java
| Violation |
Line |
| Avoid unused private fields such as 'UTF_8'. |
93 |
| Avoid unused private fields such as 'MAGNOLIA'. |
95 |
| Avoid unused method parameters such as 'urlBasePath'. |
197 |
| Useless parentheses. |
225 |
| Avoid unused local variables such as 'writer'. |
291 |
| Avoid unused local variables such as 'cid'. |
313 |
info/magnolia/module/mail/templates/impl/SimpleEmail.java
| Violation |
Line |
| Overriding method merely calls super |
54–56 |
info/magnolia/module/mail/util/MailUtil.java
| Violation |
Line |
| Avoid empty catch blocks |
116–118 |
