| Violation | Line |
|---|---|
| Avoid reassigning parameters such as 'response' | 41 |
| Avoid using 'if...else' statements without curly braces | 64 |
| Violation | Line |
|---|---|
| The class 'PortletServlet' has a Cyclomatic Complexity of 4 (Highest = 12). | 45 |
| The method 'performActionRenderRequest' has a Cyclomatic Complexity of 12. | 125 |
| Avoid if (x != y) ..; else ..; | 146 |
| Deeply nested if..then statements are hard to read | 147 |
| Deeply nested if..then statements are hard to read | 155 |
| Avoid using if statements without curly braces | 161 |
| Avoid using if statements without curly braces | 163 |
| Avoid using if statements without curly braces | 166 |
| Avoid using 'if...else' statements without curly braces | 173 |
| Avoid using 'if...else' statements without curly braces | 176 |
| Violation | Line |
|---|---|
| Avoid using if statements without curly braces | 49 |
| Violation | Line |
|---|---|
| Avoid using 'if...else' statements without curly braces | 56 |
| Avoid using 'if...else' statements without curly braces | 58 |
| Avoid using if statements without curly braces | 83 |
| Avoid using if statements without curly braces | 95 |
| Violation | Line |
|---|---|
| Avoid unused formal parameters such as 'include' | 55 |
| Avoid using if statements without curly braces | 86 |
| Deeply nested if..then statements are hard to read | 86 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log' | 36 |
| Avoid if (x != y) ..; else ..; | 88 |
| Avoid using 'if...else' statements without curly braces | 94 |
| Avoid if (x != y) ..; else ..; | 103 |
| Avoid using 'if...else' statements without curly braces | 104 |
| Avoid using 'if...else' statements without curly braces | 106 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log' | 36 |
| Avoid using 'if...else' statements without curly braces | 51 |
| Avoid using 'if...else' statements without curly braces | 53 |
| Avoid using 'if...else' statements without curly braces | 62 |
| Avoid using 'if...else' statements without curly braces | 64 |
| Violation | Line |
|---|---|
| Avoid reassigning parameters such as 'response' | 41 |
| Avoid using 'if...else' statements without curly braces | 64 |
| Violation | Line |
|---|---|
| The class 'StrutsPortlet' has a Cyclomatic Complexity of 3 (Highest = 32). | 52 |
| The method 'init' has a Cyclomatic Complexity of 13. | 127 |
| Avoid using if statements without curly braces | 131 |
| Avoid using 'if...else' statements without curly braces | 148 |
| Avoid using if statements without curly braces | 152 |
| Avoid using if statements without curly braces | 158 |
| Avoid using if statements without curly braces | 180 |
| Avoid using if statements without curly braces | 182 |
| Avoid using if statements without curly braces | 184 |
| Avoid using if statements without curly braces | 186 |
| Avoid really long methods. | 309 |
| The method 'processRequest' has a Cyclomatic Complexity of 32. | 309 |
| Avoid using if statements without curly braces | 375 |
| Avoid using if statements without curly braces | 431 |
| Avoid using if statements without curly braces | 436 |
| Avoid if (x != y) ..; else ..; | 442 |
| Avoid using if statements without curly braces | 449 |
| Avoid empty catch blocks | 460 |
| Avoid if (x != y) ..; else ..; | 464 |
| Avoid using if statements without curly braces | 467 |
| Avoid using if statements without curly braces | 495 |
| Avoid using if statements without curly braces | 502 |
| Avoid using if statements without curly braces | 513 |
| Avoid using if statements without curly braces | 524 |
| Avoid using if statements without curly braces | 527 |
| Avoid using if statements without curly braces | 533 |
| Avoid using 'for' statements without curly braces | 543 |
| Violation | Line |
|---|---|
| Avoid reassigning parameters such as 'pageURL' | 46 |
| Avoid using 'if...else' statements without curly braces | 52 |
| Avoid using 'if...else' statements without curly braces | 54 |
| Avoid using if statements without curly braces | 59 |
| Avoid using if statements without curly braces | 66 |
| All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor to silence this warning. | 86 |
| Violation | Line |
|---|---|
| Avoid using if statements without curly braces | 48 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 70 |
| Avoid if (x != y) ..; else ..; | 72 |
| Violation | Line |
|---|---|
| The class 'RenderContextAttributes' has a Cyclomatic Complexity of 3 (Highest = 13). | 30 |
| The method 'saveAttributes' has a Cyclomatic Complexity of 13. | 194 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 64 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'ServletRequest' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'ServletRequest' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'ServletRequest' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'ServletRequest' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'ServletRequest' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'ServletRequest' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'ServletRequest' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'ServletRequest' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'ServletRequest' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'ServletRequest' | 19 |