Checkstyle Results
The following document contains the results of Checkstyle 10.14.0 with struts_checks.xml ruleset.
Summary
Files |
Info |
Warnings |
Errors |
12 |
0 |
0 |
106 |
Rules
Category |
Rule |
Violations |
Severity |
design |
DesignForExtension |
4 |
Error |
header |
RegexpHeader
- header
1: / 2: * 3: s* 4: 5: * 6: s* 7: $Id.* 8: $ 9: 10: * 11: s* 12: 13: * 14: s*Copyright 15: d{4}(- 16: d{4}|, 17: d{4})* The Apache Software Fou 18: datio 19: . 20: 21: * 22: s* 23: 24: * 25: s*Lice 26: sed u 27: der the Apache Lice 28: se, Versio 29: 2 30: .0 31: (the .Lice 32: se. 33: ); 34: 35: * 36: s*you may 37: ot use this file except i 38: complia 39: ce with the Lice 40: se 41: . 42: 43: * 44: s*You may obtai 45: a copy of the Lice 46: se at 47: 48: * 49: s* 50: 51: * 52: s*http://www 53: .apache 54: .org/lice 55: ses/LICENSE 56: -2 57: .0 58: 59: * 60: s* 61: 62: * 63: s*U 64: less required by applicable law or agreed to i 65: writi 66: g, software 67: 68: * 69: s*distributed u 70: der the Lice 71: se is distributed o 72: a 73: .AS IS. BASIS, 74: 75: * 76: s*WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied 77: . 78: 79: * 80: s*See the Lice 81: se for the specific la 82: guage gover 83: i 84: g permissio 85: s a 86: d 87: 88: * 89: s*limitatio 90: s u 91: der the Lice 92: se 93: . 94: 95: */ 96: s* |
12 |
Error |
imports |
UnusedImports
|
1 |
Error |
javadoc |
JavadocMethod
|
2 |
Error |
|
JavadocPackage
|
4 |
Error |
misc |
FinalParameters |
57 |
Error |
|
NewlineAtEndOfFile |
9 |
Error |
modifier |
ModifierOrder |
4 |
Error |
naming |
LocalVariableName |
1 |
Error |
sizes |
LineLength
- fileExtensions:
"java"
- max:
"120"
- ignorePattern:
"^ \* @version .*$" |
11 |
Error |
whitespace |
ParenPad |
1 |
Error |
Details
org/apache/struts/tiles2/TilesPlugin.java
Severity |
Category |
Rule |
Message |
Line |
Error |
misc |
NewlineAtEndOfFile |
File does not end with a newline. |
1 |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
modifier |
ModifierOrder |
'static' modifier out of order with the JLS suggestions. |
75 |
Error |
misc |
FinalParameters |
Parameter moduleAware should be final. |
116 |
Error |
misc |
FinalParameters |
Parameter servlet should be final. |
132 |
Error |
misc |
FinalParameters |
Parameter moduleConfig should be final. |
132 |
Error |
javadoc |
JavadocMethod |
Expected @throws tag for 'IllegalStateException'. |
190 |
Error |
misc |
FinalParameters |
Parameter config should be final. |
215 |
Error |
misc |
FinalParameters |
Parameter plugInConfigObject should be final. |
265 |
org/apache/struts/tiles2/TilesPluginContainer.java
Severity |
Category |
Rule |
Message |
Line |
Error |
misc |
NewlineAtEndOfFile |
File does not end with a newline. |
1 |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
misc |
FinalParameters |
Parameter localeResolver should be final. |
58 |
Error |
misc |
FinalParameters |
Parameter key should be final. |
81 |
Error |
misc |
FinalParameters |
Parameter key should be final. |
99 |
Error |
misc |
FinalParameters |
Parameter definitionsFactory should be final. |
100 |
Error |
design |
DesignForExtension |
Class 'TilesPluginContainer' looks like designed for extension (can be subclassed), but the method 'getDefinition' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'TilesPluginContainer' final or making the method 'getDefinition' static/final/abstract/empty, or adding allowed annotation for the method. |
108 |
Error |
misc |
FinalParameters |
Parameter definitionName should be final. |
109 |
Error |
misc |
FinalParameters |
Parameter request should be final. |
110 |
Error |
misc |
FinalParameters |
Parameter request should be final. |
133 |
org/apache/struts/tiles2/TilesPluginContainerFactory.java
Severity |
Category |
Rule |
Message |
Line |
Error |
misc |
NewlineAtEndOfFile |
File does not end with a newline. |
1 |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
imports |
UnusedImports |
Unused import - org.apache.tiles.definition.UnresolvingLocaleDefinitionsFactory. |
32 |
Error |
modifier |
ModifierOrder |
'final' modifier out of order with the JLS suggestions. |
46 |
Error |
design |
DesignForExtension |
Class 'TilesPluginContainerFactory' looks like designed for extension (can be subclassed), but the method 'createContainer' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'TilesPluginContainerFactory' final or making the method 'createContainer' static/final/abstract/empty, or adding allowed annotation for the method. |
49 |
Error |
misc |
FinalParameters |
Parameter applicationContext should be final. |
50 |
Error |
design |
DesignForExtension |
Class 'TilesPluginContainerFactory' looks like designed for extension (can be subclassed), but the method 'instantiateContainer' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'TilesPluginContainerFactory' final or making the method 'instantiateContainer' static/final/abstract/empty, or adding allowed annotation for the method. |
67 |
Error |
misc |
FinalParameters |
Parameter context should be final. |
69 |
Error |
misc |
FinalParameters |
Parameter applicationContext should be final. |
82 |
Error |
whitespace |
ParenPad |
')' is preceded with whitespace. |
95 |
Error |
misc |
FinalParameters |
Parameter container should be final. |
132 |
Error |
misc |
FinalParameters |
Parameter applicationContext should be final. |
133 |
org/apache/struts/tiles2/TilesRequestProcessor.java
Severity |
Category |
Rule |
Message |
Line |
Error |
misc |
NewlineAtEndOfFile |
File does not end with a newline. |
1 |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
modifier |
ModifierOrder |
'final' modifier out of order with the JLS suggestions. |
69 |
Error |
misc |
FinalParameters |
Parameter servlet should be final. |
84 |
Error |
misc |
FinalParameters |
Parameter moduleConfig should be final. |
84 |
Error |
misc |
FinalParameters |
Parameter definitionName should be final. |
108 |
Error |
misc |
FinalParameters |
Parameter req should be final. |
109 |
Error |
misc |
FinalParameters |
Parameter res should be final. |
110 |
Error |
misc |
FinalParameters |
Parameter uri should be final. |
154 |
Error |
misc |
FinalParameters |
Parameter request should be final. |
155 |
Error |
misc |
FinalParameters |
Parameter response should be final. |
156 |
Error |
misc |
FinalParameters |
Parameter request should be final. |
182 |
Error |
misc |
FinalParameters |
Parameter response should be final. |
183 |
Error |
misc |
FinalParameters |
Parameter forward should be final. |
184 |
Error |
misc |
FinalParameters |
Parameter uri should be final. |
226 |
Error |
misc |
FinalParameters |
Parameter request should be final. |
227 |
Error |
misc |
FinalParameters |
Parameter response should be final. |
228 |
Error |
misc |
FinalParameters |
Parameter uri should be final. |
254 |
Error |
misc |
FinalParameters |
Parameter request should be final. |
255 |
Error |
misc |
FinalParameters |
Parameter response should be final. |
256 |
org/apache/struts/tiles2/actions/DefinitionDispatcherAction.java
Severity |
Category |
Rule |
Message |
Line |
Error |
misc |
NewlineAtEndOfFile |
File does not end with a newline. |
1 |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
modifier |
ModifierOrder |
'final' modifier out of order with the JLS suggestions. |
75 |
Error |
misc |
FinalParameters |
Parameter mapping should be final. |
94 |
Error |
misc |
FinalParameters |
Parameter form should be final. |
95 |
Error |
misc |
FinalParameters |
Parameter req should be final. |
96 |
Error |
misc |
FinalParameters |
Parameter res should be final. |
97 |
org/apache/struts/tiles2/actions/TilesAction.java
Severity |
Category |
Rule |
Message |
Line |
Error |
misc |
NewlineAtEndOfFile |
File does not end with a newline. |
1 |
Error |
javadoc |
JavadocPackage |
Missing package-info.java file. |
1 |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
misc |
FinalParameters |
Parameter mapping should be final. |
69 |
Error |
misc |
FinalParameters |
Parameter form should be final. |
70 |
Error |
misc |
FinalParameters |
Parameter req should be final. |
71 |
Error |
misc |
FinalParameters |
Parameter res should be final. |
72 |
Error |
javadoc |
JavadocMethod |
Expected @throws tag for 'ServletException'. |
83 |
Error |
misc |
FinalParameters |
Parameter context should be final. |
111 |
Error |
misc |
FinalParameters |
Parameter mapping should be final. |
112 |
Error |
misc |
FinalParameters |
Parameter form should be final. |
113 |
Error |
misc |
FinalParameters |
Parameter request should be final. |
114 |
Error |
misc |
FinalParameters |
Parameter response should be final. |
115 |
org/apache/struts/tiles2/commands/TilesPreProcessor.java
Severity |
Category |
Rule |
Message |
Line |
Error |
misc |
NewlineAtEndOfFile |
File does not end with a newline. |
1 |
Error |
javadoc |
JavadocPackage |
Missing package-info.java file. |
1 |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
misc |
FinalParameters |
Parameter sacontext should be final. |
88 |
org/apache/struts/tiles2/package-info.java
Severity |
Category |
Rule |
Message |
Line |
Error |
misc |
NewlineAtEndOfFile |
File does not end with a newline. |
1 |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 312). |
34 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 455). |
44 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 121). |
55 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 154). |
56 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 236). |
73 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 134). |
89 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 127). |
90 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 142). |
122 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 132). |
209 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 167). |
235 |
Error |
sizes |
LineLength |
Line is longer than 120 characters (found 164). |
261 |
org/apache/struts/tiles2/preparer/ActionPreparer.java
Severity |
Category |
Rule |
Message |
Line |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
misc |
FinalParameters |
Parameter action should be final. |
48 |
Error |
misc |
FinalParameters |
Parameter tilesContext should be final. |
53 |
Error |
misc |
FinalParameters |
Parameter attributeContext should be final. |
54 |
org/apache/struts/tiles2/preparer/StrutsPreparerFactory.java
Severity |
Category |
Rule |
Message |
Line |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
misc |
FinalParameters |
Parameter name should be final. |
38 |
org/apache/struts/tiles2/preparer/UrlPreparer.java
Severity |
Category |
Rule |
Message |
Line |
Error |
javadoc |
JavadocPackage |
Missing package-info.java file. |
1 |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
misc |
FinalParameters |
Parameter url should be final. |
52 |
Error |
misc |
FinalParameters |
Parameter tilesContext should be final. |
57 |
Error |
misc |
FinalParameters |
Parameter attributeContext should be final. |
58 |
org/apache/struts/tiles2/util/PlugInConfigContextAdapter.java
Severity |
Category |
Rule |
Message |
Line |
Error |
misc |
NewlineAtEndOfFile |
File does not end with a newline. |
1 |
Error |
javadoc |
JavadocPackage |
Missing package-info.java file. |
1 |
Error |
header |
RegexpHeader |
Line does not match expected header line of ' \*\s*Copyright \d{4}(-\d{4}|,\d{4})* The Apache Software Foundation\.'. |
4 |
Error |
misc |
FinalParameters |
Parameter plugInConfigObject should be final. |
56 |
Error |
misc |
FinalParameters |
Parameter servletContext should be final. |
57 |
Error |
naming |
LocalVariableName |
Name 'initParam_' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
61 |
Error |
design |
DesignForExtension |
Class 'PlugInConfigContextAdapter' looks like designed for extension (can be subclassed), but the method 'getInitParams' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'PlugInConfigContextAdapter' final or making the method 'getInitParams' static/final/abstract/empty, or adding allowed annotation for the method. |
68 |