The following document contains the results of PMD 5.5.1.
| Violation | Line |
|---|---|
| A high number of imports can indicate a high degree of coupling within an object. | 34–1129 |
| Possible God class (WMC=128, ATFD=333, TCC=0.0) | 34–1129 |
| The class 'DocumentRest' has a Modified Cyclomatic Complexity of 4 (Highest = 18). | 115–1129 |
| The class 'DocumentRest' has a Standard Cyclomatic Complexity of 4 (Highest = 18). | 115–1129 |
| The class 'DocumentRest' has a Cyclomatic Complexity of 6 (Highest = 23). | 115–1129 |
| Avoid really long classes. | 115–1129 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 138 |
| Avoid variables with short names like t | 141 |
| The method 'doSubmitOrUnsubmitDocumentToValidate' has a Cyclomatic Complexity of 13. | 459–502 |
| Deeply nested if..then statements are hard to read | 479–486 |
| The method 'doValidateDocument' has a Cyclomatic Complexity of 13. | 521–572 |
| The method 'getDocumentData' has a Cyclomatic Complexity of 23. | 813–930 |
| The method 'getDocumentData' has a Standard Cyclomatic Complexity of 18. | 813–930 |
| The method getDocumentData() has an NPath complexity of 583200 | 813–930 |
| The method 'getDocumentData' has a Modified Cyclomatic Complexity of 18. | 813–930 |
| Avoid really long methods. | 813–930 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 817 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 818 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 819 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 822 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 825 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 826 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 854 |
| These nested if statements could be combined | 893–896 |
| The method 'setAttribute' has a Cyclomatic Complexity of 15. | 943–1013 |
| The method 'setAttribute' has a Modified Cyclomatic Complexity of 10. | 943–1013 |
| The method 'setAttribute' has a Standard Cyclomatic Complexity of 10. | 943–1013 |
| The method setAttribute() has an NPath complexity of 660 | 943–1013 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 948 |
| The method 'modifyDocument' has a Cyclomatic Complexity of 10. | 1063–1128 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 42 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 42 |
| Violation | Line |
|---|---|
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 70 |
| Violation | Line |
|---|---|
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 100 |
| Violation | Line |
|---|---|
| This statement may have some unnecessary parentheses | 76 |
| Violation | Line |
|---|---|
| This statement may have some unnecessary parentheses | 45 |
| Violation | Line |
|---|---|
| This statement may have some unnecessary parentheses | 76 |
| Violation | Line |
|---|---|
| This statement may have some unnecessary parentheses | 75 |
| Violation | Line |
|---|---|
| This statement may have some unnecessary parentheses | 76 |
| Violation | Line |
|---|---|
| This statement may have some unnecessary parentheses | 76 |
| Violation | Line |
|---|---|
| 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. | 47–70 |
| Document empty constructor | 51–53 |
| Violation | Line |
|---|---|
| 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. | 57–285 |
| Document empty constructor | 62–64 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 112 |
| The String literal "text" appears 6 times in this file; the first occurrence is on line 114 | 114 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 225 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 245 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 266 |
| Violation | Line |
|---|---|
| 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. | 46–79 |
| Document empty constructor | 50–52 |
| Violation | Line |
|---|---|
| A class which only has private constructors should be final | 41–201 |
| Avoid excessively long variable names like PATH_GET_SPACES_ALLOWING_DOCUMENT_CREATION_BY_CODE_DOCUMENT_TYPE | 52 |
| Avoid excessively long variable names like TAG_CREATE_DOCUMENT_FIELDS_FORM_PARAMETER_ATTRIBUT_VALUE | 120 |
