Modules
Project Documentation
The following document contains the results of PMD 4.2.2.
| Violation | Line |
|---|---|
| Avoid unused private methods such as '_toString()'. | 1066 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 66 - 68 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 349 - 351 |
| These nested if statements could be combined | 366 - 368 |
| These nested if statements could be combined | 382 - 384 |
| These nested if statements could be combined | 399 - 401 |
| These nested if statements could be combined | 415 - 417 |
| These nested if statements could be combined | 431 - 433 |
| These nested if statements could be combined | 437 - 439 |
| These nested if statements could be combined | 452 - 454 |
| These nested if statements could be combined | 458 - 460 |
| Avoid empty catch blocks | 471 - 473 |
| Avoid empty catch blocks | 485 - 487 |
| Avoid empty catch blocks | 499 - 501 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'calcNotSoSimpleName(String)'. | 247 |
| These nested if statements could be combined | 260 - 262 |
| These nested if statements could be combined | 266 - 268 |
| These nested if statements could be combined | 550 - 553 |
| These nested if statements could be combined | 582 - 585 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 380 - 383 |
| These nested if statements could be combined | 396 - 404 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 117 - 125 |
| These nested if statements could be combined | 134 - 141 |
| Violation | Line |
|---|---|
| Error while processing /opt/hudson/home/jobs/1.2-site-deploy/workspace/trunk/ks-common/ks-common-impl/src/main/java/org/kuali/student/common/assembly/transform/IdTranslatorFilter.java: null | 0 |
| Error while processing /opt/hudson/home/jobs/1.2-site-deploy/workspace/trunk/ks-common/ks-common-impl/src/main/java/org/kuali/student/common/assembly/transform/IdTranslatorFilter.java: null | 0 |
| Error while processing /opt/hudson/home/jobs/1.2-site-deploy/workspace/trunk/ks-common/ks-common-impl/src/main/java/org/kuali/student/common/assembly/transform/IdTranslatorFilter.java: null | 0 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 113 - 116 |
| These nested if statements could be combined | 120 - 123 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'BASE_KUALI_TYPE'. | 53 |
| Avoid unused private fields such as 'BASE_KUALI_STATE'. | 54 |
| Avoid unused private fields such as 'BASE_KUALI_EFFECTIVE_DATE'. | 55 |
| Avoid unused private fields such as 'BASE_KUALI_EXPIRATION_DATE'. | 57 |
| Avoid unused method parameters such as 'processed'. | 160 - 296 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'subClazz'. | 197 |
| These nested if statements could be combined | 271 - 276 |
| These nested if statements could be combined | 346 - 352 |
| These nested if statements could be combined | 348 - 351 |
| These nested if statements could be combined | 357 - 360 |
| Avoid unused local variables such as 'and'. | 472 |
| These nested if statements could be combined | 603 - 607 |
| These nested if statements could be combined | 741 - 744 |
| Avoid unused method parameters such as 'attribute'. | 755 - 771 |
| Avoid unused method parameters such as 'attribute'. | 773 - 789 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'ac'. | 136 - 156 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 70 - 73 |
| These nested if statements could be combined | 121 - 126 |
| Avoid unused local variables such as 'pattern'. | 312 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'i'. | 319 |
| Avoid unused method parameters such as 'dataType'. | 387 - 431 |
| Avoid empty catch blocks | 394 - 395 |
| Avoid empty catch blocks | 403 - 404 |
| Avoid empty catch blocks | 410 - 411 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'os'. | 14 |
| Avoid unused private methods such as 'pad(String,int)'. | 49 |
| Avoid unused private methods such as 'escapeWiki(String)'. | 223 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'searchInfoTypeMap'. | 10 |
| Avoid unused private fields such as 'queryMap'. | 13 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 285 - 287 |
| These nested if statements could be combined | 325 - 332 |
| Avoid empty catch blocks | 422 - 423 |
| Avoid empty catch blocks | 423 - 424 |
| Avoid empty catch blocks | 424 - 425 |
| Avoid empty catch blocks | 445 - 446 |
| Avoid empty catch blocks | 446 - 447 |
| Avoid empty catch blocks | 447 - 448 |
| These nested if statements could be combined | 488 - 495 |
| These nested if statements could be combined | 660 - 662 |
| These nested if statements could be combined | 699 - 701 |
| These nested if statements could be combined | 738 - 740 |
| These nested if statements could be combined | 779 - 781 |
| These nested if statements could be combined | 819 - 821 |
| These nested if statements could be combined | 849 - 851 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 36 - 38 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 52 - 54 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 30 - 32 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 226 - 230 |
| Avoid empty catch blocks | 344 - 346 |
| Avoid empty if statements | 361 - 366 |
| These nested if statements could be combined | 404 - 406 |
| These nested if statements could be combined | 416 - 418 |
| These nested if statements could be combined | 453 - 462 |
| Avoid unused method parameters such as 'objStructure'. | 495 - 524 |
| Avoid unused method parameters such as 'valResults'. | 660 - 661 |
| Avoid unused private methods such as 'processTypeStateCaseConstraint(List)'. | 664 - 665 |
| Avoid unused method parameters such as 'valResults'. | 664 - 666 |
| These nested if statements could be combined | 675 - 682 |
| Avoid unused method parameters such as 'bcb'. | 705 - 716 |
| These nested if statements could be combined | 751 - 755 |
| These nested if statements could be combined | 796 - 800 |
| These nested if statements could be combined | 841 - 845 |
| These nested if statements could be combined | 888 - 892 |
| These nested if statements could be combined | 936 - 940 |
| These nested if statements could be combined | 973 - 976 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 55 - 57 |
| Avoid empty catch blocks | 70 - 72 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 89 - 90 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.MetadataInterrogator' | 20 |
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.MetadataInterrogator.ConstraintIds' | 21 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 365 - 367 |
| These nested if statements could be combined | 417 - 419 |
| These nested if statements could be combined | 429 - 432 |
| These nested if statements could be combined | 436 - 438 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.list.KSCheckBoxList' | 28 |
| Avoid empty if statements | 78 - 80 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'controllerId'. | 18 - 21 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'controllerId'. | 22 - 28 |
| Violation | Line |
|---|---|
| Do not use if statements that are always true or always false | 155 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'controllerId'. | 28 - 35 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'controllerId'. | 251 - 256 |
| Avoid unused constructor parameters such as 'controllerId'. | 258 - 265 |
| These nested if statements could be combined | 491 - 495 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'style'. | 45 - 48 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'fieldMap'. | 65 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 354 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 53 - 56 |
| These nested if statements could be combined | 71 - 74 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 627 - 629 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'title'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.FieldDescriptor' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.LayoutController' | 9 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.event.SectionUpdateEvent' | 10 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.ui.Hidden' | 33 |
| Avoid unused private fields such as 'timer'. | 224 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 34 - 36 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.ui.SimplePanel' | 48 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.event.dom.client.BlurEvent' | 24 |
| Avoid unused imports such as 'com.google.gwt.event.dom.client.FocusEvent' | 26 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'PANEL_CONTENT_REMOVE_LINK'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.ui.Composite' | 38 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.FocusWidget' | 42 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.HasWidgets' | 43 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyDownEvent' | 23 |
| Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyDownHandler' | 24 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 68 - 70 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'mouseOver'. | 53 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.application.ApplicationContext' | 24 |
| Avoid unused private fields such as 'appUrl'. | 86 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'title'. | 16 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'title'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'margin'. | 80 |
| Avoid empty if statements | 413 - 415 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'fieldTitle'. | 15 - 17 |
| Avoid unused constructor parameters such as 'widget'. | 15 - 17 |
| Avoid unused constructor parameters such as 'fieldTitle'. | 19 - 21 |
| Avoid unused constructor parameters such as 'widget'. | 19 - 21 |
| Avoid unused constructor parameters such as 'key'. | 19 - 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'lineCount'. | 50 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'fieldArea'. | 25 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'top'. | 23 |
| Avoid unused private fields such as 'buttonPanel'. | 24 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 66 - 68 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'allowedKeyPress'. | 57 |
| Avoid unused private fields such as 'hasWatermark'. | 58 |
| Avoid unused private fields such as 'watermarkShowing'. | 59 |
| Avoid unused private fields such as 'watermarkText'. | 60 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 134 - 137 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 51 - 53 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 278 - 282 |
| These nested if statements could be combined | 297 - 301 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'w'. | 74 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 142 - 145 |
| These nested if statements could be combined | 153 - 156 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 149 - 152 |
| These nested if statements could be combined | 160 - 163 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'ids'. | 178 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'buttonLabel'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'value'. | 472 |
| These nested if statements could be combined | 658 - 673 |
| These nested if statements could be combined | 684 - 715 |
| These nested if statements could be combined | 703 - 706 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'key'. | 42 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 330 - 340 |
| These nested if statements could be combined | 375 - 377 |
| These nested if statements could be combined | 449 - 454 |
| These nested if statements could be combined | 512 - 522 |
| An empty statement (semicolon) not part of a loop | 625 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 803 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.service.CachingSearchService' | 29 |
| Violation | Line |
|---|---|
| The null check here is misplaced; if the variable is null therell be a NullPointerException | 79 |
| The null check here is misplaced; if the variable is null therell be a NullPointerException | 79 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'searchId'. | 152 - 167 |
| Avoid unused method parameters such as 'query'. | 169 - 184 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 71 - 73 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'focusType'. | 47 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'createSummaryTableFromDataModel()'. | 30 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 196 - 199 |
| These nested if statements could be combined | 212 - 215 |
| These nested if statements could be combined | 226 - 229 |
| These nested if statements could be combined | 240 - 243 |
| Avoid unused constructor parameters such as 'style'. | 264 - 279 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 42 - 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.exceptions.MissingParameterException' | 21 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.kim.service.IdentityManagementService' | 26 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'request'. | 45 - 224 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 31 - 33 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Properties' | 35 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.messagebuilder.impl.exceptions.MessageBuilderException' | 22 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 42 |
| Avoid empty finally blocks | 67 - 68 |
| Avoid empty finally blocks | 87 - 88 |
| Avoid empty finally blocks | 107 - 108 |
| Avoid empty finally blocks | 127 - 128 |
| Avoid empty finally blocks | 183 - 184 |
| Avoid empty finally blocks | 232 - 233 |
| Avoid empty finally blocks | 290 - 291 |
| Avoid empty finally blocks | 310 - 311 |
| Avoid empty finally blocks | 326 - 327 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 96 - 97 |
| Avoid empty finally blocks | 146 - 147 |
| Avoid empty finally blocks | 232 - 233 |
| Avoid empty finally blocks | 318 - 319 |
| Avoid unused local variables such as 'LP9'. | 338 |
| Avoid unused local variables such as 'RP11'. | 339 |
| Avoid unused local variables such as 'LP9_tree'. | 344 |
| Avoid unused local variables such as 'RP11_tree'. | 345 |
| Avoid empty finally blocks | 411 - 412 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.messagebuilder.impl.exceptions.MessageBuilderException' | 24 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 90 |
| Avoid empty catch blocks | 141 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 83 - 84 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 100 - 102 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 54 - 56 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 55 - 57 |
| Avoid empty catch blocks | 74 - 76 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 54 - 55 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 19 |
| Avoid unused imports such as 'java.util.Date' | 20 |
| Avoid unused imports such as 'java.util.HashMap' | 21 |
| Avoid unused imports such as 'java.util.List' | 22 |
| Avoid unused imports such as 'java.util.Map' | 23 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlAttribute' | 27 |
| Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' | 29 |
| Avoid unused imports such as 'org.kuali.student.common.dto.MetaInfo' | 31 |
| Avoid unused imports such as 'org.kuali.student.common.dto.RichTextInfo' | 32 |
| Avoid unused imports such as 'org.kuali.student.core.ws.binding.JaxbAttributeMapListAdapter' | 33 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 19 |
| Avoid unused imports such as 'java.util.List' | 22 |
| Avoid unused imports such as 'org.kuali.student.common.dto.MetaInfo' | 33 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 19 |
| Avoid unused imports such as 'java.util.List' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.exceptions.CircularRelationshipException' | 28 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 37 - 39 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 90 |
| Avoid empty catch blocks | 90 |
| Avoid empty catch blocks | 134 |
| Avoid empty catch blocks | 134 |
| Avoid empty catch blocks | 164 |
| Avoid empty catch blocks | 164 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 41 - 43 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 45 - 47 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 296 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 41 - 43 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'value'. | 65 - 67 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 41 - 43 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 102 - 103 |
| Avoid empty catch blocks | 139 - 140 |
| Avoid empty catch blocks | 181 - 182 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.service.IdentityService' | 27 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 43 - 45 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 36 - 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'statementTypeId'. | 16 |
| Avoid unused private fields such as 'reqComponentTypeId'. | 19 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'statementTypeId'. | 16 |
| Avoid unused private fields such as 'childStatementTypeId'. | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashSet' | 19 |
| Avoid unused imports such as 'java.util.Set' | 22 |
| Avoid unused imports such as 'org.kuali.student.common.exceptions.DoesNotExistException' | 24 |
| Avoid unused imports such as 'org.kuali.student.core.statement.dto.ReqCompFieldTypeInfo' | 27 |
| Avoid unused imports such as 'org.kuali.student.core.statement.dto.ReqComponentTypeInfo' | 29 |
| Avoid unused imports such as 'org.kuali.student.core.statement.service.StatementService' | 30 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 497 - 499 |
| Avoid empty catch blocks | 528 - 530 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'NO_SELECTION_TEXT'. | 50 |
| Avoid empty catch blocks | 566 - 567 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'ruleChanged'. | 54 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSPlaceholder' | 23 |
| Avoid unused private fields such as 'separator1'. | 50 |
| Avoid unused private fields such as 'separator2'. | 51 |
| Avoid unused private fields such as 'isOperatorChecked'. | 68 |
| These nested if statements could be combined | 144 - 150 |
| These nested if statements could be combined | 160 - 166 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'token'. | 90 |
| These nested if statements could be combined | 353 - 356 |
| These nested if statements could be combined | 774 - 779 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 168 - 173 |
| These nested if statements could be combined | 210 - 216 |
| Avoid unused private methods such as 'getDeeperNode(List)'. | 265 |
| Avoid unused private methods such as 'findNodeIndex(List,int)'. | 382 |
| Avoid unused private methods such as 'hasParenthesis(List)'. | 393 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'rf'. | 87 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'viewEnum'. | 66 |
| Avoid unused private fields such as 'viewName'. | 67 |
| Avoid unused private fields such as 'referenceAttributes'. | 75 |
| Avoid unused private fields such as 'buttonPanel'. | 106 |
| Avoid empty if statements | 110 - 111 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'titlePanel'. | 56 |
| Avoid unused private fields such as 'title'. | 58 |
| Avoid unused private fields such as 'proposalNameHeader'. | 59 |
| Avoid unused private fields such as 'decisionTypeKey'. | 64 |
| Avoid unused local variables such as 'columnIndex'. | 178 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'swappableFieldsDefinition'. | 311 - 359 |
| Avoid unused method parameters such as 'deletionParentKeys'. | 311 - 359 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 16 - 297 |
| Avoid unused local variables such as 'validationResults'. | 110 |
| Avoid empty if statements | 145 - 146 |
| Avoid empty if statements | 147 - 148 |
| Avoid unused local variables such as 'result'. | 187 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 16 - 244 |
| These nested if statements could be combined | 183 - 187 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 16 - 226 |
| Avoid unused local variables such as 'validationResults'. | 103 |
| Avoid unused local variables such as 'result'. | 143 |
| Avoid empty if statements | 209 - 210 |
| Avoid empty if statements | 211 - 212 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 16 - 321 |
| Avoid unused local variables such as 'positions'. | 108 |
| Avoid unused local variables such as 'relations'. | 109 |
| Avoid unused local variables such as 'validationResults'. | 153 |
| Avoid unused local variables such as 'relationData'. | 170 |
| Avoid unused local variables such as 'positionData'. | 177 |
| Avoid unused local variables such as 'relationData'. | 183 |
| These nested if statements could be combined | 244 - 253 |
| Avoid unused private methods such as 'copyIfExists(Data,T,String)'. | 266 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'WITH_DIVIDER'. | 56 |
| Avoid unused private methods such as 'initSection(SectionTitle,boolean)'. | 163 |
| Avoid unused private methods such as 'getH3Title(String)'. | 183 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'orgPosTypeList'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'lbl'. | 57 |
| Avoid unused local variables such as 'orgName'. | 66 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 30 - 32 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 60 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 98 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Date' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Date' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 20 |
| Avoid unused imports such as 'java.util.List' | 21 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 224 - 225 |
| Avoid empty catch blocks | 242 - 243 |
| Avoid empty catch blocks | 257 - 258 |
| Avoid empty catch blocks | 412 - 413 |
| These nested if statements could be combined | 426 - 428 |
| Avoid empty catch blocks | 546 |
| Avoid empty catch blocks | 549 |
| Avoid empty catch blocks | 552 |
| Avoid empty catch blocks | 762 - 763 |
| Avoid empty catch blocks | 957 - 958 |
| Avoid empty catch blocks | 1118 - 1119 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 87 - 88 |
| Avoid empty catch blocks | 193 - 194 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 46 - 47 |
| Avoid empty catch blocks | 61 - 62 |
| Avoid empty catch blocks | 103 - 104 |
| Avoid empty catch blocks | 133 - 134 |
| Avoid empty catch blocks | 189 - 190 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'clu'. | 284 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.util.MessageUtils' | 9 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 41 - 43 |
| Avoid unused private methods such as 'getIncludingLos(String)'. | 122 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 34 - 36 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 44 - 46 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lu.LUConstants' | 65 |
| Avoid empty catch blocks | 1117 - 1119 |
| Avoid empty catch blocks | 2597 - 2598 |
| Avoid empty catch blocks | 3169 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 122 - 123 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 144 - 145 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 211 - 212 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 189 - 191 |
| Avoid empty catch blocks | 485 - 486 |
| Avoid empty catch blocks | 565 - 566 |
| Avoid empty catch blocks | 566 - 567 |
| Avoid empty catch blocks | 567 - 568 |
| Avoid unused local variables such as 'credentialClu'. | 705 |
| Avoid empty catch blocks | 706 - 707 |
| Avoid empty catch blocks | 721 - 722 |
| Avoid empty catch blocks | 722 - 723 |
| Avoid empty catch blocks | 723 - 724 |
| Avoid empty catch blocks | 867 - 868 |
| Avoid empty catch blocks | 868 - 869 |
| Avoid empty catch blocks | 869 - 870 |
| Avoid empty catch blocks | 888 - 889 |
| Avoid empty catch blocks | 889 - 890 |
| Avoid empty catch blocks | 890 - 891 |
| Avoid empty catch blocks | 976 - 977 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 137 - 138 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.exceptions.InvalidParameterException' | 20 |
| Avoid unused imports such as 'org.kuali.student.common.exceptions.MissingParameterException' | 21 |
| Avoid unused imports such as 'org.kuali.student.common.exceptions.OperationFailedException' | 22 |
| Avoid empty catch blocks | 213 - 214 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'statement'. | 664 |
| Avoid empty catch blocks | 1439 |
| These nested if statements could be combined | 1442 - 1450 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 104 - 105 |
| Avoid empty catch blocks | 186 - 187 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'name'. | 80 - 87 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 13 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.major.edit.MajorEditController' | 8 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'id'. | 13 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.ProgramStatus' | 6 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.Metadata' | 5 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSErrorDialog' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.application.Application' | 3 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 13 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 13 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 14 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'originalReqCompNL'. | 83 |
| Avoid unused private fields such as 'originalLogicExpression'. | 84 |
| Violation | Line |
|---|---|
| Avoid unnecessary return statements | 107 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 26 - 28 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'data'. | 98 - 103 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.Metadata' | 3 |
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.QueryPath' | 4 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityConfiguration' | 6 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.MultiplicitySection' | 8 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.Section' | 9 |
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.ModelDefinition' | 3 |
| Avoid unused private fields such as 'initialized'. | 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'viewContext'. | 51 |
| Avoid unused method parameters such as 'versionaLabel'. | 113 - 121 |
| Avoid unused private methods such as 'createHistoryPanel()'. | 172 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.ProgramClientConstants' | 24 |
| Avoid unused imports such as 'org.kuali.student.lum.program.dto.CoreProgramInfo' | 26 |
| Avoid unused imports such as 'org.kuali.student.lum.program.dto.CredentialProgramInfo' | 27 |
| Avoid unused private fields such as 'metadataService'. | 38 |
| Avoid unused private fields such as 'programService'. | 39 |
| Avoid unused private fields such as 'mapper'. | 41 |
| Avoid unused private fields such as 'credPgmMetadata'. | 42 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 129 - 132 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 133 - 135 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 243 - 250 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'resolvedOrgShortName'. | 157 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.student.bo.KualiStudentKimAttributes' | 22 |
| Avoid unused imports such as 'org.kuali.student.common.search.dto.SearchParam' | 23 |
| Avoid unused imports such as 'org.kuali.student.common.search.dto.SearchRequest' | 24 |
| Avoid unused imports such as 'org.kuali.student.common.search.dto.SearchResult' | 25 |
| Avoid unused imports such as 'org.kuali.student.common.search.dto.SearchResultCell' | 26 |
| These nested if statements could be combined | 113 - 115 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 51 |
| These nested if statements could be combined | 77 - 79 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.namespace.QName' | 9 |
| Avoid unused imports such as 'org.kuali.rice.core.resourceloader.GlobalResourceLoader' | 12 |
| Avoid unused imports such as 'org.kuali.rice.kew.role.QualifierResolver' | 14 |
| These nested if statements could be combined | 62 - 73 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 6 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.Data.StringKey' | 4 |
| Avoid unused imports such as 'java.util.Date' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.ui.VerticalPanel' | 39 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 461 - 463 |
| Avoid unused local variables such as 'name'. | 590 |
| Avoid unused local variables such as 'typeKey'. | 591 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyUpEvent' | 29 |
| Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyUpHandler' | 30 |
| Avoid unused imports such as 'com.google.gwt.user.client.Window' | 34 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.TextArea' | 36 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.dto.MetaInfo' | 5 |
| Violation | Line |
|---|---|
| Error while parsing /opt/hudson/home/jobs/1.2-site-deploy/workspace/trunk/ks-lum/ks-lum-ui-common/src/main/java/org/kuali/student/lum/common/client/lo/LoDisplayInfoSortedSet.java: Encountered "(" at line 13, column 14. Was expecting one of: "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... | 0 |
| Error while parsing /opt/hudson/home/jobs/1.2-site-deploy/workspace/trunk/ks-lum/ks-lum-ui-common/src/main/java/org/kuali/student/lum/common/client/lo/LoDisplayInfoSortedSet.java: Encountered "(" at line 13, column 14. Was expecting one of: "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... | 0 |
| Error while parsing /opt/hudson/home/jobs/1.2-site-deploy/workspace/trunk/ks-lum/ks-lum-ui-common/src/main/java/org/kuali/student/lum/common/client/lo/LoDisplayInfoSortedSet.java: Encountered "(" at line 13, column 14. Was expecting one of: "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... | 0 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 24 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 163 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 164 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 236 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 295 - 297 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 7 |
| Avoid unused imports such as 'com.google.gwt.user.client.Window' | 51 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 95 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 105 |
| Avoid unused method parameters such as 'searchParams'. | 490 - 532 |
| Overriding method merely calls super | 535 - 538 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 144 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 151 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 159 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 166 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 35 - 41 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'enabled'. | 126 - 139 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'enabled'. | 126 - 139 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.SwapSection' | 11 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.event.SectionUpdateEvent' | 12 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 22 |
| Avoid unused imports such as 'org.kuali.student.lum.program.dto.MajorDisciplineInfo' | 25 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'a'. | 60 |
| These nested if statements could be combined | 114 - 122 |
| These nested if statements could be combined | 131 - 138 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'fieldWidget'. | 1234 - 1239 |
| Avoid unused constructor parameters such as 'modelWidgetBinding'. | 1234 - 1239 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.progress.KSBlockingProgressIndicator' | 26 |
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.controllers.CourseProposalController' | 48 |
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.requirements.CourseRequirementsDataModel' | 49 |
| Avoid unused private fields such as 'loadDataTask'. | 94 |
| These nested if statements could be combined | 461 - 463 |
| These nested if statements could be combined | 718 - 720 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'STATEMENTS_PATH'. | 78 |
| Avoid unused private fields such as 'ID_TRANSLATION'. | 79 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 685 - 687 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'lastId1'. | 61 |
| Avoid unused private fields such as 'lastId2'. | 62 |
| Avoid unused method parameters such as 'id1Model'. | 131 - 183 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'idType'. | 174 |
| Avoid unused local variables such as 'viewContextId'. | 175 |
| Avoid unused private methods such as 'getCurrentVersion(ModelRequestCallback,Callback)'. | 312 |
| Avoid unused private methods such as 'getQuitButton()'. | 385 |
| Avoid unused private methods such as 'createActionSubmitSuccessHandler()'. | 417 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.breadcrumb.BreadcrumbManager' | 6 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.history.HistoryManager' | 7 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'originalReqCompNL'. | 92 |
| Avoid unused private fields such as 'originalLogicExpression'. | 93 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'modelId'. | 10 - 12 |
| Avoid unused constructor parameters such as 'viewEnum'. | 10 - 12 |
| Avoid unused constructor parameters such as 'name'. | 10 - 12 |
| Avoid unused constructor parameters such as 'controller'. | 10 - 12 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.widgets.CourseWorkflowActionList' | 13 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 125 - 127 |
| Avoid empty if statements | 131 - 133 |
| Avoid unused method parameters such as 'newState'. | 147 - 155 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'onSubmitSuccessHandler'. | 57 - 66 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 53 - 55 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 57 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 59 |
| Avoid unused local variables such as 'currVerPrevState'. | 81 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 25 |
| Avoid unused imports such as 'java.util.Date' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.application.Application' | 7 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.ApplicationPanel' | 14 |
| Avoid unused imports such as 'com.google.gwt.user.client.DOM' | 19 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'controllerId'. | 33 - 38 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.controllers.CurriculumHomeController.LUMViews' | 6 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.application.ApplicationContext' | 24 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.ApplicationPanel' | 32 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.HTML' | 55 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.Hidden' | 56 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.PopupPanel' | 57 |
| Avoid unused private fields such as 'versionAnchor'. | 80 |
| Avoid unused private fields such as 'appUrl'. | 88 |
| Avoid unused private fields such as 'appVersion'. | 92 |
| Avoid unused private fields such as 'codeServer'. | 93 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getLabel(String)'. | 136 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'container'. | 39 |
| Overriding method merely calls super | 133 - 135 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'cluRangeFieldEditDescriptor'. | 135 |
| Avoid unused private methods such as 'createCluSetSection()'. | 225 |
| Avoid unused private methods such as 'editCluSetSection()'. | 311 |
| Avoid unused private methods such as 'viewCluSetSection()'. | 352 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'modelDefinition'. | 255 - 274 |
| Overriding method merely calls super | 277 - 281 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 314 |
| Avoid unused method parameters such as 'saveActionEvent'. | 326 - 366 |
| Overriding method merely calls super | 392 - 394 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 110 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.layout.VerticalFlowPanel' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSErrorDialog' | 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.core.client.GWT' | 34 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.AssemblyException' | 6 |
| Avoid unused imports such as 'org.kuali.student.common.assembly.transform.MetadataFilter' | 12 |
| Avoid unused imports such as 'org.kuali.student.common.assembly.transform.TransformFilter' | 13 |
| Avoid unused imports such as 'org.kuali.student.common.ui.shared.IdAttributes' | 22 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 73 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 30 - 32 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 113 - 115 |
| Overriding method merely calls super | 118 - 120 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.xpath.XPathAPI' | 35 |
| Avoid unused imports such as 'org.w3c.dom.NodeList' | 51 |
| Avoid unused local variables such as 'principal'. | 69 |
| Avoid empty if statements | 92 - 94 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.io.StringWriter' | 5 |
| Avoid unused imports such as 'javax.xml.transform.Transformer' | 18 |
| Avoid unused imports such as 'javax.xml.transform.TransformerFactory' | 19 |
| Avoid unused imports such as 'javax.xml.transform.dom.DOMSource' | 20 |
| Avoid unused imports such as 'javax.xml.transform.stream.StreamResult' | 21 |