The following document contains the results of Checkstyle. 
| Files | Infos |
Warnings |
Errors |
|---|---|---|---|
| 40 | 16 | 154 | 178 |
| Rules | Violations | Severity |
|---|---|---|
| JavadocPackage | 1 | |
| Translation | 0 | |
| FileLength | 0 | |
FileTabCharacter
|
0 | |
RegexpSingleline
|
133 | |
| AnnotationUseStyle | 0 | |
| MissingDeprecated | 0 | |
MissingOverride
|
0 | |
| PackageAnnotation | 0 | |
JavadocMethod
|
4 | |
JavadocType
|
0 | |
JavadocVariable
|
17 | |
JavadocStyle
|
50 | |
| ConstantName | 4 | |
| LocalFinalVariableName | 0 | |
| LocalVariableName | 0 | |
| MemberName | 0 | |
| MethodName | 0 | |
| PackageName | 0 | |
| ParameterName | 0 | |
| StaticVariableName | 0 | |
| TypeName | 0 | |
| AvoidStarImport | 0 | |
| IllegalImport | 0 | |
ImportOrder
|
20 | |
| RedundantImport | 0 | |
| UnusedImports | 2 | |
LineLength
|
21 | |
| MethodLength | 0 | |
| ParameterNumber | 0 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 0 | |
| NoWhitespaceAfter | 0 | |
| NoWhitespaceBefore | 0 | |
| OperatorWrap | 6 | |
| ParenPad | 0 | |
| TypecastParenPad | 0 | |
| WhitespaceAfter | 3 | |
| WhitespaceAround | 2 | |
| ModifierOrder | 0 | |
| RedundantModifier | 2 | |
| AvoidNestedBlocks | 0 | |
| EmptyBlock | 0 | |
LeftCurly
|
4 | |
| NeedBraces | 4 | |
RightCurly
|
0 | |
| DeclarationOrder | 2 | |
| DefaultComesLast | 0 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 0 | |
| EqualsHashCode | 0 | |
| ExplicitInitialization | 4 | |
| FallThrough | 0 | |
HiddenField
|
32 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 0 | |
| MagicNumber | 1 | |
| MissingSwitchDefault | 0 | |
| ParameterAssignment | 0 | |
RedundantThrows
|
0 | |
| SimplifyBooleanExpression | 0 | |
| SimplifyBooleanReturn | 0 | |
| StringLiteralEquality | 0 | |
| DesignForExtension | 16 | |
| FinalClass | 6 | |
| HideUtilityClassConstructor | 0 | |
| InterfaceIsType | 0 | |
| VisibilityModifier | 0 | |
| BooleanExpressionComplexity | 0 | |
| ClassDataAbstractionCoupling | 0 | |
ClassFanOutComplexity
|
3 | |
| CyclomaticComplexity | 0 | |
NPathComplexity
|
2 | |
| JavaNCSS | 0 | |
| ArrayTypeStyle | 0 | |
| Indentation | 9 | |
| TodoComment | 0 | |
| UpperEll | 0 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 | |
| Line has trailing spaces. | 28 | |
| First sentence should end with a period. | 35 | |
| Line has trailing spaces. | 37 | |
| 'if' construct must use '{}'s. | 54 | |
| 'if' construct must use '{}'s. | 55 | |
| Line has trailing spaces. | 70 |
| Violation | Message | Line |
|---|---|---|
| Method 'getGadgetId' is not designed for extension - needs to be abstract, final or empty. | 16 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 12 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 120 characters. | 9 | |
| Line is longer than 120 characters. | 10 | |
| Line is longer than 120 characters. | 18 | |
| Missing a Javadoc comment. | 18 | |
| Line has trailing spaces. | 23 | |
| Line has trailing spaces. | 59 | |
| Line has trailing spaces. | 64 | |
| Line is longer than 120 characters. | 73 | |
| Variable 'ignoreCache' explicitly initialized to 'false' (default value for its type). | 79 | |
| Variable 'viewer' explicitly initialized to 'null' (default value for its type). | 80 | |
| Variable 'debug' explicitly initialized to 'false' (default value for its type). | 81 | |
| Method 'build' is not designed for extension - needs to be abstract, final or empty. | 98 | |
| Method 'locale' is not designed for extension - needs to be abstract, final or empty. | 110 | |
| 'locale' hides a field. | 110 | |
| Method 'ignoreCache' is not designed for extension - needs to be abstract, final or empty. | 123 | |
| 'ignoreCache' hides a field. | 123 | |
| Method 'viewer' is not designed for extension - needs to be abstract, final or empty. | 136 | |
| 'viewer' hides a field. | 136 | |
| Line has trailing spaces. | 141 | |
| Line has trailing spaces. | 145 | |
| Method 'debug' is not designed for extension - needs to be abstract, final or empty. | 149 | |
| 'debug' hides a field. | 149 |
| Violation | Message | Line |
|---|---|---|
| Extra HTML tag found: </interface> | 21 | |
| Extra HTML tag found: </component> | 22 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'java.util.Collections.emptyMap' import. | 4 | |
| Wrong order for 'java.io.IOException' import. | 6 | |
| Wrong order for 'org.apache.commons.lang.builder.EqualsBuilder' import. | 18 | |
| Wrong order for 'com.atlassian.gadgets.dashboard.Color' import. | 22 | |
| Line has trailing spaces. | 32 | |
| Line has trailing spaces. | 35 | |
| Line has trailing spaces. | 38 | |
| Line has trailing spaces. | 42 | |
| Line has trailing spaces. | 44 | |
| Line has trailing spaces. | 48 | |
| Line is longer than 120 characters. | 49 | |
| Class Fan-Out Complexity is 15 (max allowed is 10). | 54 | |
| Line has trailing spaces. | 71 | |
| Line has trailing spaces. | 75 | |
| Line has trailing spaces. | 85 | |
| Line has trailing spaces. | 99 | |
| Line has trailing spaces. | 102 | |
| Line has trailing spaces. | 112 | |
| Line has trailing spaces. | 113 | |
| Line has trailing spaces. | 123 | |
| Line has trailing spaces. | 133 | |
| Line has trailing spaces. | 140 | |
| Line has trailing spaces. | 160 | |
| Line has trailing spaces. | 171 | |
| Line has trailing spaces. | 186 | |
| Line has trailing spaces. | 198 | |
| First sentence should end with a period. | 207 | |
| Class SpecUriBuilder should be declared as final. | 210 | |
| Line has trailing spaces. | 213 | |
| Line has trailing spaces. | 222 | |
| Line has trailing spaces. | 231 | |
| Line has trailing spaces. | 235 | |
| Line has trailing spaces. | 244 | |
| Line has trailing spaces. | 262 | |
| Line has trailing spaces. | 275 | |
| Method 'color' is not designed for extension - needs to be abstract, final or empty. | 279 | |
| 'color' hides a field. | 279 | |
| Line has trailing spaces. | 284 | |
| Line has trailing spaces. | 288 | |
| Method 'userPrefs' is not designed for extension - needs to be abstract, final or empty. | 292 | |
| 'userPrefs' hides a field. | 292 | |
| First sentence should end with a period. | 298 | |
| Line has trailing spaces. | 300 | |
| Method 'build' is not designed for extension - needs to be abstract, final or empty. | 303 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 120 characters. | 20 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on a new line. | 6 | |
| Missing a Javadoc comment. | 7 | |
| Missing a Javadoc comment. | 7 | |
| Missing a Javadoc comment. | 7 | |
| Missing a Javadoc comment. | 7 | |
| Missing a Javadoc comment. | 7 | |
| Missing a Javadoc comment. | 7 | |
| Missing a Javadoc comment. | 7 | |
| Missing a Javadoc comment. | 7 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 | |
| Line has trailing spaces. | 28 | |
| Line has trailing spaces. | 37 | |
| Line has trailing spaces. | 44 | |
| 'if' construct must use '{}'s. | 54 | |
| 'if' construct must use '{}'s. | 55 | |
| Line has trailing spaces. | 70 |
| Violation | Message | Line |
|---|---|---|
| Method 'getDashboardId' is not designed for extension - needs to be abstract, final or empty. | 15 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 13 | |
| Line has trailing spaces. | 20 | |
| Line has trailing spaces. | 23 | |
| Line has trailing spaces. | 26 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'java.util.Iterator' import. | 11 | |
| 'net.jcip.annotations.Immutable' should be separated from previous imports. | 25 | |
| Line has trailing spaces. | 28 | |
| Line has trailing spaces. | 37 | |
| Line has trailing spaces. | 40 | |
| Line has trailing spaces. | 43 | |
| Line has trailing spaces. | 47 | |
| Line has trailing spaces. | 49 | |
| Line has trailing spaces. | 54 | |
| Line is longer than 120 characters. | 55 | |
| Class Fan-Out Complexity is 19 (max allowed is 10). | 60 | |
| Line has trailing spaces. | 70 | |
| Line has trailing spaces. | 77 | |
| Line has trailing spaces. | 80 | |
| 'columns' hides a field. | 81 | |
| Line has trailing spaces. | 94 | |
| Line has trailing spaces. | 99 | |
| Line has trailing spaces. | 103 | |
| NPath Complexity is 32 (max allowed is 20). | 108 | |
| Line has trailing spaces. | 111 | |
| Line has trailing spaces. | 113 | |
| Line has trailing spaces. | 134 | |
| Line has trailing spaces. | 144 | |
| Line has trailing spaces. | 151 | |
| Line has trailing spaces. | 154 | |
| Line has trailing spaces. | 161 | |
| Line has trailing spaces. | 165 | |
| First sentence should end with a period. | 183 | |
| 'if' is not followed by whitespace. | 217 | |
| 'if' is not followed by whitespace. | 236 | |
| First sentence should end with a period. | 254 | |
| First sentence should end with a period. | 266 | |
| Line has trailing spaces. | 277 | |
| NPath Complexity is 24 (max allowed is 20). | 278 | |
| Line has trailing spaces. | 309 | |
| Line has trailing spaces. | 323 | |
| Line has trailing spaces. | 334 | |
| Line has trailing spaces. | 339 | |
| Line is longer than 120 characters. | 350 | |
| Line has trailing spaces. | 351 | |
| Line has trailing spaces. | 352 | |
| Class TitleBuilder should be declared as final. | 364 | |
| Line has trailing spaces. | 370 | |
| Line has trailing spaces. | 372 | |
| Line has trailing spaces. | 376 | |
| Line has trailing spaces. | 385 | |
| Class Builder should be declared as final. | 391 | |
| Variable 'version' explicitly initialized to '0' (default value for its type). | 397 | |
| Line has trailing spaces. | 398 | |
| Line has trailing spaces. | 404 | |
| 'columns' hides a field. | 410 | |
| Line has trailing spaces. | 418 | |
| Line has trailing spaces. | 422 | |
| 'layout' hides a field. | 426 | |
| 'title' hides a field. | 439 | |
| Line has trailing spaces. | 444 | |
| Line has trailing spaces. | 448 | |
| 'columns' hides a field. | 452 | |
| 'version' hides a field. | 465 | |
| First sentence should end with a period. | 471 | |
| Line has trailing spaces. | 473 | |
| Line has trailing spaces. | 481 | |
| Line is longer than 120 characters. | 483 | |
| Line has trailing spaces. | 484 | |
| Line has trailing spaces. | 490 | |
| First sentence should end with a period. | 491 | |
| Line has trailing spaces. | 493 | |
| First sentence should end with a period. | 494 | |
| Line has trailing spaces. | 496 | |
| Line has trailing spaces. | 498 | |
| Line has trailing spaces. | 503 | |
| Line has trailing spaces. | 506 | |
| Line has trailing spaces. | 513 | |
| Line has trailing spaces. | 516 | |
| Line has trailing spaces. | 523 | |
| Line has trailing spaces. | 527 | |
| Line has trailing spaces. | 539 | |
| Line has trailing spaces. | 542 | |
| Line has trailing spaces. | 561 | |
| Line has trailing spaces. | 563 | |
| Line has trailing spaces. | 565 | |
| Line has trailing spaces. | 568 | |
| Missing a Javadoc comment. | 575 | |
| Missing a Javadoc comment. | 581 | |
| Missing a Javadoc comment. | 586 | |
| Missing a Javadoc comment. | 604 |
| Violation | Message | Line |
|---|---|---|
| First sentence should end with a period. | 9 |
| Violation | Message | Line |
|---|---|---|
| First sentence should end with a period. | 13 | |
| First sentence should end with a period. | 15 | |
| First sentence should end with a period. | 17 | |
| First sentence should end with a period. | 19 | |
| First sentence should end with a period. | 21 | |
| First sentence should end with a period. | 23 | |
| Line has trailing spaces. | 35 | |
| Line has trailing spaces. | 45 | |
| Line has trailing spaces. | 56 | |
| Line has trailing spaces. | 63 | |
| Line has trailing spaces. | 67 | |
| Line has trailing spaces. | 86 | |
| Line has trailing spaces. | 88 | |
| Line has trailing spaces. | 89 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 5 | |
| Line has trailing spaces. | 8 | |
| Line has trailing spaces. | 11 | |
| 'com.atlassian.gadgets.GadgetId' should be separated from previous imports. | 28 | |
| Unused import - com.atlassian.gadgets.GadgetId. | 28 | |
| Unused import - com.atlassian.gadgets.GadgetState. | 29 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'java.util.Iterator' import. | 5 | |
| First sentence should end with a period. | 9 | |
| Line is longer than 120 characters. | 10 | |
| Class Iterables should be declared as final. | 13 | |
| Line is longer than 120 characters. | 21 | |
| Line is longer than 120 characters. | 24 | |
| First sentence should end with a period. | 48 |
| Violation | Message | Line |
|---|---|---|
| Missing package-info.java file. | 0 | |
| Line is longer than 120 characters. | 33 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'java.io.InputStream' import. | 5 | |
| Line is longer than 120 characters. | 18 | |
| Line has trailing spaces. | 53 | |
| Line has trailing spaces. | 125 | |
| '+' should be on a new line. | 157 | |
| '+' should be on a new line. | 158 | |
| '+' should be on a new line. | 159 | |
| '31' is a magic number. | 227 | |
| '+' should be on a new line. | 233 | |
| '+' should be on a new line. | 234 | |
| '+' should be on a new line. | 235 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 7 | |
| Missing a Javadoc comment. | 9 | |
| Missing a Javadoc comment. | 9 | |
| Missing a Javadoc comment. | 9 | |
| Missing a Javadoc comment. | 9 | |
| Missing a Javadoc comment. | 9 | |
| Missing a Javadoc comment. | 9 | |
| '{' should be on a new line. | 17 | |
| for at indentation level 6 not at correct indentation, 8 | 18 | |
| '{' should be on a new line. | 18 | |
| if at indentation level 8 not at correct indentation, 12 | 19 | |
| '{' should be on a new line. | 19 | |
| if child at indentation level 10 not at correct indentation, 16 | 20 | |
| if rcurly at indentation level 8 not at correct indentation, 12 | 21 | |
| for rcurly at indentation level 6 not at correct indentation, 8 | 22 | |
| method def child at indentation level 6 not at correct indentation, 8 | 23 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 5 | |
| First sentence should end with a period. | 9 | |
| Redundant 'public' modifier. | 13 | |
| First sentence should end with a period. | 15 | |
| Redundant 'public' modifier. | 20 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'java.util.Collections' import. | 6 | |
| Wrong order for 'java.util.LinkedList' import. | 11 | |
| Wrong order for 'java.net.URI' import. | 14 | |
| Wrong order for 'org.apache.commons.lang.builder.ToStringBuilder' import. | 17 | |
| First sentence should end with a period. | 19 | |
| Class Fan-Out Complexity is 13 (max allowed is 10). | 22 | |
| ',' is not followed by whitespace. | 37 | |
| First sentence should end with a period. | 74 | |
| First sentence should end with a period. | 83 | |
| First sentence should end with a period. | 92 | |
| First sentence should end with a period. | 120 | |
| First sentence should end with a period. | 129 | |
| First sentence should end with a period. | 138 | |
| First sentence should end with a period. | 147 | |
| First sentence should end with a period. | 156 | |
| First sentence should end with a period. | 165 | |
| First sentence should end with a period. | 174 | |
| First sentence should end with a period. | 183 | |
| First sentence should end with a period. | 192 | |
| First sentence should end with a period. | 201 | |
| First sentence should end with a period. | 210 | |
| ',' is not followed by whitespace. | 214 | |
| First sentence should end with a period. | 219 | |
| Line is longer than 120 characters. | 221 | |
| Class Builder should be declared as final. | 265 | |
| ',' is not followed by whitespace. | 279 | |
| Line is longer than 120 characters. | 322 | |
| 'userPrefs' hides a field. | 325 | |
| 'scrolling' hides a field. | 338 | |
| 'height' hides a field. | 351 | |
| 'width' hides a field. | 364 | |
| 'title' hides a field. | 377 | |
| 'titleUrl' hides a field. | 390 | |
| 'thumbnail' hides a field. | 403 | |
| 'author' hides a field. | 416 | |
| 'authorEmail' hides a field. | 429 | |
| 'description' hides a field. | 442 | |
| 'directoryTitle' hides a field. | 455 | |
| 'features' hides a field. | 468 | |
| 'unsupportedFeatureNames' hides a field. | 481 | |
| 'viewsNames' hides a field. | 494 | |
| First sentence should end with a period. | 500 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'com.atlassian.gadgets.GadgetParsingException' import. | 4 | |
| First sentence should end with a period. | 7 | |
| First sentence should end with a period. | 12 | |
| Line is longer than 120 characters. | 19 | |
| First sentence should end with a period. | 21 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'java.util.Collections' import. | 5 | |
| Wrong order for 'org.apache.commons.lang.builder.ToStringBuilder' import. | 11 | |
| First sentence should end with a period. | 13 | |
| Class Builder should be declared as final. | 135 | |
| 'displayName' hides a field. | 177 | |
| 'required' hides a field. | 191 | |
| 'dataType' hides a field. | 204 | |
| 'enumValues' hides a field. | 218 | |
| 'defaultValue' hides a field. | 231 | |
| First sentence should end with a period. | 237 |
| Violation | Message | Line |
|---|---|---|
| First sentence should end with a period. | 5 | |
| Line has trailing spaces. | 29 | |
| Method 'getGadgetState' is not designed for extension - needs to be abstract, final or empty. | 30 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 9 | |
| Line has trailing spaces. | 17 | |
| Line has trailing spaces. | 19 | |
| method def throws at indentation level 12 not at correct indentation, 8 | 31 | |
| Line has trailing spaces. | 34 | |
| Line is longer than 120 characters. | 34 | |
| Line has trailing spaces. | 36 | |
| method def throws at indentation level 12 not at correct indentation, 8 | 45 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 6 | |
| Line has trailing spaces. | 13 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'java.util.Map' import. | 5 | |
| Wrong order for 'java.util.HashMap' import. | 6 | |
| Wrong order for 'java.util.Collections' import. | 7 | |
| Line has trailing spaces. | 17 | |
| First sentence should end with a period. | 74 | |
| Method 'viewType' is not designed for extension - needs to be abstract, final or empty. | 79 | |
| 'viewType' hides a field. | 79 | |
| Method 'writable' is not designed for extension - needs to be abstract, final or empty. | 92 | |
| First sentence should end with a period. | 98 | |
| Method 'addViewParam' is not designed for extension - needs to be abstract, final or empty. | 104 | |
| First sentence should end with a period. | 110 | |
| Method 'addViewParams' is not designed for extension - needs to be abstract, final or empty. | 115 | |
| Method 'build' is not designed for extension - needs to be abstract, final or empty. | 126 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 9 | |
| Line has trailing spaces. | 15 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'java.util.concurrent.locks.Lock' import. | 11 | |
| Name 'allViewTypes' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 19 | |
| Name 'viewTypeRegistrationLock' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 20 | |
| Name 'readLock' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 21 | |
| Name 'writeLock' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 22 | |
| Missing a Javadoc comment. | 24 | |
| Variable access definition in wrong order. | 24 | |
| Missing a Javadoc comment. | 25 | |
| Variable access definition in wrong order. | 25 | |
| Line is longer than 120 characters. | 40 | |
| Line is longer than 120 characters. | 54 | |
| Line is longer than 120 characters. | 61 | |
| for child at indentation level 19 not at correct indentation, 20 | 97 | |
| First sentence should end with a period. | 109 | |
| First sentence should end with a period. | 118 |