Project Documentation
The following document contains the results of Checkstyle. 
| Files | Infos  | Warnings  | Errors  |
|---|---|---|---|
| 22 | 0 | 0 | 1381 |
| Rules | Violations | Severity |
|---|---|---|
| PackageHtml | 3 | Error |
| NewlineAtEndOfFile | 0 | Error |
| Translation | 0 | Error |
| JavadocMethod | 262 | Error |
| JavadocType | 7 | Error |
| JavadocVariable | 30 | Error |
| JavadocStyle | 17 | Error |
| ConstantName | 2 | Error |
| LocalFinalVariableName | 0 | Error |
| LocalVariableName | 2 | Error |
| MemberName | 0 | Error |
| MethodName | 0 | Error |
| PackageName | 0 | Error |
| ParameterName | 2 | Error |
| StaticVariableName | 0 | Error |
| TypeName | 2 | Error |
| AvoidStarImport | 5 | Error |
| IllegalImport | 0 | Error |
| RedundantImport | 0 | Error |
| UnusedImports | 3 | Error |
| FileLength | 2 | Error |
| LineLength | 85 | Error |
| MethodLength | 0 | Error |
| ParameterNumber | 1 | Error |
| EmptyForIteratorPad | 0 | Error |
| MethodParamPad | 0 | Error |
| NoWhitespaceAfter | 0 | Error |
| NoWhitespaceBefore | 0 | Error |
| OperatorWrap | 0 | Error |
| ParenPad | 0 | Error |
| TypecastParenPad | 0 | Error |
| TabCharacter | 0 | Error |
| WhitespaceAfter | 0 | Error |
| WhitespaceAround | 0 | Error |
| ModifierOrder | 0 | Error |
| RedundantModifier | 0 | Error |
| AvoidNestedBlocks | 0 | Error |
| EmptyBlock | 0 | Error |
| LeftCurly | 0 | Error |
| NeedBraces | 35 | Error |
| RightCurly | 0 | Error |
| AvoidInlineConditionals | 7 | Error |
| DoubleCheckedLocking | 0 | Error |
| EmptyStatement | 0 | Error |
| EqualsHashCode | 0 | Error |
| HiddenField | 101 | Error |
| IllegalInstantiation | 0 | Error |
| InnerAssignment | 0 | Error |
| MagicNumber | 20 | Error |
| MissingSwitchDefault | 0 | Error |
| RedundantThrows | 4 | Error |
| SimplifyBooleanExpression | 0 | Error |
| SimplifyBooleanReturn | 0 | Error |
| DesignForExtension | 291 | Error |
| FinalClass | 0 | Error |
| HideUtilityClassConstructor | 0 | Error |
| InterfaceIsType | 0 | Error |
| VisibilityModifier | 2 | Error |
| ArrayTypeStyle | 0 | Error |
| FinalParameters | 306 | Error |
GenericIllegalRegexp
| 192 | Error |
| TodoComment | 0 | Error |
| UpperEll | 0 | Error |
| Violation | Message | Line |
|---|---|---|
| Unused import - java.util.Date. | 20 |
| Line has trailing spaces. | 25 |
| Line has trailing spaces. | 26 |
| Line has trailing spaces. | 29 |
| Line has trailing spaces. | 43 |
| Parameter id should be final. | 46 |
| Expected @param tag for 'id'. | 46 |
| Missing a Javadoc comment. | 52 |
| More than 7 parameters. | 52 |
| Parameter id should be final. | 52 |
| Parameter mimeType should be final. | 52 |
| Parameter title should be final. | 53 |
| Parameter description should be final. | 53 |
| Parameter keywords should be final. | 54 |
| Parameter imageLicense should be final. | 54 |
| Parameter imageSource should be final. | 55 |
| Parameter thumbnailUri should be final. | 55 |
| Parameter mediumImageUri should be final. | 56 |
| Parameter largeImageUri should be final. | 56 |
| Parameter creator should be final. | 57 |
| Method 'addCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 71 |
| Missing a Javadoc comment. | 71 |
| Parameter courseCollectionItem should be final. | 72 |
| Method 'removeCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 80 |
| Missing a Javadoc comment. | 80 |
| Parameter courseCollectionItem should be final. | 81 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 |
| Line has trailing spaces. | 27 |
| Line has trailing spaces. | 41 |
| Parameter id should be final. | 44 |
| Expected @param tag for 'id'. | 44 |
| Missing a Javadoc comment. | 50 |
| Parameter sitehandle should be final. | 50 |
| Missing a Javadoc comment. | 55 |
| Parameter sitehandle should be final. | 55 |
| Parameter siteservice should be final. | 55 |
| Method 'addCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 61 |
| Missing a Javadoc comment. | 61 |
| Parameter courseCollection should be final. | 61 |
| Method 'removeCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 69 |
| Missing a Javadoc comment. | 69 |
| Parameter courseCollection should be final. | 69 |
| Violation | Message | Line |
|---|---|---|
| Unused import - java.util.List. | 20 |
| Using the '.*' form of import should be avoided - edu.mit.stellar.hibernate.*. | 22 |
| Missing a Javadoc comment. | 24 |
| Missing a Javadoc comment. | 26 |
| Missing a Javadoc comment. | 28 |
| Missing a Javadoc comment. | 31 |
| Missing a Javadoc comment. | 34 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 |
| Line has trailing spaces. | 63 |
| Parameter id should be final. | 66 |
| 'id' hides a field. | 66 |
| Expected @param tag for 'id'. | 66 |
| Line has trailing spaces. | 72 |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 75 |
| Line has trailing spaces. | 81 |
| Method 'setId' is not designed for extension - needs to be abstract, final or empty. | 84 |
| Parameter id should be final. | 84 |
| 'id' hides a field. | 84 |
| Expected @param tag for 'id'. | 84 |
| Line has trailing spaces. | 91 |
| Method 'getCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 94 |
| Line has trailing spaces. | 100 |
| Method 'setCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 103 |
| Parameter courseCollectionItem should be final. | 104 |
| 'courseCollectionItem' hides a field. | 104 |
| Expected @param tag for 'courseCollectionItem'. | 104 |
| Line has trailing spaces. | 110 |
| Method 'getTitle' is not designed for extension - needs to be abstract, final or empty. | 113 |
| Line has trailing spaces. | 119 |
| Method 'setTitle' is not designed for extension - needs to be abstract, final or empty. | 122 |
| Parameter title should be final. | 122 |
| 'title' hides a field. | 122 |
| Expected @param tag for 'title'. | 122 |
| Line has trailing spaces. | 128 |
| Method 'getUser' is not designed for extension - needs to be abstract, final or empty. | 131 |
| Line has trailing spaces. | 137 |
| Method 'setUser' is not designed for extension - needs to be abstract, final or empty. | 140 |
| Parameter user should be final. | 140 |
| 'user' hides a field. | 140 |
| Expected @param tag for 'user'. | 140 |
| Line has trailing spaces. | 146 |
| Method 'getComments' is not designed for extension - needs to be abstract, final or empty. | 149 |
| Line has trailing spaces. | 155 |
| Method 'setComments' is not designed for extension - needs to be abstract, final or empty. | 158 |
| Parameter comments should be final. | 158 |
| 'comments' hides a field. | 158 |
| Expected @param tag for 'comments'. | 158 |
| Line has trailing spaces. | 164 |
| Method 'getCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 167 |
| Line has trailing spaces. | 173 |
| Method 'setCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 176 |
| Parameter createdOn should be final. | 176 |
| 'createdOn' hides a field. | 176 |
| Expected @param tag for 'createdOn'. | 176 |
| Line has trailing spaces. | 183 |
| Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 187 |
| Parameter rhs should be final. | 187 |
| Expected @param tag for 'rhs'. | 187 |
| 'if' construct must use '{}'s. | 188 |
| 'if' construct must use '{}'s. | 190 |
| 'if' construct must use '{}'s. | 193 |
| Line has trailing spaces. | 202 |
| Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 205 |
| '17' is a magic number. | 207 |
| Avoid inline conditionals. | 208 |
| '37' is a magic number. | 209 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 |
| Line has trailing spaces. | 27 |
| Line has trailing spaces. | 41 |
| Parameter id should be final. | 44 |
| Expected @param tag for 'id'. | 44 |
| Missing a Javadoc comment. | 50 |
| Parameter name should be final. | 50 |
| Parameter domain should be final. | 50 |
| Missing a Javadoc comment. | 56 |
| Parameter name should be final. | 56 |
| Parameter domain should be final. | 56 |
| Parameter userservice should be final. | 56 |
| Method 'addCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 63 |
| Missing a Javadoc comment. | 63 |
| Parameter courseCollectionItem should be final. | 64 |
| Method 'removeCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 72 |
| Missing a Javadoc comment. | 72 |
| Parameter courseCollectionItem should be final. | 73 |
| Method 'addCourseCollectionAnnotation' is not designed for extension - needs to be abstract, final or empty. | 79 |
| Missing a Javadoc comment. | 79 |
| Parameter courseCollectionAnnotation should be final. | 80 |
| Method 'removeCourseCollectionAnnotation' is not designed for extension - needs to be abstract, final or empty. | 88 |
| Missing a Javadoc comment. | 88 |
| Parameter courseCollectionAnnotation should be final. | 89 |
| Method 'addItemAnnotation' is not designed for extension - needs to be abstract, final or empty. | 96 |
| Missing a Javadoc comment. | 96 |
| Parameter itemAnnotation should be final. | 96 |
| Method 'removeItemAnnotation' is not designed for extension - needs to be abstract, final or empty. | 104 |
| Missing a Javadoc comment. | 104 |
| Parameter itemAnnotation should be final. | 104 |
| Violation | Message | Line |
|---|---|---|
| Unused import - java.sql.Types. | 20 |
| Line has trailing spaces. | 25 |
| Line has trailing spaces. | 29 |
| Line has trailing spaces. | 32 |
| Line has trailing spaces. | 47 |
| Parameter id should be final. | 50 |
| Expected @param tag for 'id'. | 50 |
| Missing a Javadoc comment. | 56 |
| Parameter name should be final. | 56 |
| Parameter description should be final. | 56 |
| Parameter nexusInfo should be final. | 57 |
| Parameter type should be final. | 57 |
| Method 'addCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 67 |
| Missing a Javadoc comment. | 67 |
| Parameter courseCollectionItem should be final. | 68 |
| Method 'removeCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 76 |
| Missing a Javadoc comment. | 76 |
| Parameter courseCollectionItem should be final. | 77 |
| Method 'addCourseCollectionPage' is not designed for extension - needs to be abstract, final or empty. | 84 |
| Missing a Javadoc comment. | 84 |
| Parameter courseCollectionPage should be final. | 85 |
| Method 'removeCourseCollectionPage' is not designed for extension - needs to be abstract, final or empty. | 93 |
| Missing a Javadoc comment. | 93 |
| Parameter courseCollectionPage should be final. | 94 |
| Method 'addCourseCollectionAnnotation' is not designed for extension - needs to be abstract, final or empty. | 101 |
| Missing a Javadoc comment. | 101 |
| Parameter courseCollectionAnnotation should be final. | 102 |
| Method 'removeCourseCollectionAnnotation' is not designed for extension - needs to be abstract, final or empty. | 110 |
| Missing a Javadoc comment. | 110 |
| Parameter courseCollectionAnnotation should be final. | 111 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 25 |
| Line has trailing spaces. | 65 |
| Parameter id should be final. | 68 |
| 'id' hides a field. | 68 |
| Expected @param tag for 'id'. | 68 |
| Line has trailing spaces. | 74 |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 77 |
| Line has trailing spaces. | 83 |
| Method 'setId' is not designed for extension - needs to be abstract, final or empty. | 86 |
| Parameter id should be final. | 86 |
| 'id' hides a field. | 86 |
| Expected @param tag for 'id'. | 86 |
| Line has trailing spaces. | 93 |
| Method 'getCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 96 |
| Line has trailing spaces. | 102 |
| Method 'setCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 105 |
| Parameter courseCollection should be final. | 105 |
| 'courseCollection' hides a field. | 105 |
| Expected @param tag for 'courseCollection'. | 105 |
| Line has trailing spaces. | 111 |
| Method 'getTitle' is not designed for extension - needs to be abstract, final or empty. | 114 |
| Line has trailing spaces. | 120 |
| Method 'setTitle' is not designed for extension - needs to be abstract, final or empty. | 123 |
| Parameter title should be final. | 123 |
| 'title' hides a field. | 123 |
| Expected @param tag for 'title'. | 123 |
| Line has trailing spaces. | 129 |
| Method 'getUser' is not designed for extension - needs to be abstract, final or empty. | 132 |
| Line has trailing spaces. | 138 |
| Method 'setUser' is not designed for extension - needs to be abstract, final or empty. | 141 |
| Parameter user should be final. | 141 |
| 'user' hides a field. | 141 |
| Expected @param tag for 'user'. | 141 |
| Line has trailing spaces. | 147 |
| Method 'getComments' is not designed for extension - needs to be abstract, final or empty. | 150 |
| Line has trailing spaces. | 156 |
| Method 'setComments' is not designed for extension - needs to be abstract, final or empty. | 159 |
| Parameter comments should be final. | 159 |
| 'comments' hides a field. | 159 |
| Expected @param tag for 'comments'. | 159 |
| Line has trailing spaces. | 165 |
| Method 'getCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 168 |
| Line has trailing spaces. | 174 |
| Method 'setCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 177 |
| Parameter createdOn should be final. | 177 |
| 'createdOn' hides a field. | 177 |
| Expected @param tag for 'createdOn'. | 177 |
| Line has trailing spaces. | 184 |
| Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 188 |
| Parameter rhs should be final. | 188 |
| Expected @param tag for 'rhs'. | 188 |
| 'if' construct must use '{}'s. | 189 |
| 'if' construct must use '{}'s. | 191 |
| 'if' construct must use '{}'s. | 194 |
| Line has trailing spaces. | 203 |
| Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 206 |
| '17' is a magic number. | 208 |
| Avoid inline conditionals. | 209 |
| '37' is a magic number. | 210 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 |
| Line has trailing spaces. | 39 |
| Parameter id should be final. | 42 |
| Expected @param tag for 'id'. | 42 |
| Line has trailing spaces. | 49 |
| Parameter id should be final. | 53 |
| Expected @param tag for 'id'. | 53 |
| Parameter courseCollection should be final. | 54 |
| Expected @param tag for 'courseCollection'. | 54 |
| First sentence should end with a period. | 58 |
| Parameter id should be final. | 59 |
| Expected @param tag for 'id'. | 59 |
| Parameter courseCollection should be final. | 60 |
| Expected @param tag for 'courseCollection'. | 60 |
| Parameter pageTemplate should be final. | 60 |
| Expected @param tag for 'pageTemplate'. | 60 |
| Parameter pageOrder should be final. | 61 |
| Expected @param tag for 'pageOrder'. | 61 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 |
| Line has trailing spaces. | 29 |
| Line has trailing spaces. | 30 |
| Line has trailing spaces. | 31 |
| Line has trailing spaces. | 46 |
| Parameter id should be final. | 49 |
| Expected @param tag for 'id'. | 49 |
| Missing a Javadoc comment. | 55 |
| Parameter user should be final. | 55 |
| Parameter courseCollection should be final. | 55 |
| Parameter item should be final. | 56 |
| Line is longer than 80 characters. | 66 |
| Method 'updateCourseColletionItemInfo' is not designed for extension - needs to be abstract, final or empty. | 66 |
| Missing a Javadoc comment. | 66 |
| Parameter title should be final. | 66 |
| Parameter description should be final. | 66 |
| Method 'addItemAnnotation' is not designed for extension - needs to be abstract, final or empty. | 71 |
| Missing a Javadoc comment. | 71 |
| Parameter itemAnnotation should be final. | 71 |
| Method 'removeItemAnnotation' is not designed for extension - needs to be abstract, final or empty. | 79 |
| Missing a Javadoc comment. | 79 |
| Parameter itemAnnotation should be final. | 79 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 |
| Line has trailing spaces. | 59 |
| Parameter id should be final. | 62 |
| 'id' hides a field. | 62 |
| Expected @param tag for 'id'. | 62 |
| Line has trailing spaces. | 68 |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 71 |
| Line has trailing spaces. | 77 |
| Method 'setId' is not designed for extension - needs to be abstract, final or empty. | 80 |
| Parameter id should be final. | 80 |
| 'id' hides a field. | 80 |
| Expected @param tag for 'id'. | 80 |
| Line has trailing spaces. | 87 |
| Method 'getSiteHandle' is not designed for extension - needs to be abstract, final or empty. | 90 |
| Line has trailing spaces. | 96 |
| Method 'setSiteHandle' is not designed for extension - needs to be abstract, final or empty. | 99 |
| Parameter siteHandle should be final. | 99 |
| 'siteHandle' hides a field. | 99 |
| Expected @param tag for 'siteHandle'. | 99 |
| Line has trailing spaces. | 105 |
| Method 'getSiteService' is not designed for extension - needs to be abstract, final or empty. | 108 |
| Line has trailing spaces. | 114 |
| Method 'setSiteService' is not designed for extension - needs to be abstract, final or empty. | 117 |
| Parameter siteService should be final. | 117 |
| 'siteService' hides a field. | 117 |
| Expected @param tag for 'siteService'. | 117 |
| Line has trailing spaces. | 123 |
| Method 'getCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 126 |
| Line has trailing spaces. | 132 |
| Method 'setCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 135 |
| Parameter createdOn should be final. | 135 |
| 'createdOn' hides a field. | 135 |
| Expected @param tag for 'createdOn'. | 135 |
| Line has trailing spaces. | 141 |
| Method 'getCourseCollectionSet' is not designed for extension - needs to be abstract, final or empty. | 144 |
| Line has trailing spaces. | 150 |
| Method 'setCourseCollectionSet' is not designed for extension - needs to be abstract, final or empty. | 153 |
| Parameter courseCollectionSet should be final. | 153 |
| 'courseCollectionSet' hides a field. | 153 |
| Expected @param tag for 'courseCollectionSet'. | 153 |
| Line has trailing spaces. | 160 |
| Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 164 |
| Parameter rhs should be final. | 164 |
| Expected @param tag for 'rhs'. | 164 |
| 'if' construct must use '{}'s. | 165 |
| 'if' construct must use '{}'s. | 167 |
| 'if' construct must use '{}'s. | 170 |
| Line has trailing spaces. | 179 |
| Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 182 |
| '17' is a magic number. | 184 |
| Avoid inline conditionals. | 185 |
| '37' is a magic number. | 186 |
| Violation | Message | Line |
|---|---|---|
| File length is 2,447 lines (max allowed is 2,000). | 1 |
| Using the '.*' form of import should be avoided - edu.mit.stellar.hibernate.*. | 34 |
| Using the '.*' form of import should be avoided - org.hibernate.*. | 36 |
| Line has trailing spaces. | 40 |
| First sentence should end with a period. | 48 |
| Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 51 |
| Missing a Javadoc comment. | 56 |
| Name 'lectureByDate' must match pattern '^[A-Z][a-zA-Z0-9]*$'. | 56 |
| Method 'compare' is not designed for extension - needs to be abstract, final or empty. | 58 |
| Missing a Javadoc comment. | 58 |
| Parameter arg0 should be final. | 58 |
| Parameter arg1 should be final. | 58 |
| Missing a Javadoc comment. | 70 |
| Missing a Javadoc comment. | 72 |
| Missing a Javadoc comment. | 74 |
| Missing a Javadoc comment. | 76 |
| Method 'setImages' is not designed for extension - needs to be abstract, final or empty. | 80 |
| Missing a Javadoc comment. | 80 |
| Parameter mItem should be final. | 80 |
| Method 'setImage' is not designed for extension - needs to be abstract, final or empty. | 86 |
| Missing a Javadoc comment. | 86 |
| Parameter mItem should be final. | 86 |
| Method 'setPage' is not designed for extension - needs to be abstract, final or empty. | 90 |
| Missing a Javadoc comment. | 90 |
| Parameter mPage should be final. | 90 |
| Method 'getImages' is not designed for extension - needs to be abstract, final or empty. | 94 |
| Missing a Javadoc comment. | 94 |
| Method 'getPage' is not designed for extension - needs to be abstract, final or empty. | 98 |
| Missing a Javadoc comment. | 98 |
| Missing a Javadoc comment. | 103 |
| Missing a Javadoc comment. | 105 |
| Missing a Javadoc comment. | 107 |
| Missing a Javadoc comment. | 109 |
| Missing a Javadoc comment. | 111 |
| Line is longer than 80 characters. | 114 |
| Missing a Javadoc comment. | 114 |
| Parameter mLP should be final. | 114 |
| Parameter mOldOrder should be final. | 114 |
| Parameter mNewOrder should be final. | 114 |
| Method 'setLecturePage' is not designed for extension - needs to be abstract, final or empty. | 120 |
| Missing a Javadoc comment. | 120 |
| Parameter mLP should be final. | 120 |
| Method 'setOldOrder' is not designed for extension - needs to be abstract, final or empty. | 124 |
| Missing a Javadoc comment. | 124 |
| Parameter mOldOrder should be final. | 124 |
| Method 'setNewOrder' is not designed for extension - needs to be abstract, final or empty. | 128 |
| Missing a Javadoc comment. | 128 |
| Parameter mNewOrder should be final. | 128 |
| Method 'getNewOrder' is not designed for extension - needs to be abstract, final or empty. | 132 |
| Missing a Javadoc comment. | 132 |
| Method 'getOldOrder' is not designed for extension - needs to be abstract, final or empty. | 136 |
| Missing a Javadoc comment. | 136 |
| Method 'getPage' is not designed for extension - needs to be abstract, final or empty. | 140 |
| Missing a Javadoc comment. | 140 |
| Method 'compareTo' is not designed for extension - needs to be abstract, final or empty. | 144 |
| Missing a Javadoc comment. | 144 |
| Parameter other_o should be final. | 144 |
| Name 'other_o' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 144 |
| Redundant throws: 'ClassCastException' is unchecked exception. | 144 |
| 'if' construct must use '{}'s. | 147 |
| 'if' construct must use '{}'s. | 149 |
| 'if' construct must use '{}'s. | 152 |
| 'if' construct must use '{}'s. | 154 |
| 'if' construct must use '{}'s. | 156 |
| 'if' construct must use '{}'s. | 158 |
| 'else' construct must use '{}'s. | 160 |
| Method 'getCourseLecturePage' is not designed for extension - needs to be abstract, final or empty. | 167 |
| Missing a Javadoc comment. | 167 |
| Line is longer than 80 characters. | 171 |
| Missing a Javadoc comment. | 171 |
| Missing a Javadoc comment. | 173 |
| Expected an @return tag. | 178 |
| Method 'getDownloadJournal' is not designed for extension - needs to be abstract, final or empty. | 178 |
| Method 'setDownloadJournal' is not designed for extension - needs to be abstract, final or empty. | 188 |
| Parameter newDownloadJournal should be final. | 188 |
| Expected @param tag for 'newDownloadJournal'. | 188 |
| Missing a Javadoc comment. | 214 |
| Variable 'mySessionFactory' must be private and have accessor methods. | 214 |
| Method 'getMySessionFactory' is not designed for extension - needs to be abstract, final or empty. | 216 |
| Missing a Javadoc comment. | 216 |
| Method 'setMySessionFactory' is not designed for extension - needs to be abstract, final or empty. | 222 |
| Missing a Javadoc comment. | 222 |
| Parameter sesf should be final. | 222 |
| Missing a Javadoc comment. | 232 |
| Method 'setUserName' is not designed for extension - needs to be abstract, final or empty. | 234 |
| Missing a Javadoc comment. | 234 |
| Parameter userName should be final. | 234 |
| 'userName' hides a field. | 234 |
| Method 'getUserName' is not designed for extension - needs to be abstract, final or empty. | 241 |
| First sentence should end with a period. | 245 |
| Expected an @return tag. | 248 |
| Method 'getUser' is not designed for extension - needs to be abstract, final or empty. | 248 |
| Missing a Javadoc comment. | 256 |
| Method 'getSiteHandle' is not designed for extension - needs to be abstract, final or empty. | 258 |
| Missing a Javadoc comment. | 258 |
| First sentence should end with a period. | 262 |
| Method 'setSiteHandle' is not designed for extension - needs to be abstract, final or empty. | 265 |
| Parameter sitehandle should be final. | 265 |
| Expected @param tag for 'sitehandle'. | 265 |
| Method 'getLectureId' is not designed for extension - needs to be abstract, final or empty. | 269 |
| Missing a Javadoc comment. | 269 |
| Missing a Javadoc comment. | 273 |
| First sentence should end with a period. | 275 |
| Method 'setLectureId' is not designed for extension - needs to be abstract, final or empty. | 278 |
| Parameter lectureId should be final. | 278 |
| 'lectureId' hides a field. | 278 |
| Expected @param tag for 'lectureId'. | 278 |
| Missing a Javadoc comment. | 282 |
| Method 'getImageItemInfo' is not designed for extension - needs to be abstract, final or empty. | 284 |
| Missing a Javadoc comment. | 284 |
| Method 'setImageItemInfo' is not designed for extension - needs to be abstract, final or empty. | 288 |
| Missing a Javadoc comment. | 288 |
| Parameter imageItemInfo should be final. | 288 |
| 'imageItemInfo' hides a field. | 288 |
| Missing a Javadoc comment. | 292 |
| Method 'getCourseCollectionItemId' is not designed for extension - needs to be abstract, final or empty. | 297 |
| Method 'setCourseCollectionItemId' is not designed for extension - needs to be abstract, final or empty. | 301 |
| Missing a Javadoc comment. | 301 |
| Parameter crsCollectionItemId should be final. | 301 |
| Missing a Javadoc comment. | 308 |
| Missing a Javadoc comment. | 310 |
| Missing a Javadoc comment. | 312 |
| Method 'createUser' is not designed for extension - needs to be abstract, final or empty. | 326 |
| Missing a Javadoc comment. | 326 |
| Parameter username should be final. | 326 |
| Parameter domain should be final. | 326 |
| Parameter userservice should be final. | 326 |
| Line is longer than 80 characters. | 337 |
| Method 'getUserByUserName' is not designed for extension - needs to be abstract, final or empty. | 376 |
| Missing a Javadoc comment. | 376 |
| Parameter username should be final. | 376 |
| Line is longer than 80 characters. | 390 |
| Method 'createNexusInfo' is not designed for extension - needs to be abstract, final or empty. | 413 |
| Missing a Javadoc comment. | 413 |
| Parameter siteHandle should be final. | 413 |
| 'siteHandle' hides a field. | 413 |
| Parameter siteService should be final. | 413 |
| Line is longer than 80 characters. | 423 |
| Method 'getNexusInfoBySiteHandle' is not designed for extension - needs to be abstract, final or empty. | 461 |
| Missing a Javadoc comment. | 461 |
| Parameter siteHandle should be final. | 461 |
| 'siteHandle' hides a field. | 461 |
| Line is longer than 80 characters. | 472 |
| Method 'createCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 505 |
| Missing a Javadoc comment. | 505 |
| Parameter name should be final. | 505 |
| Parameter description should be final. | 506 |
| Parameter siteHandle should be final. | 506 |
| 'siteHandle' hides a field. | 506 |
| Parameter type should be final. | 506 |
| Line is longer than 80 characters. | 516 |
| Method 'getCourseCollectionBySiteHandle' is not designed for extension - needs to be abstract, final or empty. | 574 |
| Missing a Javadoc comment. | 574 |
| Parameter sitehandle should be final. | 574 |
| Method 'getCourseCollectionById' is not designed for extension - needs to be abstract, final or empty. | 611 |
| Missing a Javadoc comment. | 611 |
| Parameter id should be final. | 611 |
| Line is longer than 80 characters. | 623 |
| Line is longer than 80 characters. | 649 |
| Method 'getCourseCollectionLectureVector' is not designed for extension - needs to be abstract, final or empty. | 649 |
| Parameter siteHandle should be final. | 649 |
| 'siteHandle' hides a field. | 649 |
| Expected @param tag for 'siteHandle'. | 649 |
| Line is longer than 80 characters. | 661 |
| Method 'createCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 707 |
| Missing a Javadoc comment. | 707 |
| Parameter courseCollection should be final. | 708 |
| Parameter item should be final. | 708 |
| Parameter user should be final. | 708 |
| Line is longer than 80 characters. | 719 |
| Method 'getCourseCollectionItemById' is not designed for extension - needs to be abstract, final or empty. | 753 |
| Missing a Javadoc comment. | 753 |
| Parameter id should be final. | 753 |
| Line is longer than 80 characters. | 765 |
| Line has trailing spaces. | 790 |
| Method 'getCourseCollectionItemVector' is not designed for extension - needs to be abstract, final or empty. | 793 |
| Parameter siteHandle should be final. | 793 |
| 'siteHandle' hides a field. | 793 |
| Expected @param tag for 'siteHandle'. | 793 |
| Line is longer than 80 characters. | 814 |
| Line is longer than 80 characters. | 815 |
| Line has trailing spaces. | 873 |
| Line is longer than 80 characters. | 876 |
| Method 'getCourseCollectionLectureItemsVector' is not designed for extension - needs to be abstract, final or empty. | 876 |
| Parameter lectureId should be final. | 876 |
| 'lectureId' hides a field. | 876 |
| Expected @param tag for 'lectureId'. | 876 |
| Line is longer than 80 characters. | 900 |
| Line is longer than 80 characters. | 902 |
| Line has trailing spaces. | 963 |
| Method 'addImageToCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 965 |
| Missing a Javadoc comment. | 965 |
| Parameter imageInfo should be final. | 965 |
| Parameter username should be final. | 966 |
| Parameter siteHandle should be final. | 966 |
| 'siteHandle' hides a field. | 966 |
| Line is longer than 80 characters. | 983 |
| Line is longer than 80 characters. | 996 |
| Line is longer than 80 characters. | 1081 |
| Method 'addImageToLecture' is not designed for extension - needs to be abstract, final or empty. | 1090 |
| Missing a Javadoc comment. | 1090 |
| Parameter imageId should be final. | 1090 |
| Parameter lectureId should be final. | 1090 |
| 'lectureId' hides a field. | 1090 |
| Line is longer than 80 characters. | 1104 |
| Line is longer than 80 characters. | 1115 |
| Line is longer than 80 characters. | 1121 |
| Line is longer than 80 characters. | 1184 |
| Line is longer than 80 characters. | 1194 |
| Method 'updateCouseCollectionPageTemplate' is not designed for extension - needs to be abstract, final or empty. | 1206 |
| Missing a Javadoc comment. | 1206 |
| Parameter crlPageTemplate should be final. | 1206 |
| Parameter crlLectureId should be final. | 1207 |
| Parameter crlPageId should be final. | 1207 |
| Line is longer than 80 characters. | 1229 |
| Line is longer than 80 characters. | 1263 |
| Method 'updateCourseCollectionItemTitle' is not designed for extension - needs to be abstract, final or empty. | 1321 |
| Missing a Javadoc comment. | 1321 |
| Parameter crlItem should be final. | 1321 |
| Parameter title should be final. | 1322 |
| Method 'changeCourseCollectionTitle' is not designed for extension - needs to be abstract, final or empty. | 1367 |
| Missing a Javadoc comment. | 1367 |
| Parameter crlLectureId should be final. | 1367 |
| Parameter title should be final. | 1368 |
| Parameter date should be final. | 1368 |
| Method 'changeCourseCollectionItemTitle' is not designed for extension - needs to be abstract, final or empty. | 1424 |
| Missing a Javadoc comment. | 1424 |
| Parameter crlItemId should be final. | 1424 |
| Parameter title should be final. | 1425 |
| Method 'updateCourseCollectionItemOrder' is not designed for extension - needs to be abstract, final or empty. | 1500 |
| Missing a Javadoc comment. | 1500 |
| Parameter crlItemId should be final. | 1500 |
| Parameter order should be final. | 1501 |
| Line is longer than 80 characters. | 1545 |
| Method 'updateCourseCollectionPageOrder' is not designed for extension - needs to be abstract, final or empty. | 1549 |
| Missing a Javadoc comment. | 1549 |
| Parameter crlPageId should be final. | 1549 |
| Parameter order should be final. | 1550 |
| Line is longer than 80 characters. | 1602 |
| Method 'updateCourseCollectionItemDescription' is not designed for extension - needs to be abstract, final or empty. | 1606 |
| Missing a Javadoc comment. | 1606 |
| Parameter crlItem should be final. | 1607 |
| Parameter description should be final. | 1607 |
| Method 'changeCourseCollectionItemDescription' is not designed for extension - needs to be abstract, final or empty. | 1651 |
| Missing a Javadoc comment. | 1651 |
| Parameter crlItemId should be final. | 1651 |
| Parameter description should be final. | 1652 |
| Line is longer than 80 characters. | 1699 |
| Method 'deleteLectureFromCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 1717 |
| Missing a Javadoc comment. | 1717 |
| Parameter crlLectureId should be final. | 1717 |
| Line is longer than 80 characters. | 1740 |
| Line is longer than 80 characters. | 1746 |
| Method 'deleteImageFromLecture' is not designed for extension - needs to be abstract, final or empty. | 1801 |
| Missing a Javadoc comment. | 1801 |
| Parameter crlItemId should be final. | 1801 |
| Parameter crlPageId should be final. | 1801 |
| Method 'deleteImageFromCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 1899 |
| Missing a Javadoc comment. | 1899 |
| Parameter crlItemId should be final. | 1899 |
| Parameter baseDir should be final. | 1900 |
| Method 'deleteImageFromCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 1953 |
| Missing a Javadoc comment. | 1953 |
| Parameter crlItemId should be final. | 1953 |
| Line is longer than 80 characters. | 2000 |
| Method 'getLecturePageSet' is not designed for extension - needs to be abstract, final or empty. | 2048 |
| Line has trailing spaces. | 2058 |
| Method 'getLecturePageVector' is not designed for extension - needs to be abstract, final or empty. | 2061 |
| Parameter lectureId should be final. | 2061 |
| 'lectureId' hides a field. | 2061 |
| Expected @param tag for 'lectureId'. | 2061 |
| Line is longer than 80 characters. | 2084 |
| Line is longer than 80 characters. | 2095 |
| First sentence should end with a period. | 2196 |
| Expected an @return tag. | 2199 |
| Method 'getCourseCollectionItemSet' is not designed for extension - needs to be abstract, final or empty. | 2199 |
| First sentence should end with a period. | 2208 |
| Expected an @return tag. | 2211 |
| Method 'getCourseCollectionLectureSet' is not designed for extension - needs to be abstract, final or empty. | 2211 |
| First sentence should end with a period. | 2220 |
| Expected an @return tag. | 2223 |
| Method 'getCourseCollectionLectureItemsSet' is not designed for extension - needs to be abstract, final or empty. | 2223 |
| Method 'isImageSaved' is not designed for extension - needs to be abstract, final or empty. | 2232 |
| Missing a Javadoc comment. | 2232 |
| Method 'isImageRemoved' is not designed for extension - needs to be abstract, final or empty. | 2250 |
| Missing a Javadoc comment. | 2250 |
| Method 'downloadImage' is not designed for extension - needs to be abstract, final or empty. | 2271 |
| Missing a Javadoc comment. | 2271 |
| Parameter largeImageURL should be final. | 2271 |
| Parameter imageId should be final. | 2271 |
| Parameter repositoryId should be final. | 2272 |
| Parameter baseDir should be final. | 2272 |
| '300000' is a magic number. | 2290 |
| Method 'createLocalImageFiles' is not designed for extension - needs to be abstract, final or empty. | 2311 |
| Missing a Javadoc comment. | 2311 |
| Parameter largeImageURL should be final. | 2311 |
| Parameter imageId should be final. | 2312 |
| Parameter baseDir should be final. | 2312 |
| Parameter repositoryId should be final. | 2312 |
| Parameter imageMagickDir should be final. | 2313 |
| Parameter urlBase should be final. | 2313 |
| Redundant throws: 'java.io.IOException' is subclass of 'Exception'. | 2313 |
| Name 'URL_BASE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2321 |
| '100' is a magic number. | 2351 |
| '500' is a magic number. | 2357 |
| Method 'saveLecturePageVector' is not designed for extension - needs to be abstract, final or empty. | 2371 |
| Missing a Javadoc comment. | 2371 |
| Parameter mLecturePageVector should be final. | 2371 |
| Line is longer than 80 characters. | 2388 |
| Line is longer than 80 characters. | 2393 |
| Line is longer than 80 characters. | 2396 |
| Line is longer than 80 characters. | 2399 |
| Method 'createLocalItemFromImageMap' is not designed for extension - needs to be abstract, final or empty. | 2434 |
| Missing a Javadoc comment. | 2434 |
| Parameter imageInfo should be final. | 2434 |
| Violation | Message | Line |
|---|---|---|
| File length is 2,447 lines (max allowed is 2,000). | 1 |
| Using the '.*' form of import should be avoided - edu.mit.stellar.hibernate.*. | 34 |
| Using the '.*' form of import should be avoided - org.hibernate.*. | 36 |
| Line has trailing spaces. | 40 |
| First sentence should end with a period. | 48 |
| Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 51 |
| Missing a Javadoc comment. | 56 |
| Name 'lectureByDate' must match pattern '^[A-Z][a-zA-Z0-9]*$'. | 56 |
| Method 'compare' is not designed for extension - needs to be abstract, final or empty. | 58 |
| Missing a Javadoc comment. | 58 |
| Parameter arg0 should be final. | 58 |
| Parameter arg1 should be final. | 58 |
| Missing a Javadoc comment. | 70 |
| Missing a Javadoc comment. | 72 |
| Missing a Javadoc comment. | 74 |
| Missing a Javadoc comment. | 76 |
| Method 'setImages' is not designed for extension - needs to be abstract, final or empty. | 80 |
| Missing a Javadoc comment. | 80 |
| Parameter mItem should be final. | 80 |
| Method 'setImage' is not designed for extension - needs to be abstract, final or empty. | 86 |
| Missing a Javadoc comment. | 86 |
| Parameter mItem should be final. | 86 |
| Method 'setPage' is not designed for extension - needs to be abstract, final or empty. | 90 |
| Missing a Javadoc comment. | 90 |
| Parameter mPage should be final. | 90 |
| Method 'getImages' is not designed for extension - needs to be abstract, final or empty. | 94 |
| Missing a Javadoc comment. | 94 |
| Method 'getPage' is not designed for extension - needs to be abstract, final or empty. | 98 |
| Missing a Javadoc comment. | 98 |
| Missing a Javadoc comment. | 103 |
| Missing a Javadoc comment. | 105 |
| Missing a Javadoc comment. | 107 |
| Missing a Javadoc comment. | 109 |
| Missing a Javadoc comment. | 111 |
| Line is longer than 80 characters. | 114 |
| Missing a Javadoc comment. | 114 |
| Parameter mLP should be final. | 114 |
| Parameter mOldOrder should be final. | 114 |
| Parameter mNewOrder should be final. | 114 |
| Method 'setLecturePage' is not designed for extension - needs to be abstract, final or empty. | 120 |
| Missing a Javadoc comment. | 120 |
| Parameter mLP should be final. | 120 |
| Method 'setOldOrder' is not designed for extension - needs to be abstract, final or empty. | 124 |
| Missing a Javadoc comment. | 124 |
| Parameter mOldOrder should be final. | 124 |
| Method 'setNewOrder' is not designed for extension - needs to be abstract, final or empty. | 128 |
| Missing a Javadoc comment. | 128 |
| Parameter mNewOrder should be final. | 128 |
| Method 'getNewOrder' is not designed for extension - needs to be abstract, final or empty. | 132 |
| Missing a Javadoc comment. | 132 |
| Method 'getOldOrder' is not designed for extension - needs to be abstract, final or empty. | 136 |
| Missing a Javadoc comment. | 136 |
| Method 'getPage' is not designed for extension - needs to be abstract, final or empty. | 140 |
| Missing a Javadoc comment. | 140 |
| Method 'compareTo' is not designed for extension - needs to be abstract, final or empty. | 144 |
| Missing a Javadoc comment. | 144 |
| Parameter other_o should be final. | 144 |
| Name 'other_o' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 144 |
| Redundant throws: 'ClassCastException' is unchecked exception. | 144 |
| 'if' construct must use '{}'s. | 147 |
| 'if' construct must use '{}'s. | 149 |
| 'if' construct must use '{}'s. | 152 |
| 'if' construct must use '{}'s. | 154 |
| 'if' construct must use '{}'s. | 156 |
| 'if' construct must use '{}'s. | 158 |
| 'else' construct must use '{}'s. | 160 |
| Method 'getCourseLecturePage' is not designed for extension - needs to be abstract, final or empty. | 167 |
| Missing a Javadoc comment. | 167 |
| Line is longer than 80 characters. | 171 |
| Missing a Javadoc comment. | 171 |
| Missing a Javadoc comment. | 173 |
| Expected an @return tag. | 178 |
| Method 'getDownloadJournal' is not designed for extension - needs to be abstract, final or empty. | 178 |
| Method 'setDownloadJournal' is not designed for extension - needs to be abstract, final or empty. | 188 |
| Parameter newDownloadJournal should be final. | 188 |
| Expected @param tag for 'newDownloadJournal'. | 188 |
| Missing a Javadoc comment. | 214 |
| Variable 'mySessionFactory' must be private and have accessor methods. | 214 |
| Method 'getMySessionFactory' is not designed for extension - needs to be abstract, final or empty. | 216 |
| Missing a Javadoc comment. | 216 |
| Method 'setMySessionFactory' is not designed for extension - needs to be abstract, final or empty. | 222 |
| Missing a Javadoc comment. | 222 |
| Parameter sesf should be final. | 222 |
| Missing a Javadoc comment. | 232 |
| Method 'setUserName' is not designed for extension - needs to be abstract, final or empty. | 234 |
| Missing a Javadoc comment. | 234 |
| Parameter userName should be final. | 234 |
| 'userName' hides a field. | 234 |
| Method 'getUserName' is not designed for extension - needs to be abstract, final or empty. | 241 |
| First sentence should end with a period. | 245 |
| Expected an @return tag. | 248 |
| Method 'getUser' is not designed for extension - needs to be abstract, final or empty. | 248 |
| Missing a Javadoc comment. | 256 |
| Method 'getSiteHandle' is not designed for extension - needs to be abstract, final or empty. | 258 |
| Missing a Javadoc comment. | 258 |
| First sentence should end with a period. | 262 |
| Method 'setSiteHandle' is not designed for extension - needs to be abstract, final or empty. | 265 |
| Parameter sitehandle should be final. | 265 |
| Expected @param tag for 'sitehandle'. | 265 |
| Method 'getLectureId' is not designed for extension - needs to be abstract, final or empty. | 269 |
| Missing a Javadoc comment. | 269 |
| Missing a Javadoc comment. | 273 |
| First sentence should end with a period. | 275 |
| Method 'setLectureId' is not designed for extension - needs to be abstract, final or empty. | 278 |
| Parameter lectureId should be final. | 278 |
| 'lectureId' hides a field. | 278 |
| Expected @param tag for 'lectureId'. | 278 |
| Missing a Javadoc comment. | 282 |
| Method 'getImageItemInfo' is not designed for extension - needs to be abstract, final or empty. | 284 |
| Missing a Javadoc comment. | 284 |
| Method 'setImageItemInfo' is not designed for extension - needs to be abstract, final or empty. | 288 |
| Missing a Javadoc comment. | 288 |
| Parameter imageItemInfo should be final. | 288 |
| 'imageItemInfo' hides a field. | 288 |
| Missing a Javadoc comment. | 292 |
| Method 'getCourseCollectionItemId' is not designed for extension - needs to be abstract, final or empty. | 297 |
| Method 'setCourseCollectionItemId' is not designed for extension - needs to be abstract, final or empty. | 301 |
| Missing a Javadoc comment. | 301 |
| Parameter crsCollectionItemId should be final. | 301 |
| Missing a Javadoc comment. | 308 |
| Missing a Javadoc comment. | 310 |
| Missing a Javadoc comment. | 312 |
| Method 'createUser' is not designed for extension - needs to be abstract, final or empty. | 326 |
| Missing a Javadoc comment. | 326 |
| Parameter username should be final. | 326 |
| Parameter domain should be final. | 326 |
| Parameter userservice should be final. | 326 |
| Line is longer than 80 characters. | 337 |
| Method 'getUserByUserName' is not designed for extension - needs to be abstract, final or empty. | 376 |
| Missing a Javadoc comment. | 376 |
| Parameter username should be final. | 376 |
| Line is longer than 80 characters. | 390 |
| Method 'createNexusInfo' is not designed for extension - needs to be abstract, final or empty. | 413 |
| Missing a Javadoc comment. | 413 |
| Parameter siteHandle should be final. | 413 |
| 'siteHandle' hides a field. | 413 |
| Parameter siteService should be final. | 413 |
| Line is longer than 80 characters. | 423 |
| Method 'getNexusInfoBySiteHandle' is not designed for extension - needs to be abstract, final or empty. | 461 |
| Missing a Javadoc comment. | 461 |
| Parameter siteHandle should be final. | 461 |
| 'siteHandle' hides a field. | 461 |
| Line is longer than 80 characters. | 472 |
| Method 'createCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 505 |
| Missing a Javadoc comment. | 505 |
| Parameter name should be final. | 505 |
| Parameter description should be final. | 506 |
| Parameter siteHandle should be final. | 506 |
| 'siteHandle' hides a field. | 506 |
| Parameter type should be final. | 506 |
| Line is longer than 80 characters. | 516 |
| Method 'getCourseCollectionBySiteHandle' is not designed for extension - needs to be abstract, final or empty. | 574 |
| Missing a Javadoc comment. | 574 |
| Parameter sitehandle should be final. | 574 |
| Method 'getCourseCollectionById' is not designed for extension - needs to be abstract, final or empty. | 611 |
| Missing a Javadoc comment. | 611 |
| Parameter id should be final. | 611 |
| Line is longer than 80 characters. | 623 |
| Line is longer than 80 characters. | 649 |
| Method 'getCourseCollectionLectureVector' is not designed for extension - needs to be abstract, final or empty. | 649 |
| Parameter siteHandle should be final. | 649 |
| 'siteHandle' hides a field. | 649 |
| Expected @param tag for 'siteHandle'. | 649 |
| Line is longer than 80 characters. | 661 |
| Method 'createCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 707 |
| Missing a Javadoc comment. | 707 |
| Parameter courseCollection should be final. | 708 |
| Parameter item should be final. | 708 |
| Parameter user should be final. | 708 |
| Line is longer than 80 characters. | 719 |
| Method 'getCourseCollectionItemById' is not designed for extension - needs to be abstract, final or empty. | 753 |
| Missing a Javadoc comment. | 753 |
| Parameter id should be final. | 753 |
| Line is longer than 80 characters. | 765 |
| Line has trailing spaces. | 790 |
| Method 'getCourseCollectionItemVector' is not designed for extension - needs to be abstract, final or empty. | 793 |
| Parameter siteHandle should be final. | 793 |
| 'siteHandle' hides a field. | 793 |
| Expected @param tag for 'siteHandle'. | 793 |
| Line is longer than 80 characters. | 814 |
| Line is longer than 80 characters. | 815 |
| Line has trailing spaces. | 873 |
| Line is longer than 80 characters. | 876 |
| Method 'getCourseCollectionLectureItemsVector' is not designed for extension - needs to be abstract, final or empty. | 876 |
| Parameter lectureId should be final. | 876 |
| 'lectureId' hides a field. | 876 |
| Expected @param tag for 'lectureId'. | 876 |
| Line is longer than 80 characters. | 900 |
| Line is longer than 80 characters. | 902 |
| Line has trailing spaces. | 963 |
| Method 'addImageToCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 965 |
| Missing a Javadoc comment. | 965 |
| Parameter imageInfo should be final. | 965 |
| Parameter username should be final. | 966 |
| Parameter siteHandle should be final. | 966 |
| 'siteHandle' hides a field. | 966 |
| Line is longer than 80 characters. | 983 |
| Line is longer than 80 characters. | 996 |
| Line is longer than 80 characters. | 1081 |
| Method 'addImageToLecture' is not designed for extension - needs to be abstract, final or empty. | 1090 |
| Missing a Javadoc comment. | 1090 |
| Parameter imageId should be final. | 1090 |
| Parameter lectureId should be final. | 1090 |
| 'lectureId' hides a field. | 1090 |
| Line is longer than 80 characters. | 1104 |
| Line is longer than 80 characters. | 1115 |
| Line is longer than 80 characters. | 1121 |
| Line is longer than 80 characters. | 1184 |
| Line is longer than 80 characters. | 1194 |
| Method 'updateCouseCollectionPageTemplate' is not designed for extension - needs to be abstract, final or empty. | 1206 |
| Missing a Javadoc comment. | 1206 |
| Parameter crlPageTemplate should be final. | 1206 |
| Parameter crlLectureId should be final. | 1207 |
| Parameter crlPageId should be final. | 1207 |
| Line is longer than 80 characters. | 1229 |
| Line is longer than 80 characters. | 1263 |
| Method 'updateCourseCollectionItemTitle' is not designed for extension - needs to be abstract, final or empty. | 1321 |
| Missing a Javadoc comment. | 1321 |
| Parameter crlItem should be final. | 1321 |
| Parameter title should be final. | 1322 |
| Method 'changeCourseCollectionTitle' is not designed for extension - needs to be abstract, final or empty. | 1367 |
| Missing a Javadoc comment. | 1367 |
| Parameter crlLectureId should be final. | 1367 |
| Parameter title should be final. | 1368 |
| Parameter date should be final. | 1368 |
| Method 'changeCourseCollectionItemTitle' is not designed for extension - needs to be abstract, final or empty. | 1424 |
| Missing a Javadoc comment. | 1424 |
| Parameter crlItemId should be final. | 1424 |
| Parameter title should be final. | 1425 |
| Method 'updateCourseCollectionItemOrder' is not designed for extension - needs to be abstract, final or empty. | 1500 |
| Missing a Javadoc comment. | 1500 |
| Parameter crlItemId should be final. | 1500 |
| Parameter order should be final. | 1501 |
| Line is longer than 80 characters. | 1545 |
| Method 'updateCourseCollectionPageOrder' is not designed for extension - needs to be abstract, final or empty. | 1549 |
| Missing a Javadoc comment. | 1549 |
| Parameter crlPageId should be final. | 1549 |
| Parameter order should be final. | 1550 |
| Line is longer than 80 characters. | 1602 |
| Method 'updateCourseCollectionItemDescription' is not designed for extension - needs to be abstract, final or empty. | 1606 |
| Missing a Javadoc comment. | 1606 |
| Parameter crlItem should be final. | 1607 |
| Parameter description should be final. | 1607 |
| Method 'changeCourseCollectionItemDescription' is not designed for extension - needs to be abstract, final or empty. | 1651 |
| Missing a Javadoc comment. | 1651 |
| Parameter crlItemId should be final. | 1651 |
| Parameter description should be final. | 1652 |
| Line is longer than 80 characters. | 1699 |
| Method 'deleteLectureFromCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 1717 |
| Missing a Javadoc comment. | 1717 |
| Parameter crlLectureId should be final. | 1717 |
| Line is longer than 80 characters. | 1740 |
| Line is longer than 80 characters. | 1746 |
| Method 'deleteImageFromLecture' is not designed for extension - needs to be abstract, final or empty. | 1801 |
| Missing a Javadoc comment. | 1801 |
| Parameter crlItemId should be final. | 1801 |
| Parameter crlPageId should be final. | 1801 |
| Method 'deleteImageFromCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 1899 |
| Missing a Javadoc comment. | 1899 |
| Parameter crlItemId should be final. | 1899 |
| Parameter baseDir should be final. | 1900 |
| Method 'deleteImageFromCourseCollectionItem' is not designed for extension - needs to be abstract, final or empty. | 1953 |
| Missing a Javadoc comment. | 1953 |
| Parameter crlItemId should be final. | 1953 |
| Line is longer than 80 characters. | 2000 |
| Method 'getLecturePageSet' is not designed for extension - needs to be abstract, final or empty. | 2048 |
| Line has trailing spaces. | 2058 |
| Method 'getLecturePageVector' is not designed for extension - needs to be abstract, final or empty. | 2061 |
| Parameter lectureId should be final. | 2061 |
| 'lectureId' hides a field. | 2061 |
| Expected @param tag for 'lectureId'. | 2061 |
| Line is longer than 80 characters. | 2084 |
| Line is longer than 80 characters. | 2095 |
| First sentence should end with a period. | 2196 |
| Expected an @return tag. | 2199 |
| Method 'getCourseCollectionItemSet' is not designed for extension - needs to be abstract, final or empty. | 2199 |
| First sentence should end with a period. | 2208 |
| Expected an @return tag. | 2211 |
| Method 'getCourseCollectionLectureSet' is not designed for extension - needs to be abstract, final or empty. | 2211 |
| First sentence should end with a period. | 2220 |
| Expected an @return tag. | 2223 |
| Method 'getCourseCollectionLectureItemsSet' is not designed for extension - needs to be abstract, final or empty. | 2223 |
| Method 'isImageSaved' is not designed for extension - needs to be abstract, final or empty. | 2232 |
| Missing a Javadoc comment. | 2232 |
| Method 'isImageRemoved' is not designed for extension - needs to be abstract, final or empty. | 2250 |
| Missing a Javadoc comment. | 2250 |
| Method 'downloadImage' is not designed for extension - needs to be abstract, final or empty. | 2271 |
| Missing a Javadoc comment. | 2271 |
| Parameter largeImageURL should be final. | 2271 |
| Parameter imageId should be final. | 2271 |
| Parameter repositoryId should be final. | 2272 |
| Parameter baseDir should be final. | 2272 |
| '300000' is a magic number. | 2290 |
| Method 'createLocalImageFiles' is not designed for extension - needs to be abstract, final or empty. | 2311 |
| Missing a Javadoc comment. | 2311 |
| Parameter largeImageURL should be final. | 2311 |
| Parameter imageId should be final. | 2312 |
| Parameter baseDir should be final. | 2312 |
| Parameter repositoryId should be final. | 2312 |
| Parameter imageMagickDir should be final. | 2313 |
| Parameter urlBase should be final. | 2313 |
| Redundant throws: 'java.io.IOException' is subclass of 'Exception'. | 2313 |
| Name 'URL_BASE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2321 |
| '100' is a magic number. | 2351 |
| '500' is a magic number. | 2357 |
| Method 'saveLecturePageVector' is not designed for extension - needs to be abstract, final or empty. | 2371 |
| Missing a Javadoc comment. | 2371 |
| Parameter mLecturePageVector should be final. | 2371 |
| Line is longer than 80 characters. | 2388 |
| Line is longer than 80 characters. | 2393 |
| Line is longer than 80 characters. | 2396 |
| Line is longer than 80 characters. | 2399 |
| Method 'createLocalItemFromImageMap' is not designed for extension - needs to be abstract, final or empty. | 2434 |
| Missing a Javadoc comment. | 2434 |
| Parameter imageInfo should be final. | 2434 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 |
| Line has trailing spaces. | 40 |
| Parameter id should be final. | 43 |
| Expected @param tag for 'id'. | 43 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 |
| Line has trailing spaces. | 39 |
| Parameter id should be final. | 42 |
| Expected @param tag for 'id'. | 42 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 |
| Line has trailing spaces. | 86 |
| Parameter id should be final. | 89 |
| 'id' hides a field. | 89 |
| Expected @param tag for 'id'. | 89 |
| Line has trailing spaces. | 95 |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 98 |
| Line has trailing spaces. | 104 |
| Method 'setId' is not designed for extension - needs to be abstract, final or empty. | 107 |
| Parameter id should be final. | 107 |
| 'id' hides a field. | 107 |
| Expected @param tag for 'id'. | 107 |
| Line has trailing spaces. | 114 |
| Method 'getMimeType' is not designed for extension - needs to be abstract, final or empty. | 117 |
| Line has trailing spaces. | 123 |
| Method 'setMimeType' is not designed for extension - needs to be abstract, final or empty. | 126 |
| Parameter mimeType should be final. | 126 |
| 'mimeType' hides a field. | 126 |
| Expected @param tag for 'mimeType'. | 126 |
| Line has trailing spaces. | 132 |
| Method 'getTitle' is not designed for extension - needs to be abstract, final or empty. | 135 |
| Line has trailing spaces. | 141 |
| Method 'setTitle' is not designed for extension - needs to be abstract, final or empty. | 144 |
| Parameter title should be final. | 144 |
| 'title' hides a field. | 144 |
| Expected @param tag for 'title'. | 144 |
| Line has trailing spaces. | 150 |
| Method 'getDescription' is not designed for extension - needs to be abstract, final or empty. | 153 |
| Line has trailing spaces. | 159 |
| Method 'setDescription' is not designed for extension - needs to be abstract, final or empty. | 162 |
| Parameter description should be final. | 162 |
| 'description' hides a field. | 162 |
| Expected @param tag for 'description'. | 162 |
| Line has trailing spaces. | 168 |
| Method 'getKeywords' is not designed for extension - needs to be abstract, final or empty. | 171 |
| Line has trailing spaces. | 177 |
| Method 'setKeywords' is not designed for extension - needs to be abstract, final or empty. | 180 |
| Parameter keywords should be final. | 180 |
| 'keywords' hides a field. | 180 |
| Expected @param tag for 'keywords'. | 180 |
| Line has trailing spaces. | 186 |
| Method 'getImageLicense' is not designed for extension - needs to be abstract, final or empty. | 189 |
| Line has trailing spaces. | 195 |
| Method 'setImageLicense' is not designed for extension - needs to be abstract, final or empty. | 198 |
| Parameter imageLicense should be final. | 198 |
| 'imageLicense' hides a field. | 198 |
| Expected @param tag for 'imageLicense'. | 198 |
| Line has trailing spaces. | 204 |
| Method 'getImageSource' is not designed for extension - needs to be abstract, final or empty. | 207 |
| Line has trailing spaces. | 213 |
| Method 'setImageSource' is not designed for extension - needs to be abstract, final or empty. | 216 |
| Parameter imageSource should be final. | 216 |
| 'imageSource' hides a field. | 216 |
| Expected @param tag for 'imageSource'. | 216 |
| Line has trailing spaces. | 222 |
| Method 'getCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 225 |
| Line has trailing spaces. | 231 |
| Method 'setCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 234 |
| Parameter createdOn should be final. | 234 |
| 'createdOn' hides a field. | 234 |
| Expected @param tag for 'createdOn'. | 234 |
| Line has trailing spaces. | 240 |
| Method 'getCreator' is not designed for extension - needs to be abstract, final or empty. | 243 |
| Line has trailing spaces. | 249 |
| Method 'setCreator' is not designed for extension - needs to be abstract, final or empty. | 252 |
| Parameter creator should be final. | 252 |
| 'creator' hides a field. | 252 |
| Expected @param tag for 'creator'. | 252 |
| Line has trailing spaces. | 258 |
| Method 'getCreationPeriod' is not designed for extension - needs to be abstract, final or empty. | 261 |
| Line has trailing spaces. | 267 |
| Method 'setCreationPeriod' is not designed for extension - needs to be abstract, final or empty. | 270 |
| Parameter creationPeriod should be final. | 270 |
| 'creationPeriod' hides a field. | 270 |
| Expected @param tag for 'creationPeriod'. | 270 |
| Line has trailing spaces. | 276 |
| Method 'getThumbnailUri' is not designed for extension - needs to be abstract, final or empty. | 279 |
| Line has trailing spaces. | 285 |
| Method 'setThumbnailUri' is not designed for extension - needs to be abstract, final or empty. | 288 |
| Parameter thumbnailUri should be final. | 288 |
| 'thumbnailUri' hides a field. | 288 |
| Expected @param tag for 'thumbnailUri'. | 288 |
| Line has trailing spaces. | 294 |
| Method 'getMediumImageUri' is not designed for extension - needs to be abstract, final or empty. | 297 |
| Line has trailing spaces. | 303 |
| Method 'setMediumImageUri' is not designed for extension - needs to be abstract, final or empty. | 306 |
| Parameter mediumImageUri should be final. | 306 |
| 'mediumImageUri' hides a field. | 306 |
| Expected @param tag for 'mediumImageUri'. | 306 |
| Line has trailing spaces. | 312 |
| Method 'getLargeImageUri' is not designed for extension - needs to be abstract, final or empty. | 315 |
| Line has trailing spaces. | 321 |
| Method 'setLargeImageUri' is not designed for extension - needs to be abstract, final or empty. | 324 |
| Parameter largeImageUri should be final. | 324 |
| 'largeImageUri' hides a field. | 324 |
| Expected @param tag for 'largeImageUri'. | 324 |
| Line has trailing spaces. | 330 |
| Method 'getCourseCollectionItemSet' is not designed for extension - needs to be abstract, final or empty. | 333 |
| Line has trailing spaces. | 339 |
| Line is longer than 80 characters. | 342 |
| Method 'setCourseCollectionItemSet' is not designed for extension - needs to be abstract, final or empty. | 342 |
| Parameter courseCollectionItemSet should be final. | 342 |
| 'courseCollectionItemSet' hides a field. | 342 |
| Expected @param tag for 'courseCollectionItemSet'. | 342 |
| Line has trailing spaces. | 349 |
| Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 353 |
| Parameter rhs should be final. | 353 |
| Expected @param tag for 'rhs'. | 353 |
| 'if' construct must use '{}'s. | 354 |
| 'if' construct must use '{}'s. | 356 |
| 'if' construct must use '{}'s. | 359 |
| Line has trailing spaces. | 368 |
| Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 371 |
| '17' is a magic number. | 373 |
| Avoid inline conditionals. | 374 |
| '37' is a magic number. | 375 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 25 |
| Line is longer than 80 characters. | 43 |
| Line has trailing spaces. | 73 |
| Parameter id should be final. | 76 |
| 'id' hides a field. | 76 |
| Expected @param tag for 'id'. | 76 |
| Line has trailing spaces. | 82 |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 85 |
| Line has trailing spaces. | 91 |
| Method 'setId' is not designed for extension - needs to be abstract, final or empty. | 94 |
| Parameter id should be final. | 94 |
| 'id' hides a field. | 94 |
| Expected @param tag for 'id'. | 94 |
| Line has trailing spaces. | 101 |
| Method 'getNexusInfo' is not designed for extension - needs to be abstract, final or empty. | 104 |
| Line has trailing spaces. | 110 |
| Method 'setNexusInfo' is not designed for extension - needs to be abstract, final or empty. | 113 |
| Parameter nexusInfo should be final. | 113 |
| 'nexusInfo' hides a field. | 113 |
| Expected @param tag for 'nexusInfo'. | 113 |
| Line has trailing spaces. | 119 |
| Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 122 |
| Line has trailing spaces. | 128 |
| Method 'setName' is not designed for extension - needs to be abstract, final or empty. | 131 |
| Parameter name should be final. | 131 |
| 'name' hides a field. | 131 |
| Expected @param tag for 'name'. | 131 |
| Line has trailing spaces. | 137 |
| Method 'getDescription' is not designed for extension - needs to be abstract, final or empty. | 140 |
| Line has trailing spaces. | 146 |
| Method 'setDescription' is not designed for extension - needs to be abstract, final or empty. | 149 |
| Parameter description should be final. | 149 |
| 'description' hides a field. | 149 |
| Expected @param tag for 'description'. | 149 |
| Line has trailing spaces. | 155 |
| Method 'getType' is not designed for extension - needs to be abstract, final or empty. | 158 |
| Line has trailing spaces. | 164 |
| Method 'setType' is not designed for extension - needs to be abstract, final or empty. | 167 |
| Parameter type should be final. | 167 |
| 'type' hides a field. | 167 |
| Expected @param tag for 'type'. | 167 |
| Line has trailing spaces. | 173 |
| Method 'getCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 176 |
| Line has trailing spaces. | 182 |
| Method 'setCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 185 |
| Parameter createdOn should be final. | 185 |
| 'createdOn' hides a field. | 185 |
| Expected @param tag for 'createdOn'. | 185 |
| Line has trailing spaces. | 191 |
| Method 'getCourseCollectionAnnotationSet' is not designed for extension - needs to be abstract, final or empty. | 194 |
| Line has trailing spaces. | 200 |
| Method 'setCourseCollectionAnnotationSet' is not designed for extension - needs to be abstract, final or empty. | 203 |
| Parameter courseCollectionAnnotationSet should be final. | 204 |
| 'courseCollectionAnnotationSet' hides a field. | 204 |
| Expected @param tag for 'courseCollectionAnnotationSet'. | 204 |
| Line has trailing spaces. | 210 |
| Method 'getCourseCollectionItemSet' is not designed for extension - needs to be abstract, final or empty. | 213 |
| Line has trailing spaces. | 219 |
| Line is longer than 80 characters. | 222 |
| Method 'setCourseCollectionItemSet' is not designed for extension - needs to be abstract, final or empty. | 222 |
| Parameter courseCollectionItemSet should be final. | 222 |
| 'courseCollectionItemSet' hides a field. | 222 |
| Expected @param tag for 'courseCollectionItemSet'. | 222 |
| Line has trailing spaces. | 228 |
| Method 'getCourseCollectionPageSet' is not designed for extension - needs to be abstract, final or empty. | 231 |
| Line has trailing spaces. | 237 |
| Line is longer than 80 characters. | 240 |
| Method 'setCourseCollectionPageSet' is not designed for extension - needs to be abstract, final or empty. | 240 |
| Parameter courseCollectionPageSet should be final. | 240 |
| 'courseCollectionPageSet' hides a field. | 240 |
| Expected @param tag for 'courseCollectionPageSet'. | 240 |
| Line has trailing spaces. | 247 |
| Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 251 |
| Parameter rhs should be final. | 251 |
| Expected @param tag for 'rhs'. | 251 |
| 'if' construct must use '{}'s. | 252 |
| 'if' construct must use '{}'s. | 254 |
| 'if' construct must use '{}'s. | 257 |
| Line has trailing spaces. | 266 |
| Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 269 |
| '17' is a magic number. | 271 |
| Avoid inline conditionals. | 272 |
| '37' is a magic number. | 273 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 24 |
| Line is longer than 80 characters. | 39 |
| Line has trailing spaces. | 68 |
| Parameter id should be final. | 71 |
| 'id' hides a field. | 71 |
| Expected @param tag for 'id'. | 71 |
| Line has trailing spaces. | 77 |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 80 |
| Line has trailing spaces. | 86 |
| Method 'setId' is not designed for extension - needs to be abstract, final or empty. | 89 |
| Parameter id should be final. | 89 |
| 'id' hides a field. | 89 |
| Expected @param tag for 'id'. | 89 |
| Line has trailing spaces. | 96 |
| Method 'getUsername' is not designed for extension - needs to be abstract, final or empty. | 99 |
| Line has trailing spaces. | 105 |
| Method 'setUsername' is not designed for extension - needs to be abstract, final or empty. | 108 |
| Parameter username should be final. | 108 |
| 'username' hides a field. | 108 |
| Expected @param tag for 'username'. | 108 |
| Line has trailing spaces. | 114 |
| Method 'getDomain' is not designed for extension - needs to be abstract, final or empty. | 117 |
| Line has trailing spaces. | 123 |
| Method 'setDomain' is not designed for extension - needs to be abstract, final or empty. | 126 |
| Parameter domain should be final. | 126 |
| 'domain' hides a field. | 126 |
| Expected @param tag for 'domain'. | 126 |
| Line has trailing spaces. | 132 |
| Method 'getUserService' is not designed for extension - needs to be abstract, final or empty. | 135 |
| Line has trailing spaces. | 141 |
| Method 'setUserService' is not designed for extension - needs to be abstract, final or empty. | 144 |
| Parameter userService should be final. | 144 |
| 'userService' hides a field. | 144 |
| Expected @param tag for 'userService'. | 144 |
| Line has trailing spaces. | 150 |
| Method 'getCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 153 |
| Line has trailing spaces. | 159 |
| Method 'setCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 162 |
| Parameter createdOn should be final. | 162 |
| 'createdOn' hides a field. | 162 |
| Expected @param tag for 'createdOn'. | 162 |
| Line has trailing spaces. | 168 |
| Method 'getCourseCollectionAnnotationSet' is not designed for extension - needs to be abstract, final or empty. | 171 |
| Line has trailing spaces. | 177 |
| Method 'setCourseCollectionAnnotationSet' is not designed for extension - needs to be abstract, final or empty. | 180 |
| Parameter courseCollectionAnnotationSet should be final. | 181 |
| 'courseCollectionAnnotationSet' hides a field. | 181 |
| Expected @param tag for 'courseCollectionAnnotationSet'. | 181 |
| Line has trailing spaces. | 187 |
| Method 'getCourseCollectionItemSet' is not designed for extension - needs to be abstract, final or empty. | 190 |
| Line has trailing spaces. | 196 |
| Line is longer than 80 characters. | 199 |
| Method 'setCourseCollectionItemSet' is not designed for extension - needs to be abstract, final or empty. | 199 |
| Parameter courseCollectionItemSet should be final. | 199 |
| 'courseCollectionItemSet' hides a field. | 199 |
| Expected @param tag for 'courseCollectionItemSet'. | 199 |
| Line has trailing spaces. | 205 |
| Method 'getItemAnnotationSet' is not designed for extension - needs to be abstract, final or empty. | 208 |
| Line has trailing spaces. | 214 |
| Method 'setItemAnnotationSet' is not designed for extension - needs to be abstract, final or empty. | 217 |
| Parameter itemAnnotationSet should be final. | 217 |
| 'itemAnnotationSet' hides a field. | 217 |
| Expected @param tag for 'itemAnnotationSet'. | 217 |
| Line has trailing spaces. | 224 |
| Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 228 |
| Parameter rhs should be final. | 228 |
| Expected @param tag for 'rhs'. | 228 |
| 'if' construct must use '{}'s. | 229 |
| 'if' construct must use '{}'s. | 231 |
| 'if' construct must use '{}'s. | 234 |
| Line has trailing spaces. | 243 |
| Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 246 |
| '17' is a magic number. | 248 |
| Avoid inline conditionals. | 249 |
| '37' is a magic number. | 250 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 25 |
| Line has trailing spaces. | 79 |
| Parameter id should be final. | 82 |
| 'id' hides a field. | 82 |
| Expected @param tag for 'id'. | 82 |
| Line has trailing spaces. | 88 |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 91 |
| Line has trailing spaces. | 97 |
| Method 'setId' is not designed for extension - needs to be abstract, final or empty. | 100 |
| Parameter id should be final. | 100 |
| 'id' hides a field. | 100 |
| Expected @param tag for 'id'. | 100 |
| Line has trailing spaces. | 107 |
| Method 'getCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 110 |
| Line has trailing spaces. | 116 |
| Method 'setCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 119 |
| Parameter courseCollection should be final. | 119 |
| 'courseCollection' hides a field. | 119 |
| Expected @param tag for 'courseCollection'. | 119 |
| Line has trailing spaces. | 125 |
| Method 'getItem' is not designed for extension - needs to be abstract, final or empty. | 128 |
| Line has trailing spaces. | 134 |
| Method 'setItem' is not designed for extension - needs to be abstract, final or empty. | 137 |
| Parameter item should be final. | 137 |
| 'item' hides a field. | 137 |
| Expected @param tag for 'item'. | 137 |
| Line has trailing spaces. | 143 |
| Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 146 |
| Line has trailing spaces. | 152 |
| Method 'setName' is not designed for extension - needs to be abstract, final or empty. | 155 |
| Parameter name should be final. | 155 |
| 'name' hides a field. | 155 |
| Expected @param tag for 'name'. | 155 |
| Line has trailing spaces. | 161 |
| Method 'getCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 164 |
| Line has trailing spaces. | 170 |
| Method 'setCreatedOn' is not designed for extension - needs to be abstract, final or empty. | 173 |
| Parameter createdOn should be final. | 173 |
| 'createdOn' hides a field. | 173 |
| Expected @param tag for 'createdOn'. | 173 |
| Line has trailing spaces. | 179 |
| Method 'getUser' is not designed for extension - needs to be abstract, final or empty. | 182 |
| Line has trailing spaces. | 188 |
| Method 'setUser' is not designed for extension - needs to be abstract, final or empty. | 191 |
| Parameter user should be final. | 191 |
| 'user' hides a field. | 191 |
| Expected @param tag for 'user'. | 191 |
| Line has trailing spaces. | 197 |
| Method 'getItemTitle' is not designed for extension - needs to be abstract, final or empty. | 200 |
| Line has trailing spaces. | 206 |
| Method 'setItemTitle' is not designed for extension - needs to be abstract, final or empty. | 209 |
| Parameter itemTitle should be final. | 209 |
| 'itemTitle' hides a field. | 209 |
| Expected @param tag for 'itemTitle'. | 209 |
| Line has trailing spaces. | 215 |
| Method 'getItemDescription' is not designed for extension - needs to be abstract, final or empty. | 218 |
| Line has trailing spaces. | 224 |
| Method 'setItemDescription' is not designed for extension - needs to be abstract, final or empty. | 227 |
| Parameter itemDescription should be final. | 227 |
| 'itemDescription' hides a field. | 227 |
| Expected @param tag for 'itemDescription'. | 227 |
| Line has trailing spaces. | 233 |
| Method 'getItemOrder' is not designed for extension - needs to be abstract, final or empty. | 236 |
| Line has trailing spaces. | 242 |
| Method 'setItemOrder' is not designed for extension - needs to be abstract, final or empty. | 245 |
| Parameter itemOrder should be final. | 245 |
| 'itemOrder' hides a field. | 245 |
| Expected @param tag for 'itemOrder'. | 245 |
| Line has trailing spaces. | 251 |
| Method 'getItemSource' is not designed for extension - needs to be abstract, final or empty. | 254 |
| Line has trailing spaces. | 260 |
| Method 'setItemSource' is not designed for extension - needs to be abstract, final or empty. | 263 |
| Parameter itemSource should be final. | 263 |
| 'itemSource' hides a field. | 263 |
| Expected @param tag for 'itemSource'. | 263 |
| Line has trailing spaces. | 269 |
| Method 'getItemAnnotationSet' is not designed for extension - needs to be abstract, final or empty. | 272 |
| Line has trailing spaces. | 278 |
| Method 'setItemAnnotationSet' is not designed for extension - needs to be abstract, final or empty. | 281 |
| Parameter itemAnnotationSet should be final. | 281 |
| 'itemAnnotationSet' hides a field. | 281 |
| Expected @param tag for 'itemAnnotationSet'. | 281 |
| Line has trailing spaces. | 288 |
| Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 292 |
| Parameter rhs should be final. | 292 |
| Expected @param tag for 'rhs'. | 292 |
| 'if' construct must use '{}'s. | 293 |
| 'if' construct must use '{}'s. | 295 |
| 'if' construct must use '{}'s. | 298 |
| Line has trailing spaces. | 307 |
| Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 310 |
| '17' is a magic number. | 312 |
| Avoid inline conditionals. | 313 |
| '37' is a magic number. | 314 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 25 |
| Line has trailing spaces. | 58 |
| Parameter id should be final. | 61 |
| 'id' hides a field. | 61 |
| Expected @param tag for 'id'. | 61 |
| First sentence should end with a period. | 65 |
| Parameter id should be final. | 66 |
| 'id' hides a field. | 66 |
| Expected @param tag for 'id'. | 66 |
| Parameter courseCollection should be final. | 67 |
| 'courseCollection' hides a field. | 67 |
| Expected @param tag for 'courseCollection'. | 67 |
| First sentence should end with a period. | 72 |
| Parameter id should be final. | 73 |
| 'id' hides a field. | 73 |
| Expected @param tag for 'id'. | 73 |
| Parameter courseCollection should be final. | 74 |
| 'courseCollection' hides a field. | 74 |
| Expected @param tag for 'courseCollection'. | 74 |
| Parameter pageTemplate should be final. | 74 |
| 'pageTemplate' hides a field. | 74 |
| Expected @param tag for 'pageTemplate'. | 74 |
| Parameter pageOrder should be final. | 75 |
| 'pageOrder' hides a field. | 75 |
| Expected @param tag for 'pageOrder'. | 75 |
| Line has trailing spaces. | 84 |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 87 |
| Line has trailing spaces. | 93 |
| Method 'setId' is not designed for extension - needs to be abstract, final or empty. | 96 |
| Parameter id should be final. | 96 |
| 'id' hides a field. | 96 |
| Expected @param tag for 'id'. | 96 |
| Line has trailing spaces. | 102 |
| Method 'getCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 105 |
| Line has trailing spaces. | 111 |
| Method 'setCourseCollection' is not designed for extension - needs to be abstract, final or empty. | 114 |
| Parameter courseCollection should be final. | 114 |
| 'courseCollection' hides a field. | 114 |
| Expected @param tag for 'courseCollection'. | 114 |
| Line has trailing spaces. | 120 |
| Method 'getPageTemplate' is not designed for extension - needs to be abstract, final or empty. | 123 |
| Line has trailing spaces. | 129 |
| Method 'setPageTemplate' is not designed for extension - needs to be abstract, final or empty. | 132 |
| Parameter pageTemplate should be final. | 132 |
| 'pageTemplate' hides a field. | 132 |
| Expected @param tag for 'pageTemplate'. | 132 |
| Line has trailing spaces. | 138 |
| Method 'getPageOrder' is not designed for extension - needs to be abstract, final or empty. | 141 |
| Line has trailing spaces. | 147 |
| Method 'setPageOrder' is not designed for extension - needs to be abstract, final or empty. | 150 |
| Parameter pageOrder should be final. | 150 |
| 'pageOrder' hides a field. | 150 |
| Expected @param tag for 'pageOrder'. | 150 |