The following document contains the results of Checkstyle. 
| Files | Infos |
Warnings |
Errors |
|---|---|---|---|
| 35 | 10 | 157 | 187 |
| Rules | Violations | Severity |
|---|---|---|
| JavadocPackage | 6 | |
| Translation | 0 | |
| FileLength | 0 | |
FileTabCharacter
|
0 | |
RegexpSingleline
|
141 | |
| AnnotationUseStyle | 0 | |
| MissingDeprecated | 0 | |
MissingOverride
|
0 | |
| PackageAnnotation | 0 | |
JavadocMethod
|
39 | |
JavadocType
|
5 | |
JavadocVariable
|
17 | |
JavadocStyle
|
39 | |
| ConstantName | 0 | |
| LocalFinalVariableName | 0 | |
| LocalVariableName | 0 | |
| MemberName | 0 | |
| MethodName | 0 | |
| PackageName | 0 | |
| ParameterName | 0 | |
| StaticVariableName | 0 | |
| TypeName | 0 | |
| AvoidStarImport | 0 | |
| IllegalImport | 0 | |
ImportOrder
|
17 | |
| RedundantImport | 0 | |
| UnusedImports | 5 | |
LineLength
|
16 | |
| MethodLength | 0 | |
| ParameterNumber | 0 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 0 | |
| NoWhitespaceAfter | 0 | |
| NoWhitespaceBefore | 0 | |
| ParenPad | 0 | |
| TypecastParenPad | 0 | |
| WhitespaceAfter | 1 | |
| WhitespaceAround | 2 | |
| ModifierOrder | 0 | |
| RedundantModifier | 19 | |
| AvoidNestedBlocks | 0 | |
| EmptyBlock | 0 | |
LeftCurly
|
4 | |
| NeedBraces | 4 | |
RightCurly
|
0 | |
| DeclarationOrder | 0 | |
| DefaultComesLast | 0 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 0 | |
| EqualsHashCode | 0 | |
| ExplicitInitialization | 1 | |
| FallThrough | 0 | |
HiddenField
|
11 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 0 | |
| MagicNumber | 1 | |
| MissingSwitchDefault | 0 | |
| ParameterAssignment | 0 | |
RedundantThrows
|
0 | |
| SimplifyBooleanExpression | 0 | |
| SimplifyBooleanReturn | 0 | |
| StringLiteralEquality | 0 | |
| DesignForExtension | 10 | |
| FinalClass | 4 | |
| HideUtilityClassConstructor | 0 | |
| InterfaceIsType | 0 | |
| VisibilityModifier | 0 | |
| BooleanExpressionComplexity | 0 | |
| ClassDataAbstractionCoupling | 0 | |
ClassFanOutComplexity
|
2 | |
| CyclomaticComplexity | 0 | |
NPathComplexity
|
2 | |
| JavaNCSS | 0 | |
| ArrayTypeStyle | 0 | |
| Indentation | 8 | |
| TodoComment | 0 | |
| UpperEll | 0 |
| Violation | Message | Line |
|---|---|---|
| Unused import - javax.annotation.Nullable. | 3 | |
| First sentence should end with a period. | 5 | |
| First sentence should end with a period. | 10 | |
| Redundant 'public' modifier. | 15 |
| 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 |
|---|---|---|
| First sentence should end with a period. | 4 | |
| Missing a Javadoc comment. | 9 | |
| Missing a Javadoc comment. | 14 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 10 | |
| Method 'getGadgetId' is not designed for extension - needs to be abstract, final or empty. | 16 | |
| Missing a Javadoc comment. | 16 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 8 | |
| Line has trailing spaces. | 12 | |
| Missing a Javadoc comment. | 13 |
| 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 is longer than 120 characters. | 59 | |
| Variable 'viewer' explicitly initialized to 'null' (default value for its type). | 66 | |
| Missing a Javadoc comment. | 68 | |
| Method 'build' is not designed for extension - needs to be abstract, final or empty. | 73 | |
| Missing a Javadoc comment. | 73 | |
| Method 'locale' is not designed for extension - needs to be abstract, final or empty. | 85 | |
| 'locale' hides a field. | 85 | |
| Method 'ignoreCache' is not designed for extension - needs to be abstract, final or empty. | 98 | |
| 'ignoreCache' hides a field. | 98 | |
| Method 'viewer' is not designed for extension - needs to be abstract, final or empty. | 111 | |
| 'viewer' hides a field. | 111 |
| Violation | Message | Line |
|---|---|---|
| Extra HTML tag found: </interface> | 21 | |
| Extra HTML tag found: </component> | 22 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 8 | |
| Missing a Javadoc comment. | 13 |
| Violation | Message | Line |
|---|---|---|
| Missing package-info.java file. | 0 | |
| Unused @throws tag for 'GadgetSpecUriNotAllowedException'. | 11 | |
| Unused @throws tag for 'NullPointerException'. | 12 |
| 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 has trailing spaces. | 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 | |
| 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. | 247 | |
| Line has trailing spaces. | 262 | |
| Missing a Javadoc comment. | 263 | |
| 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 |
|---|---|---|
| '{' 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 |
|---|---|---|
| Missing a Javadoc comment. | 10 | |
| Method 'getDashboardId' is not designed for extension - needs to be abstract, final or empty. | 15 | |
| Missing a Javadoc comment. | 15 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 7 | |
| 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.Collections.nCopies' import. | 7 | |
| Wrong order for 'java.io.IOException' import. | 9 | |
| Wrong order for 'org.apache.commons.lang.builder.EqualsBuilder' import. | 23 | |
| Wrong order for 'com.atlassian.gadgets.GadgetState' import. | 27 | |
| Line has trailing spaces. | 30 | |
| Line has trailing spaces. | 39 | |
| Line has trailing spaces. | 42 | |
| Line has trailing spaces. | 45 | |
| Line has trailing spaces. | 49 | |
| Line has trailing spaces. | 51 | |
| Line has trailing spaces. | 56 | |
| Line has trailing spaces. | 57 | |
| Class Fan-Out Complexity is 19 (max allowed is 10). | 62 | |
| Line has trailing spaces. | 72 | |
| Line has trailing spaces. | 79 | |
| Line has trailing spaces. | 82 | |
| 'columns' hides a field. | 83 | |
| Line has trailing spaces. | 96 | |
| Line has trailing spaces. | 101 | |
| Line has trailing spaces. | 105 | |
| NPath Complexity is 32 (max allowed is 20). | 110 | |
| Line has trailing spaces. | 113 | |
| Line has trailing spaces. | 115 | |
| Line has trailing spaces. | 136 | |
| Line has trailing spaces. | 146 | |
| Line has trailing spaces. | 164 | |
| Line has trailing spaces. | 167 | |
| Line has trailing spaces. | 174 | |
| Line has trailing spaces. | 178 | |
| 'if' is not followed by whitespace. | 211 | |
| 'if' is not followed by whitespace. | 230 | |
| First sentence should end with a period. | 248 | |
| First sentence should end with a period. | 260 | |
| Line has trailing spaces. | 271 | |
| NPath Complexity is 24 (max allowed is 20). | 272 | |
| Line has trailing spaces. | 303 | |
| Line has trailing spaces. | 317 | |
| Line has trailing spaces. | 328 | |
| Line has trailing spaces. | 333 | |
| Line is longer than 120 characters. | 344 | |
| Line has trailing spaces. | 345 | |
| Line has trailing spaces. | 346 | |
| Class TitleBuilder should be declared as final. | 358 | |
| Line has trailing spaces. | 364 | |
| Line has trailing spaces. | 366 | |
| Line has trailing spaces. | 370 | |
| Line has trailing spaces. | 379 | |
| Class Builder should be declared as final. | 385 | |
| Line has trailing spaces. | 392 | |
| Line has trailing spaces. | 398 | |
| 'columns' hides a field. | 404 | |
| Line has trailing spaces. | 411 | |
| Line has trailing spaces. | 415 | |
| 'layout' hides a field. | 419 | |
| 'title' hides a field. | 432 | |
| 'maxGadgets' hides a field. | 445 | |
| Line has trailing spaces. | 450 | |
| Line has trailing spaces. | 454 | |
| 'columns' hides a field. | 458 | |
| First sentence should end with a period. | 464 | |
| Line has trailing spaces. | 466 | |
| Line has trailing spaces. | 474 | |
| Line has trailing spaces. | 484 | |
| First sentence should end with a period. | 485 | |
| Line has trailing spaces. | 487 | |
| First sentence should end with a period. | 488 | |
| Line has trailing spaces. | 490 | |
| Line has trailing spaces. | 492 | |
| Line has trailing spaces. | 497 | |
| Line has trailing spaces. | 500 | |
| Line has trailing spaces. | 507 | |
| Line has trailing spaces. | 510 | |
| Line has trailing spaces. | 521 | |
| Line has trailing spaces. | 525 | |
| Unused @throws tag for 'IllegalStateException'. | 527 | |
| Line has trailing spaces. | 537 | |
| Line has trailing spaces. | 540 | |
| Unused @throws tag for 'IllegalArgumentException'. | 543 | |
| Line has trailing spaces. | 559 | |
| Line has trailing spaces. | 561 | |
| Line has trailing spaces. | 563 | |
| Line has trailing spaces. | 566 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 6 | |
| Line has trailing spaces. | 18 | |
| Line has trailing spaces. | 27 | |
| Line has trailing spaces. | 35 |
| 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 | |
| Unused @throws tag for 'IllegalArgumentException'. | 70 | |
| 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 |
|---|---|---|
| Missing package-info.java file. | 0 | |
| Wrong order for 'java.util.Iterator' import. | 5 | |
| Class Iterables should be declared as final. | 9 | |
| Missing a Javadoc comment. | 9 | |
| Missing a Javadoc comment. | 16 | |
| Missing a Javadoc comment. | 38 |
| Violation | Message | Line |
|---|---|---|
| Missing package-info.java file. | 0 | |
| Wrong order for 'com.atlassian.gadgets.GadgetRequestContext' import. | 7 | |
| Line has trailing spaces. | 11 | |
| Line has trailing spaces. | 19 | |
| Line has trailing spaces. | 21 | |
| Line is longer than 120 characters. | 27 |
| Violation | Message | Line |
|---|---|---|
| Missing package-info.java file. | 0 | |
| Line is longer than 120 characters. | 14 | |
| Unused @throws tag for 'NullPointerException'. | 36 | |
| Missing a Javadoc comment. | 44 | |
| Missing a Javadoc comment. | 49 | |
| Missing a Javadoc comment. | 54 | |
| Missing a Javadoc comment. | 59 | |
| Unused @throws tag for 'NullPointerException'. | 118 | |
| Missing a Javadoc comment. | 126 | |
| Missing a Javadoc comment. | 131 | |
| '31' is a magic number. | 157 |
| Violation | Message | Line |
|---|---|---|
| Unused @throws tag for 'NullPointerException'. | 13 | |
| Unused @throws tag for 'NullPointerException'. | 22 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 3 | |
| Missing a Javadoc comment. | 3 | |
| Missing a Javadoc comment. | 5 | |
| Missing a Javadoc comment. | 5 | |
| Missing a Javadoc comment. | 5 | |
| Missing a Javadoc comment. | 5 | |
| Missing a Javadoc comment. | 5 | |
| Missing a Javadoc comment. | 5 | |
| Expected @param tag for 'value'. | 13 | |
| '{' should be on a new line. | 13 | |
| for at indentation level 6 not at correct indentation, 8 | 14 | |
| '{' should be on a new line. | 14 | |
| if at indentation level 8 not at correct indentation, 12 | 15 | |
| '{' should be on a new line. | 15 | |
| if child at indentation level 10 not at correct indentation, 16 | 16 | |
| if rcurly at indentation level 8 not at correct indentation, 12 | 17 | |
| for rcurly at indentation level 6 not at correct indentation, 8 | 18 | |
| method def child at indentation level 6 not at correct indentation, 8 | 19 |
| Violation | Message | Line |
|---|---|---|
| Unused import - java.util.Map. | 3 | |
| Missing a Javadoc comment. | 5 | |
| First sentence should end with a period. | 7 | |
| Redundant 'public' modifier. | 11 | |
| First sentence should end with a period. | 13 | |
| Redundant 'public' modifier. | 18 |
| Violation | Message | Line |
|---|---|---|
| Missing package-info.java file. | 0 | |
| Wrong order for 'java.util.Map' import. | 5 | |
| Wrong order for 'java.util.Collection' import. | 6 | |
| Unused import - java.util.Collection. | 6 | |
| Wrong order for 'java.net.URI' import. | 7 | |
| Line has trailing spaces. | 9 | |
| Missing a Javadoc comment. | 9 | |
| First sentence should end with a period. | 11 | |
| Redundant 'public' modifier. | 15 | |
| First sentence should end with a period. | 17 | |
| Redundant 'public' modifier. | 21 | |
| First sentence should end with a period. | 23 | |
| Redundant 'public' modifier. | 28 | |
| First sentence should end with a period. | 30 | |
| Redundant 'public' modifier. | 34 | |
| First sentence should end with a period. | 36 | |
| Redundant 'public' modifier. | 40 | |
| First sentence should end with a period. | 42 | |
| Redundant 'public' modifier. | 46 | |
| First sentence should end with a period. | 48 | |
| Redundant 'public' modifier. | 52 | |
| Line has trailing spaces. | 53 | |
| First sentence should end with a period. | 54 | |
| Redundant 'public' modifier. | 58 | |
| First sentence should end with a period. | 60 | |
| Redundant 'public' modifier. | 64 | |
| First sentence should end with a period. | 66 | |
| Redundant 'public' modifier. | 70 | |
| First sentence should end with a period. | 72 | |
| Redundant 'public' modifier. | 76 | |
| First sentence should end with a period. | 78 | |
| Redundant 'public' modifier. | 82 | |
| First sentence should end with a period. | 84 | |
| Redundant 'public' modifier. | 88 | |
| First sentence should end with a period. | 90 | |
| Redundant 'public' modifier. | 94 | |
| ',' is not followed by whitespace. | 94 | |
| First sentence should end with a period. | 96 | |
| Line is longer than 120 characters. | 98 | |
| Redundant 'public' modifier. | 100 |
| 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 | |
| Expected @throws tag for 'GadgetParsingException'. | 19 | |
| First sentence should end with a period. | 21 | |
| Expected @throws tag for 'GadgetParsingException'. | 28 |
| Violation | Message | Line |
|---|---|---|
| First sentence should end with a period. | 5 | |
| Redundant 'public' modifier. | 46 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 12 | |
| Missing a Javadoc comment. | 18 | |
| Missing a Javadoc comment. | 24 | |
| Line has trailing spaces. | 29 | |
| Method 'getGadgetState' is not designed for extension - needs to be abstract, final or empty. | 30 | |
| Missing a Javadoc comment. | 30 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'com.atlassian.gadgets.GadgetRequestContext' import. | 5 | |
| Line has trailing spaces. | 9 | |
| Line has trailing spaces. | 17 | |
| Line has trailing spaces. | 19 | |
| Line is longer than 120 characters. | 23 | |
| Line is longer than 120 characters. | 25 | |
| method def throws at indentation level 12 not at correct indentation, 8 | 28 | |
| Line has trailing spaces. | 33 | |
| Line has trailing spaces. | 35 | |
| Line is longer than 120 characters. | 40 | |
| Line is longer than 120 characters. | 42 | |
| Line is longer than 120 characters. | 44 | |
| method def throws at indentation level 12 not at correct indentation, 8 | 45 | |
| Line has trailing spaces. | 48 | |
| Line has trailing spaces. | 50 | |
| Line is longer than 120 characters. | 54 | |
| Line is longer than 120 characters. | 57 |
| Violation | Message | Line |
|---|---|---|
| Wrong order for 'com.atlassian.gadgets.GadgetRequestContext' import. | 7 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 6 | |
| Line has trailing spaces. | 13 |
| Violation | Message | Line |
|---|---|---|
| Missing package-info.java file. | 0 | |
| Missing a Javadoc comment. | 7 | |
| Missing a Javadoc comment. | 9 | |
| Missing a Javadoc comment. | 10 | |
| Line has trailing spaces. | 11 | |
| Line has trailing spaces. | 14 | |
| Line has trailing spaces. | 20 | |
| Missing a Javadoc comment. | 21 | |
| Line has trailing spaces. | 25 | |
| Missing a Javadoc comment. | 26 |