The following document contains the results of Checkstyle
. 
| Files | Infos  |
Warnings  |
Errors  |
|---|---|---|---|
| 31 | 0 | 0 | 1672 |
| Rules | Violations | Severity |
|---|---|---|
| PackageHtml | 7 | Error |
| NewlineAtEndOfFile | 3 | Error |
| Translation | 0 | Error |
| JavadocMethod | 269 | Error |
| JavadocType | 15 | Error |
| JavadocVariable | 86 | Error |
| JavadocStyle | 13 | Error |
| ConstantName | 9 | Error |
| LocalFinalVariableName | 0 | Error |
| LocalVariableName | 0 | Error |
| MemberName | 1 | Error |
| MethodName | 0 | Error |
| PackageName | 0 | Error |
| ParameterName | 0 | Error |
| StaticVariableName | 2 | Error |
| TypeName | 0 | Error |
| AvoidStarImport | 2 | Error |
| IllegalImport | 0 | Error |
| RedundantImport | 0 | Error |
| UnusedImports | 4 | Error |
| FileLength | 0 | Error |
| LineLength | 228 | Error |
| MethodLength | 0 | Error |
| ParameterNumber | 3 | Error |
| EmptyForIteratorPad | 0 | Error |
| MethodParamPad | 0 | Error |
| NoWhitespaceAfter | 0 | Error |
| NoWhitespaceBefore | 0 | Error |
| OperatorWrap | 3 | Error |
| ParenPad | 2 | Error |
| TypecastParenPad | 0 | Error |
| TabCharacter | 3 | Error |
| WhitespaceAfter | 4 | Error |
| WhitespaceAround | 48 | Error |
| ModifierOrder | 15 | Error |
| RedundantModifier | 6 | Error |
| AvoidNestedBlocks | 0 | Error |
| EmptyBlock | 6 | Error |
| LeftCurly | 362 | Error |
| NeedBraces | 72 | Error |
| RightCurly | 68 | Error |
| AvoidInlineConditionals | 44 | Error |
| DoubleCheckedLocking | 1 | Error |
| EmptyStatement | 0 | Error |
| EqualsHashCode | 0 | Error |
| HiddenField | 48 | Error |
| IllegalInstantiation | 0 | Error |
| InnerAssignment | 0 | Error |
| MagicNumber | 23 | Error |
| MissingSwitchDefault | 0 | Error |
| RedundantThrows | 2 | Error |
| SimplifyBooleanExpression | 0 | Error |
| SimplifyBooleanReturn | 0 | Error |
| DesignForExtension | 137 | Error |
| FinalClass | 0 | Error |
| HideUtilityClassConstructor | 2 | Error |
| InterfaceIsType | 0 | Error |
| VisibilityModifier | 14 | Error |
| ArrayTypeStyle | 0 | Error |
| FinalParameters | 164 | Error |
GenericIllegalRegexp
|
5 | Error |
| TodoComment | 1 | Error |
| UpperEll | 0 | Error |
| Violation | Message | Line |
|---|---|---|
|
Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
|
File does not end with a newline. | 0 |
|
'{' should be on the previous line. | 27 |
|
Line is longer than 80 characters. | 28 |
|
Missing a Javadoc comment. | 28 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 28 |
|
Missing a Javadoc comment. | 29 |
|
Missing a Javadoc comment. | 30 |
|
Missing a Javadoc comment. | 31 |
|
Missing a Javadoc comment. | 32 |
|
Missing a Javadoc comment. | 33 |
|
Missing a Javadoc comment. | 35 |
|
'{' should be on the previous line. | 36 |
|
Method 'sendBuffer' is not designed for extension - needs to be abstract, final or empty. | 41 |
|
Missing a Javadoc comment. | 41 |
|
'{' should be on the previous line. | 42 |
|
'{' should be on the previous line. | 44 |
|
'+' is not preceded with whitespace. | 45 |
|
'+' is not followed by whitespace. | 45 |
|
'+' is not preceded with whitespace. | 45 |
|
'+' is not followed by whitespace. | 45 |
|
'+' is not preceded with whitespace. | 49 |
|
'+' is not followed by whitespace. | 49 |
|
'+' is not preceded with whitespace. | 49 |
|
'+' is not followed by whitespace. | 49 |
|
'+' is not preceded with whitespace. | 49 |
|
'+' is not followed by whitespace. | 49 |
|
'{' should be on the previous line. | 58 |
|
'{' should be on the previous line. | 60 |
|
'{' should be on the previous line. | 66 |
|
'+' is not preceded with whitespace. | 67 |
|
'+' is not followed by whitespace. | 67 |
|
'}' should be on the same line. | 69 |
|
'{' should be on the previous line. | 71 |
|
'if' construct must use '{}'s. | 72 |
|
'10' is a magic number. | 72 |
|
'else' construct must use '{}'s. | 74 |
|
'}' should be on the same line. | 78 |
|
'{' should be on the previous line. | 80 |
|
'}' should be on the same line. | 86 |
|
'{' should be on the previous line. | 88 |
|
Method 'size' is not designed for extension - needs to be abstract, final or empty. | 94 |
|
Missing a Javadoc comment. | 94 |
|
'{' should be on the previous line. | 95 |
|
Method 'errorSize' is not designed for extension - needs to be abstract, final or empty. | 99 |
|
Missing a Javadoc comment. | 99 |
|
'{' should be on the previous line. | 100 |
|
Method 'addItem' is not designed for extension - needs to be abstract, final or empty. | 104 |
|
Missing a Javadoc comment. | 104 |
|
Parameter item should be final. | 104 |
|
'{' should be on the previous line. | 105 |
|
Method 'addErrorItem' is not designed for extension - needs to be abstract, final or empty. | 110 |
|
Missing a Javadoc comment. | 110 |
|
Parameter item should be final. | 110 |
|
'{' should be on the previous line. | 111 |
|
Method 'getQueue' is not designed for extension - needs to be abstract, final or empty. | 116 |
|
Missing a Javadoc comment. | 116 |
|
'{' should be on the previous line. | 117 |
|
Method 'getErrorQueue' is not designed for extension - needs to be abstract, final or empty. | 121 |
|
Missing a Javadoc comment. | 121 |
|
'{' should be on the previous line. | 122 |
|
Method 'isSending' is not designed for extension - needs to be abstract, final or empty. | 126 |
|
Missing a Javadoc comment. | 126 |
|
'{' should be on the previous line. | 127 |
|
Method 'getSendingStarted' is not designed for extension - needs to be abstract, final or empty. | 131 |
|
Missing a Javadoc comment. | 131 |
|
'{' should be on the previous line. | 132 |
|
Method 'getItemBeingSent' is not designed for extension - needs to be abstract, final or empty. | 136 |
|
Missing a Javadoc comment. | 136 |
|
'{' should be on the previous line. | 137 |
|
Method 'unstickQueue' is not designed for extension - needs to be abstract, final or empty. | 141 |
|
Missing a Javadoc comment. | 141 |
|
Method 'emptyErrorQueue' is not designed for extension - needs to be abstract, final or empty. | 146 |
|
Missing a Javadoc comment. | 146 |
|
'{' should be on the previous line. | 147 |
|
Method 'resendErrorQueue' is not designed for extension - needs to be abstract, final or empty. | 151 |
|
Missing a Javadoc comment. | 151 |
|
'{' should be on the previous line. | 152 |
|
Method 'sendingStarted' is not designed for extension - needs to be abstract, final or empty. | 157 |
|
Missing a Javadoc comment. | 157 |
|
'{' should be on the previous line. | 158 |
|
Method 'sendingStopped' is not designed for extension - needs to be abstract, final or empty. | 163 |
|
Missing a Javadoc comment. | 163 |
|
'{' should be on the previous line. | 164 |
| Violation | Message | Line |
|---|---|---|
|
Missing a Javadoc comment. | 21 |
|
'{' should be on the previous line. | 22 |
|
Missing a Javadoc comment. | 23 |
|
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 23 |
|
Variable 'LOG' must be private and have accessor methods. | 23 |
|
Missing a Javadoc comment. | 25 |
|
Missing a Javadoc comment. | 26 |
|
Missing a Javadoc comment. | 27 |
|
Missing a Javadoc comment. | 28 |
|
Missing a Javadoc comment. | 29 |
|
Missing a Javadoc comment. | 30 |
|
Missing a Javadoc comment. | 32 |
|
'{' should be on the previous line. | 33 |
|
Line is longer than 80 characters. | 36 |
|
Missing a Javadoc comment. | 36 |
|
Parameter id should be final. | 36 |
|
'id' hides a field. | 36 |
|
Parameter name should be final. | 36 |
|
'name' hides a field. | 36 |
|
Parameter description should be final. | 36 |
|
'description' hides a field. | 36 |
|
Parameter hostName should be final. | 36 |
|
Parameter username should be final. | 36 |
|
'username' hides a field. | 36 |
|
Parameter password should be final. | 36 |
|
'password' hides a field. | 36 |
|
Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 46 |
|
Missing a Javadoc comment. | 46 |
|
'{' should be on the previous line. | 47 |
|
Method 'setId' is not designed for extension - needs to be abstract, final or empty. | 51 |
|
Missing a Javadoc comment. | 51 |
|
Parameter id should be final. | 51 |
|
'id' hides a field. | 51 |
|
'{' should be on the previous line. | 52 |
|
Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 57 |
|
Missing a Javadoc comment. | 57 |
|
'{' should be on the previous line. | 58 |
|
Method 'setName' is not designed for extension - needs to be abstract, final or empty. | 62 |
|
Missing a Javadoc comment. | 62 |
|
Parameter name should be final. | 62 |
|
'name' hides a field. | 62 |
|
'{' should be on the previous line. | 63 |
|
Method 'getDescription' is not designed for extension - needs to be abstract, final or empty. | 68 |
|
Missing a Javadoc comment. | 68 |
|
'{' should be on the previous line. | 69 |
|
Method 'setDescription' is not designed for extension - needs to be abstract, final or empty. | 73 |
|
Missing a Javadoc comment. | 73 |
|
Parameter description should be final. | 73 |
|
'description' hides a field. | 73 |
|
'{' should be on the previous line. | 74 |
|
Method 'getHostname' is not designed for extension - needs to be abstract, final or empty. | 79 |
|
Missing a Javadoc comment. | 79 |
|
'{' should be on the previous line. | 80 |
|
Method 'setHostname' is not designed for extension - needs to be abstract, final or empty. | 84 |
|
Missing a Javadoc comment. | 84 |
|
Parameter serverName should be final. | 84 |
|
'{' should be on the previous line. | 85 |
|
Method 'getUsername' is not designed for extension - needs to be abstract, final or empty. | 90 |
|
Missing a Javadoc comment. | 90 |
|
'{' should be on the previous line. | 91 |
|
Method 'setUsername' is not designed for extension - needs to be abstract, final or empty. | 95 |
|
Missing a Javadoc comment. | 95 |
|
Parameter username should be final. | 95 |
|
'username' hides a field. | 95 |
|
'{' should be on the previous line. | 96 |
|
'if' construct must use '{}'s. | 97 |
|
'else' construct must use '{}'s. | 99 |
|
Method 'getPassword' is not designed for extension - needs to be abstract, final or empty. | 104 |
|
Missing a Javadoc comment. | 104 |
|
'{' should be on the previous line. | 105 |
|
Method 'setPassword' is not designed for extension - needs to be abstract, final or empty. | 109 |
|
Missing a Javadoc comment. | 109 |
|
Parameter password should be final. | 109 |
|
'password' hides a field. | 109 |
|
'{' should be on the previous line. | 110 |
|
'if' construct must use '{}'s. | 111 |
|
'else' construct must use '{}'s. | 113 |
|
Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 119 |
|
Missing a Javadoc comment. | 119 |
|
Parameter o should be final. | 119 |
|
'{' should be on the previous line. | 120 |
|
'if' construct must use '{}'s. | 121 |
|
'if' construct must use '{}'s. | 122 |
|
'if' construct must use '{}'s. | 126 |
|
Line is longer than 80 characters. | 126 |
|
Avoid inline conditionals. | 126 |
|
'if' construct must use '{}'s. | 127 |
|
Line is longer than 80 characters. | 127 |
|
Avoid inline conditionals. | 127 |
|
'if' construct must use '{}'s. | 128 |
|
Line is longer than 80 characters. | 128 |
|
Avoid inline conditionals. | 128 |
|
'if' construct must use '{}'s. | 129 |
|
Line is longer than 80 characters. | 129 |
|
Avoid inline conditionals. | 129 |
|
'if' construct must use '{}'s. | 130 |
|
Line is longer than 80 characters. | 130 |
|
Avoid inline conditionals. | 130 |
|
'if' construct must use '{}'s. | 131 |
|
Line is longer than 80 characters. | 131 |
|
Avoid inline conditionals. | 131 |
|
Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 136 |
|
Missing a Javadoc comment. | 136 |
|
'{' should be on the previous line. | 137 |
|
Avoid inline conditionals. | 139 |
|
'29' is a magic number. | 140 |
|
Avoid inline conditionals. | 140 |
|
Line is longer than 80 characters. | 141 |
|
'29' is a magic number. | 141 |
|
Avoid inline conditionals. | 141 |
|
'29' is a magic number. | 142 |
|
Avoid inline conditionals. | 142 |
|
'29' is a magic number. | 143 |
|
Avoid inline conditionals. | 143 |
|
'29' is a magic number. | 144 |
|
Avoid inline conditionals. | 144 |
|
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 148 |
|
Missing a Javadoc comment. | 148 |
|
'{' should be on the previous line. | 149 |
|
Line is longer than 80 characters. | 150 |
|
'{' should be on the previous line. | 158 |
|
Line is longer than 80 characters. | 162 |
|
Missing a Javadoc comment. | 162 |
|
Parameter ois should be final. | 162 |
| Violation | Message | Line |
|---|---|---|
|
Missing a Javadoc comment. | 13 |
|
'{' should be on the previous line. | 14 |
| Violation | Message | Line |
|---|---|---|
|
Missing a Javadoc comment. | 18 |
|
'{' should be on the previous line. | 19 |
|
Missing a Javadoc comment. | 20 |
|
Missing a Javadoc comment. | 22 |
|
Missing a Javadoc comment. | 24 |
|
Missing a Javadoc comment. | 26 |
|
Missing a Javadoc comment. | 28 |
|
Missing a Javadoc comment. | 30 |
|
Missing a Javadoc comment. | 32 |
|
Missing a Javadoc comment. | 34 |
|
Missing a Javadoc comment. | 36 |
|
Missing a Javadoc comment. | 38 |
|
Missing a Javadoc comment. | 40 |
|
Missing a Javadoc comment. | 42 |
|
Missing a Javadoc comment. | 44 |
|
Missing a Javadoc comment. | 46 |
| Violation | Message | Line |
|---|---|---|
|
Unused import - alt.javax.mail.Transport. | 11 |
|
Missing a Javadoc comment. | 19 |
|
'{' should be on the previous line. | 20 |
|
Missing a Javadoc comment. | 27 |
|
Missing a Javadoc comment. | 29 |
|
Missing a Javadoc comment. | 31 |
|
Missing a Javadoc comment. | 33 |
|
Missing a Javadoc comment. | 35 |
|
Missing a Javadoc comment. | 37 |
|
Missing a Javadoc comment. | 39 |
|
Missing a Javadoc comment. | 41 |
|
Missing a Javadoc comment. | 43 |
|
Missing a Javadoc comment. | 45 |
|
Line is longer than 80 characters. | 47 |
|
Missing a Javadoc comment. | 47 |
|
Missing a Javadoc comment. | 49 |
| Violation | Message | Line |
|---|---|---|
|
Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
|
Missing a Javadoc comment. | 19 |
|
'{' should be on the previous line. | 20 |
|
Line is longer than 80 characters. | 21 |
|
Method 'updateMimeMessage' is not designed for extension - needs to be abstract, final or empty. | 21 |
|
Missing a Javadoc comment. | 21 |
|
Parameter email should be final. | 21 |
|
Parameter defaultFrom should be final. | 21 |
|
Parameter prefix should be final. | 21 |
|
Parameter message should be final. | 21 |
|
Line is longer than 80 characters. | 37 |
|
Line is longer than 80 characters. | 39 |
|
'if' construct must use '{}'s. | 43 |
|
Line is longer than 80 characters. | 44 |
|
'if' construct must use '{}'s. | 46 |
|
Line is longer than 80 characters. | 47 |
|
'if' construct must use '{}'s. | 49 |
|
Line is longer than 80 characters. | 50 |
|
'if' construct must use '{}'s. | 52 |
|
'if' construct must use '{}'s. | 55 |
|
'{' should be on the previous line. | 59 |
|
Line is longer than 80 characters. | 62 |
|
'if' is not followed by whitespace. | 64 |
|
'{' should be on the previous line. | 65 |
|
'}' should be on the same line. | 67 |
|
'{' should be on the previous line. | 69 |
|
'}' should be on the same line. | 74 |
|
'{' should be on the previous line. | 76 |
|
Line has trailing spaces. | 77 |
|
Line is longer than 80 characters. | 77 |
|
Line is longer than 80 characters. | 79 |
|
'if' is not followed by whitespace. | 81 |
|
'{' should be on the previous line. | 82 |
|
'}' should be on the same line. | 84 |
|
'{' should be on the previous line. | 86 |
|
'}' should be on the same line. | 91 |
|
'{' should be on the previous line. | 93 |
|
Line is longer than 80 characters. | 94 |
|
'if' construct must use '{}'s. | 98 |
|
'if' construct must use '{}'s. | 101 |
|
'else' construct must use '{}'s. | 103 |
|
'if' construct must use '{}'s. | 108 |
|
'{' should be on the previous line. | 112 |
|
'}' should be on the same line. | 122 |
|
'{' should be on the previous line. | 124 |
|
'{' should be on the previous line. | 130 |
|
Line is longer than 80 characters. | 131 |
|
Line is longer than 80 characters. | 134 |
| Violation | Message | Line |
|---|---|---|
|
Line is longer than 80 characters. | 20 |
|
Missing a Javadoc comment. | 20 |
|
Missing a Javadoc comment. | 22 |
|
'{' should be on the previous line. | 23 |
|
Line is longer than 80 characters. | 27 |
|
Missing a Javadoc comment. | 27 |
|
Parameter id should be final. | 27 |
|
Parameter name should be final. | 27 |
|
Parameter description should be final. | 27 |
|
Parameter serverName should be final. | 27 |
|
Parameter username should be final. | 27 |
|
Parameter password should be final. | 27 |
|
Method 'getType' 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 'getSession' 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 |
| Violation | Message | Line |
|---|---|---|
|
Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
|
'{' should be on the previous line. | 26 |
|
Line is longer than 80 characters. | 27 |
|
Missing a Javadoc comment. | 27 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 27 |
|
Missing a Javadoc comment. | 28 |
|
Missing a Javadoc comment. | 29 |
|
Missing a Javadoc comment. | 31 |
|
Parameter file should be final. | 31 |
|
'{' should be on the previous line. | 32 |
|
Line is longer than 80 characters. | 34 |
|
'{' should be on the previous line. | 36 |
|
Line is longer than 80 characters. | 37 |
|
Line is longer than 80 characters. | 40 |
|
Line is longer than 80 characters. | 41 |
|
'{' should be on the previous line. | 46 |
|
'{' should be on the previous line. | 48 |
|
'}' should be on the same line. | 57 |
|
'{' should be on the previous line. | 59 |
|
Line is longer than 80 characters. | 61 |
|
'}' should be on the same line. | 62 |
|
'{' should be on the previous line. | 64 |
|
'{' should be on the previous line. | 66 |
|
'}' should be on the same line. | 68 |
|
'{' should be on the previous line. | 70 |
|
Missing a Javadoc comment. | 76 |
|
'{' should be on the previous line. | 77 |
|
Missing a Javadoc comment. | 81 |
|
Parameter file should be final. | 81 |
|
'{' should be on the previous line. | 82 |
|
Method 'getLoadedManager' is not designed for extension - needs to be abstract, final or empty. | 87 |
|
Missing a Javadoc comment. | 87 |
|
'{' should be on the previous line. | 88 |
|
Method 'setLoadedManager' is not designed for extension - needs to be abstract, final or empty. | 92 |
|
Missing a Javadoc comment. | 92 |
|
Parameter loadedManager should be final. | 92 |
|
'loadedManager' hides a field. | 92 |
|
'{' should be on the previous line. | 93 |
| Violation | Message | Line |
|---|---|---|
|
Line is longer than 80 characters. | 10 |
|
Line is longer than 80 characters. | 12 |
|
'{' should be on the previous line. | 15 |
|
First sentence should end with a period. | 38 |
|
Expected @param tag for 'item'. | 43 |
|
Expected an @return tag. | 50 |
|
Expected an @return tag. | 57 |
|
Line is longer than 80 characters. | 65 |
|
First sentence should end with a period. | 69 |
|
Expected an @return tag. | 82 |
|
Line is longer than 80 characters. | 85 |
| Violation | Message | Line |
|---|---|---|
|
First sentence should end with a period. | 10 |
|
'{' should be on the previous line. | 14 |
|
Line is longer than 80 characters. | 15 |
|
Missing a Javadoc comment. | 15 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 15 |
|
Line is longer than 80 characters. | 17 |
|
Missing a Javadoc comment. | 17 |
|
Missing a Javadoc comment. | 18 |
|
Name 'ATLASSIAN_MAIL_PROPERTIES' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 18 |
|
Variable 'ATLASSIAN_MAIL_PROPERTIES' must be private and have accessor methods. | 18 |
|
Line is longer than 80 characters. | 22 |
|
'{' should be on the previous line. | 25 |
|
Line is longer than 80 characters. | 28 |
|
'synchronized' is not followed by whitespace. | 30 |
|
'{' should be on the previous line. | 31 |
|
'}' should be on the same line. | 35 |
|
'{' should be on the previous line. | 37 |
|
Line is longer than 80 characters. | 38 |
| Violation | Message | Line |
|---|---|---|
|
Line is longer than 80 characters. | 39 |
|
Missing a Javadoc comment. | 39 |
|
Line is longer than 80 characters. | 41 |
|
Missing a Javadoc comment. | 41 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 41 |
|
Missing a Javadoc comment. | 43 |
|
Missing a Javadoc comment. | 44 |
|
Missing a Javadoc comment. | 45 |
|
Missing a Javadoc comment. | 46 |
|
Missing a Javadoc comment. | 47 |
|
Missing a Javadoc comment. | 48 |
|
Missing a Javadoc comment. | 49 |
|
Missing a Javadoc comment. | 51 |
|
Missing a Javadoc comment. | 52 |
|
Missing a Javadoc comment. | 54 |
|
'{' should be on the previous line. | 55 |
|
Line is longer than 80 characters. | 58 |
|
Missing a Javadoc comment. | 58 |
|
More than 7 parameters. | 58 |
|
Parameter id should be final. | 58 |
|
Parameter name should be final. | 58 |
|
Parameter description should be final. | 58 |
|
Parameter from should be final. | 58 |
|
Parameter prefix should be final. | 58 |
|
'prefix' hides a field. | 58 |
|
Parameter isSession should be final. | 58 |
|
Parameter location should be final. | 58 |
|
Parameter username should be final. | 58 |
|
Parameter password should be final. | 58 |
|
Line is longer than 80 characters. | 60 |
|
Line is longer than 80 characters. | 63 |
|
Missing a Javadoc comment. | 63 |
|
More than 7 parameters. | 63 |
|
Parameter id should be final. | 63 |
|
Parameter name should be final. | 63 |
|
Parameter description should be final. | 63 |
|
Parameter from should be final. | 63 |
|
Parameter prefix should be final. | 63 |
|
'prefix' hides a field. | 63 |
|
Parameter isSession should be final. | 63 |
|
Parameter location should be final. | 63 |
|
Parameter username should be final. | 63 |
|
Parameter password should be final. | 63 |
|
Parameter smtpPort should be final. | 63 |
|
'smtpPort' hides a field. | 63 |
|
Line is longer than 80 characters. | 65 |
|
Line is longer than 80 characters. | 68 |
|
Missing a Javadoc comment. | 68 |
|
More than 7 parameters. | 68 |
|
Parameter id should be final. | 68 |
|
Parameter name should be final. | 68 |
|
Parameter description should be final. | 68 |
|
Parameter from should be final. | 68 |
|
Parameter prefix should be final. | 68 |
|
'prefix' hides a field. | 68 |
|
Parameter isSession should be final. | 68 |
|
Parameter removePrecedence should be final. | 68 |
|
'removePrecedence' hides a field. | 68 |
|
Parameter location should be final. | 68 |
|
Parameter username should be final. | 68 |
|
Parameter password should be final. | 68 |
|
Parameter smtpPort should be final. | 68 |
|
'smtpPort' hides a field. | 68 |
|
'{' should be on the previous line. | 78 |
|
Method 'getJndiLocation' is not designed for extension - needs to be abstract, final or empty. | 84 |
|
Missing a Javadoc comment. | 84 |
|
'{' should be on the previous line. | 85 |
|
Method 'setJndiLocation' is not designed for extension - needs to be abstract, final or empty. | 89 |
|
Missing a Javadoc comment. | 89 |
|
Parameter jndiLocation should be final. | 89 |
|
'jndiLocation' hides a field. | 89 |
|
'{' should be on the previous line. | 90 |
|
Method 'getAuthenticator' is not designed for extension - needs to be abstract, final or empty. | 95 |
|
Missing a Javadoc comment. | 95 |
|
'{' should be on the previous line. | 96 |
|
First sentence should end with a period. | 100 |
|
Expected an @return tag. | 103 |
|
Method 'getSession' is not designed for extension - needs to be abstract, final or empty. | 103 |
|
Expected @throws tag for 'NamingException'. | 103 |
|
Expected @throws tag for 'MailException'. | 103 |
|
'{' should be on the previous line. | 104 |
|
'{' should be on the previous line. | 107 |
|
'{' should be on the previous line. | 109 |
|
'{' should be on the previous line. | 112 |
|
'}' should be on the same line. | 114 |
|
'{' should be on the previous line. | 116 |
|
Line is longer than 80 characters. | 117 |
|
'+' should be on a new line. | 117 |
|
Line is longer than 80 characters. | 118 |
|
Avoid inline conditionals. | 118 |
|
'+' is not preceded with whitespace. | 118 |
|
'+' is not followed by whitespace. | 118 |
|
'+' is not preceded with whitespace. | 118 |
|
'+' is not followed by whitespace. | 118 |
|
Line is longer than 80 characters. | 119 |
|
'}' should be on the same line. | 121 |
|
'{' should be on the previous line. | 123 |
|
'+' is not preceded with whitespace. | 128 |
|
'+' is not followed by whitespace. | 128 |
|
'{' should be on the previous line. | 131 |
|
'{' should be on the previous line. | 136 |
|
Line is longer than 80 characters. | 141 |
|
'{' should be on the previous line. | 143 |
|
'{' should be on the previous line. | 145 |
|
'{' should be on the previous line. | 148 |
|
Line is longer than 80 characters. | 150 |
|
Method 'getJndiSession' is not designed for extension - needs to be abstract, final or empty. | 158 |
|
Missing a Javadoc comment. | 158 |
|
'{' should be on the previous line. | 159 |
|
Method 'send' is not designed for extension - needs to be abstract, final or empty. | 165 |
|
Missing a Javadoc comment. | 165 |
|
Parameter email should be final. | 165 |
|
'{' should be on the previous line. | 166 |
|
'{' should be on the previous line. | 168 |
|
Line is longer than 80 characters. | 174 |
|
Line is longer than 80 characters. | 179 |
|
Line is longer than 80 characters. | 180 |
|
'}' should be on the same line. | 184 |
|
'{' should be on the previous line. | 186 |
|
'}' should be on the same line. | 188 |
|
'{' should be on the previous line. | 190 |
|
'}' should be on the same line. | 192 |
|
'{' should be on the previous line. | 194 |
|
Line is longer than 80 characters. | 195 |
|
First sentence should end with a period. | 201 |
|
Method 'quietSend' is not designed for extension - needs to be abstract, final or empty. | 204 |
|
Parameter email should be final. | 204 |
|
Expected @param tag for 'email'. | 204 |
|
Expected @throws tag for 'MailException'. | 204 |
|
'{' should be on the previous line. | 205 |
|
'{' should be on the previous line. | 207 |
|
'}' should be on the same line. | 209 |
|
'{' should be on the previous line. | 211 |
|
Line is longer than 80 characters. | 212 |
|
Method 'getType' is not designed for extension - needs to be abstract, final or empty. | 216 |
|
Missing a Javadoc comment. | 216 |
|
'{' should be on the previous line. | 217 |
|
Method 'getDefaultFrom' is not designed for extension - needs to be abstract, final or empty. | 221 |
|
Missing a Javadoc comment. | 221 |
|
'{' should be on the previous line. | 222 |
|
Method 'setDefaultFrom' is not designed for extension - needs to be abstract, final or empty. | 226 |
|
Missing a Javadoc comment. | 226 |
|
Parameter defaultFrom should be final. | 226 |
|
'defaultFrom' hides a field. | 226 |
|
'{' should be on the previous line. | 227 |
|
Method 'getPrefix' is not designed for extension - needs to be abstract, final or empty. | 232 |
|
Missing a Javadoc comment. | 232 |
|
'{' should be on the previous line. | 233 |
|
Method 'setPrefix' is not designed for extension - needs to be abstract, final or empty. | 237 |
|
Missing a Javadoc comment. | 237 |
|
Parameter prefix should be final. | 237 |
|
'prefix' hides a field. | 237 |
|
'{' should be on the previous line. | 238 |
|
Method 'isRemovePrecedence' is not designed for extension - needs to be abstract, final or empty. | 242 |
|
Missing a Javadoc comment. | 242 |
|
'{' should be on the previous line. | 243 |
|
Method 'setRemovePrecedence' is not designed for extension - needs to be abstract, final or empty. | 247 |
|
Missing a Javadoc comment. | 247 |
|
Parameter precedence should be final. | 247 |
|
'{' should be on the previous line. | 248 |
|
Method 'getSmtpPort' is not designed for extension - needs to be abstract, final or empty. | 253 |
|
Missing a Javadoc comment. | 253 |
|
'{' should be on the previous line. | 254 |
|
Method 'setSmtpPort' is not designed for extension - needs to be abstract, final or empty. | 258 |
|
Missing a Javadoc comment. | 258 |
|
Parameter smtpPort should be final. | 258 |
|
'smtpPort' hides a field. | 258 |
|
'{' should be on the previous line. | 259 |
|
Method 'setDebug' is not designed for extension - needs to be abstract, final or empty. | 264 |
|
Missing a Javadoc comment. | 264 |
|
Parameter debug should be final. | 264 |
|
'debug' hides a field. | 264 |
|
Method 'setDebugStream' is not designed for extension - needs to be abstract, final or empty. | 269 |
|
Missing a Javadoc comment. | 269 |
|
Parameter debugStream should be final. | 269 |
|
'debugStream' hides a field. | 269 |
|
Method 'getDebug' is not designed for extension - needs to be abstract, final or empty. | 275 |
|
Missing a Javadoc comment. | 275 |
|
Method 'getDebugStream' is not designed for extension - needs to be abstract, final or empty. | 279 |
|
Missing a Javadoc comment. | 279 |
|
Method 'isSessionServer' is not designed for extension - needs to be abstract, final or empty. | 283 |
|
Missing a Javadoc comment. | 283 |
|
'{' should be on the previous line. | 284 |
|
Method 'setSessionServer' is not designed for extension - needs to be abstract, final or empty. | 288 |
|
Missing a Javadoc comment. | 288 |
|
Parameter sessionServer should be final. | 288 |
|
'{' should be on the previous line. | 289 |
|
Missing a Javadoc comment. | 294 |
|
'{' should be on the previous line. | 295 |
|
Missing a Javadoc comment. | 296 |
|
'{' should be on the previous line. | 297 |
|
Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 303 |
|
Missing a Javadoc comment. | 303 |
|
Parameter o should be final. | 303 |
|
'{' should be on the previous line. | 304 |
|
'{' should be on the previous line. | 306 |
|
'{' should be on the previous line. | 310 |
|
'{' should be on the previous line. | 314 |
|
'{' should be on the previous line. | 321 |
|
Line is longer than 80 characters. | 324 |
|
Avoid inline conditionals. | 324 |
|
Line is longer than 80 characters. | 328 |
|
Avoid inline conditionals. | 328 |
|
Line is longer than 80 characters. | 332 |
|
Avoid inline conditionals. | 332 |
|
'{' should be on the previous line. | 337 |
|
Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 344 |
|
Missing a Javadoc comment. | 344 |
|
'{' should be on the previous line. | 345 |
|
'29' is a magic number. | 347 |
|
Avoid inline conditionals. | 347 |
|
Line is longer than 80 characters. | 348 |
|
'29' is a magic number. | 348 |
|
Avoid inline conditionals. | 348 |
|
'29' is a magic number. | 349 |
|
Avoid inline conditionals. | 349 |
|
'29' is a magic number. | 350 |
|
Avoid inline conditionals. | 350 |
|
'29' is a magic number. | 351 |
|
Avoid inline conditionals. | 351 |
|
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 355 |
|
Missing a Javadoc comment. | 355 |
|
'{' should be on the previous line. | 356 |
|
Line is longer than 80 characters. | 357 |
|
Method 'propertyChanged' is not designed for extension - needs to be abstract, final or empty. | 363 |
|
'{' should be on the previous line. | 364 |
| Violation | Message | Line |
|---|---|---|
|
Using the '.*' form of import should be avoided - java.util.*. | 16 |
|
Missing a Javadoc comment. | 18 |
|
'{' should be on the previous line. | 19 |
|
Line is longer than 80 characters. | 20 |
|
Missing a Javadoc comment. | 20 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 20 |
|
Missing a Javadoc comment. | 23 |
|
Variable 'serverIds' must be private and have accessor methods. | 23 |
|
Missing a Javadoc comment. | 25 |
|
Name 'DEFAULT_CONFIG_FILE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 25 |
|
Missing a Javadoc comment. | 27 |
|
Variable 'configFile' must be private and have accessor methods. | 27 |
|
Method 'init' is not designed for extension - needs to be abstract, final or empty. | 29 |
|
Missing a Javadoc comment. | 29 |
|
Parameter params should be final. | 29 |
|
'{' should be on the previous line. | 30 |
|
'{' should be on the previous line. | 35 |
|
Missing a Javadoc comment. | 42 |
|
'{' should be on the previous line. | 43 |
|
'{' should be on the previous line. | 46 |
|
Line is longer than 80 characters. | 52 |
|
Line is longer than 80 characters. | 58 |
|
Line is longer than 80 characters. | 61 |
|
'}' should be on the same line. | 68 |
|
'{' should be on the previous line. | 70 |
|
Line is longer than 80 characters. | 72 |
|
Method 'getConfigurationInputStream' is not designed for extension - needs to be abstract, final or empty. | 76 |
|
Missing a Javadoc comment. | 76 |
|
Parameter resource should be final. | 76 |
|
'{' should be on the previous line. | 77 |
|
Line is longer than 80 characters. | 78 |
|
Method 'getConfigFile' is not designed for extension - needs to be abstract, final or empty. | 81 |
|
Missing a Javadoc comment. | 81 |
|
'{' should be on the previous line. | 82 |
|
Method 'getMailServer' is not designed for extension - needs to be abstract, final or empty. | 86 |
|
Missing a Javadoc comment. | 86 |
|
Parameter id should be final. | 86 |
|
'{' should be on the previous line. | 87 |
|
Method 'getMailServer' is not designed for extension - needs to be abstract, final or empty. | 91 |
|
Missing a Javadoc comment. | 91 |
|
Parameter name should be final. | 91 |
|
'{' should be on the previous line. | 92 |
|
'if' construct must use '{}'s. | 93 |
|
Line is longer than 80 characters. | 96 |
|
'if' construct must use '{}'s. | 99 |
|
Method 'create' is not designed for extension - needs to be abstract, final or empty. | 106 |
|
Missing a Javadoc comment. | 106 |
|
Parameter mailServer should be final. | 106 |
|
Line is longer than 80 characters. | 110 |
|
Line is longer than 80 characters. | 111 |
|
'while' construct must use '{}'s. | 112 |
|
Method 'update' is not designed for extension - needs to be abstract, final or empty. | 120 |
|
Missing a Javadoc comment. | 120 |
|
Parameter mailServer should be final. | 120 |
|
'{' should be on the previous line. | 121 |
|
Method 'delete' is not designed for extension - needs to be abstract, final or empty. | 125 |
|
Missing a Javadoc comment. | 125 |
|
Parameter mailServerId should be final. | 125 |
|
'{' should be on the previous line. | 126 |
|
'if' construct must use '{}'s. | 127 |
|
'if' construct must use '{}'s. | 129 |
|
Line is longer than 80 characters. | 130 |
|
Method 'getServerNames' is not designed for extension - needs to be abstract, final or empty. | 135 |
|
Missing a Javadoc comment. | 135 |
|
'{' should be on the previous line. | 136 |
|
Line is longer than 80 characters. | 139 |
|
Method 'getSmtpMailServers' is not designed for extension - needs to be abstract, final or empty. | 148 |
|
Missing a Javadoc comment. | 148 |
|
'{' should be on the previous line. | 149 |
|
Line is longer than 80 characters. | 152 |
|
'if' construct must use '{}'s. | 155 |
|
Method 'getPopMailServers' is not designed for extension - needs to be abstract, final or empty. | 162 |
|
Missing a Javadoc comment. | 162 |
|
'{' should be on the previous line. | 163 |
|
Line is longer than 80 characters. | 166 |
|
'if' construct must use '{}'s. | 169 |
|
Method 'getDefaultSMTPMailServer' is not designed for extension - needs to be abstract, final or empty. | 176 |
|
Missing a Javadoc comment. | 176 |
|
'{' should be on the previous line. | 177 |
|
'if' construct must use '{}'s. | 180 |
|
Method 'getDefaultPopMailServer' is not designed for extension - needs to be abstract, final or empty. | 186 |
|
Missing a Javadoc comment. | 186 |
|
'{' should be on the previous line. | 187 |
|
'if' construct must use '{}'s. | 190 |
|
Method 'getSMTPMailServerClass' is not designed for extension - needs to be abstract, final or empty. | 196 |
|
Missing a Javadoc comment. | 196 |
|
'{' should be on the previous line. | 197 |
|
Method 'getPopMailServerClass' is not designed for extension - needs to be abstract, final or empty. | 201 |
|
Missing a Javadoc comment. | 201 |
|
'{' should be on the previous line. | 202 |
| Violation | Message | Line |
|---|---|---|
|
Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
|
'{' should be on the previous line. | 14 |
|
Expected @throws tag for 'MailException'. | 18 |
|
Expected an @return tag. | 23 |
|
Redundant 'public' modifier. | 23 |
|
Expected an @return tag. | 28 |
|
Redundant 'public' modifier. | 28 |
|
First sentence should end with a period. | 30 |
|
Expected an @return tag. | 33 |
|
Redundant 'public' modifier. | 33 |
|
First sentence should end with a period. | 35 |
|
Expected an @return tag. | 38 |
|
Redundant 'public' modifier. | 38 |
| Violation | Message | Line |
|---|---|---|
|
Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
|
Comment matches to-do format 'TODO:'. | 50 |
|
Line is longer than 80 characters. | 52 |
|
Utility classes should not have a public or default constructor. | 55 |
|
'{' should be on the previous line. | 56 |
|
Missing a Javadoc comment. | 57 |
|
Line has trailing spaces. | 58 |
|
Missing a Javadoc comment. | 59 |
|
Line is longer than 80 characters. | 60 |
|
Missing a Javadoc comment. | 60 |
|
Missing a Javadoc comment. | 61 |
|
Missing a Javadoc comment. | 62 |
|
Missing a Javadoc comment. | 63 |
|
Missing a Javadoc comment. | 64 |
|
Missing a Javadoc comment. | 65 |
|
Missing a Javadoc comment. | 66 |
|
Line is longer than 80 characters. | 68 |
|
Missing a Javadoc comment. | 68 |
|
Name 'htmlConverter' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 68 |
|
Missing a Javadoc comment. | 69 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 69 |
|
Line is longer than 80 characters. | 72 |
|
Line is longer than 80 characters. | 74 |
|
First sentence should end with a period. | 76 |
|
Missing a Javadoc comment. | 86 |
|
Missing a Javadoc comment. | 87 |
|
Missing a Javadoc comment. | 88 |
|
Missing a Javadoc comment. | 90 |
|
Parameter contentType should be final. | 90 |
|
'contentType' hides a field. | 90 |
|
Parameter fileName should be final. | 90 |
|
'fileName' hides a field. | 90 |
|
Parameter contents should be final. | 90 |
|
'contents' hides a field. | 90 |
|
Method 'getContentType' is not designed for extension - needs to be abstract, final or empty. | 97 |
|
Missing a Javadoc comment. | 97 |
|
'{' should be on the previous line. | 98 |
|
Method 'getContents' is not designed for extension - needs to be abstract, final or empty. | 102 |
|
Missing a Javadoc comment. | 102 |
|
'{' should be on the previous line. | 103 |
|
Method 'getFilename' is not designed for extension - needs to be abstract, final or empty. | 107 |
|
Missing a Javadoc comment. | 107 |
|
'{' should be on the previous line. | 108 |
|
First sentence should end with a period. | 113 |
|
Line is longer than 80 characters. | 114 |
|
Expected an @return tag. | 116 |
|
Line is longer than 80 characters. | 116 |
|
Parameter addresses should be final. | 116 |
|
Expected @param tag for 'addresses'. | 116 |
|
Expected @throws tag for 'AddressException'. | 116 |
|
'{' should be on the previous line. | 122 |
|
Line is longer than 80 characters. | 125 |
|
Line is longer than 80 characters. | 129 |
|
Line is longer than 80 characters. | 131 |
|
Line is longer than 80 characters. | 132 |
|
Line is longer than 80 characters. | 134 |
|
Line is longer than 80 characters. | 136 |
|
Line is longer than 80 characters. | 137 |
|
Line is longer than 80 characters. | 142 |
|
Line is longer than 80 characters. | 147 |
|
Parameter message should be final. | 149 |
|
'{' should be on the previous line. | 150 |
|
'{' should be on the previous line. | 152 |
|
Line has trailing spaces. | 154 |
|
'{' should be on the previous line. | 156 |
|
'{' should be on the previous line. | 158 |
|
Line is longer than 80 characters. | 159 |
|
'{' should be on the previous line. | 164 |
|
Line has trailing spaces. | 168 |
|
'}' should be on the same line. | 170 |
|
'{' should be on the previous line. | 172 |
|
Line is longer than 80 characters. | 173 |
|
'}' should be on the same line. | 175 |
|
'{' should be on the previous line. | 177 |
|
Line is longer than 80 characters. | 178 |
|
Line is longer than 80 characters. | 184 |
|
Line is longer than 80 characters. | 189 |
|
Parameter message should be final. | 189 |
|
Expected @throws tag for 'MessagingException'. | 189 |
|
Expected @throws tag for 'IOException'. | 189 |
|
'{' should be on the previous line. | 194 |
|
'cast' is not followed by whitespace. | 195 |
|
Line is longer than 80 characters. | 198 |
|
Line is longer than 80 characters. | 201 |
|
Missing a Javadoc comment. | 201 |
|
Parameter attachments should be final. | 201 |
|
Parameter parts should be final. | 201 |
|
'{' should be on the previous line. | 204 |
|
'{' should be on the previous line. | 208 |
|
Line is longer than 80 characters. | 212 |
|
'}' should be on the same line. | 213 |
|
'{' should be on the previous line. | 215 |
|
'{' should be on the previous line. | 217 |
|
'{' should be on the previous line. | 219 |
|
Line is longer than 80 characters. | 220 |
|
'}' should be on the same line. | 222 |
|
'{' should be on the previous line. | 224 |
|
Line is longer than 80 characters. | 227 |
|
Missing a Javadoc comment. | 234 |
|
Parameter part should be final. | 234 |
|
'{' should be on the previous line. | 236 |
|
Line is longer than 80 characters. | 237 |
|
Parameter in should be final. | 246 |
|
Expected @param tag for 'in'. | 246 |
|
Expected @throws tag for 'IOException'. | 246 |
|
'{' should be on the previous line. | 247 |
|
'512' is a magic number. | 249 |
|
'{' should be on the previous line. | 252 |
|
Line is longer than 80 characters. | 261 |
|
Line is longer than 80 characters. | 265 |
|
Line is longer than 80 characters. | 266 |
|
Line is longer than 80 characters. | 270 |
|
Parameter message should be final. | 270 |
|
Parameter addresses should be final. | 283 |
|
'{' should be on the previous line. | 284 |
|
'if' construct must use '{}'s. | 285 |
|
'{' should be on the previous line. | 289 |
|
'{' should be on the previous line. | 291 |
|
'{' should be on the previous line. | 295 |
|
Line is longer than 80 characters. | 296 |
|
'}' should be on the same line. | 298 |
|
'{' should be on the previous line. | 300 |
|
Must have at least one statement. | 300 |
|
Line is longer than 80 characters. | 310 |
|
Line is longer than 80 characters. | 312 |
|
Parameter matchEmail should be final. | 312 |
|
Expected @param tag for 'matchEmail'. | 312 |
|
Parameter message should be final. | 312 |
|
Expected @param tag for 'message'. | 312 |
|
Expected @throws tag for 'MessagingException'. | 312 |
|
'if' construct must use '{}'s. | 316 |
|
'{' should be on the previous line. | 320 |
|
'if' construct must use '{}'s. | 323 |
|
Unclosed HTML tag found: <String> of trimmed non-null email addresses from the | 331 |
|
Line is longer than 80 characters. | 336 |
|
Line is longer than 80 characters. | 338 |
|
Parameter message should be final. | 338 |
|
'{' should be on the previous line. | 344 |
|
'{' should be on the previous line. | 346 |
|
'{' should be on the previous line. | 348 |
|
Line is longer than 80 characters. | 351 |
|
'{' should be on the previous line. | 353 |
|
Line is longer than 80 characters. | 363 |
|
Expected an @return tag. | 370 |
|
Line is longer than 80 characters. | 370 |
|
Parameter path should be final. | 370 |
|
Expected @param tag for 'path'. | 370 |
|
Expected @throws tag for 'MessagingException'. | 370 |
|
Missing a Javadoc comment. | 382 |
|
Parameter path should be final. | 382 |
|
'if' construct must use '{}'s. | 383 |
|
'{' should be on the previous line. | 388 |
|
Line is longer than 80 characters. | 395 |
|
Missing a Javadoc comment. | 395 |
|
Parameter path should be final. | 395 |
|
'(' is followed by whitespace. | 409 |
|
')' is preceded with whitespace. | 409 |
|
'+' is not preceded with whitespace. | 416 |
|
'+' is not followed by whitespace. | 416 |
|
'+' is not preceded with whitespace. | 416 |
|
'+' is not followed by whitespace. | 416 |
|
'+' is not preceded with whitespace. | 416 |
|
'+' is not followed by whitespace. | 416 |
|
'+' is not preceded with whitespace. | 422 |
|
'+' is not followed by whitespace. | 422 |
|
'+' is not preceded with whitespace. | 422 |
|
'+' is not followed by whitespace. | 422 |
|
'+' is not preceded with whitespace. | 422 |
|
'+' is not followed by whitespace. | 422 |
|
'+' is not preceded with whitespace. | 431 |
|
'+' is not followed by whitespace. | 431 |
|
Line is longer than 80 characters. | 436 |
|
Missing a Javadoc comment. | 436 |
|
Parameter multipart should be final. | 436 |
|
Line is longer than 80 characters. | 443 |
|
Missing a Javadoc comment. | 443 |
|
Parameter multipart should be final. | 443 |
|
Parameter sb should be final. | 443 |
|
Line is longer than 80 characters. | 447 |
|
Line is longer than 80 characters. | 448 |
|
'if' is not followed by whitespace. | 448 |
|
Line is longer than 80 characters. | 450 |
|
'if' is not followed by whitespace. | 451 |
|
'{' should be on the previous line. | 452 |
|
'}' should be on the same line. | 454 |
|
'{' should be on the previous line. | 456 |
|
Line is longer than 80 characters. | 457 |
|
Line is longer than 80 characters. | 463 |
|
'{' should be on the previous line. | 465 |
|
Line is longer than 80 characters. | 469 |
|
'{' should be on the previous line. | 472 |
|
'{' should be on the previous line. | 475 |
|
'}' should be on the same line. | 477 |
|
'if' is not followed by whitespace. | 478 |
|
'{' should be on the previous line. | 479 |
|
'}' should be on the same line. | 482 |
|
'{' should be on the previous line. | 484 |
|
Line is longer than 80 characters. | 485 |
|
Line is longer than 80 characters. | 487 |
|
Line is longer than 80 characters. | 493 |
|
Missing a Javadoc comment. | 493 |
|
Parameter content should be final. | 493 |
|
Parameter sb should be final. | 493 |
|
'{' should be on the previous line. | 496 |
|
'if' construct must use '{}'s. | 497 |
|
'if' is not followed by whitespace. | 497 |
|
Line is longer than 80 characters. | 502 |
|
Missing a Javadoc comment. | 502 |
|
Parameter part should be final. | 502 |
|
Redundant throws: 'UnsupportedEncodingException' is subclass of 'IOException'. | 503 |
|
'{' should be on the previous line. | 504 |
|
'if' construct must use '{}'s. | 505 |
|
'{' should be on the previous line. | 511 |
|
'{' should be on the previous line. | 513 |
|
'}' should be on the same line. | 515 |
|
'{' should be on the previous line. | 517 |
|
Line is longer than 80 characters. | 518 |
|
Line is longer than 80 characters. | 519 |
|
'}' should be on the same line. | 523 |
|
'{' should be on the previous line. | 525 |
|
'{' should be on the previous line. | 530 |
|
Line is longer than 80 characters. | 531 |
|
Line is longer than 80 characters. | 537 |
|
Missing a Javadoc comment. | 537 |
|
Parameter part should be final. | 537 |
|
Parameter charsetName should be final. | 537 |
|
Redundant throws: 'UnsupportedEncodingException' is subclass of 'IOException'. | 537 |
|
'{' should be on the previous line. | 539 |
|
'{' should be on the previous line. | 543 |
|
Line is longer than 80 characters. | 544 |
|
'}' should be on the same line. | 548 |
|
'{' should be on the previous line. | 550 |
|
Line is longer than 80 characters. | 556 |
|
Missing a Javadoc comment. | 556 |
|
Parameter multipart should be final. | 556 |
|
Parameter mimeType should be final. | 556 |
|
'{' should be on the previous line. | 559 |
|
Line is longer than 80 characters. | 562 |
|
Line is longer than 80 characters. | 570 |
|
Missing a Javadoc comment. | 570 |
|
Parameter contentType should be final. | 570 |
|
Parameter mimeType should be final. | 570 |
|
'public' modifier out of order with the JLS suggestions. | 583 |
|
Line is longer than 80 characters. | 595 |
|
'public' modifier out of order with the JLS suggestions. | 595 |
|
First sentence should end with a period. | 600 |
|
Line is longer than 80 characters. | 607 |
|
'public' modifier out of order with the JLS suggestions. | 607 |
|
First sentence should end with a period. | 613 |
|
Line is longer than 80 characters. | 620 |
|
'public' modifier out of order with the JLS suggestions. | 620 |
|
Line is longer than 80 characters. | 626 |
|
Line is longer than 80 characters. | 631 |
|
Line is longer than 80 characters. | 633 |
|
'public' modifier out of order with the JLS suggestions. | 633 |
|
Line is longer than 80 characters. | 642 |
|
'public' modifier out of order with the JLS suggestions. | 647 |
|
'{' should be on the previous line. | 648 |
|
'{' should be on the previous line. | 655 |
|
Missing a Javadoc comment. | 662 |
|
'private' modifier out of order with the JLS suggestions. | 662 |
|
'{' should be on the previous line. | 663 |
|
Line is longer than 80 characters. | 668 |
|
Line is longer than 80 characters. | 671 |
|
Line is longer than 80 characters. | 672 |
|
Line is longer than 80 characters. | 675 |
|
Line is longer than 80 characters. | 681 |
|
Line is longer than 80 characters. | 683 |
|
'public' modifier out of order with the JLS suggestions. | 683 |
|
'{' should be on the previous line. | 690 |
|
'}' should be on the same line. | 692 |
|
'{' should be on the previous line. | 694 |
|
'{' should be on the previous line. | 696 |
|
'{' should be on the previous line. | 702 |
|
'{' should be on the previous line. | 705 |
|
Line is longer than 80 characters. | 707 |
|
'}' should be on the same line. | 711 |
|
'{' should be on the previous line. | 713 |
|
Line is longer than 80 characters. | 723 |
|
'private' modifier out of order with the JLS suggestions. | 727 |
|
'{' should be on the previous line. | 728 |
|
Line is longer than 80 characters. | 733 |
|
Line is longer than 80 characters. | 734 |
|
Line is longer than 80 characters. | 737 |
|
Line is longer than 80 characters. | 738 |
|
Line is longer than 80 characters. | 744 |
|
'public' modifier out of order with the JLS suggestions. | 744 |
|
Line is longer than 80 characters. | 750 |
|
'{' should be on the previous line. | 753 |
|
'if' is not followed by whitespace. | 755 |
|
'{' should be on the previous line. | 756 |
|
'{' should be on the previous line. | 764 |
|
'{' should be on the previous line. | 769 |
|
Line is longer than 80 characters. | 776 |
|
Missing a Javadoc comment. | 776 |
|
'private' modifier out of order with the JLS suggestions. | 776 |
|
'{' should be on the previous line. | 781 |
|
'{' should be on the previous line. | 783 |
|
'{' should be on the previous line. | 786 |
|
Line is longer than 80 characters. | 796 |
|
Line is longer than 80 characters. | 802 |
|
'private' modifier out of order with the JLS suggestions. | 802 |
|
Line is longer than 80 characters. | 805 |
|
'{' should be on the previous line. | 807 |
|
'{' should be on the previous line. | 809 |
|
Line is longer than 80 characters. | 810 |
|
'{' should be on the previous line. | 812 |
|
Line is longer than 80 characters. | 823 |
|
Expected an @return tag. | 830 |
|
Line is longer than 80 characters. | 830 |
|
'public' modifier out of order with the JLS suggestions. | 830 |
|
Line is longer than 80 characters. | 837 |
|
Line is longer than 80 characters. | 838 |
|
Line is longer than 80 characters. | 840 |
|
Line is longer than 80 characters. | 846 |
|
'public' modifier out of order with the JLS suggestions. | 846 |
|
Expected @param tag for 'filename'. | 846 |
|
'{' should be on the previous line. | 850 |
|
Line is longer than 80 characters. | 858 |
|
Line is longer than 80 characters. | 860 |
|
Line is longer than 80 characters. | 861 |
|
Line is longer than 80 characters. | 862 |
|
Line is longer than 80 characters. | 864 |
|
'public' modifier out of order with the JLS suggestions. | 864 |
|
Line is longer than 80 characters. | 867 |
|
Line is longer than 80 characters. | 868 |
| Violation | Message | Line |
|---|---|---|
|
'{' should be on the previous line. | 7 |
| Violation | Message | Line |
|---|---|---|
|
'{' should be on the previous line. | 13 |
|
Missing a Javadoc comment. | 14 |
|
'{' should be on the previous line. | 15 |
|
Missing a Javadoc comment. | 18 |
|
Parameter s should be final. | 18 |
|
'{' should be on the previous line. | 19 |
|
Missing a Javadoc comment. | 23 |
|
Parameter throwable should be final. | 23 |
|
'{' should be on the previous line. | 24 |
|
Missing a Javadoc comment. | 28 |
|
Parameter s should be final. | 28 |
|
Parameter throwable should be final. | 28 |
|
'{' should be on the previous line. | 29 |
| Violation | Message | Line |
|---|---|---|
|
'{' should be on the previous line. | 24 |
|
Missing a Javadoc comment. | 25 |
|
Parameter params should be final. | 25 |
|
'{' should be on the previous line. | 26 |
|
Missing a Javadoc comment. | 30 |
|
Missing a Javadoc comment. | 32 |
|
Missing a Javadoc comment. | 34 |
|
Missing a Javadoc comment. | 36 |
|
Missing a Javadoc comment. | 38 |
|
Missing a Javadoc comment. | 40 |
|
Missing a Javadoc comment. | 42 |
|
Missing a Javadoc comment. | 44 |
|
Line is longer than 80 characters. | 46 |
|
Missing a Javadoc comment. | 46 |
|
Line is longer than 80 characters. | 48 |
|
Missing a Javadoc comment. | 48 |
|
Method 'getSession' is not designed for extension - needs to be abstract, final or empty. | 50 |
|
Missing a Javadoc comment. | 50 |
|
Parameter props should be final. | 50 |
|
Parameter auth should be final. | 50 |
|
'{' should be on the previous line. | 51 |
| Violation | Message | Line |
|---|---|---|
|
Using the '.*' form of import should be avoided - java.io.*. | 8 |
|
Line is longer than 80 characters. | 12 |
|
Line is longer than 80 characters. | 13 |
|
'{' should be on the previous line. | 17 |
|
Line is longer than 80 characters. | 18 |
|
Missing a Javadoc comment. | 18 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 18 |
|
Missing a Javadoc comment. | 20 |
|
Missing a Javadoc comment. | 22 |
|
Variable 'out' must be private and have accessor methods. | 22 |
|
Missing a Javadoc comment. | 23 |
|
Variable 'started' must be private and have accessor methods. | 23 |
|
Missing a Javadoc comment. | 24 |
|
Variable 'inBody' must be private and have accessor methods. | 24 |
|
Missing a Javadoc comment. | 25 |
|
Variable 'inList' must be private and have accessor methods. | 25 |
|
Missing a Javadoc comment. | 26 |
|
Variable 'firstTD' must be private and have accessor methods. | 26 |
|
Missing a Javadoc comment. | 27 |
|
Variable 'listCount' must be private and have accessor methods. | 27 |
|
Missing a Javadoc comment. | 28 |
|
Variable 'links' must be private and have accessor methods. | 28 |
|
Missing a Javadoc comment. | 30 |
|
Missing a Javadoc comment. | 31 |
|
Missing a Javadoc comment. | 32 |
|
Missing a Javadoc comment. | 33 |
|
Missing a Javadoc comment. | 34 |
|
Missing a Javadoc comment. | 35 |
|
Missing a Javadoc comment. | 36 |
|
Line is longer than 80 characters. | 37 |
|
Missing a Javadoc comment. | 37 |
|
Missing a Javadoc comment. | 41 |
|
Parameter writer should be final. | 41 |
|
Line is longer than 80 characters. | 45 |
|
Missing a Javadoc comment. | 45 |
|
Parameter tag should be final. | 45 |
|
Parameter set should be final. | 45 |
|
Parameter position should be final. | 45 |
|
'{' should be on the previous line. | 47 |
|
'{' should be on the previous line. | 49 |
|
'}' should be on the same line. | 51 |
|
Line is longer than 80 characters. | 52 |
|
'if' construct must use '{}'s. | 56 |
|
'if' is not followed by whitespace. | 56 |
|
'}' should be on the same line. | 58 |
|
'if' is not followed by whitespace. | 62 |
|
'{' should be on the previous line. | 63 |
|
'}' should be on the same line. | 66 |
|
'else' construct must use '{}'s. | 67 |
|
'}' should be on the same line. | 69 |
|
'{' should be on the previous line. | 71 |
|
'}' should be on the same line. | 73 |
|
'{' should be on the previous line. | 75 |
|
'}' should be on the same line. | 78 |
|
Line is longer than 80 characters. | 79 |
|
'if' is not followed by whitespace. | 81 |
|
'{' should be on the previous line. | 82 |
|
'}' should be on the same line. | 84 |
|
'{' should be on the previous line. | 86 |
|
'}' should be on the same line. | 89 |
|
'{' should be on the previous line. | 91 |
|
'}' should be on the same line. | 93 |
|
'{' should be on the previous line. | 95 |
|
'cast' is not followed by whitespace. | 97 |
|
'}' should be on the same line. | 98 |
|
'{' should be on the previous line. | 100 |
|
'cast' is not followed by whitespace. | 102 |
|
'}' should be on the same line. | 103 |
|
'{' should be on the previous line. | 105 |
|
'}' should be on the same line. | 107 |
|
Line is longer than 80 characters. | 108 |
|
'}' should be on the same line. | 111 |
|
'{' should be on the previous line. | 113 |
|
'}' should be on the same line. | 116 |
|
'{' should be on the previous line. | 118 |
|
Missing a Javadoc comment. | 124 |
|
Parameter src should be final. | 124 |
|
'{' should be on the previous line. | 125 |
|
'if' is not followed by whitespace. | 126 |
|
'{' should be on the previous line. | 127 |
|
Missing a Javadoc comment. | 133 |
|
Parameter tag should be final. | 133 |
|
Parameter position should be final. | 133 |
|
Line is longer than 80 characters. | 134 |
|
'if' construct must use '{}'s. | 137 |
|
'if' is not followed by whitespace. | 137 |
|
'}' should be on the same line. | 139 |
|
'{' should be on the previous line. | 141 |
|
'if' is not followed by whitespace. | 142 |
|
'{' should be on the previous line. | 143 |
|
'{' should be on the previous line. | 146 |
|
'{' should be on the previous line. | 149 |
|
'cast' is not followed by whitespace. | 150 |
|
Line is longer than 80 characters. | 151 |
|
'if' is not followed by whitespace. | 152 |
|
'{' should be on the previous line. | 153 |
|
'}' should be on the same line. | 157 |
|
'{' should be on the previous line. | 159 |
|
Line is longer than 80 characters. | 160 |
|
Missing a Javadoc comment. | 167 |
|
Parameter aChar should be final. | 167 |
|
Parameter position should be final. | 167 |
|
'{' should be on the previous line. | 169 |
|
'{' should be on the previous line. | 171 |
|
'}' should be on the same line. | 175 |
|
'{' should be on the previous line. | 177 |
|
Line is longer than 80 characters. | 182 |
|
Missing a Javadoc comment. | 182 |
|
Parameter tag should be final. | 182 |
|
Parameter a should be final. | 182 |
|
Parameter pos should be final. | 182 |
|
'{' should be on the previous line. | 184 |
|
'if' construct must use '{}'s. | 185 |
|
'}' should be on the same line. | 187 |
|
'{' should be on the previous line. | 189 |
|
Method 'convert' is not designed for extension - needs to be abstract, final or empty. | 197 |
|
Missing a Javadoc comment. | 197 |
|
Parameter html should be final. | 197 |
|
'{' should be on the previous line. | 198 |
|
Missing a Javadoc comment. | 205 |
|
Parameter reader should be final. | 205 |
|
Parameter writer should be final. | 205 |
|
'{' should be on the previous line. | 206 |
| Violation | Message | Line |
|---|---|---|
|
Missing a Javadoc comment. | 9 |
|
'{' should be on the previous line. | 10 |
|
Missing a Javadoc comment. | 12 |
|
Missing a Javadoc comment. | 13 |
|
Missing a Javadoc comment. | 16 |
|
Missing a Javadoc comment. | 17 |
|
Missing a Javadoc comment. | 18 |
|
Missing a Javadoc comment. | 19 |
|
Missing a Javadoc comment. | 20 |
|
Missing a Javadoc comment. | 21 |
|
Missing a Javadoc comment. | 22 |
|
Missing a Javadoc comment. | 23 |
|
Missing a Javadoc comment. | 24 |
|
Missing a Javadoc comment. | 25 |
|
Missing a Javadoc comment. | 26 |
|
Missing a Javadoc comment. | 27 |
|
Missing a Javadoc comment. | 29 |
|
Parameter to should be final. | 29 |
|
'to' hides a field. | 29 |
|
'{' should be on the previous line. | 30 |
|
Line is longer than 80 characters. | 35 |
|
Line is longer than 80 characters. | 43 |
|
Parameter to should be final. | 48 |
|
'to' hides a field. | 48 |
|
Expected @param tag for 'to'. | 48 |
|
'{' should be on the previous line. | 49 |
|
'if' construct must use '{}'s. | 50 |
|
Line is longer than 80 characters. | 57 |
|
Parameter to should be final. | 62 |
|
'to' hides a field. | 62 |
|
Expected @param tag for 'to'. | 62 |
|
Parameter cc should be final. | 62 |
|
'cc' hides a field. | 62 |
|
Expected @param tag for 'cc'. | 62 |
|
Parameter bcc should be final. | 62 |
|
'bcc' hides a field. | 62 |
|
Expected @param tag for 'bcc'. | 62 |
|
'{' should be on the previous line. | 63 |
|
'if' construct must use '{}'s. | 64 |
|
Line is longer than 80 characters. | 64 |
|
Line is longer than 80 characters. | 65 |
|
Method 'loadDefaultHeaders' is not designed for extension - needs to be abstract, final or empty. | 72 |
|
Missing a Javadoc comment. | 72 |
|
'{' should be on the previous line. | 73 |
|
Line is longer than 80 characters. | 74 |
|
Line is longer than 80 characters. | 75 |
|
Method 'setFrom' is not designed for extension - needs to be abstract, final or empty. | 81 |
|
Missing a Javadoc comment. | 81 |
|
Parameter from should be final. | 81 |
|
'from' hides a field. | 81 |
|
'{' should be on the previous line. | 82 |
|
Method 'setFromName' is not designed for extension - needs to be abstract, final or empty. | 87 |
|
Missing a Javadoc comment. | 87 |
|
Parameter fromName should be final. | 87 |
|
'fromName' hides a field. | 87 |
|
'{' should be on the previous line. | 88 |
|
Method 'setTo' is not designed for extension - needs to be abstract, final or empty. | 93 |
|
Missing a Javadoc comment. | 93 |
|
Parameter to should be final. | 93 |
|
'to' hides a field. | 93 |
|
'{' should be on the previous line. | 94 |
|
Method 'setSubject' is not designed for extension - needs to be abstract, final or empty. | 99 |
|
Missing a Javadoc comment. | 99 |
|
Parameter subject should be final. | 99 |
|
'subject' hides a field. | 99 |
|
'{' should be on the previous line. | 100 |
|
Method 'setCc' is not designed for extension - needs to be abstract, final or empty. | 105 |
|
Missing a Javadoc comment. | 105 |
|
Parameter cc should be final. | 105 |
|
'cc' hides a field. | 105 |
|
'{' should be on the previous line. | 106 |
|
Method 'setBcc' is not designed for extension - needs to be abstract, final or empty. | 111 |
|
Missing a Javadoc comment. | 111 |
|
Parameter bcc should be final. | 111 |
|
'bcc' hides a field. | 111 |
|
'{' should be on the previous line. | 112 |
|
Method 'setReplyTo' is not designed for extension - needs to be abstract, final or empty. | 117 |
|
Missing a Javadoc comment. | 117 |
|
Parameter replyTo should be final. | 117 |
|
'replyTo' hides a field. | 117 |
|
'{' should be on the previous line. | 118 |
|
Method 'setInReplyTo' is not designed for extension - needs to be abstract, final or empty. | 123 |
|
Missing a Javadoc comment. | 123 |
|
Parameter inReplyTo should be final. | 123 |
|
'inReplyTo' hides a field. | 123 |
|
'{' should be on the previous line. | 124 |
|
Method 'setBody' is not designed for extension - needs to be abstract, final or empty. | 129 |
|
Missing a Javadoc comment. | 129 |
|
Parameter body should be final. | 129 |
|
'body' hides a field. | 129 |
|
'{' should be on the previous line. | 130 |
|
Method 'setMimeType' is not designed for extension - needs to be abstract, final or empty. | 135 |
|
Missing a Javadoc comment. | 135 |
|
Parameter mimeType should be final. | 135 |
|
'mimeType' hides a field. | 135 |
|
'{' should be on the previous line. | 136 |
|
Method 'setEncoding' is not designed for extension - needs to be abstract, final or empty. | 141 |
|
Missing a Javadoc comment. | 141 |
|
Parameter encoding should be final. | 141 |
|
'encoding' hides a field. | 141 |
|
'{' should be on the previous line. | 142 |
|
Method 'setMultipart' is not designed for extension - needs to be abstract, final or empty. | 147 |
|
Missing a Javadoc comment. | 147 |
|
Parameter multipart should be final. | 147 |
|
'multipart' hides a field. | 147 |
|
'{' should be on the previous line. | 148 |
|
Method 'getFrom' is not designed for extension - needs to be abstract, final or empty. | 153 |
|
Missing a Javadoc comment. | 153 |
|
'{' should be on the previous line. | 154 |
|
Method 'getFromName' is not designed for extension - needs to be abstract, final or empty. | 158 |
|
Missing a Javadoc comment. | 158 |
|
'{' should be on the previous line. | 159 |
|
Method 'getTo' is not designed for extension - needs to be abstract, final or empty. | 163 |
|
Missing a Javadoc comment. | 163 |
|
'{' should be on the previous line. | 164 |
|
Method 'getSubject' is not designed for extension - needs to be abstract, final or empty. | 168 |
|
Missing a Javadoc comment. | 168 |
|
'{' should be on the previous line. | 169 |
|
Method 'getCc' is not designed for extension - needs to be abstract, final or empty. | 173 |
|
Missing a Javadoc comment. | 173 |
|
'{' should be on the previous line. | 174 |
|
Method 'getBcc' is not designed for extension - needs to be abstract, final or empty. | 178 |
|
Missing a Javadoc comment. | 178 |
|
'{' should be on the previous line. | 179 |
|
Method 'getReplyTo' is not designed for extension - needs to be abstract, final or empty. | 183 |
|
Missing a Javadoc comment. | 183 |
|
'{' should be on the previous line. | 184 |
|
Method 'getInReplyTo' is not designed for extension - needs to be abstract, final or empty. | 188 |
|
Missing a Javadoc comment. | 188 |
|
'{' should be on the previous line. | 189 |
|
Method 'getBody' is not designed for extension - needs to be abstract, final or empty. | 193 |
|
Missing a Javadoc comment. | 193 |
|
'{' should be on the previous line. | 194 |
|
Method 'getMimeType' is not designed for extension - needs to be abstract, final or empty. | 198 |
|
Missing a Javadoc comment. | 198 |
|
'{' should be on the previous line. | 199 |
|
Method 'getEncoding' is not designed for extension - needs to be abstract, final or empty. | 203 |
|
Missing a Javadoc comment. | 203 |
|
'{' should be on the previous line. | 204 |
|
Method 'getMultipart' is not designed for extension - needs to be abstract, final or empty. | 208 |
|
Missing a Javadoc comment. | 208 |
|
'{' should be on the previous line. | 209 |
|
Method 'getMessageId' is not designed for extension - needs to be abstract, final or empty. | 213 |
|
Missing a Javadoc comment. | 213 |
|
'{' should be on the previous line. | 214 |
|
Method 'setMessageId' is not designed for extension - needs to be abstract, final or empty. | 218 |
|
Missing a Javadoc comment. | 218 |
|
Parameter messageId should be final. | 218 |
|
'messageId' hides a field. | 218 |
|
'{' should be on the previous line. | 219 |
|
First sentence should end with a period. | 223 |
|
Expected an @return tag. | 229 |
|
Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 229 |
|
Parameter o should be final. | 229 |
|
Expected @param tag for 'o'. | 229 |
|
'{' should be on the previous line. | 230 |
|
'if' construct must use '{}'s. | 231 |
|
'if' construct must use '{}'s. | 232 |
|
'if' construct must use '{}'s. | 236 |
|
Line is longer than 80 characters. | 236 |
|
Avoid inline conditionals. | 236 |
|
'if' construct must use '{}'s. | 237 |
|
Avoid inline conditionals. | 237 |
|
'if' construct must use '{}'s. | 238 |
|
Line is longer than 80 characters. | 238 |
|
Avoid inline conditionals. | 238 |
|
'if' construct must use '{}'s. | 239 |
|
Line is longer than 80 characters. | 239 |
|
Avoid inline conditionals. | 239 |
|
'if' construct must use '{}'s. | 240 |
|
Line is longer than 80 characters. | 240 |
|
Avoid inline conditionals. | 240 |
|
'if' construct must use '{}'s. | 241 |
|
Line is longer than 80 characters. | 241 |
|
Avoid inline conditionals. | 241 |
|
'if' construct must use '{}'s. | 242 |
|
Line is longer than 80 characters. | 242 |
|
Avoid inline conditionals. | 242 |
|
'if' construct must use '{}'s. | 243 |
|
Line is longer than 80 characters. | 243 |
|
Avoid inline conditionals. | 243 |
|
'if' construct must use '{}'s. | 244 |
|
Line is longer than 80 characters. | 244 |
|
Avoid inline conditionals. | 244 |
|
'if' construct must use '{}'s. | 245 |
|
Line is longer than 80 characters. | 245 |
|
Avoid inline conditionals. | 245 |
|
'if' construct must use '{}'s. | 246 |
|
Line is longer than 80 characters. | 246 |
|
Avoid inline conditionals. | 246 |
|
'if' construct must use '{}'s. | 247 |
|
Expected an @return tag. | 257 |
|
Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 257 |
|
'{' should be on the previous line. | 258 |
|
'29' is a magic number. | 261 |
|
Avoid inline conditionals. | 261 |
|
'29' is a magic number. | 262 |
|
Avoid inline conditionals. | 262 |
|
'29' is a magic number. | 263 |
|
Avoid inline conditionals. | 263 |
|
'29' is a magic number. | 264 |
|
Avoid inline conditionals. | 264 |
|
'29' is a magic number. | 265 |
|
Avoid inline conditionals. | 265 |
|
'29' is a magic number. | 266 |
|
Avoid inline conditionals. | 266 |
|
'29' is a magic number. | 267 |
|
Avoid inline conditionals. | 267 |
|
'29' is a magic number. | 268 |
|
Avoid inline conditionals. | 268 |
|
'29' is a magic number. | 269 |
|
Avoid inline conditionals. | 269 |
|
'29' is a magic number. | 270 |
|
Avoid inline conditionals. | 270 |
|
'29' is a magic number. | 271 |
|
Avoid inline conditionals. | 271 |
|
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 275 |
|
Missing a Javadoc comment. | 275 |
|
'{' should be on the previous line. | 276 |
|
Line is longer than 80 characters. | 277 |
|
'+' should be on a new line. | 277 |
|
Line is longer than 80 characters. | 278 |
|
'+' should be on a new line. | 278 |
|
Line is longer than 80 characters. | 279 |
|
Method 'addHeader' is not designed for extension - needs to be abstract, final or empty. | 282 |
|
Missing a Javadoc comment. | 282 |
|
Parameter headerName should be final. | 282 |
|
Parameter headerValue should be final. | 282 |
|
'{' should be on the previous line. | 283 |
|
Method 'removeHeader' is not designed for extension - needs to be abstract, final or empty. | 291 |
|
Parameter headerName should be final. | 291 |
|
Expected @param tag for 'headerName'. | 291 |
|
'{' should be on the previous line. | 292 |
|
'{' should be on the previous line. | 294 |
|
'}' should be on the same line. | 296 |
|
'{' should be on the previous line. | 298 |
|
Method 'getHeaders' is not designed for extension - needs to be abstract, final or empty. | 303 |
|
Missing a Javadoc comment. | 303 |
|
'{' should be on the previous line. | 304 |
| Violation | Message | Line |
|---|---|---|
|
Missing a Javadoc comment. | 7 |
|
'{' should be on the previous line. | 8 |
|
Missing a Javadoc comment. | 9 |
|
Variable 'subject' must be private and have accessor methods. | 9 |
|
Missing a Javadoc comment. | 10 |
|
Variable 'dateQueued' must be private and have accessor methods. | 10 |
|
Missing a Javadoc comment. | 11 |
|
Missing a Javadoc comment. | 12 |
|
Variable 'mailThreader' must be private and have accessor methods. | 12 |
|
Line has trailing spaces. | 13 |
|
Missing a Javadoc comment. | 15 |
|
'{' should be on the previous line. | 16 |
|
Missing a Javadoc comment. | 20 |
|
Parameter subject should be final. | 20 |
|
'subject' hides a field. | 20 |
|
'{' should be on the previous line. | 21 |
|
Method 'getSubject' is not designed for extension - needs to be abstract, final or empty. | 26 |
|
Missing a Javadoc comment. | 26 |
|
'{' should be on the previous line. | 27 |
|
Method 'getDateQueued' is not designed for extension - needs to be abstract, final or empty. | 31 |
|
Missing a Javadoc comment. | 31 |
|
'{' should be on the previous line. | 32 |
|
Method 'getSendCount' is not designed for extension - needs to be abstract, final or empty. | 36 |
|
Missing a Javadoc comment. | 36 |
|
'{' should be on the previous line. | 37 |
|
Method 'hasError' is not designed for extension - needs to be abstract, final or empty. | 41 |
|
Missing a Javadoc comment. | 41 |
|
'{' should be on the previous line. | 42 |
|
Method 'incrementSendCount' is not designed for extension - needs to be abstract, final or empty. | 46 |
|
Missing a Javadoc comment. | 46 |
|
'{' should be on the previous line. | 47 |
|
Method 'setMailThreader' is not designed for extension - needs to be abstract, final or empty. | 51 |
|
Missing a Javadoc comment. | 51 |
|
Parameter mailThreader should be final. | 51 |
|
'mailThreader' hides a field. | 51 |
|
Method 'execute' is not designed for extension - needs to be abstract, final or empty. | 55 |
|
Missing a Javadoc comment. | 55 |
|
'{' should be on the previous line. | 56 |
| Violation | Message | Line |
|---|---|---|
|
File does not end with a newline. | 0 |
|
Missing a Javadoc comment. | 18 |
|
'{' should be on the previous line. | 19 |
|
Missing a Javadoc comment. | 20 |
|
Missing a Javadoc comment. | 22 |
|
Missing a Javadoc comment. | 24 |
|
Missing a Javadoc comment. | 26 |
|
Missing a Javadoc comment. | 28 |
|
Missing a Javadoc comment. | 30 |
|
Missing a Javadoc comment. | 32 |
|
Missing a Javadoc comment. | 34 |
|
Missing a Javadoc comment. | 36 |
|
Missing a Javadoc comment. | 38 |
|
Redundant 'public' modifier. | 38 |
|
Missing a Javadoc comment. | 40 |
|
Redundant 'public' modifier. | 40 |
|
Missing a Javadoc comment. | 42 |
|
Missing a Javadoc comment. | 44 |
|
Missing a Javadoc comment. | 46 |
|
Missing a Javadoc comment. | 48 |
|
Line is longer than 80 characters. | 59 |
|
Expected an @return tag. | 66 |
|
Expected an @return tag. | 71 |
| Violation | Message | Line |
|---|---|---|
|
Missing a Javadoc comment. | 33 |
|
'{' should be on the previous line. | 34 |
|
Method 'getMailServer' is not designed for extension - needs to be abstract, final or empty. | 35 |
|
Missing a Javadoc comment. | 35 |
|
Parameter id should be final. | 35 |
|
'{' should be on the previous line. | 36 |
|
'{' should be on the previous line. | 38 |
|
Line is longer than 80 characters. | 39 |
|
'if' construct must use '{}'s. | 40 |
|
'else' construct must use '{}'s. | 42 |
|
'}' should be on the same line. | 44 |
|
'{' should be on the previous line. | 46 |
|
Method 'getMailServer' is not designed for extension - needs to be abstract, final or empty. | 51 |
|
Missing a Javadoc comment. | 51 |
|
Parameter name should be final. | 51 |
|
'{' should be on the previous line. | 52 |
|
'{' should be on the previous line. | 54 |
|
Line is longer than 80 characters. | 55 |
|
'if' construct must use '{}'s. | 56 |
|
'else' construct must use '{}'s. | 58 |
|
'}' should be on the same line. | 60 |
|
'{' should be on the previous line. | 62 |
|
Method 'getServerNames' is not designed for extension - needs to be abstract, final or empty. | 67 |
|
Missing a Javadoc comment. | 67 |
|
'{' should be on the previous line. | 68 |
|
'{' should be on the previous line. | 70 |
|
Line is longer than 80 characters. | 71 |
|
'{' should be on the previous line. | 74 |
|
Line is longer than 80 characters. | 75 |
|
'}' should be on the same line. | 79 |
|
'{' should be on the previous line. | 81 |
|
Method 'getSmtpMailServers' 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 'getPopMailServers' 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 |
|
Method 'create' is not designed for extension - needs to be abstract, final or empty. | 96 |
|
Missing a Javadoc comment. | 96 |
|
Parameter mailServer should be final. | 96 |
|
'{' should be on the previous line. | 97 |
|
'{' should be on the previous line. | 99 |
|
Line is longer than 80 characters. | 100 |
|
'}' should be on the same line. | 102 |
|
'{' should be on the previous line. | 104 |
|
Method 'update' is not designed for extension - needs to be abstract, final or empty. | 109 |
|
Missing a Javadoc comment. | 109 |
|
Parameter mailServer should be final. | 109 |
|
'{' should be on the previous line. | 110 |
|
'{' should be on the previous line. | 112 |
|
'}' should be on the same line. | 116 |
|
'{' should be on the previous line. | 118 |
|
Method 'delete' is not designed for extension - needs to be abstract, final or empty. | 123 |
|
Missing a Javadoc comment. | 123 |
|
Parameter mailServerId should be final. | 123 |
|
'{' should be on the previous line. | 124 |
|
'{' should be on the previous line. | 126 |
|
'}' should be on the same line. | 129 |
|
'{' should be on the previous line. | 131 |
|
Method 'getDefaultSMTPMailServer' is not designed for extension - needs to be abstract, final or empty. | 137 |
|
Missing a Javadoc comment. | 137 |
|
'{' should be on the previous line. | 138 |
|
'{' should be on the previous line. | 141 |
|
Line is longer than 80 characters. | 143 |
|
'if' construct must use '{}'s. | 144 |
|
'}' should be on the same line. | 146 |
|
'{' should be on the previous line. | 148 |
|
Must have at least one statement. | 148 |
|
'{' should be on the previous line. | 152 |
|
'if' construct must use '{}'s. | 153 |
|
Line is longer than 80 characters. | 153 |
|
'}' should be on the same line. | 156 |
|
'{' should be on the previous line. | 158 |
|
Must have at least one statement. | 158 |
|
Method 'getDefaultPopMailServer' is not designed for extension - needs to be abstract, final or empty. | 164 |
|
Missing a Javadoc comment. | 164 |
|
'{' should be on the previous line. | 165 |
|
'{' should be on the previous line. | 168 |
|
Line is longer than 80 characters. | 170 |
|
'if' construct must use '{}'s. | 171 |
|
'}' should be on the same line. | 173 |
|
'{' should be on the previous line. | 175 |
|
Must have at least one statement. | 175 |
|
'{' should be on the previous line. | 179 |
|
'if' construct must use '{}'s. | 180 |
|
'}' should be on the same line. | 183 |
|
'{' should be on the previous line. | 185 |
|
Must have at least one statement. | 185 |
|
Method 'getMailServersByType' is not designed for extension - needs to be abstract, final or empty. | 191 |
|
Missing a Javadoc comment. | 191 |
|
Parameter serverType should be final. | 191 |
|
'{' should be on the previous line. | 196 |
|
'if' construct must use '{}'s. | 198 |
|
Method 'getMailServerGV' is not designed for extension - needs to be abstract, final or empty. | 204 |
|
Missing a Javadoc comment. | 204 |
|
Parameter id should be final. | 204 |
|
'{' should be on the previous line. | 205 |
|
'{' should be on the previous line. | 207 |
|
Line is longer than 80 characters. | 208 |
|
'}' should be on the same line. | 209 |
|
'{' should be on the previous line. | 211 |
|
Method 'constructMailServer' is not designed for extension - needs to be abstract, final or empty. | 216 |
|
Missing a Javadoc comment. | 216 |
|
Parameter gv should be final. | 216 |
|
'{' should be on the previous line. | 217 |
|
'if' construct must use '{}'s. | 218 |
|
Line is longer than 80 characters. | 219 |
|
'{' should be on the previous line. | 221 |
|
'if' construct must use '{}'s. | 224 |
|
'if' is not followed by whitespace. | 224 |
|
'{' should be on the previous line. | 228 |
|
Line is longer than 80 characters. | 229 |
|
'}' should be on the same line. | 230 |
|
'{' should be on the previous line. | 232 |
|
Line is longer than 80 characters. | 233 |
|
'}' should be on the same line. | 235 |
|
'else' construct must use '{}'s. | 236 |
|
Method 'getMapFromColumns' is not designed for extension - needs to be abstract, final or empty. | 240 |
|
Missing a Javadoc comment. | 240 |
|
Parameter mailServer should be final. | 240 |
|
'{' should be on the previous line. | 251 |
|
Must have at least one statement. | 251 |
|
'}' should be on the same line. | 253 |
|
'{' should be on the previous line. | 255 |
|
'{' should be on the previous line. | 261 |
|
'}' should be on the same line. | 263 |
|
'{' should be on the previous line. | 265 |
|
'}' should be on the same line. | 268 |
|
'{' should be on the previous line. | 270 |
|
Line is longer than 80 characters. | 271 |
| Violation | Message | Line |
|---|---|---|
|
Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
|
Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
|
Unused import - javax.mail.Multipart. | 9 |
|
'{' should be on the previous line. | 19 |
|
Line is longer than 80 characters. | 20 |
|
Missing a Javadoc comment. | 20 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 20 |
|
Missing a Javadoc comment. | 22 |
|
Missing a Javadoc comment. | 24 |
|
Parameter email should be final. | 24 |
|
'email' hides a field. | 24 |
|
'{' should be on the previous line. | 25 |
|
Method 'send' is not designed for extension - needs to be abstract, final or empty. | 30 |
|
Missing a Javadoc comment. | 30 |
|
'{' should be on the previous line. | 31 |
|
Line is longer than 80 characters. | 34 |
|
'{' should be on the previous line. | 37 |
|
Line is longer than 80 characters. | 38 |
|
'{' should be on the previous line. | 44 |
|
'if' construct must use '{}'s. | 46 |
|
'if' construct must use '{}'s. | 48 |
|
'}' should be on the same line. | 49 |
|
Line contains a tab character. | 50 |
|
Line contains a tab character. | 51 |
|
'{' should be on the previous line. | 51 |
|
Line is longer than 80 characters. | 52 |
|
Line contains a tab character. | 52 |
|
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 56 |
|
Missing a Javadoc comment. | 56 |
|
'{' should be on the previous line. | 57 |
|
Avoid inline conditionals. | 58 |
| Violation | Message | Line |
|---|---|---|
|
File does not end with a newline. | 0 |
|
Unused import - com.atlassian.mail.server.SMTPMailServer. | 12 |
|
Unused import - javax.mail.Multipart. | 14 |
|
Missing a Javadoc comment. | 16 |
|
Utility classes should not have a public or default constructor. | 16 |
|
'{' should be on the previous line. | 17 |
|
Line is longer than 80 characters. | 18 |
|
Missing a Javadoc comment. | 18 |
|
Missing a Javadoc comment. | 20 |
|
Missing a Javadoc comment. | 22 |
|
'{' should be on the previous line. | 23 |
|
Missing a Javadoc comment. | 27 |
|
'{' should be on the previous line. | 28 |
|
'{' should be on the previous line. | 30 |
|
'{' should be on the previous line. | 32 |
|
The double-checked locking idiom is broken and should be avoided. | 33 |
|
'{' should be on the previous line. | 34 |
|
Missing a Javadoc comment. | 43 |
|
Parameter serverManager should be final. | 43 |
|
'serverManager' hides a field. | 43 |
|
'{' should be on the previous line. | 44 |
|
Missing a Javadoc comment. | 48 |
|
'{' should be on the previous line. | 49 |
