The following document contains the results of Checkstyle. 
| Files | Infos |
Warnings |
Errors |
|---|---|---|---|
| 41 | 0 | 0 | 124 |
| Rules | Violations | Severity |
|---|---|---|
| Translation | 0 | |
| FileLength | 0 | |
FileTabCharacter
|
0 | |
JavadocMethod
|
75 | |
JavadocType
|
4 | |
JavadocVariable
|
4 | |
JavadocStyle
|
0 | |
ConstantName
|
0 | |
| LocalFinalVariableName | 1 | |
| LocalVariableName | 0 | |
| MemberName | 0 | |
| MethodName | 0 | |
| PackageName | 0 | |
| ParameterName | 0 | |
| StaticVariableName | 0 | |
| TypeName | 0 | |
| AvoidStarImport | 7 | |
| IllegalImport | 0 | |
| RedundantImport | 0 | |
| UnusedImports | 5 | |
LineLength
|
0 | |
| MethodLength | 0 | |
| ParameterNumber | 0 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 1 | |
| NoWhitespaceAfter | 0 | |
NoWhitespaceBefore
|
0 | |
| OperatorWrap | 1 | |
| ParenPad | 0 | |
| TypecastParenPad | 0 | |
| WhitespaceAfter | 0 | |
WhitespaceAround
|
0 | |
| ModifierOrder | 4 | |
| RedundantModifier | 0 | |
| AvoidNestedBlocks | 0 | |
| EmptyBlock | 0 | |
| LeftCurly | 0 | |
| NeedBraces | 0 | |
| RightCurly | 1 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 0 | |
| EqualsHashCode | 0 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 0 | |
| MissingSwitchDefault | 0 | |
| RedundantThrows | 2 | |
| SimplifyBooleanExpression | 0 | |
| SimplifyBooleanReturn | 0 | |
| FinalClass | 0 | |
| HideUtilityClassConstructor | 3 | |
| InterfaceIsType | 0 | |
VisibilityModifier
|
0 | |
| ArrayTypeStyle | 0 | |
FinalParameters
|
14 | |
| UpperEll | 0 | |
NestedIfDepth
|
0 | |
| NestedTryDepth | 0 | |
| ExplicitInitialization | 2 |
| Violation | Message | Line |
|---|---|---|
| Utility classes should not have a public or default constructor. | 9 |
| Violation | Message | Line |
|---|---|---|
| Utility classes should not have a public or default constructor. | 9 | |
| Missing a Javadoc comment. | 12 | |
| Missing a Javadoc comment. | 15 | |
| Missing a Javadoc comment. | 17 | |
| Missing a Javadoc comment. | 19 |
| Violation | Message | Line |
|---|---|---|
| Unused import - com.sun.org.apache.bcel.internal.generic.LoadClass. | 3 | |
| Unused import - org.apache.tapestry5.ioc.services.ThreadLocale. | 5 |
| Violation | Message | Line |
|---|---|---|
| Utility classes should not have a public or default constructor. | 13 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - org.apache.tapestry5.*. | 3 | |
| Using the '.*' form of import should be avoided - org.apache.tapestry5.annotations.*. | 4 | |
| Missing a Javadoc comment. | 104 | |
| Missing a Javadoc comment. | 116 | |
| Missing a Javadoc comment. | 120 | |
| Missing a Javadoc comment. | 143 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 9 | |
| Missing a Javadoc comment. | 42 |
| Violation | Message | Line |
|---|---|---|
| Unused import - org.apache.tapestry5.Asset. | 3 | |
| Using the '.*' form of import should be avoided - org.apache.tapestry5.annotations.*. | 5 | |
| Unused import - org.apache.tapestry5.ioc.annotations.Value. | 9 | |
| Missing a Javadoc comment. | 107 | |
| Missing a Javadoc comment. | 122 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - org.apache.tapestry5.annotations.*. | 8 | |
| Variable 'delete' explicitly initialized to 'false' (default value for its type). | 105 | |
| Variable 'cancel' explicitly initialized to 'false' (default value for its type). | 108 | |
| Missing a Javadoc comment. | 115 | |
| Missing a Javadoc comment. | 120 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 39 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - org.apache.tapestry5.services.*. | 13 | |
| Using the '.*' form of import should be avoided - org.appfuse.webapp.services.impl.*. | 22 | |
| Missing a Javadoc comment. | 37 | |
| Missing a Javadoc comment. | 50 | |
| Missing a Javadoc comment. | 72 | |
| Missing a Javadoc comment. | 76 | |
| Missing a Javadoc comment. | 89 | |
| Missing a Javadoc comment. | 133 | |
| Expected an @return tag. | 165 | |
| Expected @param tag for 'logger'. | 166 | |
| Expected @param tag for 'renderer'. | 167 | |
| Expected @param tag for 'componentSource'. | 168 | |
| Expected @param tag for 'productionMode'. | 170 | |
| Expected @param tag for 'service'. | 171 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 11 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 7 | |
| Expected an @return tag. | 11 | |
| Missing a Javadoc comment. | 12 | |
| Expected an @return tag. | 18 | |
| Missing a Javadoc comment. | 20 | |
| Expected an @return tag. | 27 | |
| Expected @param tag for 'roleName'. | 27 | |
| Expected an @return tag. | 33 | |
| Expected @param tag for 'user'. | 39 | |
| Expected an @return tag. | 50 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 22 | |
| Parameter environment should be final. | 22 | |
| Parameter markupWriter should be final. | 22 | |
| Missing a Javadoc comment. | 27 | |
| Missing a Javadoc comment. | 31 | |
| Missing a Javadoc comment. | 35 | |
| Missing a Javadoc comment. | 48 | |
| Missing a Javadoc comment. | 52 | |
| Missing a Javadoc comment. | 56 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.util.*. | 8 | |
| Missing a Javadoc comment. | 20 | |
| Parameter logger should be final. | 20 | |
| Parameter threadLocale should be final. | 20 | |
| '(' should be on the previous line. | 26 | |
| Name 'EMPTY' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 37 | |
| Parameter map should be final. | 67 | |
| Parameter locale should be final. | 67 |
| Violation | Message | Line |
|---|---|---|
| 'private' modifier out of order with the JLS suggestions. | 16 | |
| 'private' modifier out of order with the JLS suggestions. | 17 | |
| 'private' modifier out of order with the JLS suggestions. | 18 | |
| Missing a Javadoc comment. | 20 | |
| Parameter simpleMailMessage should be final. | 20 | |
| Parameter mailEngine should be final. | 20 | |
| Parameter messages should be final. | 20 | |
| Missing a Javadoc comment. | 27 | |
| Redundant throws: 'UsernameNotFoundException' is unchecked exception. | 28 | |
| Redundant throws: 'MailException' is unchecked exception. | 28 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 14 | |
| Missing a Javadoc comment. | 18 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 8 | |
| Missing a Javadoc comment. | 12 | |
| Parameter roleManager should be final. | 12 | |
| Missing a Javadoc comment. | 16 | |
| Missing a Javadoc comment. | 20 |
| Violation | Message | Line |
|---|---|---|
| 'static' modifier out of order with the JLS suggestions. | 25 | |
| Missing a Javadoc comment. | 27 | |
| Missing a Javadoc comment. | 39 | |
| Missing a Javadoc comment. | 52 | |
| Missing a Javadoc comment. | 63 | |
| Missing a Javadoc comment. | 68 | |
| Missing a Javadoc comment. | 92 | |
| Missing a Javadoc comment. | 96 | |
| Missing a Javadoc comment. | 103 |
| Violation | Message | Line |
|---|---|---|
| Unused import - org.apache.tapestry5.ioc.internal.util.InternalUtils. | 4 | |
| Missing a Javadoc comment. | 9 | |
| Missing a Javadoc comment. | 13 | |
| Parameter userManager should be final. | 13 | |
| Missing a Javadoc comment. | 17 | |
| Missing a Javadoc comment. | 21 | |
| '}' should be on the same line. | 30 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 23 | |
| Parameter environment should be final. | 23 | |
| Parameter iconAsset should be final. | 23 | |
| Parameter markupWriter should be final. | 24 | |
| Missing a Javadoc comment. | 30 | |
| Missing a Javadoc comment. | 41 | |
| Missing a Javadoc comment. | 45 | |
| Missing a Javadoc comment. | 56 | |
| Missing a Javadoc comment. | 59 | |
| Missing a Javadoc comment. | 78 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 28 |