The following document contains the results of Checkstyle.
| Files | Infos | Warnings | Errors |
|---|---|---|---|
| 139 | 0 | 0 | 841 |
| Error | Line | |
|---|---|---|
| First sentence should end with a period. | 96 |
|
| Name '_instance' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 99 |
|
| First sentence should end with a period. | 101 |
|
| Missing a Javadoc comment. | 112 |
|
| Missing a Javadoc comment. | 117 |
|
| First sentence should end with a period. | 122 |
|
| Missing a Javadoc comment. | 139 |
|
| First sentence should end with a period. | 217 |
|
| First sentence should end with a period. | 228 |
|
| First sentence should end with a period. | 240 |
|
| Expected @param tag for 'locale'. | 258 |
|
| First sentence should end with a period. | 282 |
|
| ';' is followed by whitespace. | 290 |
|
| 'cast' is not followed by whitespace. | 293 |
|
| ';' is followed by whitespace. | 299 |
|
| ';' is followed by whitespace. | 307 |
|
| 'cast' is not followed by whitespace. | 310 |
|
| ';' is followed by whitespace. | 316 |
|
| First sentence should end with a period. | 324 |
|
| ';' is followed by whitespace. | 332 |
|
| Name '_converter' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 335 |
|
| 'cast' is not followed by whitespace. | 335 |
|
| ';' is followed by whitespace. | 341 |
|
| First sentence should end with a period. | 349 |
|
| First sentence should end with a period. | 360 |
|
| First sentence should end with a period. | 371 |
|
| First sentence should end with a period. | 380 |
|
| 'cast' is not followed by whitespace. | 389 |
|
| Throws count is 5 (max allowed is 2). | 402 |
|
| Expected @throws tag for 'NoSuchMethodException'. | 402 |
|
| Expected @throws tag for 'IllegalArgumentException'. | 403 |
|
| Redundant throws: 'IllegalArgumentException' is unchecked exception. | 403 |
|
| Expected @throws tag for 'InstantiationException'. | 404 |
|
| Expected @throws tag for 'IllegalAccessException'. | 405 |
|
| Expected @throws tag for 'InvocationTargetException'. | 406 |
|
| Throws count is 5 (max allowed is 2). | 436 |
|
| Expected @throws tag for 'NoSuchMethodException'. | 436 |
|
| Expected @throws tag for 'IllegalArgumentException'. | 437 |
|
| Redundant throws: 'IllegalArgumentException' is unchecked exception. | 437 |
|
| Expected @throws tag for 'InstantiationException'. | 438 |
|
| Expected @throws tag for 'IllegalAccessException'. | 439 |
|
| Expected @throws tag for 'InvocationTargetException'. | 440 |
|
| 'cast' is not followed by whitespace. | 448 |
|
| Name '_converter' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 451 |
|
| 'cast' is not followed by whitespace. | 451 |
|
| '{' is followed by whitespace. | 456 |
|
| '{' is followed by whitespace. | 458 |
|
| Name '_newConverter' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 460 |
|
| 'cast' is not followed by whitespace. | 461 |
|
| 'cast' is not followed by whitespace. | 467 |
|
| 'cast' is not followed by whitespace. | 468 |
|
| 'cast' is not followed by whitespace. | 481 |
|
| Expected an @return tag. | 511 |
|
| Throws count is 5 (max allowed is 2). | 512 |
|
| Expected @throws tag for 'NoSuchMethodException'. | 512 |
|
| Expected @throws tag for 'IllegalArgumentException'. | 513 |
|
| Redundant throws: 'IllegalArgumentException' is unchecked exception. | 513 |
|
| Expected @throws tag for 'InstantiationException'. | 514 |
|
| Expected @throws tag for 'IllegalAccessException'. | 515 |
|
| Expected @throws tag for 'InvocationTargetException'. | 516 |
|
| 'cast' is not followed by whitespace. | 524 |
|
| Name '_formatter' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 528 |
|
| 'cast' is not followed by whitespace. | 528 |
|
| Name '_newFormatter' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 533 |
|
| 'cast' is not followed by whitespace. | 533 |
|
| 'cast' is not followed by whitespace. | 545 |
|
| Missing a Javadoc comment. | 557 |
|
| Avoid inline conditionals. | 560 |
|
| Avoid inline conditionals. | 561 |
|
| Avoid inline conditionals. | 562 |
|
| Missing a Javadoc comment. | 571 |
|
| Avoid inline conditionals. | 574 |
|
| Avoid inline conditionals. | 575 |
|
| Avoid inline conditionals. | 576 |
|
| Missing a Javadoc comment. | 584 |
| Error | Line | |
|---|---|---|
|
| Missing a Javadoc comment. | 58 |
|
| Missing a Javadoc comment. | 61 |
|
| Missing a Javadoc comment. | 64 |
|
| Missing a Javadoc comment. | 66 |
|
| Name 'BREAKSYMBOLS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 66 |
|
| ',' is not followed by whitespace. | 66 |
|
| First sentence should end with a period. | 68 |
|
| 'validatorRegistry' hides a field. | 74 |
|
| Expected @param tag for 'validatorRegistry'. | 74 |
|
| 'ctrl' hides a field. | 75 |
|
| Expected @param tag for 'ctrl'. | 75 |
|
| Line is longer than 100 characters. | 82 |
|
| Cyclomatic Complexity is 20 (max allowed is 9). | 84 |
|
| 'if' construct must use '{}'s. | 92 |
|
| Line is longer than 100 characters. | 105 |
|
| ';' is followed by whitespace. | 107 |
|
| 'cast' is not followed by whitespace. | 109 |
|
| 'if' construct must use '{}'s. | 111 |
|
| 'cast' is not followed by whitespace. | 123 |
|
| 'if' construct must use '{}'s. | 124 |
|
| '&&' should be on the previous line. | 135 |
|
| ';' is followed by whitespace. | 139 |
|
| 'cast' is not followed by whitespace. | 141 |
|
| Cyclomatic Complexity is 12 (max allowed is 9). | 153 |
|
| Missing a Javadoc comment. | 153 |
|
| 'cast' is not followed by whitespace. | 167 |
|
| Avoid inline conditionals. | 178 |
|
| Avoid inline conditionals. | 190 |
|
| 'if' construct must use '{}'s. | 203 |
|
| Missing a Javadoc comment. | 209 |
|
| Missing a Javadoc comment. | 240 |
|
| Missing a Javadoc comment. | 260 |
|
| 'cast' is not followed by whitespace. | 263 |
|
| 'if' construct must use '{}'s. | 266 |
|
| Cyclomatic Complexity is 12 (max allowed is 9). | 293 |
|
| Missing a Javadoc comment. | 293 |
|
| ';' is followed by whitespace. | 311 |
|
| 'cast' is not followed by whitespace. | 313 |
|
| 'cast' is not followed by whitespace. | 319 |
|
| Avoid inline conditionals. | 328 |
|
| Avoid inline conditionals. | 354 |
|
| Expected @param tag for 'cs'. | 381 |
| Error | Line | |
|---|---|---|
|
| First sentence should end with a period. | 57 |
|
| First sentence should end with a period. | 181 |
|
| First sentence should end with a period. | 190 |
|
| 'noCache' hides a field. | 194 |
|
| Expected @param tag for 'noCache'. | 194 |
|
| Expected @param tag for 'b'. | 214 |
|
| Line is longer than 100 characters. | 344 |
|
| Line is longer than 100 characters. | 357 |
|
| Expected @param tag for 'b'. | 417 |
|
| 'populateAndValidate' hides a field. | 502 |
| Error | Line | |
|---|---|---|
|
| Missing a Javadoc comment. | 62 |
|
| Missing a Javadoc comment. | 68 |
|
| Missing a Javadoc comment. | 71 |
|
| Missing a Javadoc comment. | 74 |
|
| Missing a Javadoc comment. | 77 |
|
| Missing a Javadoc comment. | 83 |
|
| Missing a Javadoc comment. | 86 |
|
| Missing a Javadoc comment. | 89 |
|
| First sentence should end with a period. | 91 |
|
| 'static' modifier out of order with the JLS suggestions. | 92 |
|
| First sentence should end with a period. | 94 |
|
| 'static' modifier out of order with the JLS suggestions. | 95 |
|
| 'bean' hides a field. | 134 |
|
| Avoid inline conditionals. | 159 |
|
| 'cast' is not followed by whitespace. | 159 |
|
| 'errors' hides a field. | 166 |
|
| 'if' construct must use '{}'s. | 243 |
|
| Missing a Javadoc comment. | 263 |
|
| Avoid inline conditionals. | 320 |
|
| 'cast' is not followed by whitespace. | 341 |
|
| Line is longer than 100 characters. | 353 |
|
| Cyclomatic Complexity is 16 (max allowed is 9). | 405 |
|
| 'if' construct must use '{}'s. | 407 |
|
| Name '_overrideFields' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 413 |
|
| 'cast' is not followed by whitespace. | 425 |
|
| Name '_value' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 460 |
|
| Cyclomatic Complexity is 11 (max allowed is 9). | 496 |
|
| 'cast' is not followed by whitespace. | 519 |
|
| 'if' construct must use '{}'s. | 589 |
|
| Avoid inline conditionals. | 629 |
|
| Expected an @return tag. | 640 |
|
| 'if' construct must use '{}'s. | 642 |
|
| First sentence should end with a period. | 646 |
|
| Avoid inline conditionals. | 653 |
|
| First sentence should end with a period. | 656 |
|
| Avoid inline conditionals. | 663 |
|
| First sentence should end with a period. | 666 |
|
| First sentence should end with a period. | 675 |
|
| Avoid inline conditionals. | 682 |
|
| First sentence should end with a period. | 685 |
|
| 'if' construct must use '{}'s. | 692 |
|
| First sentence should end with a period. | 696 |
|
| Avoid inline conditionals. | 703 |
|
| First sentence should end with a period. | 706 |
|
| Avoid inline conditionals. | 714 |
|
| First sentence should end with a period. | 717 |
|
| Avoid inline conditionals. | 724 |
|
| First sentence should end with a period. | 727 |
|
| Avoid inline conditionals. | 735 |
|
| First sentence should end with a period. | 738 |
|
| Avoid inline conditionals. | 746 |
|
| First sentence should end with a period. | 749 |
|
| First sentence should end with a period. | 758 |
|
| 'controller' hides a field. | 762 |
|
| ';' is followed by whitespace. | 781 |
|
| 'cast' is not followed by whitespace. | 783 |
|
| ';' is followed by whitespace. | 797 |
|
| 'cast' is not followed by whitespace. | 799 |
| Error | Line | |
|---|---|---|
|
| Line is longer than 100 characters. | 137 |
|
| First sentence should end with a period. | 145 |
|
| First sentence should end with a period. | 148 |
|
| Line is longer than 100 characters. | 162 |
|
| Cyclomatic Complexity is 20 (max allowed is 9). | 164 |
|
| Expected @throws tag for 'ServletException'. | 164 |
|
| 'cast' is not followed by whitespace. | 278 |
|
| Name '_execParams' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 308 |
|
| Line is longer than 100 characters. | 314 |
|
| 'cast' is not followed by whitespace. | 314 |
|
| Cyclomatic Complexity is 18 (max allowed is 9). | 353 |
|
| Expected @param tag for '_execParams'. | 356 |
|
| Name '_execParams' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 356 |
|
| Expected @throws tag for 'Exception'. | 357 |
|
| Line is longer than 100 characters. | 373 |
|
| '{' should be on a new line. | 391 |
|
| 'cast' is not followed by whitespace. | 394 |
|
| 'if' construct must use '{}'s. | 414 |
|
| 'cast' is not followed by whitespace. | 417 |
|
| '&&' should be on the previous line. | 419 |
|
| 'if' construct must use '{}'s. | 435 |
|
| Name '_succeeded' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 453 |
|
| 'cast' is not followed by whitespace. | 455 |
|
| 'if' construct must use '{}'s. | 459 |
|
| Redundant 'final' modifier. | 473 |
|
| 'if' construct must use '{}'s. | 483 |
|
| Unused @param tag for 'locale'. | 494 |
|
| Cyclomatic Complexity is 15 (max allowed is 9). | 523 |
|
| 'cast' is not followed by whitespace. | 536 |
|
| 'cast' is not followed by whitespace. | 539 |
|
| 'cast' is not followed by whitespace. | 552 |
|
| Line is longer than 100 characters. | 558 |
|
| Unused @param tag for 'locale'. | 603 |
|
| 'cast' is not followed by whitespace. | 619 |
|
| 'if' construct must use '{}'s. | 620 |
|
| Unused @param tag for 'params'. | 654 |
|
| Expected @param tag for 'execParams'. | 659 |
|
| Expected @param tag for 'e'. | 661 |
|
| Expected @throws tag for 'ServletException'. | 662 |
|
| 'if' construct must use '{}'s. | 664 |
|
| Redundant 'final' modifier. | 696 |
|
| Line is longer than 100 characters. | 701 |
|
| 'cast' is not followed by whitespace. | 752 |
|
| Line is longer than 100 characters. | 757 |
|
| '{' is followed by whitespace. | 757 |
|
| Line is longer than 100 characters. | 761 |
|
| '{' is followed by whitespace. | 761 |
|
| Cyclomatic Complexity is 11 (max allowed is 9). | 793 |
|
| Expected @param tag for 'validatorDelegate'. | 910 |
|
| Expected @param tag for 'validatorDelegate'. | 925 |
|
| First sentence should end with a period. | 1092 |
|
| First sentence should end with a period. | 1101 |
|
| 'cast' is not followed by whitespace. | 1225 |
|
| 'cast' is not followed by whitespace. | 1280 |
|
| 'cast' is not followed by whitespace. | 1283 |
|
| 'cast' is not followed by whitespace. | 1289 |
| Error | Line | |
|---|---|---|
|
| Line does not match expected header line of ' * Copyright (c) 2003, Open Edge B.V.'. | 7 |
|
| Missing a Javadoc comment. | 41 |
|
| Missing a Javadoc comment. | 44 |
|
| Missing a Javadoc comment. | 46 |
|
| 'static' modifier out of order with the JLS suggestions. | 46 |
|
| Missing a Javadoc comment. | 47 |
|
| 'static' modifier out of order with the JLS suggestions. | 47 |
|
| Missing a Javadoc comment. | 48 |
|
| 'static' modifier out of order with the JLS suggestions. | 48 |
|
| Missing a Javadoc comment. | 49 |
|
| 'static' modifier out of order with the JLS suggestions. | 49 |
|
| Missing a Javadoc comment. | 50 |
|
| 'static' modifier out of order with the JLS suggestions. | 50 |
|
| Missing a Javadoc comment. | 51 |
|
| 'static' modifier out of order with the JLS suggestions. | 51 |
|
| 'interceptorRegistry' hides a field. | 58 |
|
| Expected @throws tag for 'ServletException'. | 83 |
|
| Expected @throws tag for 'FlowException'. | 83 |
|
| Missing a Javadoc comment. | 88 |
|
| Expected @throws tag for 'ServletException'. | 111 |
|
| Expected @throws tag for 'FlowException'. | 111 |
|
| Missing a Javadoc comment. | 116 |
|
| Expected @throws tag for 'ServletException'. | 141 |
|
| Expected @throws tag for 'FlowException'. | 141 |
|
| Missing a Javadoc comment. | 146 |
|
| Expected @throws tag for 'ServletException'. | 170 |
|
| Expected @throws tag for 'FlowException'. | 170 |
|
| Missing a Javadoc comment. | 175 |
|
| Expected @throws tag for 'ServletException'. | 200 |
|
| Expected @throws tag for 'FlowException'. | 200 |
|
| Missing a Javadoc comment. | 205 |
|
| Expected @throws tag for 'ServletException'. | 229 |
|
| Expected @throws tag for 'FlowException'. | 229 |
|
| Missing a Javadoc comment. | 234 |
|
| Cyclomatic Complexity is 16 (max allowed is 9). | 247 |
|
| Missing a Javadoc comment. | 247 |
|
| 'if' construct must use '{}'s. | 257 |
|
| 'if' construct must use '{}'s. | 260 |
|
| 'cast' is not followed by whitespace. | 273 |
|
| 'cast' is not followed by whitespace. | 277 |
|
| 'cast' is not followed by whitespace. | 281 |
|
| 'cast' is not followed by whitespace. | 285 |
|
| 'cast' is not followed by whitespace. | 289 |
|
| 'cast' is not followed by whitespace. | 293 |
|
| Missing a Javadoc comment. | 320 |
|
| 'cast' is not followed by whitespace. | 334 |
|
| 'cast' is not followed by whitespace. | 343 |
|
| Line is longer than 100 characters. | 372 |
|
| Cyclomatic Complexity is 11 (max allowed is 9). | 376 |
|
| Missing a Javadoc comment. | 376 |
| Error | Line | |
|---|---|---|
|
| Missing a Javadoc comment. | 48 |
|
| Missing a Javadoc comment. | 50 |
|
| First sentence should end with a period. | 53 |
|
| First sentence should end with a period. | 66 |
|
| First sentence should end with a period. | 81 |
|
| 'if' construct must use '{}'s. | 90 |
|
| First sentence should end with a period. | 94 |
|
| ';' is followed by whitespace. | 105 |
|
| 'cast' is not followed by whitespace. | 107 |
|
| 'cast' is not followed by whitespace. | 115 |
|
| First sentence should end with a period. | 121 |
|
| Avoid inline conditionals. | 128 |
| Error | Line | |
|---|---|---|
|
| interfaces should describe a type and hence have methods. | 38 |
|
| Redundant 'public' modifier. | 43 |
|
| 'static' modifier out of order with the JLS suggestions. | 43 |
|
| Redundant 'public' modifier. | 48 |
|
| 'static' modifier out of order with the JLS suggestions. | 48 |
|
| Redundant 'public' modifier. | 53 |
|
| 'static' modifier out of order with the JLS suggestions. | 53 |
|
| Redundant 'public' modifier. | 58 |
|
| ||