Frameworks

## PMD Results

The following document contains the results of PMD.

## Summary

Files | Errors
--- | ---
5 | 25

## Files

Files | Violations
--- | ---
org/apache/portals/bridges/frameworks/ForwardTag.java | 3
org/apache/portals/bridges/frameworks/Forwarder.java | 2
org/apache/portals/bridges/frameworks/GenericFrameworkPortlet.java | 9
org/apache/portals/bridges/frameworks/spring/PortletApplicationModelImpl.java | 3
org/apache/portals/bridges/frameworks/spring/validation/ValidationSupport.java | 8

### org/apache/portals/bridges/frameworks/ForwardTag.java

Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 66
Avoid if (x != y) ..; else ..; | 70
Avoid if (x != y) ..; else ..; | 72

### org/apache/portals/bridges/frameworks/Forwarder.java

Violation | Line
--- | ---
Avoid empty catch blocks | 177
Avoid empty catch blocks | 203

### org/apache/portals/bridges/frameworks/GenericFrameworkPortlet.java

Violation | Line
--- | ---
The class 'GenericFrameworkPortlet' has a Cyclomatic Complexity of 3 (Highest = 11). | 59
Avoid empty catch blocks | 231
Avoid empty catch blocks | 257
The method 'determineLogicalView' has a Cyclomatic Complexity of 11. | 295
Switch statements should have a default label | 479
Avoid unused formal parameters such as 'view' | 502
Avoid if (x != y) ..; else ..; | 512
Avoid using if statements without curly braces | 515
Avoid using if statements without curly braces | 556

### org/apache/portals/bridges/frameworks/spring/PortletApplicationModelImpl.java

Violation | Line
--- | ---
No need to import a type that's in the same package | 42
Avoid empty catch blocks | 171
Avoid if (x != y) ..; else ..; | 224

### org/apache/portals/bridges/frameworks/spring/validation/ValidationSupport.java

Violation | Line
--- | ---
Avoid empty catch blocks | 70
Avoid empty catch blocks | 74
Avoid empty catch blocks | 78
Avoid unused local variables such as 'action' | 362
Avoid unused local variables such as 'f1' | 376
Avoid unused local variables such as 'f2' | 377
Avoid unused local variables such as 'f0' | 378
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor to silence this warning. | 404