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 | 490 - 493 |
| 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'. | 235 |
| Avoid unused method parameters such as 'dataType'. | 303 - 347 |
| Avoid empty catch blocks | 310 - 311 |
| Avoid empty catch blocks | 319 - 320 |
| Avoid empty catch blocks | 326 - 327 |
| 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 | 44 - 46 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 52 - 54 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 39 - 41 |
| 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 |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.assembly.data.QueryPath' | 24 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.layouts.MenuSectionController' | 26 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.layouts.TabMenuController' | 27 |
| These nested if statements could be combined | 373 - 375 |
| These nested if statements could be combined | 428 - 430 |
| These nested if statements could be combined | 440 - 443 |
| These nested if statements could be combined | 447 - 449 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.list.KSCheckBoxList' | 31 |
| Avoid empty if statements | 89 - 91 |
| 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 constructor parameters such as 'controllerId'. | 31 - 38 |
| Overriding method merely calls super | 117 - 120 |
| 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 | 356 |
| 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 local variables such as 'InvocationException'. | 61 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 123 - 125 |
| Avoid empty if statements | 157 - 159 |
| These nested if statements could be combined | 431 - 443 |
| 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 methods such as 'getStackTrace(Throwable)'. | 60 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'PANEL_CONTENT_REMOVE_LINK'. | 40 |
| Avoid empty if statements | 282 - 284 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 68 - 70 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'mouseOver'. | 63 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'appUrl'. | 85 |
| 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 | 81 - 86 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.ui.UIObject' | 38 |
| These nested if statements could be combined | 478 - 483 |
| Avoid empty if statements | 504 - 506 |
| 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 |
|---|---|
| Avoid unused private fields such as 'loading'. | 45 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 181 - 185 |
| 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 | 280 - 284 |
| These nested if statements could be combined | 299 - 303 |
| 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 | 150 - 153 |
| These nested if statements could be combined | 161 - 164 |
| 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'. | 474 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'key'. | 42 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 356 - 366 |
| These nested if statements could be combined | 402 - 404 |
| These nested if statements could be combined | 489 - 494 |
| These nested if statements could be combined | 566 - 576 |
| An empty statement (semicolon) not part of a loop | 711 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 899 |
| 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'. | 164 - 180 |
| Avoid unused method parameters such as 'query'. | 182 - 198 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 71 - 73 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'focusType'. | 49 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'createSummaryTableFromDataModel()'. | 30 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 167 - 170 |
| These nested if statements could be combined | 183 - 186 |
| These nested if statements could be combined | 196 - 199 |
| These nested if statements could be combined | 209 - 212 |
| Avoid unused constructor parameters such as 'style'. | 233 - 248 |
| 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'. | 182 - 196 |
| Avoid unused method parameters such as 'template'. | 213 - 226 |
| Avoid unused method parameters such as 'template'. | 240 - 253 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.util.DateFormatThread' | 15 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 68 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 76 |
| 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 | 59 - 61 |
| Avoid empty catch blocks | 78 - 80 |
| 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 | 89 |
| Avoid empty catch blocks | 89 |
| Avoid empty catch blocks | 133 |
| Avoid empty catch blocks | 133 |
| Avoid empty catch blocks | 163 |
| Avoid empty catch blocks | 163 |
| 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 | 309 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 41 - 43 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'value'. | 64 - 66 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 42 - 44 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 96 |
| Avoid empty catch blocks | 126 |
| Avoid empty catch blocks | 161 |
| 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 | 514 - 516 |
| Avoid empty catch blocks | 545 - 547 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'viewEnum'. | 67 |
| Avoid unused private fields such as 'viewName'. | 68 |
| Avoid unused private fields such as 'referenceAttributes'. | 76 |
| Avoid unused private fields such as 'buttonPanel'. | 114 |
| Avoid empty if statements | 118 - 119 |
| 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 imports such as 'org.kuali.student.common.ui.client.util.DebugIdUtils' | 22 |
| Avoid unused private fields such as 'NO_SELECTION_TEXT'. | 51 |
| Avoid empty catch blocks | 568 - 569 |
| 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 | 260 - 272 |
| These nested if statements could be combined | 312 - 338 |
| These nested if statements could be combined | 549 - 551 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 59 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 97 |
| 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 | 230 - 231 |
| Avoid empty catch blocks | 248 - 249 |
| Avoid empty catch blocks | 263 - 264 |
| Avoid empty catch blocks | 437 - 438 |
| These nested if statements could be combined | 451 - 453 |
| Avoid empty catch blocks | 571 |
| Avoid empty catch blocks | 574 |
| Avoid empty catch blocks | 577 |
| Avoid empty catch blocks | 787 - 788 |
| Avoid empty catch blocks | 982 - 983 |
| Avoid empty catch blocks | 1151 - 1152 |
| 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'. | 289 |
| 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 | 1195 - 1197 |
| Avoid empty catch blocks | 2681 - 2682 |
| Avoid empty catch blocks | 4033 |
| 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 | 1555 |
| These nested if statements could be combined | 1558 - 1566 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 109 - 110 |
| Avoid empty catch blocks | 203 - 204 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'name'. | 85 - 92 |
| Avoid empty if statements | 562 - 564 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 144 - 146 |
| Avoid returning from a finally block | 147 |
| 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 |
|---|---|
| Overriding method merely calls super | 283 - 285 |
| Avoid empty if statements | 789 - 791 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 417 - 419 |
| These nested if statements could be combined | 979 - 981 |
| Avoid empty if statements | 1080 - 1082 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.ui.HorizontalPanel' | 41 |
| Avoid empty if statements | 93 - 95 |
| Avoid unused method parameters such as 'model'. | 179 - 290 |
| Avoid unused method parameters such as 'modifyPath'. | 179 - 290 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'originalReqCompNL'. | 84 |
| Avoid unused private fields such as 'originalLogicExpression'. | 85 |
| Violation | Line |
|---|---|
| Avoid unnecessary return statements | 108 |
| 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'. | 159 - 164 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'initialized'. | 20 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'viewContext'. | 54 |
| Avoid unused method parameters such as 'versionaLabel'. | 116 - 124 |
| Avoid unused private methods such as 'createHistoryPanel()'. | 175 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'endInstAdmitTerm'. | 98 - 133 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.dto.MajorDisciplineInfo' | 14 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'endInstAdmitTerm'. | 100 - 135 |
| 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.exceptions.DoesNotExistException' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 6 |
| 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 |
|---|---|
| These nested if statements could be combined | 108 - 110 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 76 - 78 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 58 - 69 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 33 - 35 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 54 - 56 |
| 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)'. | 334 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.application.ApplicationContext' | 25 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 519 - 521 |
| Avoid unused local variables such as 'name'. | 655 |
| Avoid unused local variables such as 'typeKey'. | 656 |
| 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 'com.google.gwt.user.client.ui.Widget' | 10 |
| 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 | 305 - 307 |
| 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 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'. | 1273 - 1278 |
| Avoid unused constructor parameters such as 'modelWidgetBinding'. | 1273 - 1278 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 715 - 718 |
| These nested if statements could be combined | 1103 - 1106 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'STATEMENTS_PATH'. | 79 |
| Avoid unused private fields such as 'ID_TRANSLATION'. | 80 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 239 - 241 |
| Violation | Line |
|---|---|
| Method call on object which may be null | 653 - 655 |
| Overriding method merely calls super | 976 - 978 |
| 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.common.ui.client.widgets.field.layout.layouts.VerticalFieldLayout' | 45 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.table.summary.SummaryTableSection' | 50 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.ComplexPanel' | 69 |
| Avoid unused local variables such as 'idType'. | 182 |
| Avoid unused local variables such as 'viewContextId'. | 183 |
| 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.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 unused private fields such as 'inactivateCourseActionItem'. | 58 |
| Avoid empty if statements | 165 - 167 |
| Avoid empty if statements | 171 - 173 |
| Avoid unused method parameters such as 'newState'. | 177 - 185 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'onSubmitSuccessHandler'. | 57 - 66 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'statementService'. | 224 - 237 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.service.ProgramServiceConstants' | 40 |
| 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 | 298 |
| Avoid unused method parameters such as 'cluCode'. | 344 - 428 |
| Avoid unused method parameters such as 'dependencyType'. | 344 - 428 |
| Avoid unused method parameters such as 'dependencySectionKey'. | 448 - 516 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 448 - 452 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'controllerId'. | 34 - 39 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.controllers.CourseProposalController' | 9 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'appUrl'. | 82 |
| Avoid unused private fields such as 'appVersion'. | 86 |
| Avoid unused private fields such as 'codeServer'. | 87 |
| 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 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 |