The following document contains the results of Checkstyle.
Files | Infos | Warnings | Errors |
---|---|---|---|
32 | 63 | 100 | 145 |
Rules | Violations | Severity |
---|---|---|
JavadocPackage | 3 | Error |
Translation | 0 | Error |
FileLength | 0 | Error |
FileTabCharacter
|
0 | Error |
RegexpSingleline
|
86 | Warning |
AnnotationUseStyle | 0 | Error |
MissingDeprecated | 0 | Error |
MissingOverride
|
0 | Error |
PackageAnnotation | 0 | Error |
JavadocMethod
|
15 | Error |
JavadocType
|
19 | Error |
JavadocVariable
|
2 | Error |
JavadocStyle
|
7 | Error |
ConstantName | 1 | Error |
LocalFinalVariableName | 0 | Error |
LocalVariableName | 0 | Error |
MemberName | 0 | Error |
MethodName | 0 | Error |
PackageName | 0 | Error |
ParameterName | 0 | Error |
StaticVariableName | 0 | Error |
TypeName | 0 | Error |
AvoidStarImport | 0 | Error |
IllegalImport | 0 | Error |
ImportOrder
|
51 | Error |
RedundantImport | 0 | Error |
UnusedImports | 0 | Error |
LineLength
|
14 | Warning |
MethodLength | 0 | Error |
ParameterNumber | 0 | Error |
EmptyForIteratorPad | 0 | Error |
MethodParamPad | 0 | Error |
NoWhitespaceAfter | 0 | Error |
NoWhitespaceBefore | 0 | Error |
OperatorWrap | 6 | Error |
ParenPad | 2 | Error |
TypecastParenPad | 0 | Error |
WhitespaceAfter | 0 | Error |
WhitespaceAround | 17 | Error |
ModifierOrder | 1 | Error |
RedundantModifier | 0 | Error |
AvoidNestedBlocks | 0 | Error |
EmptyBlock | 0 | Error |
LeftCurly
|
0 | Error |
NeedBraces | 0 | Error |
RightCurly
|
0 | Error |
DeclarationOrder | 2 | Error |
DefaultComesLast | 0 | Error |
DoubleCheckedLocking | 0 | Error |
EmptyStatement | 0 | Error |
EqualsHashCode | 0 | Error |
ExplicitInitialization | 0 | Error |
FallThrough | 0 | Error |
HiddenField
|
0 | Error |
IllegalInstantiation | 0 | Error |
InnerAssignment | 0 | Error |
MagicNumber | 4 | Error |
MissingSwitchDefault | 0 | Error |
ParameterAssignment | 0 | Error |
RedundantThrows
|
0 | Error |
SimplifyBooleanExpression | 0 | Error |
SimplifyBooleanReturn | 0 | Error |
StringLiteralEquality | 0 | Error |
DesignForExtension | 63 | Info |
FinalClass | 1 | Error |
HideUtilityClassConstructor | 1 | Error |
InterfaceIsType | 0 | Error |
VisibilityModifier | 1 | Error |
BooleanExpressionComplexity | 0 | Error |
ClassDataAbstractionCoupling | 0 | Error |
ClassFanOutComplexity
|
8 | Error |
CyclomaticComplexity | 0 | Error |
NPathComplexity
|
0 | Error |
JavaNCSS | 0 | Error |
ArrayTypeStyle | 0 | Error |
Indentation | 0 | Error |
TodoComment | 4 | Error |
UpperEll | 0 | Error |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 19 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.io.IOException' import. | 5 | |
Missing a Javadoc comment. | 17 | |
Line has trailing spaces. | 29 | |
Method 'doGet' is not designed for extension - needs to be abstract, final or empty. | 30 | |
Line is longer than 120 characters. | 35 |
Violation | Message | Line |
---|---|---|
Line is longer than 120 characters. | 23 | |
Method 'checkForPermissionToConfigureDirectory' is not designed for extension - needs to be abstract, final or empty. | 37 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.net.URI' import. | 6 | |
Missing a Javadoc comment. | 8 |
Violation | Message | Line |
---|---|---|
Wrong order for 'javax.annotation.Nullable' import. | 5 | |
'com.google.common.base.Predicate' should be separated from previous imports. | 9 | |
First sentence should end with a period. | 11 | |
Line is longer than 120 characters. | 12 | |
'{' is not followed by whitespace. | 16 | |
'}' is not preceded with whitespace. | 16 | |
Line has trailing spaces. | 17 | |
'(' is followed by whitespace. | 48 | |
')' is preceded with whitespace. | 48 | |
Line has trailing spaces. | 54 | |
'if' is not followed by whitespace. | 55 |
Violation | Message | Line |
---|---|---|
Missing a Javadoc comment. | 5 | |
First sentence should end with a period. | 16 | |
Line has trailing spaces. | 27 | |
Line has trailing spaces. | 31 | |
Line has trailing spaces. | 34 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 5 | |
Line has trailing spaces. | 11 | |
Line has trailing spaces. | 17 | |
Method 'getFeedUri' is not designed for extension - needs to be abstract, final or empty. | 18 |
Violation | Message | Line |
---|---|---|
'com.google.common.base.Preconditions.checkNotNull' should be separated from previous imports. | 5 | |
'org.apache.commons.lang.StringUtils.isBlank' should be separated from previous imports. | 9 | |
Wrong order for 'java.io.InputStream' import. | 11 | |
Wrong order for 'com.atlassian.gadgets.GadgetSpecProvider' import. | 29 | |
'com.google.common.base.Function' should be separated from previous imports. | 32 | |
Missing a Javadoc comment. | 43 | |
Class Fan-Out Complexity is 13 (max allowed is 10). | 43 | |
Line has trailing spaces. | 46 | |
Line has trailing spaces. | 48 | |
Line has trailing spaces. | 52 | |
Line has trailing spaces. | 59 | |
Method 'contains' is not designed for extension - needs to be abstract, final or empty. | 60 | |
Method 'entries' is not designed for extension - needs to be abstract, final or empty. | 65 | |
Line has trailing spaces. | 72 | |
Method 'addFeed' is not designed for extension - needs to be abstract, final or empty. | 77 | |
Line has trailing spaces. | 82 | |
Missing a Javadoc comment. | 86 | |
Line has trailing spaces. | 96 | |
Method 'getFeeds' is not designed for extension - needs to be abstract, final or empty. | 99 | |
Missing a Javadoc comment. | 103 | |
Line has trailing spaces. | 109 | |
Line has trailing spaces. | 112 | |
Method 'containsFeed' is not designed for extension - needs to be abstract, final or empty. | 116 | |
Missing a Javadoc comment. | 120 | |
Line has trailing spaces. | 129 | |
Method 'getFeedProvider' is not designed for extension - needs to be abstract, final or empty. | 133 | |
Missing a Javadoc comment. | 137 | |
Line has trailing spaces. | 150 | |
Method 'removeFeed' is not designed for extension - needs to be abstract, final or empty. | 153 | |
Missing a Javadoc comment. | 157 | |
Method 'getFeedProviders' is not designed for extension - needs to be abstract, final or empty. | 164 | |
Line has trailing spaces. | 168 | |
Line has trailing spaces. | 173 | |
Line is longer than 120 characters. | 174 | |
Instance variable definition in wrong order. | 174 | |
Missing a Javadoc comment. | 176 | |
Line has trailing spaces. | 181 | |
Line has trailing spaces. | 186 | |
Missing a Javadoc comment. | 187 | |
Class Fan-Out Complexity is 16 (max allowed is 10). | 187 | |
Line has trailing spaces. | 193 | |
Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 200 | |
Line has trailing spaces. | 204 | |
Method 'getUri' is not designed for extension - needs to be abstract, final or empty. | 205 | |
Line has trailing spaces. | 209 | |
Method 'getApplicationName' is not designed for extension - needs to be abstract, final or empty. | 210 | |
Line has trailing spaces. | 219 | |
Method 'getTitle' is not designed for extension - needs to be abstract, final or empty. | 220 | |
Line has trailing spaces. | 224 | |
Method 'getIcon' is not designed for extension - needs to be abstract, final or empty. | 225 | |
Method 'getBaseUri' is not designed for extension - needs to be abstract, final or empty. | 235 | |
Method 'contains' is not designed for extension - needs to be abstract, final or empty. | 261 | |
Method 'entries' is not designed for extension - needs to be abstract, final or empty. | 266 | |
Line has trailing spaces. | 270 | |
Line has trailing spaces. | 321 | |
Class Fan-Out Complexity is 13 (max allowed is 10). | 328 | |
Line has trailing spaces. | 356 | |
Line is longer than 120 characters. | 374 | |
Line has trailing spaces. | 379 |
Violation | Message | Line |
---|---|---|
Wrong order for 'com.atlassian.gadgets.GadgetSpecProvider' import. | 8 | |
'com.google.common.base.Function' should be separated from previous imports. | 9 | |
Missing a Javadoc comment. | 13 | |
Utility classes should not have a public or default constructor. | 13 | |
Line has trailing spaces. | 23 | |
Line is longer than 120 characters. | 74 |
Violation | Message | Line |
---|---|---|
Missing a Javadoc comment. | 8 |
Violation | Message | Line |
---|---|---|
Line has trailing spaces. | 16 | |
Method 'getFeedUri' is not designed for extension - needs to be abstract, final or empty. | 17 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.net.URI' import. | 5 | |
Missing a Javadoc comment. | 7 | |
Line has trailing spaces. | 17 | |
Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 18 | |
Line has trailing spaces. | 22 | |
Method 'getUri' is not designed for extension - needs to be abstract, final or empty. | 23 | |
Line has trailing spaces. | 27 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 28 |
Violation | Message | Line |
---|---|---|
Missing package-info.java file. | 0 | |
Line has trailing spaces. | 10 | |
Line has trailing spaces. | 17 | |
Line has trailing spaces. | 21 | |
Line has trailing spaces. | 24 | |
Line has trailing spaces. | 29 | |
First sentence should end with a period. | 30 | |
Line has trailing spaces. | 32 | |
First sentence should end with a period. | 37 | |
Line has trailing spaces. | 39 |
Violation | Message | Line |
---|---|---|
Wrong order for 'com.atlassian.gadgets.directory.Directory' import. | 6 | |
Wrong order for 'com.google.common.base.Function' import. | 9 | |
Wrong order for 'com.atlassian.gadgets.spec.GadgetSpecFactory' import. | 13 | |
Wrong order for 'com.atlassian.gadgets.spec.GadgetSpec' import. | 14 | |
Wrong order for 'com.atlassian.gadgets.GadgetParsingException' import. | 15 | |
Missing a Javadoc comment. | 18 | |
Line is longer than 120 characters. | 37 |
Violation | Message | Line |
---|---|---|
Wrong order for 'com.atlassian.gadgets.directory.Directory' import. | 9 | |
'com.google.common.base.Function' should be separated from previous imports. | 18 | |
Comment matches to-do format 'TODO:'. | 26 | |
Class Fan-Out Complexity is 17 (max allowed is 10). | 27 | |
Method 'contains' is not designed for extension - needs to be abstract, final or empty. | 46 | |
Method 'internalEntries' is not designed for extension - needs to be abstract, final or empty. | 51 | |
Method 'add' is not designed for extension - needs to be abstract, final or empty. | 57 | |
Missing a Javadoc comment. | 61 | |
Method 'remove' is not designed for extension - needs to be abstract, final or empty. | 73 | |
Missing a Javadoc comment. | 77 | |
Method 'convertToLocalizedDirectoryEntry' is not designed for extension - needs to be abstract, final or empty. | 85 | |
Missing a Javadoc comment. | 91 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 137 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.net.URI' import. | 7 | |
Wrong order for 'com.atlassian.gadgets.GadgetRequestContext' import. | 12 | |
'com.google.common.base.Function' should be separated from previous imports. | 15 | |
Missing a Javadoc comment. | 19 | |
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 21 | |
Method 'getEntries' is not designed for extension - needs to be abstract, final or empty. | 30 | |
Method 'contains' is not designed for extension - needs to be abstract, final or empty. | 35 | |
Missing a Javadoc comment. | 45 | |
Missing a Javadoc comment. | 71 |
Violation | Message | Line |
---|---|---|
Missing a Javadoc comment. | 10 | |
Method 'buildDirectoryResourceUrl' is not designed for extension - needs to be abstract, final or empty. | 18 | |
Method 'buildDirectoryGadgetResourceUrl' is not designed for extension - needs to be abstract, final or empty. | 24 | |
Method 'buildSubscribedGadgetFeedsUrl' is not designed for extension - needs to be abstract, final or empty. | 30 | |
Line has trailing spaces. | 34 | |
Method 'buildSubscribedGadgetFeedUrl' is not designed for extension - needs to be abstract, final or empty. | 35 |
Violation | Message | Line |
---|---|---|
Missing package-info.java file. | 0 | |
'org.apache.commons.lang.StringUtils.defaultString' should be separated from previous imports. | 11 | |
Wrong order for 'com.atlassian.gadgets.spec.Feature' import. | 12 | |
Missing a Javadoc comment. | 15 | |
'static' modifier out of order with the JLS suggestions. | 17 | |
'&&' should be on a new line. | 123 | |
'&&' should be on a new line. | 124 | |
'31' is a magic number. | 133 | |
'31' is a magic number. | 134 | |
'+' should be on a new line. | 140 | |
'+' should be on a new line. | 141 | |
'+' should be on a new line. | 142 | |
'+' should be on a new line. | 143 |
Violation | Message | Line |
---|---|---|
Wrong order for 'com.atlassian.gadgets.GadgetSpecProvider' import. | 7 | |
'org.apache.commons.logging.Log' should be separated from previous imports. | 14 | |
Wrong order for 'com.atlassian.gadgets.spec.GadgetSpecFactory' import. | 16 | |
Wrong order for 'com.atlassian.gadgets.GadgetParsingException' import. | 17 | |
Class Fan-Out Complexity is 11 (max allowed is 10). | 24 | |
Method 'contains' is not designed for extension - needs to be abstract, final or empty. | 42 | |
Missing a Javadoc comment. | 46 | |
Method 'internalEntries' is not designed for extension - needs to be abstract, final or empty. | 65 | |
Missing a Javadoc comment. | 70 | |
Method 'convertToLocalizedDirectoryEntry' is not designed for extension - needs to be abstract, final or empty. | 93 | |
Missing a Javadoc comment. | 99 | |
Line is longer than 120 characters. | 103 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 119 |
Violation | Message | Line |
---|---|---|
Line has trailing spaces. | 27 | |
Method 'voteOn' is not designed for extension - needs to be abstract, final or empty. | 28 |
Violation | Message | Line |
---|---|---|
Wrong order for 'com.google.common.collect.ImmutableSet' import. | 18 | |
Class Fan-Out Complexity is 13 (max allowed is 10). | 24 | |
Method 'contains' is not designed for extension - needs to be abstract, final or empty. | 41 | |
Method 'pluginGadgetSpecEnabled' is not designed for extension - needs to be abstract, final or empty. | 54 | |
Method 'pluginGadgetSpecDisabled' is not designed for extension - needs to be abstract, final or empty. | 62 | |
Method 'internalEntries' is not designed for extension - needs to be abstract, final or empty. | 70 | |
Method 'convertToLocalizedDirectoryEntry' is not designed for extension - needs to be abstract, final or empty. | 84 | |
Missing a Javadoc comment. | 90 | |
Line is longer than 120 characters. | 137 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 141 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.net.URI' import. | 6 | |
'com.google.common.base.Function' should be separated from previous imports. | 14 | |
Missing a Javadoc comment. | 19 | |
Missing a Javadoc comment. | 21 | |
Line has trailing spaces. | 22 | |
Line has trailing spaces. | 29 | |
Method 'add' is not designed for extension - needs to be abstract, final or empty. | 30 | |
Method 'contains' is not designed for extension - needs to be abstract, final or empty. | 35 | |
Method 'get' is not designed for extension - needs to be abstract, final or empty. | 40 | |
Method 'getAll' is not designed for extension - needs to be abstract, final or empty. | 45 | |
Line has trailing spaces. | 49 | |
Method 'remove' is not designed for extension - needs to be abstract, final or empty. | 50 | |
Line has trailing spaces. | 71 | |
Line has trailing spaces. | 80 | |
Line has trailing spaces. | 85 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 3 |
Violation | Message | Line |
---|---|---|
'org.apache.commons.lang.builder.EqualsBuilder' should be separated from previous imports. | 4 | |
Wrong order for 'javax.xml.bind.annotation.XmlElement' import. | 6 | |
Comment matches to-do format 'TODO:'. | 11 | |
'}' is not preceded with whitespace. | 20 |
Violation | Message | Line |
---|---|---|
Wrong order for 'com.atlassian.gadgets.GadgetRequestContext' import. | 5 | |
Wrong order for 'javax.xml.bind.annotation.XmlElement' import. | 7 | |
Wrong order for 'java.util.Collection' import. | 9 | |
Wrong order for 'com.google.common.collect.ImmutableList' import. | 14 | |
Comment matches to-do format 'TODO:'. | 18 | |
Class JAXBDirectoryContents should be declared as final. | 20 | |
'}' is not preceded with whitespace. | 23 | |
'}' is not preceded with whitespace. | 26 | |
Line is longer than 120 characters. | 44 |
Violation | Message | Line |
---|---|---|
Wrong order for 'javax.xml.bind.annotation.XmlElement' import. | 9 | |
Wrong order for 'java.net.URI' import. | 11 | |
Comment matches to-do format 'TODO:'. | 16 | |
'}' is not preceded with whitespace. | 44 | |
Line is longer than 120 characters. | 60 | |
'17' is a magic number. | 201 | |
'29' is a magic number. | 201 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.net.URI' import. | 5 | |
'com.google.common.collect.ImmutableList' should be separated from previous imports. | 16 | |
Missing a Javadoc comment. | 18 | |
Missing a Javadoc comment. | 23 | |
Variable access definition in wrong order. | 23 | |
'=' is not preceded with whitespace. | 23 | |
'=' is not followed by whitespace. | 23 | |
Variable 'gadgets' must be private and have accessor methods. | 23 | |
Line has trailing spaces. | 24 | |
Line has trailing spaces. | 30 | |
Line has trailing spaces. | 36 | |
Line has trailing spaces. | 38 | |
Line has trailing spaces. | 47 | |
Method 'getFeed' is not designed for extension - needs to be abstract, final or empty. | 48 | |
Line has trailing spaces. | 52 | |
Method 'getSelf' is not designed for extension - needs to be abstract, final or empty. | 53 | |
Line has trailing spaces. | 57 | |
Method 'getGadgets' is not designed for extension - needs to be abstract, final or empty. | 58 | |
Line has trailing spaces. | 62 | |
Missing a Javadoc comment. | 63 | |
Line has trailing spaces. | 66 | |
Line has trailing spaces. | 71 | |
Line has trailing spaces. | 78 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.net.URI' import. | 5 | |
'com.google.common.collect.ImmutableList' should be separated from previous imports. | 14 | |
Missing a Javadoc comment. | 16 | |
'=' is not preceded with whitespace. | 16 | |
'=' is not followed by whitespace. | 16 | |
'=' is not preceded with whitespace. | 20 | |
'=' is not followed by whitespace. | 20 | |
Line has trailing spaces. | 26 | |
Line has trailing spaces. | 28 | |
Line has trailing spaces. | 36 | |
Line has trailing spaces. | 44 | |
Method 'getSelf' is not designed for extension - needs to be abstract, final or empty. | 45 | |
Line has trailing spaces. | 49 | |
Method 'getSubscribedGadgetFeeds' is not designed for extension - needs to be abstract, final or empty. | 50 | |
Line has trailing spaces. | 54 | |
Missing a Javadoc comment. | 55 | |
Line has trailing spaces. | 75 | |
Line has trailing spaces. | 88 | |
Line has trailing spaces. | 93 | |
Line has trailing spaces. | 98 | |
Line has trailing spaces. | 103 | |
Line has trailing spaces. | 108 | |
Line has trailing spaces. | 113 | |
Line has trailing spaces. | 118 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.io.IOException' import. | 5 | |
Wrong order for 'com.atlassian.gadgets.GadgetParsingException' import. | 27 | |
Wrong order for 'com.atlassian.gadgets.directory.Directory' import. | 36 | |
Class Fan-Out Complexity is 18 (max allowed is 10). | 43 | |
Method 'getDirectory' is not designed for extension - needs to be abstract, final or empty. | 84 | |
'}' is not preceded with whitespace. | 86 | |
Method 'putGadgetInDirectory' is not designed for extension - needs to be abstract, final or empty. | 102 | |
Method 'deleteGadgetFromDirectory' is not designed for extension - needs to be abstract, final or empty. | 171 |
Violation | Message | Line |
---|---|---|
Wrong order for 'javax.servlet.http.HttpServletRequest' import. | 5 | |
Missing a Javadoc comment. | 24 | |
Line has trailing spaces. | 42 | |
Method 'get' is not designed for extension - needs to be abstract, final or empty. | 43 | |
'}' is not preceded with whitespace. | 44 | |
Method 'remove' is not designed for extension - needs to be abstract, final or empty. | 52 | |
Line has trailing spaces. | 68 |
Violation | Message | Line |
---|---|---|
Missing package-info.java file. | 0 | |
'org.apache.commons.lang.StringUtils.isEmpty' should be separated from previous imports. | 4 | |
Wrong order for 'java.io.IOException' import. | 6 | |
Wrong order for 'com.atlassian.gadgets.dashboard.PermissionException' import. | 28 | |
Wrong order for 'com.atlassian.gadgets.directory.internal.GadgetFeedsSpecProvider.FeedSpecProvider' import. | 34 | |
Missing a Javadoc comment. | 39 | |
Class Fan-Out Complexity is 20 (max allowed is 10). | 39 | |
Line has trailing spaces. | 43 | |
Line has trailing spaces. | 48 | |
Line has trailing spaces. | 60 | |
Method 'get' is not designed for extension - needs to be abstract, final or empty. | 61 | |
'}' is not preceded with whitespace. | 62 | |
Line has trailing spaces. | 67 | |
Method 'add' is not designed for extension - needs to be abstract, final or empty. | 68 | |
'}' is not preceded with whitespace. | 70 | |
Line has trailing spaces. | 74 | |
Line is longer than 120 characters. | 77 | |
Line is longer than 120 characters. | 111 | |
Line has trailing spaces. | 118 | |
Line has trailing spaces. | 132 | |
Line is longer than 120 characters. | 139 |