The following document contains the results of Checkstyle. 
| Files | Infos | Warnings | Errors |
|---|---|---|---|
| 36 | 0 | 316 | 0 |
| Rules | Violations | Severity |
|---|---|---|
| ConstantName | 25 | |
| LocalFinalVariableName | 0 | |
| LocalVariableName | 1 | |
| MemberName | 0 | |
| MethodName | 3 | |
| PackageName | 0 | |
| ParameterName | 0 | |
| StaticVariableName | 1 | |
| TypeName | 0 | |
| AvoidStarImport | 2 | |
| IllegalImport | 0 | |
| RedundantImport | 0 | |
| UnusedImports | 16 | |
| FileLength | 0 | |
LineLength
| 24 | |
| MethodLength | 1 | |
| ParameterNumber | 0 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 0 | |
| OperatorWrap | 1 | |
| ParenPad | 37 | |
| TypecastParenPad | 0 | |
| TabCharacter | 2 | |
| ModifierOrder | 18 | |
| RedundantModifier | 2 | |
| EmptyBlock | 1 | |
| LeftCurly | 0 | |
| NeedBraces | 35 | |
| RightCurly | 9 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 0 | |
| EqualsHashCode | 0 | |
HiddenField
| 1 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 0 | |
| MissingSwitchDefault | 1 | |
| RedundantThrows | 11 | |
| SimplifyBooleanExpression | 1 | |
| SimplifyBooleanReturn | 1 | |
| FinalClass | 0 | |
| HideUtilityClassConstructor | 1 | |
| InterfaceIsType | 0 | |
| VisibilityModifier | 27 | |
| ArrayTypeStyle | 0 | |
| FinalParameters | 41 | |
GenericIllegalRegexp
| 0 | |
| UpperEll | 0 | |
| FinalLocalVariable | 51 | |
| PackageHtml | 3 | |
| NewlineAtEndOfFile | 0 | |
| Translation | 0 |
| Violation | Message | Line |
|---|---|---|
| Redundant 'public' modifier. | 31 | |
| Redundant 'public' modifier. | 32 |
| Violation | Message | Line |
|---|---|---|
| Variable 'upperLeft' must be private and have accessor methods. | 27 | |
| Variable 'lowerRight' must be private and have accessor methods. | 28 | |
| Variable 'maxX' must be private and have accessor methods. | 29 | |
| Variable 'minX' must be private and have accessor methods. | 30 | |
| Variable 'maxY' must be private and have accessor methods. | 31 | |
| Variable 'minY' must be private and have accessor methods. | 32 | |
| Variable 'centerX' must be private and have accessor methods. | 33 | |
| Variable 'centerY' must be private and have accessor methods. | 34 | |
| Parameter maxX should be final. | 47 | |
| Parameter minX should be final. | 47 | |
| Parameter maxY should be final. | 47 | |
| Parameter minY should be final. | 47 | |
| Parameter maxX should be final. | 58 | |
| Parameter minX should be final. | 70 | |
| Parameter maxY should be final. | 82 | |
| Parameter centerX should be final. | 94 | |
| Parameter centerY should be final. | 106 | |
| Parameter minY should be final. | 119 | |
| Parameter ul should be final. | 133 | |
| Parameter lr should be final. | 133 | |
| Parameter ul should be final. | 140 | |
| Parameter lr should be final. | 146 |
| Violation | Message | Line |
|---|---|---|
| Variable 'context' must be private and have accessor methods. | 35 |
| Violation | Message | Line |
|---|---|---|
| Name 'NAMESPACES' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 37 |
| Violation | Message | Line |
|---|---|---|
| Unused import - no.sesat.search.query.Clause. | 48 | |
| Unused import - no.sesat.search.query.token.EvaluationException. | 52 | |
| Unused import - no.sesat.search.mode.config.BaseSearchConfiguration. | 57 | |
| Unused import - no.sesat.search.query.token.TokenEvaluator. | 66 | |
| Unused import - no.sesat.search.query.token.TokenPredicate. | 67 | |
| Line is longer than 120 characters. | 106 | |
| Variable 'allCancelled' must be private and have accessor methods. | 128 | |
| Variable 'datamodel' must be private and have accessor methods. | 130 | |
| Variable 'engine' must be private and have accessor methods. | 132 | |
| Method length is 163 lines (max allowed is 150). | 237 | |
| '(' is followed by whitespace. | 336 | |
| ')' is preceded with whitespace. | 336 | |
| '(' is followed by whitespace. | 363 | |
| ')' is preceded with whitespace. | 363 | |
| '||' should be on a new line. | 440 | |
| Line is longer than 120 characters. | 601 | |
| Line is longer than 120 characters. | 611 | |
| Line is longer than 120 characters. | 615 | |
| Line is longer than 120 characters. | 619 | |
| '(' is followed by whitespace. | 675 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| Unused import - java.util.Hashtable. | 23 | |
| Unused import - java.util.Set. | 25 | |
| Unused import - no.sesat.search.datamodel.DataModel. | 28 | |
| Unused import - org.apache.commons.lang.StringEscapeUtils. | 30 | |
| Unused import - org.apache.commons.lang.StringUtils. | 31 | |
| Variable 'e' should be declared final. | 117 | |
| '(' is followed by whitespace. | 164 | |
| ')' is preceded with whitespace. | 164 | |
| '(' is followed by whitespace. | 176 | |
| ')' is preceded with whitespace. | 176 | |
| '(' is followed by whitespace. | 178 | |
| ')' is preceded with whitespace. | 178 |
| Violation | Message | Line |
|---|---|---|
| Parameter algorithm should be final. | 62 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| 'static' modifier out of order with the JLS suggestions. | 35 | |
| Name 'secretCode' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 35 | |
| 'static' modifier out of order with the JLS suggestions. | 36 | |
| Name 'organization' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 36 | |
| 'static' modifier out of order with the JLS suggestions. | 37 | |
| Name 'event' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 37 | |
| Parameter request should be final. | 40 | |
| Redundant throws: 'RuntimeException' is unchecked exception. | 46 | |
| Parameter name should be final. | 91 |
| Violation | Message | Line |
|---|---|---|
| Unused import - java.util.Date. | 20 | |
| Name 'yip_decoder' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 39 | |
| Parameter s should be final. | 39 | |
| Name 'yip_decoder' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 49 | |
| Parameter s should be final. | 49 | |
| Parameter length should be final. | 49 | |
| Line contains a tab character. | 61 | |
| Parameter s should be final. | 70 | |
| Name 'yip_checknames' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 78 | |
| Parameter s1 should be final. | 78 | |
| Parameter s2 should be final. | 78 | |
| Conditional logic can be removed. | 80 | |
| 'else' construct must use '{}'s. | 82 | |
| Line contains a tab character. | 83 | |
| Parameter s should be final. | 86 | |
| Parameter i should be final. | 86 | |
| Variable 'length' should be declared final. | 92 | |
| 'if' construct must use '{}'s. | 93 | |
| Variable 'cal' should be declared final. | 101 | |
| Variable 'day' should be declared final. | 102 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.util.*. | 27 | |
| 'static' modifier out of order with the JLS suggestions. | 39 | |
| Name 'zoom_1' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 39 | |
| 'static' modifier out of order with the JLS suggestions. | 40 | |
| Name 'zoom_2' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 40 | |
| 'static' modifier out of order with the JLS suggestions. | 41 | |
| Name 'zoom_3' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 41 | |
| 'static' modifier out of order with the JLS suggestions. | 42 | |
| Name 'zoom_4' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 42 | |
| 'static' modifier out of order with the JLS suggestions. | 43 | |
| Name 'zoom_5' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 43 | |
| 'static' modifier out of order with the JLS suggestions. | 44 | |
| Name 'zoom_6' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 44 | |
| 'static' modifier out of order with the JLS suggestions. | 45 | |
| Name 'zoom_7' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 45 | |
| 'static' modifier out of order with the JLS suggestions. | 47 | |
| Name 'zoomCount' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 47 | |
| Variable 'defaultZoom' must be private and have accessor methods. | 49 | |
| Variable 'imgWidth' must be private and have accessor methods. | 50 | |
| Variable 'imgHeight' must be private and have accessor methods. | 51 | |
| Variable 'envFactor' must be private and have accessor methods. | 52 | |
| 'static' modifier out of order with the JLS suggestions. | 54 | |
| Name 'iconPxSizeHeigth' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 54 | |
| 'static' modifier out of order with the JLS suggestions. | 55 | |
| Name 'iconPxSizeWidth' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 55 | |
| Variable 'iconOffsetHeigth' must be private and have accessor methods. | 57 | |
| Variable 'iconOffsetWidth' must be private and have accessor methods. | 58 | |
| Line is longer than 120 characters. | 59 | |
| 'static' modifier out of order with the JLS suggestions. | 59 | |
| Name 'iconOverlapOffsetWidth' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 59 | |
| Line is longer than 120 characters. | 60 | |
| 'static' modifier out of order with the JLS suggestions. | 60 | |
| Name 'iconOverlapOffsetHeigth' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 60 | |
| Variable 'mapCenterPxX' must be private and have accessor methods. | 61 | |
| Variable 'mapCenterPxY' must be private and have accessor methods. | 62 | |
| Line is longer than 120 characters. | 63 | |
| 'static' modifier out of order with the JLS suggestions. | 63 | |
| Name 'panFactor' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 63 | |
| Line is longer than 120 characters. | 64 | |
| 'static' modifier out of order with the JLS suggestions. | 64 | |
| Name 'zoomFactor' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 64 | |
| 'static' modifier out of order with the JLS suggestions. | 65 | |
| Name 'defaultNoCoord' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 65 | |
| Variable 'maxX' must be private and have accessor methods. | 67 | |
| Variable 'minX' must be private and have accessor methods. | 69 | |
| Variable 'maxY' must be private and have accessor methods. | 70 | |
| Variable 'minY' must be private and have accessor methods. | 71 | |
| Variable 'mapCenterCoordX' must be private and have accessor methods. | 72 | |
| Variable 'mapCenterCoordY' must be private and have accessor methods. | 73 | |
| Name 'pixelSize' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 78 | |
| Variable 'imgMeterSize' must be private and have accessor methods. | 79 | |
| Parameter defaultZoom should be final. | 85 | |
| 'defaultZoom' hides a field. | 85 | |
| Parameter sCoord should be final. | 97 | |
| Variable 'length' should be declared final. | 104 | |
| 'if' construct must use '{}'s. | 115 | |
| Variable 'mp' should be declared final. | 122 | |
| Variable 'zoomscale' should be declared final. | 140 | |
| Variable 'metersPrPixel' should be declared final. | 142 | |
| Variable 'mapEnvelope' should be declared final. | 147 | |
| Variable 'mp' should be declared final. | 158 | |
| Variable 'zoomnivaa' should be declared final. | 162 | |
| '}' should be on the same line. | 164 | |
| Variable 'mps' should be declared final. | 186 | |
| Variable 'mp' should be declared final. | 205 | |
| Variable 'company' should be declared final. | 206 | |
| Variable 'mps' should be declared final. | 224 | |
| Variable 'mps' should be declared final. | 235 | |
| Variable 'mp' should be declared final. | 238 | |
| Variable 'company' should be declared final. | 240 | |
| Variable 'me' should be declared final. | 263 | |
| Parameter vMapPoints should be final. | 273 | |
| Parameter vMapPoints should be final. | 282 | |
| Parameter zoomlevel should be final. | 282 | |
| ')' is preceded with whitespace. | 293 | |
| Line is longer than 120 characters. | 295 | |
| 'if' construct must use '{}'s. | 306 | |
| 'if' construct must use '{}'s. | 308 | |
| 'if' construct must use '{}'s. | 310 | |
| 'if' construct must use '{}'s. | 312 | |
| ')' is preceded with whitespace. | 316 | |
| 'if' construct must use '{}'s. | 320 | |
| '}' should be on the same line. | 332 | |
| '}' should be on the same line. | 336 | |
| '}' should be on the same line. | 352 | |
| Line is longer than 120 characters. | 357 | |
| Line is longer than 120 characters. | 366 | |
| '}' should be on the same line. | 372 | |
| 'if' construct must use '{}'s. | 374 | |
| Line is longer than 120 characters. | 374 | |
| 'if' construct must use '{}'s. | 376 | |
| Variable 'factorHW' should be declared final. | 379 | |
| Name 'factor_dXdY' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 380 | |
| Variable 'factor_dXdY' should be declared final. | 380 | |
| Line is longer than 120 characters. | 381 | |
| ')' is preceded with whitespace. | 381 | |
| '}' should be on the same line. | 383 | |
| Line is longer than 120 characters. | 384 | |
| Line is longer than 120 characters. | 387 | |
| Variable 'extFact' should be declared final. | 388 | |
| Variable 'extendTop' should be declared final. | 389 | |
| Line is longer than 120 characters. | 390 | |
| Variable 'extendBottom' should be declared final. | 390 | |
| Variable 'extendLeft' should be declared final. | 391 | |
| Line is longer than 120 characters. | 392 | |
| Variable 'extendRigth' should be declared final. | 392 | |
| Variable 'me' should be declared final. | 410 | |
| ')' is preceded with whitespace. | 413 | |
| Line is longer than 120 characters. | 415 | |
| 'if' construct must use '{}'s. | 425 | |
| 'if' construct must use '{}'s. | 427 | |
| 'if' construct must use '{}'s. | 429 | |
| 'if' construct must use '{}'s. | 431 | |
| Line is longer than 120 characters. | 439 | |
| 'if' construct must use '{}'s. | 449 | |
| 'if' construct must use '{}'s. | 452 | |
| Variable 'xOffset' should be declared final. | 455 | |
| Variable 'yOffset' should be declared final. | 456 | |
| Parameter vMapPoints should be final. | 495 | |
| Variable 'zoomscale' should be declared final. | 500 | |
| Variable 'arrList' should be declared final. | 503 | |
| Variable 'upperLeftX' should be declared final. | 506 | |
| Variable 'upperLeftY' should be declared final. | 507 | |
| Line is longer than 120 characters. | 508 | |
| Line is longer than 120 characters. | 511 | |
| Variable 'vMapPointsNew' should be declared final. | 512 | |
| 'if' construct must use '{}'s. | 519 | |
| 'if' construct must use '{}'s. | 521 | |
| 'if' construct must use '{}'s. | 523 | |
| 'if' construct must use '{}'s. | 525 | |
| 'if' construct must use '{}'s. | 527 | |
| Variable 'deltaX' should be declared final. | 533 | |
| Variable 'deltaY' should be declared final. | 534 | |
| Variable 'yFactor' should be declared final. | 535 | |
| Variable 'xFactor' should be declared final. | 536 | |
| '}' should be on the same line. | 545 | |
| Variable 'deltaX' should be declared final. | 566 | |
| Variable 'deltaX' should be declared final. | 576 | |
| Variable 'length' should be declared final. | 579 | |
| Variable 'zoomscale' should be declared final. | 589 | |
| 'if' construct must use '{}'s. | 591 | |
| 'if' construct must use '{}'s. | 593 | |
| 'if' construct must use '{}'s. | 595 | |
| 'if' construct must use '{}'s. | 597 | |
| 'if' construct must use '{}'s. | 599 | |
| 'if' construct must use '{}'s. | 601 | |
| 'if' construct must use '{}'s. | 603 | |
| 'else' construct must use '{}'s. | 605 | |
| 'if' construct must use '{}'s. | 617 | |
| 'if' construct must use '{}'s. | 619 | |
| 'if' construct must use '{}'s. | 621 | |
| 'if' construct must use '{}'s. | 623 | |
| 'if' construct must use '{}'s. | 625 | |
| 'if' construct must use '{}'s. | 627 | |
| Variable 'zoomLevels' should be declared final. | 637 | |
| Parameter imgWidth should be final. | 663 | |
| Parameter imgHeigth should be final. | 671 | |
| Parameter env should be final. | 680 | |
| Variable 'tmp' should be declared final. | 714 |
| Violation | Message | Line |
|---|---|---|
| Expression can be simplified. | 76 |
| Violation | Message | Line |
|---|---|---|
| Parameter e should be final. | 33 | |
| Parameter s should be final. | 43 | |
| Parameter e should be final. | 43 |
| Violation | Message | Line |
|---|---|---|
| Name 'PROPERTY_KEY___QR_SERVER' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 33 | |
| Name 'PROPERTY_KEY___TV_PATTERN' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 34 | |
| Name '_COLLECTION_KEY' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 35 | |
| Name '_SPELLCHECK' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 36 | |
| Name '_LANGUAGE' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 37 | |
| Name '_WEBCRAWL_TEMPLATE' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 38 |
| Violation | Message | Line |
|---|---|---|
| Unused import - java.util.Iterator. | 20 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| Unused import - java.io.IOException. | 25 | |
| Unused import - no.sesat.search.view.FindResource. | 51 | |
| Redundant throws: 'ResourceNotFoundException' is unchecked exception. | 126 | |
| Redundant throws: 'ResourceNotFoundException' is subclass of 'Exception'. | 274 | |
| Redundant throws: 'ResourceNotFoundException' is unchecked exception. | 274 | |
| Redundant throws: 'ParseErrorException' is subclass of 'Exception'. | 274 | |
| Redundant throws: 'ParseErrorException' is unchecked exception. | 274 |
| Violation | Message | Line |
|---|---|---|
| Unused import - no.sesat.search.site.config.SiteConfiguration. | 21 | |
| Utility classes should not have a public or default constructor. | 33 |
| Violation | Message | Line |
|---|---|---|
| Variable 'm' should be declared final. | 57 | |
| Parameter newsSource should be final. | 70 |
| Violation | Message | Line |
|---|---|---|
| switch without "default" clause. | 80 | |
| '(' is followed by whitespace. | 98 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.io.*. | 19 | |
| Redundant throws: 'ResourceNotFoundException' is unchecked exception. | 71 | |
| Variable 'content' should be declared final. | 97 | |
| Must have at least one statement. | 115 | |
| Variable 'builder' should be declared final. | 153 | |
| Variable 'isr' should be declared final. | 156 | |
| Variable 'in' should be declared final. | 157 | |
| Variable 'fileStream' should be declared final. | 201 | |
| Parameter ctx should be final. | 252 | |
| Variable 'style' should be declared final. | 255 | |
| '}' should be on the same line. | 259 | |
| '}' should be on the same line. | 262 | |
| Line is longer than 120 characters. | 268 | |
| Line is longer than 120 characters. | 278 |
| Violation | Message | Line |
|---|---|---|
| Unused import - javax.resource.NotSupportedException. | 66 | |
| Redundant throws: 'ResourceNotFoundException' is unchecked exception. | 313 |
| Violation | Message | Line |
|---|---|---|
| Redundant throws: 'ResourceNotFoundException' is unchecked exception. | 110 | |
| '(' is followed by whitespace. | 114 | |
| ')' is preceded with whitespace. | 114 | |
| '(' is followed by whitespace. | 118 | |
| ')' is preceded with whitespace. | 118 | |
| '(' is followed by whitespace. | 119 | |
| ')' is preceded with whitespace. | 119 | |
| Parameter resource should be final. | 125 | |
| Parameter resource should be final. | 134 | |
| '(' is followed by whitespace. | 140 | |
| ')' is preceded with whitespace. | 140 | |
| '(' is followed by whitespace. | 141 | |
| ')' is preceded with whitespace. | 141 | |
| '(' is followed by whitespace. | 142 | |
| ')' is preceded with whitespace. | 142 | |
| '(' is followed by whitespace. | 143 | |
| ')' is preceded with whitespace. | 143 | |
| Redundant throws: 'ResourceNotFoundException' is unchecked exception. | 155 | |
| ')' is preceded with whitespace. | 158 | |
| '(' is followed by whitespace. | 161 | |
| ')' is preceded with whitespace. | 161 | |
| '(' is followed by whitespace. | 162 | |
| ')' is preceded with whitespace. | 162 | |
| Redundant throws: 'ResourceNotFoundException' is unchecked exception. | 167 | |
| '(' is followed by whitespace. | 177 | |
| ')' is preceded with whitespace. | 177 |