The following document contains the results of Checkstyle. 
| Files | Infos |
Warnings |
Errors |
|---|---|---|---|
| 11 | 0 | 0 | 367 |
| Rules | Violations | Severity |
|---|---|---|
| PackageHtml | 1 | |
| NewlineAtEndOfFile | 1 | |
| Translation | 0 | |
| JavadocMethod | 35 | |
| JavadocType | 6 | |
| JavadocVariable | 17 | |
| JavadocStyle | 0 | |
| ConstantName | 0 | |
| LocalFinalVariableName | 0 | |
| LocalVariableName | 0 | |
| MemberName | 0 | |
| MethodName | 0 | |
| PackageName | 0 | |
| ParameterName | 0 | |
| StaticVariableName | 0 | |
| TypeName | 0 | |
| AvoidStarImport | 0 | |
| IllegalImport | 0 | |
| RedundantImport | 0 | |
| UnusedImports | 2 | |
| FileLength | 0 | |
| LineLength | 65 | |
| MethodLength | 0 | |
| ParameterNumber | 0 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 0 | |
| NoWhitespaceAfter | 0 | |
| NoWhitespaceBefore | 0 | |
| OperatorWrap | 0 | |
| ParenPad | 0 | |
| TypecastParenPad | 0 | |
| TabCharacter | 0 | |
| WhitespaceAfter | 0 | |
| WhitespaceAround | 63 | |
| ModifierOrder | 0 | |
| RedundantModifier | 0 | |
| AvoidNestedBlocks | 0 | |
| EmptyBlock | 0 | |
| LeftCurly | 70 | |
| NeedBraces | 0 | |
| RightCurly | 4 | |
| AvoidInlineConditionals | 1 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 0 | |
| EqualsHashCode | 0 | |
| HiddenField | 13 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 0 | |
| MagicNumber | 0 | |
| MissingSwitchDefault | 0 | |
| RedundantThrows | 2 | |
| SimplifyBooleanExpression | 0 | |
| SimplifyBooleanReturn | 0 | |
| DesignForExtension | 15 | |
| FinalClass | 1 | |
| HideUtilityClassConstructor | 1 | |
| InterfaceIsType | 0 | |
| VisibilityModifier | 2 | |
| ArrayTypeStyle | 0 | |
| FinalParameters | 38 | |
GenericIllegalRegexp
|
30 | |
| TodoComment | 0 | |
| UpperEll | 0 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 6 | |
| '{' should be on the previous line. | 7 | |
| Line has trailing spaces. | 8 | |
| Line is longer than 80 characters. | 8 | |
| Missing a Javadoc comment. | 8 | |
| Missing a Javadoc comment. | 10 | |
| Variable 'webResourceManager' must be private and have accessor methods. | 10 | |
| Missing a Javadoc comment. | 11 | |
| Variable 'applicationProperties' must be private and have accessor methods. | 11 | |
| Missing a Javadoc comment. | 12 | |
| Line has trailing spaces. | 13 | |
| Missing a Javadoc comment. | 14 | |
| Parameter applicationProperties should be final. | 14 | |
| 'applicationProperties' hides a field. | 14 | |
| Parameter webResourceManager should be final. | 15 | |
| 'webResourceManager' hides a field. | 15 | |
| Parameter pluginModuleKey should be final. | 16 | |
| 'pluginModuleKey' hides a field. | 16 | |
| '{' should be on the previous line. | 17 | |
| Method 'buildImageUrl' is not designed for extension - needs to be abstract, final or empty. | 23 | |
| Missing a Javadoc comment. | 23 | |
| Parameter path should be final. | 23 | |
| '{' should be on the previous line. | 24 | |
| '{' should be on the previous line. | 26 | |
| Line has trailing spaces. | 29 | |
| Method 'buildRpcJsUrl' is not designed for extension - needs to be abstract, final or empty. | 32 | |
| Missing a Javadoc comment. | 32 | |
| '{' should be on the previous line. | 33 | |
| Method 'getBaseUrl' is not designed for extension - needs to be abstract, final or empty. | 37 | |
| Missing a Javadoc comment. | 37 | |
| '{' should be on the previous line. | 38 | |
| Missing a Javadoc comment. | 42 | |
| '{' should be on the previous line. | 43 | |
| Line is longer than 80 characters. | 44 |
| Violation | Message | Line |
|---|---|---|
| Unused import - com.atlassian.plugin.Plugin. | 7 | |
| '{' should be on the previous line. | 17 | |
| Missing a Javadoc comment. | 18 | |
| Line is longer than 80 characters. | 21 | |
| Line is longer than 80 characters. | 24 | |
| Line is longer than 80 characters. | 25 | |
| Line is longer than 80 characters. | 26 | |
| Unused @throws tag for 'NullPointerException'. | 26 | |
| Parameter applicationProperties should be final. | 28 | |
| 'applicationProperties' hides a field. | 28 | |
| '{' should be on the previous line. | 29 | |
| Line is longer than 80 characters. | 30 | |
| Line is longer than 80 characters. | 36 | |
| Unused @throws tag for 'NullPointerException'. | 39 | |
| Method 'buildGadgetSpecUrl' is not designed for extension - needs to be abstract, final or empty. | 41 | |
| Parameter key should be final. | 41 | |
| '{' should be on the previous line. | 42 | |
| Line is longer than 80 characters. | 49 | |
| Line is longer than 80 characters. | 51 | |
| Line is longer than 80 characters. | 53 | |
| Unused @throws tag for 'NullPointerException'. | 56 | |
| Method 'buildGadgetSpecUrl' is not designed for extension - needs to be abstract, final or empty. | 58 | |
| Parameter pluginKey should be final. | 58 | |
| Parameter location should be final. | 58 | |
| '{' should be on the previous line. | 59 | |
| '{' should be on the previous line. | 67 | |
| Line is longer than 80 characters. | 75 | |
| Line is longer than 80 characters. | 76 | |
| Line is longer than 80 characters. | 77 | |
| Line is longer than 80 characters. | 79 | |
| Line is longer than 80 characters. | 81 | |
| Line is longer than 80 characters. | 82 | |
| Line is longer than 80 characters. | 83 | |
| Line is longer than 80 characters. | 84 | |
| Line is longer than 80 characters. | 86 | |
| Unused @throws tag for 'com.atlassian.gadgets.GadgetSpecUriNotAllowedException'. | 86 | |
| Line is longer than 80 characters. | 87 | |
| Line is longer than 80 characters. | 89 | |
| Unused @throws tag for 'NullPointerException'. | 89 | |
| Method 'parseGadgetSpecUrl' is not designed for extension - needs to be abstract, final or empty. | 91 | |
| Parameter gadgetSpecUrl should be final. | 91 | |
| '{' should be on the previous line. | 92 | |
| '{' should be on the previous line. | 97 | |
| Line is longer than 80 characters. | 98 | |
| Line is longer than 80 characters. | 102 | |
| Line is longer than 80 characters. | 103 | |
| '{' should be on the previous line. | 105 | |
| Line is longer than 80 characters. | 106 | |
| '{' should be on the previous line. | 110 | |
| Line is longer than 80 characters. | 111 | |
| Line is longer than 80 characters. | 113 | |
| '{' should be on the previous line. | 115 | |
| Line is longer than 80 characters. | 116 | |
| '{' should be on the previous line. | 119 | |
| Missing a Javadoc comment. | 126 | |
| '{' should be on the previous line. | 127 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 7 | |
| Missing a Javadoc comment. | 8 | |
| Parameter message should be final. | 8 | |
| Parameter cause should be final. | 8 | |
| '{' should be on the previous line. | 9 | |
| Missing a Javadoc comment. | 13 | |
| Parameter message should be final. | 13 | |
| '{' should be on the previous line. | 14 |
| Violation | Message | Line |
|---|---|---|
| File does not end with a newline. | 0 | |
| Unused import - java.net.URI. | 3 | |
| Line is longer than 80 characters. | 6 | |
| '{' should be on the previous line. | 10 | |
| Missing a Javadoc comment. | 11 | |
| Parameter e should be final. | 11 | |
| '{' should be on the previous line. | 12 | |
| Missing a Javadoc comment. | 16 | |
| Parameter message should be final. | 16 | |
| '{' should be on the previous line. | 17 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 14 | |
| Class Uri should be declared as final. | 16 | |
| '{' should be on the previous line. | 17 | |
| Missing a Javadoc comment. | 18 | |
| '{' is not followed by whitespace. | 18 | |
| '}' is not preceded with whitespace. | 18 | |
| Line is longer than 80 characters. | 21 | |
| Line has trailing spaces. | 23 | |
| Parameter uriComponent should be final. | 27 | |
| '{' should be on the previous line. | 28 | |
| '{' should be on the previous line. | 30 | |
| '}' should be on the same line. | 32 | |
| '{' should be on the previous line. | 34 | |
| Line is longer than 80 characters. | 40 | |
| Line has trailing spaces. | 42 | |
| Parameter uriComponent should be final. | 46 | |
| '{' should be on the previous line. | 47 | |
| '{' should be on the previous line. | 49 | |
| '}' should be on the same line. | 51 | |
| '{' should be on the previous line. | 53 | |
| Line has trailing spaces. | 57 | |
| Line has trailing spaces. | 59 | |
| Line is longer than 80 characters. | 59 | |
| Line has trailing spaces. | 61 | |
| Line is longer than 80 characters. | 63 | |
| Parameter gadgetUri should be final. | 66 | |
| '{' should be on the previous line. | 67 | |
| '{' should be on the previous line. | 69 | |
| Line has trailing spaces. | 72 | |
| '{' should be on the previous line. | 74 | |
| Line is longer than 80 characters. | 76 | |
| '}' should be on the same line. | 80 | |
| '{' should be on the previous line. | 82 | |
| Line has trailing spaces. | 87 | |
| Line is longer than 80 characters. | 89 | |
| Line is longer than 80 characters. | 90 | |
| Line has trailing spaces. | 91 | |
| Line has trailing spaces. | 92 | |
| Line is longer than 80 characters. | 94 | |
| Line is longer than 80 characters. | 97 | |
| Parameter gadgetUri should be final. | 97 | |
| Redundant throws: 'GadgetSpecUriNotAllowedException' is unchecked exception. | 97 | |
| '{' should be on the previous line. | 100 | |
| Line is longer than 80 characters. | 101 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 80 characters. | 4 | |
| Line is longer than 80 characters. | 5 | |
| Line has trailing spaces. | 6 | |
| Line is longer than 80 characters. | 6 | |
| Utility classes should not have a public or default constructor. | 8 | |
| '{' should be on the previous line. | 9 | |
| Missing a Javadoc comment. | 10 | |
| Parameter value should be final. | 10 | |
| Redundant throws: 'IllegalUriParameterException' is unchecked exception. | 10 | |
| '{' should be on the previous line. | 13 | |
| '}' should be on the same line. | 15 | |
| '{' should be on the previous line. | 17 | |
| Line is longer than 80 characters. | 18 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 80 characters. | 10 | |
| Line has trailing spaces. | 11 | |
| Line is longer than 80 characters. | 11 | |
| Line is longer than 80 characters. | 12 | |
| '{' should be on the previous line. | 16 | |
| Missing a Javadoc comment. | 17 | |
| Missing a Javadoc comment. | 19 | |
| Parameter uriTemplate should be final. | 19 | |
| '{' should be on the previous line. | 20 | |
| Line has trailing spaces. | 23 | |
| Line has trailing spaces. | 26 | |
| Line is longer than 80 characters. | 28 | |
| Method 'matches' is not designed for extension - needs to be abstract, final or empty. | 30 | |
| Parameter uri should be final. | 30 | |
| '{' should be on the previous line. | 31 | |
| '{' should be on the previous line. | 34 | |
| '{' should be on the previous line. | 38 | |
| Line is longer than 80 characters. | 39 | |
| Line has trailing spaces. | 46 | |
| Line has trailing spaces. | 49 | |
| Line is longer than 80 characters. | 52 | |
| Unused @throws tag for 'IllegalArgumentException'. | 52 | |
| Method 'extractParameters' is not designed for extension - needs to be abstract, final or empty. | 54 | |
| '<' is not preceded with whitespace. | 54 | |
| '<' is not followed by whitespace. | 54 | |
| '>' is not preceded with whitespace. | 54 | |
| Parameter uri should be final. | 54 | |
| '{' should be on the previous line. | 55 | |
| '{' should be on the previous line. | 57 | |
| Line is longer than 80 characters. | 58 | |
| '<' is not preceded with whitespace. | 60 | |
| '<' is not followed by whitespace. | 60 | |
| '>' is not preceded with whitespace. | 60 | |
| '<' is not preceded with whitespace. | 60 | |
| '<' is not followed by whitespace. | 60 | |
| '>' is not preceded with whitespace. | 60 | |
| '>' is not followed by whitespace. | 60 | |
| '{' should be on the previous line. | 63 | |
| '{' should be on the previous line. | 65 | |
| Line is longer than 80 characters. | 66 | |
| Missing a Javadoc comment. | 72 | |
| Parameter s should be final. | 72 | |
| '{' should be on the previous line. | 73 | |
| Missing a Javadoc comment. | 77 | |
| Parameter s should be final. | 77 | |
| '{' should be on the previous line. | 78 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 7 | |
| Line has trailing spaces. | 10 | |
| Line has trailing spaces. | 17 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 3 | |
| '{' should be on the previous line. | 4 | |
| Missing a Javadoc comment. | 5 | |
| '{' should be on the previous line. | 6 | |
| Missing a Javadoc comment. | 10 | |
| Parameter url should be final. | 10 | |
| '{' should be on the previous line. | 11 | |
| Avoid inline conditionals. | 12 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 17 | |
| '{' should be on the previous line. | 18 | |
| Line has trailing spaces. | 19 | |
| Line is longer than 80 characters. | 19 | |
| Missing a Javadoc comment. | 19 | |
| Line is longer than 80 characters. | 20 | |
| Missing a Javadoc comment. | 20 | |
| Missing a Javadoc comment. | 22 | |
| Missing a Javadoc comment. | 24 | |
| Missing a Javadoc comment. | 26 | |
| Line is longer than 80 characters. | 28 | |
| Missing a Javadoc comment. | 28 | |
| Parameter webInterfaceManager should be final. | 28 | |
| 'webInterfaceManager' hides a field. | 28 | |
| Parameter applicationProperties should be final. | 28 | |
| 'applicationProperties' hides a field. | 28 | |
| Parameter i18n should be final. | 28 | |
| 'i18n' hides a field. | 28 | |
| Line is longer than 80 characters. | 35 | |
| Method 'findMenuItemsInSection' is not designed for extension - needs to be abstract, final or empty. | 35 | |
| Missing a Javadoc comment. | 35 | |
| '<' is not preceded with whitespace. | 35 | |
| '<' is not followed by whitespace. | 35 | |
| '>' is not preceded with whitespace. | 35 | |
| Parameter section should be final. | 35 | |
| Parameter initialContext should be final. | 35 | |
| '<' is not preceded with whitespace. | 35 | |
| '<' is not followed by whitespace. | 35 | |
| '>' is not preceded with whitespace. | 35 | |
| '?' is not followed by whitespace. | 35 | |
| Line is longer than 80 characters. | 37 | |
| '<' is not preceded with whitespace. | 37 | |
| '<' is not followed by whitespace. | 37 | |
| '>' is not preceded with whitespace. | 37 | |
| '<' is not preceded with whitespace. | 37 | |
| '<' is not followed by whitespace. | 37 | |
| '>' is not preceded with whitespace. | 37 | |
| '>' is not followed by whitespace. | 37 | |
| Line is longer than 80 characters. | 38 | |
| '<' is not preceded with whitespace. | 38 | |
| '<' is not followed by whitespace. | 38 | |
| '>' is not preceded with whitespace. | 38 | |
| Line is longer than 80 characters. | 39 | |
| '<' is not preceded with whitespace. | 39 | |
| '<' is not followed by whitespace. | 39 | |
| '>' is not preceded with whitespace. | 39 | |
| '>' is not followed by whitespace. | 39 | |
| Parameter descriptor should be final. | 40 | |
| '{' should be on the previous line. | 41 | |
| Line has trailing spaces. | 46 | |
| Missing a Javadoc comment. | 47 | |
| '{' should be on the previous line. | 48 | |
| Missing a Javadoc comment. | 49 | |
| Missing a Javadoc comment. | 50 | |
| '<' is not preceded with whitespace. | 50 | |
| '<' is not followed by whitespace. | 50 | |
| '>' is not preceded with whitespace. | 50 | |
| Missing a Javadoc comment. | 52 | |
| Line is longer than 80 characters. | 53 | |
| Parameter descriptor should be final. | 53 | |
| 'descriptor' hides a field. | 53 | |
| Parameter context should be final. | 53 | |
| '<' is not preceded with whitespace. | 53 | |
| '<' is not followed by whitespace. | 53 | |
| '>' is not preceded with whitespace. | 53 | |
| 'context' hides a field. | 53 | |
| Line has trailing spaces. | 58 | |
| Method 'getLink' is not designed for extension - needs to be abstract, final or empty. | 59 | |
| Missing a Javadoc comment. | 59 | |
| '{' should be on the previous line. | 60 | |
| Line has trailing spaces. | 63 | |
| Method 'getLabel' is not designed for extension - needs to be abstract, final or empty. | 64 | |
| Missing a Javadoc comment. | 64 | |
| '{' should be on the previous line. | 65 | |
| Line has trailing spaces. | 68 | |
| Method 'getStyleClass' is not designed for extension - needs to be abstract, final or empty. | 69 | |
| Missing a Javadoc comment. | 69 | |
| '{' should be on the previous line. | 70 | |
| Line has trailing spaces. | 74 | |
| Missing a Javadoc comment. | 75 | |
| '{' should be on the previous line. | 76 | |
| Missing a Javadoc comment. | 77 | |
| '<' is not preceded with whitespace. | 77 | |
| '<' is not followed by whitespace. | 77 | |
| '>' is not preceded with whitespace. | 77 | |
| Missing a Javadoc comment. | 78 | |
| Missing a Javadoc comment. | 80 | |
| Parameter link should be final. | 80 | |
| 'link' hides a field. | 80 | |
| Parameter context should be final. | 80 | |
| '<' is not preceded with whitespace. | 80 | |
| '<' is not followed by whitespace. | 80 | |
| '>' is not preceded with whitespace. | 80 | |
| 'context' hides a field. | 80 | |
| '{' should be on the previous line. | 81 | |
| Line is longer than 80 characters. | 83 | |
| '<' is not preceded with whitespace. | 83 | |
| '<' is not followed by whitespace. | 83 | |
| '>' is not preceded with whitespace. | 83 | |
| '>' is not followed by whitespace. | 83 | |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 86 | |
| Missing a Javadoc comment. | 86 | |
| '{' should be on the previous line. | 87 | |
| Method 'getDisplayableUrl' is not designed for extension - needs to be abstract, final or empty. | 91 | |
| Missing a Javadoc comment. | 91 | |
| '{' should be on the previous line. | 92 | |
| Line is longer than 80 characters. | 93 | |
| Line is longer than 80 characters. | 94 | |
| '<' is not preceded with whitespace. | 94 | |
| '<' is not followed by whitespace. | 94 | |
| '>' is not preceded with whitespace. | 94 | |
| '>' is not followed by whitespace. | 94 | |
| '{' should be on the previous line. | 96 | |
| Line has trailing spaces. | 101 | |
| Missing a Javadoc comment. | 102 | |
| Parameter url should be final. | 102 | |
| '{' should be on the previous line. | 103 | |
| Line has trailing spaces. | 107 | |
| Missing a Javadoc comment. | 108 | |
| '{' should be on the previous line. | 109 | |
| Missing a Javadoc comment. | 110 | |
| Missing a Javadoc comment. | 111 | |
| '<' is not preceded with whitespace. | 111 | |
| '<' is not followed by whitespace. | 111 | |
| '>' is not preceded with whitespace. | 111 | |
| Missing a Javadoc comment. | 113 | |
| Parameter webLabel should be final. | 113 | |
| 'webLabel' hides a field. | 113 | |
| Parameter context should be final. | 113 | |
| '<' is not preceded with whitespace. | 113 | |
| '<' is not followed by whitespace. | 113 | |
| '>' is not preceded with whitespace. | 113 | |
| 'context' hides a field. | 113 | |
| '{' should be on the previous line. | 114 | |
| Line has trailing spaces. | 118 | |
| Method 'getDisplayableLabel' is not designed for extension - needs to be abstract, final or empty. | 119 | |
| Missing a Javadoc comment. | 119 | |
| '{' should be on the previous line. | 120 | |
| Line is longer than 80 characters. | 121 | |
| Line is longer than 80 characters. | 122 | |
| '<' is not preceded with whitespace. | 122 | |
| '<' is not followed by whitespace. | 122 | |
| '>' is not preceded with whitespace. | 122 | |
| '>' is not followed by whitespace. | 122 |