Checkstyle Results
The following document contains the results of Checkstyle 10.14.0 with struts_checks.xml ruleset.
Summary
| Files | |||
|---|---|---|---|
| 15 | 0 | 0 | 168 |
Files
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | RightCurly | 2 | |
| coding | MagicNumber | 3 | |
| design | HideUtilityClassConstructor | 1 | |
VisibilityModifier
|
4 | ||
| header | RegexpHeader
|
15 | |
| javadoc | JavadocMethod
|
2 | |
JavadocPackage
|
3 | ||
| JavadocVariable | 1 | ||
| MissingJavadocMethod | 5 | ||
| misc | ArrayTypeStyle | 1 | |
| FinalParameters | 96 | ||
| NewlineAtEndOfFile | 9 | ||
| modifier | ModifierOrder | 14 | |
| naming | LocalFinalVariableName | 1 | |
| MemberName | 2 | ||
| MethodName | 5 | ||
| StaticVariableName | 2 | ||
| whitespace | MethodParamPad | 1 | |
| OperatorWrap | 1 |
Details
org/apache/struts/apps/mailreader/Constants.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | 1 | |
| header | RegexpHeader | Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. | 4 | |
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 34 |
org/apache/struts/apps/mailreader/actions/BaseAction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | 1 | |
| header | RegexpHeader | Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. | 4 | |
| design | VisibilityModifier | Variable 'USERNAME' must be private and have accessor methods. | 67 | |
| naming | StaticVariableName | Name 'USERNAME' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 67 | |
| design | VisibilityModifier | Variable 'PASSWORD' must be private and have accessor methods. | 74 | |
| naming | StaticVariableName | Name 'PASSWORD' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 74 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 81 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 88 | |
| misc | FinalParameters | Parameter request should be final. | 102 | |
| misc | FinalParameters | Parameter user should be final. | 102 | |
| misc | FinalParameters | Parameter session should be final. | 119 | |
| misc | FinalParameters | Parameter method should be final. | 119 | |
| misc | FinalParameters | Parameter key should be final. | 119 | |
| misc | FinalParameters | Parameter mapping should be final. | 135 | |
| misc | FinalParameters | Parameter mapping should be final. | 149 | |
| misc | FinalParameters | Parameter mapping should be final. | 164 | |
| misc | FinalParameters | Parameter form should be final. | 182 | |
| misc | FinalParameters | Parameter property should be final. | 182 | |
| misc | FinalParameters | Parameter session should be final. | 207 | |
| misc | FinalParameters | Parameter request should be final. | 219 | |
| misc | FinalParameters | Parameter database should be final. | 239 | |
| misc | FinalParameters | Parameter username should be final. | 239 | |
| misc | FinalParameters | Parameter password should be final. | 240 | |
| misc | FinalParameters | Parameter errors should be final. | 240 | |
| misc | FinalParameters | Parameter username should be final. | 282 | |
| misc | FinalParameters | Parameter password should be final. | 283 | |
| misc | FinalParameters | Parameter errors should be final. | 283 | |
| misc | FinalParameters | Parameter session should be final. | 311 | |
| misc | FinalParameters | Parameter request should be final. | 323 | |
| misc | FinalParameters | Parameter mapping should be final. | 339 | |
| misc | FinalParameters | Parameter request should be final. | 340 | |
| misc | FinalParameters | Parameter errors should be final. | 341 | |
| misc | FinalParameters | Parameter mapping should be final. | 355 | |
| misc | FinalParameters | Parameter method should be final. | 355 | |
| misc | FinalParameters | Parameter request should be final. | 366 | |
| misc | FinalParameters | Parameter user should be final. | 379 | |
| naming | LocalFinalVariableName | Name 'LOG_DATABASE_SAVE_ERROR' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 381 | |
| misc | FinalParameters | Parameter form should be final. | 404 | |
| misc | FinalParameters | Parameter property should be final. | 404 | |
| misc | FinalParameters | Parameter value should be final. | 404 |
org/apache/struts/apps/mailreader/actions/LocaleAction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | 1 | |
| javadoc | JavadocPackage | Missing package-info.java file. | 1 | |
| header | RegexpHeader | Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. | 4 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 48 | |
| misc | FinalParameters | Parameter string should be final. | 59 | |
| misc | FinalParameters | Parameter mapping should be final. | 118 | |
| misc | FinalParameters | Parameter form should be final. | 119 | |
| misc | FinalParameters | Parameter request should be final. | 120 | |
| misc | FinalParameters | Parameter response should be final. | 121 |
org/apache/struts/apps/mailreader/actions/LogoffAction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | 1 | |
| header | RegexpHeader | Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. | 4 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 49 | |
| javadoc | MissingJavadocMethod | Missing a Javadoc comment. | 53 | |
| misc | FinalParameters | Parameter mapping should be final. | 54 | |
| misc | FinalParameters | Parameter form should be final. | 55 | |
| misc | FinalParameters | Parameter request should be final. | 56 | |
| misc | FinalParameters | Parameter response should be final. | 57 |
org/apache/struts/apps/mailreader/actions/LogonAction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | 1 | |
| header | RegexpHeader | Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. | 4 | |
| javadoc | JavadocMethod | @return tag should be present and have description. | 56 | |
| misc | FinalParameters | Parameter mapping should be final. | 57 | |
| misc | FinalParameters | Parameter form should be final. | 58 | |
| misc | FinalParameters | Parameter request should be final. | 59 | |
| misc | FinalParameters | Parameter response should be final. | 60 |
org/apache/struts/apps/mailreader/actions/MainMenuAction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | 1 | |
| header | RegexpHeader | Line does not match expected header line of ' \*\s*\$Id.*\$'. | 2 | |
| javadoc | MissingJavadocMethod | Missing a Javadoc comment. | 36 | |
| misc | FinalParameters | Parameter mapping should be final. | 37 | |
| misc | FinalParameters | Parameter form should be final. | 38 | |
| misc | FinalParameters | Parameter request should be final. | 39 | |
| misc | FinalParameters | Parameter response should be final. | 40 |
org/apache/struts/apps/mailreader/actions/RegistrationAction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | 1 | |
| header | RegexpHeader | Line does not match expected header line of ' \*\s*\$Id.*\$'. | 2 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 58 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 68 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 75 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 82 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 89 | |
| design | VisibilityModifier | Variable 'LOG_REGISTRATION_POPULATE' must be private and have accessor methods. | 98 | |
| naming | MemberName | Name 'LOG_REGISTRATION_POPULATE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 98 | |
| misc | FinalParameters | Parameter username should be final. | 108 | |
| misc | FinalParameters | Parameter errors should be final. | 109 | |
| misc | FinalParameters | Parameter form should be final. | 129 | |
| misc | FinalParameters | Parameter request should be final. | 130 | |
| misc | FinalParameters | Parameter errors should be final. | 131 | |
| blocks | RightCurly | '}' at column 9 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | 141 | |
| misc | FinalParameters | Parameter form should be final. | 183 | |
| misc | FinalParameters | Parameter user should be final. | 183 | |
| misc | FinalParameters | Parameter user should be final. | 218 | |
| misc | FinalParameters | Parameter form should be final. | 218 | |
| misc | FinalParameters | Parameter request should be final. | 257 | |
| misc | FinalParameters | Parameter errors should be final. | 258 | |
| naming | MethodName | Name 'Edit' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 286 | |
| misc | FinalParameters | Parameter mapping should be final. | 287 | |
| misc | FinalParameters | Parameter form should be final. | 288 | |
| misc | FinalParameters | Parameter request should be final. | 289 | |
| misc | FinalParameters | Parameter response should be final. | 290 | |
| naming | MethodName | Name 'Save' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 323 | |
| misc | FinalParameters | Parameter mapping should be final. | 324 | |
| misc | FinalParameters | Parameter form should be final. | 325 | |
| misc | FinalParameters | Parameter request should be final. | 326 | |
| misc | FinalParameters | Parameter response should be final. | 327 |
org/apache/struts/apps/mailreader/actions/SubscriptionAction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | 1 | |
| header | RegexpHeader | Line does not match expected header line of ' \*\s*\$Id.*\$'. | 2 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 48 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 58 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 65 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 72 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 76 | |
| design | VisibilityModifier | Variable 'LOG_SUBSCRIPTION_POPULATE' must be private and have accessor methods. | 76 | |
| naming | MemberName | Name 'LOG_SUBSCRIPTION_POPULATE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 76 | |
| misc | FinalParameters | Parameter user should be final. | 88 | |
| misc | FinalParameters | Parameter host should be final. | 88 | |
| blocks | RightCurly | '}' at column 9 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | 94 | |
| misc | FinalParameters | Parameter subscription should be final. | 116 | |
| misc | FinalParameters | Parameter form should be final. | 116 | |
| misc | FinalParameters | Parameter form should be final. | 145 | |
| misc | FinalParameters | Parameter subscription should be final. | 145 | |
| misc | FinalParameters | Parameter mapping should be final. | 182 | |
| misc | FinalParameters | Parameter session should be final. | 183 | |
| misc | FinalParameters | Parameter user should be final. | 184 | |
| misc | FinalParameters | Parameter subscription should be final. | 185 | |
| naming | MethodName | Name 'Delete' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 221 | |
| misc | FinalParameters | Parameter mapping should be final. | 222 | |
| misc | FinalParameters | Parameter form should be final. | 223 | |
| misc | FinalParameters | Parameter request should be final. | 224 | |
| misc | FinalParameters | Parameter response should be final. | 225 | |
| naming | MethodName | Name 'Edit' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 254 | |
| misc | FinalParameters | Parameter mapping should be final. | 255 | |
| misc | FinalParameters | Parameter form should be final. | 256 | |
| misc | FinalParameters | Parameter request should be final. | 257 | |
| misc | FinalParameters | Parameter response should be final. | 258 | |
| naming | MethodName | Name 'Save' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 299 | |
| misc | FinalParameters | Parameter mapping should be final. | 300 | |
| misc | FinalParameters | Parameter form should be final. | 301 | |
| misc | FinalParameters | Parameter request should be final. | 302 | |
| misc | FinalParameters | Parameter response should be final. | 303 |
org/apache/struts/apps/mailreader/actions/WelcomeAction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | 1 | |
| header | RegexpHeader | Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. | 4 | |
| javadoc | MissingJavadocMethod | Missing a Javadoc comment. | 56 | |
| misc | FinalParameters | Parameter mapping should be final. | 57 | |
| misc | FinalParameters | Parameter form should be final. | 58 | |
| misc | FinalParameters | Parameter request should be final. | 59 | |
| misc | FinalParameters | Parameter response should be final. | 60 |
org/apache/struts/apps/mailreader/plugin/MemoryDatabasePlugIn.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | 1 | |
| javadoc | JavadocPackage | Missing package-info.java file. | 1 | |
| header | RegexpHeader | Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. | 4 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 74 | |
| javadoc | MissingJavadocMethod | Missing a Javadoc comment. | 93 | |
| javadoc | MissingJavadocMethod | Missing a Javadoc comment. | 97 | |
| misc | FinalParameters | Parameter pathname should be final. | 97 | |
| misc | FinalParameters | Parameter servlet should be final. | 136 | |
| misc | FinalParameters | Parameter config should be final. | 136 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 154 | |
| javadoc | JavadocMethod | @return tag should be present and have description. | 179 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 190 | |
| coding | MagicNumber | '1024' is a magic number. | 198 | |
| coding | MagicNumber | '1024' is a magic number. | 199 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 201 | |
| coding | MagicNumber | '1024' is a magic number. | 201 |
org/apache/struts/apps/mailreader/resources/AlternateApplicationResources.properties
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | RegexpHeader | Line does not match expected header line of '/\*\s*'. | 1 |
org/apache/struts/apps/mailreader/resources/AlternateApplicationResources_ja.properties
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | RegexpHeader | Missing a header - not enough lines in file. | 1 |
org/apache/struts/apps/mailreader/resources/ApplicationResources.properties
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | RegexpHeader | Line does not match expected header line of '/\*\s*'. | 1 |
org/apache/struts/apps/mailreader/resources/ApplicationResources_ja.properties
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | RegexpHeader | Line does not match expected header line of '/\*\s*'. | 1 |
org/apache/struts/apps/mailreader/resources/ApplicationResources_ru.properties
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | RegexpHeader | Line does not match expected header line of '/\*\s*'. | 1 |


