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 | 553 - 556 |
| These nested if statements could be combined | 585 - 588 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 470 - 473 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 117 - 125 |
| These nested if statements could be combined | 134 - 141 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.Data.DataValue' | 25 |
| 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 | 607 - 611 |
| These nested if statements could be combined | 745 - 748 |
| Avoid unused method parameters such as 'attribute'. | 759 - 775 |
| Avoid unused method parameters such as 'attribute'. | 777 - 793 |
| 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 | 288 - 290 |
| These nested if statements could be combined | 326 - 335 |
| Avoid empty catch blocks | 424 - 425 |
| Avoid empty catch blocks | 425 - 426 |
| Avoid empty catch blocks | 426 - 427 |
| Avoid empty catch blocks | 447 - 448 |
| Avoid empty catch blocks | 448 - 449 |
| Avoid empty catch blocks | 449 - 450 |
| These nested if statements could be combined | 502 - 509 |
| Avoid empty catch blocks | 599 - 600 |
| Avoid empty catch blocks | 600 - 601 |
| Avoid empty catch blocks | 601 - 602 |
| These nested if statements could be combined | 703 - 709 |
| These nested if statements could be combined | 807 - 809 |
| These nested if statements could be combined | 846 - 848 |
| These nested if statements could be combined | 885 - 887 |
| These nested if statements could be combined | 926 - 928 |
| These nested if statements could be combined | 966 - 968 |
| These nested if statements could be combined | 996 - 998 |
| 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.ui.client.widgets.notification.KSNotification' | 22 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.notification.KSNotifier' | 23 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.progress.KSBlockingProgressIndicator' | 24 |
| 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 |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.layouts.MenuSectionController' | 25 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.layouts.TabMenuController' | 26 |
| These nested if statements could be combined | 368 - 370 |
| These nested if statements could be combined | 423 - 425 |
| These nested if statements could be combined | 435 - 438 |
| These nested if statements could be combined | 442 - 444 |
| 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 method parameters such as 'valueWithTranslations'. | 82 - 86 |
| 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 |
|---|---|
| Overriding method merely calls super | 361 - 364 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 3 |
| Avoid unused constructor parameters such as 'controllerId'. | 32 - 39 |
| Overriding method merely calls super | 118 - 121 |
| 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 | 644 - 646 |
| 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 |
|---|---|
| These nested if statements could be combined | 154 - 156 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 641 - 643 |
| Avoid empty if statements | 644 - 646 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.ui.Hidden' | 33 |
| Avoid unused private fields such as 'timer'. | 234 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 3 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 53 - 55 |
| Avoid empty if statements | 107 - 111 |
| These nested if statements could be combined | 310 - 334 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.ui.Widget' | 7 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 161 - 189 |
| These nested if statements could be combined | 177 - 180 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 34 - 36 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 617 - 619 |
| 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'. | 40 |
| Avoid empty if statements | 282 - 284 |
| 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 |
|---|---|
| These nested if statements could be combined | 68 - 70 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'mouseOver'. | 62 |
| 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'. | 17 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'title'. | 38 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 80 - 85 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 436 - 441 |
| Avoid empty if statements | 462 - 464 |
| 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 imports such as 'java.util.ArrayList' | 11 |
| 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 |
|---|---|
| Avoid unused private fields such as 'loading'. | 45 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 181 - 183 |
| 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 |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| 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'. | 75 |
| Overriding method merely calls super | 98 - 100 |
| 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 imports such as 'java.util.ArrayList' | 11 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'value'. | 467 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'key'. | 42 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 350 - 360 |
| These nested if statements could be combined | 396 - 398 |
| These nested if statements could be combined | 475 - 480 |
| These nested if statements could be combined | 552 - 562 |
| An empty statement (semicolon) not part of a loop | 693 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 881 |
| 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'. | 161 - 176 |
| Avoid unused method parameters such as 'query'. | 178 - 193 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 71 - 73 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'focusType'. | 48 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'createSummaryTableFromDataModel()'. | 30 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 202 - 205 |
| These nested if statements could be combined | 220 - 223 |
| These nested if statements could be combined | 234 - 237 |
| These nested if statements could be combined | 248 - 251 |
| Avoid unused constructor parameters such as 'style'. | 272 - 287 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 42 - 44 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 27 - 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.exceptions.MissingParameterException' | 26 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.kim.service.IdentityManagementService' | 26 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'template'. | 177 - 190 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 64 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 72 |
| 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 | 42 - 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 12 |
| Avoid empty catch blocks | 98 |
| Avoid empty catch blocks | 128 |
| Avoid empty catch blocks | 163 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.exceptions.MissingParameterException' | 9 |
| 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 | 503 - 505 |
| Avoid empty catch blocks | 534 - 536 |
| 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'. | 112 |
| Avoid empty if statements | 116 - 117 |
| 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 imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.WarnContainer' | 38 |
| Avoid unused method parameters such as 'swappableFieldsDefinition'. | 314 - 362 |
| Avoid unused method parameters such as 'deletionParentKeys'. | 314 - 362 |
| 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 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 | 778 - 783 |
| 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 |
|---|---|
| These nested if statements could be combined | 254 - 266 |
| These nested if statements could be combined | 306 - 332 |
| These nested if statements could be combined | 524 - 526 |
| 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 | 229 - 230 |
| Avoid empty catch blocks | 247 - 248 |
| Avoid empty catch blocks | 262 - 263 |
| Avoid empty catch blocks | 436 - 437 |
| These nested if statements could be combined | 450 - 452 |
| Avoid empty catch blocks | 570 |
| Avoid empty catch blocks | 573 |
| Avoid empty catch blocks | 576 |
| Avoid empty catch blocks | 786 - 787 |
| Avoid empty catch blocks | 981 - 982 |
| Avoid empty catch blocks | 1150 - 1151 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 87 - 88 |
| Avoid empty catch blocks | 193 - 194 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 47 - 48 |
| Avoid empty catch blocks | 62 - 63 |
| Avoid empty catch blocks | 104 - 105 |
| Avoid empty catch blocks | 134 - 135 |
| Avoid empty catch blocks | 190 - 191 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'clu'. | 283 |
| 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 empty catch blocks | 1126 - 1128 |
| Avoid empty catch blocks | 2607 - 2608 |
| Avoid empty catch blocks | 3916 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 122 - 123 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 144 - 145 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 212 - 213 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils.isEmpty' | 18 |
| Avoid empty catch blocks | 193 - 195 |
| Avoid empty catch blocks | 490 - 491 |
| Avoid empty catch blocks | 570 - 571 |
| Avoid empty catch blocks | 571 - 572 |
| Avoid empty catch blocks | 572 - 573 |
| Avoid unused local variables such as 'credentialClu'. | 711 |
| Avoid empty catch blocks | 712 - 713 |
| Avoid empty catch blocks | 727 - 728 |
| Avoid empty catch blocks | 728 - 729 |
| Avoid empty catch blocks | 729 - 730 |
| Avoid empty catch blocks | 875 - 876 |
| Avoid empty catch blocks | 876 - 877 |
| Avoid empty catch blocks | 877 - 878 |
| Avoid empty catch blocks | 896 - 897 |
| Avoid empty catch blocks | 897 - 898 |
| Avoid empty catch blocks | 898 - 899 |
| Avoid empty catch blocks | 989 - 990 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 138 - 139 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 215 - 216 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 1544 |
| These nested if statements could be combined | 1547 - 1555 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 109 - 110 |
| Avoid empty catch blocks | 203 - 204 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.field.layout.element.MessageKeyInfo' | 6 |
| Avoid unused imports such as 'org.kuali.student.lum.program.client.major.proposal.MajorProposalSummaryConfiguration' | 7 |
| Avoid unused imports such as 'org.kuali.student.lum.program.client.properties.ProgramProperties' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.buttongroups.ButtonEnumerations.YesNoCancelEnum' | 31 |
| Avoid unused imports such as 'com.google.gwt.event.logical.shared.CloseEvent' | 52 |
| Avoid unused imports such as 'com.google.gwt.event.logical.shared.CloseHandler' | 53 |
| Avoid unused constructor parameters such as 'name'. | 90 - 97 |
| Avoid empty if statements | 555 - 557 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.QueryPath' | 11 |
| 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.common.ui.client.mvc.Controller' | 10 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.DataModel' | 11 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.ModelRequestCallback' | 12 |
| Avoid unused imports such as 'org.kuali.student.core.workflow.ui.client.widgets.WorkflowUtilities' | 14 |
| 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.common.ui.client.application.Application' | 3 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.common.client.lu.LUUIConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.Configurer' | 3 |
| Avoid unused imports such as 'org.kuali.student.lum.common.client.lu.LUUIConstants' | 14 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.VerticalSection' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSCheckBox' | 11 |
| Avoid unused imports such as 'org.kuali.student.lum.common.client.widgets.AppLocations' | 19 |
| Avoid unused imports such as 'org.kuali.student.lum.program.client.variation.VariationsBinding' | 26 |
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 27 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.FlexTable' | 29 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 94 - 96 |
| Avoid unused method parameters such as 'model'. | 147 - 269 |
| Avoid unused method parameters such as 'modifyPath'. | 147 - 269 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.Metadata' | 5 |
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.QueryPath' | 6 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.views.SectionView' | 10 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.Callback' | 12 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.progress.BlockingTask' | 16 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.progress.KSBlockingProgressIndicator' | 17 |
| Avoid unused imports such as 'org.kuali.student.core.document.ui.client.widgets.documenttool.DocumentList' | 21 |
| Avoid unused imports such as 'org.kuali.student.core.document.ui.client.widgets.documenttool.DocumentListBinding' | 22 |
| Avoid unused imports such as 'org.kuali.student.core.statement.dto.StatementTreeViewInfo' | 23 |
| Avoid unused imports such as 'org.kuali.student.lum.common.client.lu.LUUIConstants' | 27 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.Widget' | 41 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.binding.ListToTextBinding' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.menus.KSListPanel' | 12 |
| Avoid unused imports such as 'org.kuali.student.lum.common.client.lo.TreeStringBinding' | 19 |
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 27 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.core.statement.dto.ReqCompFieldInfo' | 22 |
| Avoid unused imports such as 'org.kuali.student.core.statement.dto.ReqComponentInfo' | 23 |
| 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 imports such as 'org.kuali.student.common.ui.client.service.BaseDataOrchestrationRpcService' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.service.BaseDataOrchestrationRpcServiceAsync' | 7 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'data'. | 126 - 131 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.major.edit.MajorEditController' | 30 |
| 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 private fields such as 'initialized'. | 20 |
| 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.common.ui.client.event.ActionEvent' | 6 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.event.SaveActionEvent' | 7 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.ActionCompleteCallback' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.dto.MajorDisciplineInfo' | 14 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 225 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 229 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 255 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 259 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.search.dto.SearchParam' | 23 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 129 - 132 |
| Violation | Line |
|---|---|
| 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 |
| 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 private methods such as 'filterResults(List)'. | 332 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 517 - 519 |
| Avoid unused local variables such as 'name'. | 646 |
| Avoid unused local variables such as 'typeKey'. | 647 |
| 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 | 299 - 301 |
| 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' | 23 |
| Avoid unused imports such as 'org.kuali.student.lum.program.dto.MajorDisciplineInfo' | 26 |
| 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'. | 1270 - 1275 |
| Avoid unused constructor parameters such as 'modelWidgetBinding'. | 1270 - 1275 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.LayoutController' | 17 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.table.summary.SummaryTableBlock' | 35 |
| These nested if statements could be combined | 695 - 698 |
| These nested if statements could be combined | 1083 - 1086 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'STATEMENTS_PATH'. | 78 |
| Avoid unused private fields such as 'ID_TRANSLATION'. | 79 |
| Violation | Line |
|---|---|
| Method call on object which may be null | 668 - 670 |
| Overriding method merely calls super | 980 - 982 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'lastId1'. | 62 |
| Avoid unused private fields such as 'lastId2'. | 63 |
| Avoid unused method parameters such as 'id1Model'. | 137 - 202 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'java.util.List' | 19 |
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.Data' | 21 |
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.Metadata' | 22 |
| Avoid unused imports such as 'org.kuali.student.common.dto.DtoConstants' | 23 |
| Avoid unused imports such as 'org.kuali.student.common.rice.authorization.PermissionType' | 24 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.application.Application' | 25 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.application.KSAsyncCallback' | 26 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.application.ViewContext' | 27 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.layouts.DocumentLayoutController' | 28 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.Callback' | 30 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.DataModel' | 31 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.DataModelDefinition' | 32 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.ModelProvider' | 33 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.ModelRequestCallback' | 34 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.WorkQueue' | 35 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.WorkQueue.WorkItem' | 36 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.dto.ReferenceModel' | 37 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.util.WindowTitleUtils' | 38 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSButton' | 39 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSLabel' | 40 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSLightBox' | 41 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSButtonAbstract.ButtonStyle' | 42 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.notification.KSNotification' | 43 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.notification.KSNotifier' | 44 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.progress.BlockingTask' | 45 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.progress.KSBlockingProgressIndicator' | 46 |
| Avoid unused imports such as 'org.kuali.student.common.ui.shared.IdAttributes.IdType' | 47 |
| Avoid unused imports such as 'org.kuali.student.core.statement.dto.StatementTypeInfo' | 48 |
| Avoid unused imports such as 'org.kuali.student.lum.common.client.helpers.RecentlyViewedHelper' | 49 |
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.configuration.CourseProposalConfigurer' | 50 |
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.configuration.ViewCourseConfigurer' | 51 |
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.configuration.CourseProposalConfigurer.CourseSections' | 52 |
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.service.CourseRpcService' | 55 |
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.service.CourseRpcServiceAsync' | 56 |
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.widgets.CourseWorkflowActionList' | 57 |
| Avoid unused imports such as 'com.google.gwt.core.client.GWT' | 59 |
| Avoid unused imports such as 'com.google.gwt.event.dom.client.ClickEvent' | 60 |
| Avoid unused imports such as 'com.google.gwt.event.dom.client.ClickHandler' | 61 |
| Avoid unused imports such as 'com.google.gwt.event.logical.shared.CloseEvent' | 62 |
| Avoid unused imports such as 'com.google.gwt.event.logical.shared.CloseHandler' | 63 |
| Avoid unused imports such as 'com.google.gwt.user.client.Window' | 64 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.Widget' | 65 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.configuration.AbstractCourseConfigurer' | 53 |
| Avoid unused local variables such as 'idType'. | 181 |
| Avoid unused local variables such as 'viewContextId'. | 182 |
| Avoid unused private methods such as 'getCurrentVersion(ModelRequestCallback,Callback)'. | 311 |
| Avoid unused private methods such as 'getQuitButton()'. | 384 |
| Avoid unused private methods such as 'createActionSubmitSuccessHandler()'. | 416 |
| 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.common.ui.client.configurable.mvc.layouts.TabMenuController' | 6 |
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.widgets.CourseWorkflowActionList' | 14 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 167 - 169 |
| Avoid empty if statements | 173 - 175 |
| Avoid unused method parameters such as 'newState'. | 179 - 187 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'onSubmitSuccessHandler'. | 57 - 66 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'statementService'. | 223 - 236 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 49 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 51 |
| 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 |
|---|---|
| Overriding method merely calls super | 36 - 39 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 297 |
| Avoid unused method parameters such as 'cluCode'. | 343 - 427 |
| Avoid unused method parameters such as 'dependencyType'. | 343 - 427 |
| Avoid unused method parameters such as 'dependencySectionKey'. | 447 - 515 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.student.common.ui.client.service.SecurityRpcService' | 44 |
| Avoid duplicate imports such as 'org.kuali.student.common.ui.client.service.SecurityRpcServiceAsync' | 45 |
| Avoid unused imports such as 'org.kuali.student.common.util.security.SecurityUtils' | 46 |
| Avoid empty if statements | 82 - 84 |
| Avoid unused private methods such as 'createNavigationWidget(String)'. | 622 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'controllerId'. | 33 - 38 |
| 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 imports such as 'org.kuali.student.common.ui.client.mvc.history.HistoryManager' | 26 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'container'. | 44 |
| Overriding method merely calls super | 147 - 149 |
| 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 - 361 |
| Overriding method merely calls super | 387 - 389 |
| 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.service.CachingSearchService' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.AssemblyException' | 7 |
| Avoid unused imports such as 'org.kuali.student.common.assembly.transform.MetadataFilter' | 13 |
| Avoid unused imports such as 'org.kuali.student.common.assembly.transform.TransformFilter' | 14 |
| Avoid unused imports such as 'org.kuali.student.common.exceptions.DataValidationErrorException' | 15 |
| Avoid unused imports such as 'org.kuali.student.common.ui.shared.IdAttributes' | 23 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 75 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.exceptions.DataValidationErrorException' | 23 |
| 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 duplicate imports such as 'org.kuali.student.common.util.security.UserWithId' | 27 |
| 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 |