Modules
Project Documentation
The following document contains the results of PMD 4.2.2.
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 274 - 276 |
| These nested if statements could be combined | 314 - 321 |
| These nested if statements could be combined | 429 - 436 |
| These nested if statements could be combined | 601 - 603 |
| These nested if statements could be combined | 640 - 642 |
| These nested if statements could be combined | 679 - 681 |
| These nested if statements could be combined | 720 - 722 |
| These nested if statements could be combined | 760 - 762 |
| These nested if statements could be combined | 790 - 792 |
| 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 unused private methods such as '_toString()'. | 1050 |
| 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)'. | 246 |
| These nested if statements could be combined | 259 - 261 |
| These nested if statements could be combined | 265 - 267 |
| These nested if statements could be combined | 549 - 552 |
| These nested if statements could be combined | 581 - 584 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 386 - 389 |
| These nested if statements could be combined | 402 - 410 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 117 - 125 |
| These nested if statements could be combined | 134 - 141 |
| Violation | Line |
|---|---|
| Error while processing /opt/hudson/home/jobs/1-1-site-deploy-perf/workspace/ks-1.1-perf/ks-common/ks-common-impl/src/main/java/org/kuali/student/core/assembly/transform/IdTranslatorFilter.java: null | 0 |
| Error while processing /opt/hudson/home/jobs/1-1-site-deploy-perf/workspace/ks-1.1-perf/ks-common/ks-common-impl/src/main/java/org/kuali/student/core/assembly/transform/IdTranslatorFilter.java: null | 0 |
| Error while processing /opt/hudson/home/jobs/1-1-site-deploy-perf/workspace/ks-1.1-perf/ks-common/ks-common-impl/src/main/java/org/kuali/student/core/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'. | 196 |
| These nested if statements could be combined | 270 - 275 |
| These nested if statements could be combined | 336 - 342 |
| These nested if statements could be combined | 338 - 341 |
| These nested if statements could be combined | 347 - 350 |
| Avoid unused local variables such as 'and'. | 462 |
| These nested if statements could be combined | 593 - 597 |
| These nested if statements could be combined | 731 - 734 |
| Avoid unused method parameters such as 'attribute'. | 745 - 761 |
| Avoid unused method parameters such as 'attribute'. | 763 - 779 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'ac'. | 135 - 155 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 69 - 72 |
| These nested if statements could be combined | 120 - 125 |
| Avoid unused local variables such as 'pattern'. | 311 |
| 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'. | 13 |
| Avoid unused private methods such as 'pad(String,int)'. | 48 |
| Avoid unused private methods such as 'escapeWiki(String)'. | 222 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'searchInfoTypeMap'. | 11 |
| Avoid unused private fields such as 'queryMap'. | 14 |
| 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.demo.HeaderDemo' | 21 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.table.scroll.TableDemoPanel' | 26 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 327 - 329 |
| These nested if statements could be combined | 365 - 367 |
| These nested if statements could be combined | 377 - 380 |
| These nested if statements could be combined | 384 - 386 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 67 - 69 |
| Violation | Line |
|---|---|
| Do not use if statements that are always true or always false | 155 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 13 - 15 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 532 - 536 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 94 - 97 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'style'. | 45 - 48 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'fieldMap'. | 64 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 347 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 53 - 56 |
| These nested if statements could be combined | 71 - 74 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.Window' | 21 |
| Overriding method merely calls super | 499 - 501 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'title'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.FieldDescriptor' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.ui.Hidden' | 33 |
| Avoid unused private fields such as 'timer'. | 167 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 34 - 36 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 16 - 720 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.event.dom.client.BlurEvent' | 24 |
| Avoid unused imports such as 'com.google.gwt.event.dom.client.FocusEvent' | 26 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'PANEL_CONTENT_REMOVE_LINK'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyDownEvent' | 23 |
| Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyDownHandler' | 24 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'mouseOver'. | 53 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'appUrl'. | 85 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'title'. | 16 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'title'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'margin'. | 49 |
| Avoid empty if statements | 344 - 346 |
| 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'. | 40 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'fieldArea'. | 16 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'top'. | 18 |
| Avoid unused private fields such as 'buttonPanel'. | 19 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 66 - 68 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'allowedKeyPress'. | 57 |
| Avoid unused private fields such as 'hasWatermark'. | 58 |
| Avoid unused private fields such as 'watermarkShowing'. | 59 |
| Avoid unused private fields such as 'watermarkText'. | 60 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 46 - 48 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 278 - 282 |
| These nested if statements could be combined | 297 - 301 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'w'. | 74 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 142 - 145 |
| These nested if statements could be combined | 153 - 156 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 154 - 157 |
| These nested if statements could be combined | 165 - 168 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'ids'. | 178 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'buttonLabel'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'value'. | 455 |
| These nested if statements could be combined | 634 - 644 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'key'. | 42 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 310 - 320 |
| These nested if statements could be combined | 355 - 357 |
| These nested if statements could be combined | 429 - 434 |
| These nested if statements could be combined | 492 - 502 |
| An empty statement (semicolon) not part of a loop | 598 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 775 |
| Violation | Line |
|---|---|
| The null check here is misplaced; if the variable is null therell be a NullPointerException | 76 |
| The null check here is misplaced; if the variable is null therell be a NullPointerException | 76 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'searchId'. | 149 - 163 |
| Avoid unused method parameters such as 'query'. | 165 - 179 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 263 - 265 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'createSummaryTableFromDataModel()'. | 30 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 196 - 199 |
| These nested if statements could be combined | 212 - 215 |
| These nested if statements could be combined | 226 - 229 |
| These nested if statements could be combined | 240 - 243 |
| Avoid unused constructor parameters such as 'style'. | 264 - 279 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 42 - 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.core.exceptions.MissingParameterException' | 20 |
| 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 | 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.core.dto.MetaInfo' | 31 |
| Avoid unused imports such as 'org.kuali.student.core.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.core.dto.MetaInfo' | 34 |
| 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.core.exceptions.CircularRelationshipException' | 28 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 37 - 39 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 83 - 84 |
| Avoid empty catch blocks | 84 - 85 |
| Avoid empty catch blocks | 122 - 123 |
| Avoid empty catch blocks | 123 - 124 |
| Avoid empty catch blocks | 146 - 147 |
| Avoid empty catch blocks | 147 - 148 |
| 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 | 290 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.core.exceptions.DoesNotExistException' | 25 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 41 - 43 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'value'. | 65 - 67 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 41 - 43 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 101 - 102 |
| Avoid empty catch blocks | 137 - 138 |
| Avoid empty catch blocks | 178 - 179 |
| 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 'javax.persistence.ManyToOne' | 25 |
| 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.core.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 | 350 |
| Do not use if statements that are always true or always false | 432 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 492 - 494 |
| Avoid empty catch blocks | 522 - 524 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'viewEnum'. | 66 |
| Avoid unused private fields such as 'viewName'. | 67 |
| Avoid unused private fields such as 'referenceAttributes'. | 75 |
| Avoid unused private fields such as 'buttonPanel'. | 106 |
| Avoid empty if statements | 110 - 111 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'swappableFieldsDefinition'. | 310 - 358 |
| Avoid unused method parameters such as 'deletionParentKeys'. | 310 - 358 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'NO_SELECTION_TEXT'. | 42 |
| Avoid empty catch blocks | 558 - 559 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'ruleChanged'. | 54 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSPlaceholder' | 23 |
| Avoid unused private fields such as 'separator1'. | 50 |
| Avoid unused private fields such as 'separator2'. | 51 |
| Avoid unused private fields such as 'isOperatorChecked'. | 68 |
| These nested if statements could be combined | 144 - 150 |
| These nested if statements could be combined | 160 - 166 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'token'. | 90 |
| These nested if statements could be combined | 353 - 356 |
| These nested if statements could be combined | 774 - 779 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 194 - 197 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 168 - 173 |
| These nested if statements could be combined | 210 - 216 |
| Avoid unused private methods such as 'getDeeperNode(List)'. | 265 |
| Avoid unused private methods such as 'findNodeIndex(List,int)'. | 382 |
| Avoid unused private methods such as 'hasParenthesis(List)'. | 393 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'rf'. | 87 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.core.proposal.ProposalConstants' | 29 |
| 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 |
|---|---|
| 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 - 320 |
| Avoid unused local variables such as 'positions'. | 108 |
| Avoid unused local variables such as 'relations'. | 109 |
| Avoid unused local variables such as 'validationResults'. | 152 |
| Avoid unused local variables such as 'relationData'. | 169 |
| Avoid unused local variables such as 'positionData'. | 176 |
| Avoid unused local variables such as 'relationData'. | 182 |
| These nested if statements could be combined | 243 - 252 |
| Avoid unused private methods such as 'copyIfExists(Data,T,String)'. | 265 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'WITH_DIVIDER'. | 56 |
| Avoid unused private methods such as 'initSection(SectionTitle,boolean)'. | 163 |
| Avoid unused private methods such as 'getH3Title(String)'. | 183 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'orgPosTypeList'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'lbl'. | 57 |
| Avoid unused local variables such as 'orgName'. | 66 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 30 - 32 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 61 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 99 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Date' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Date' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 20 |
| Avoid unused imports such as 'java.util.List' | 21 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 224 - 225 |
| Avoid empty catch blocks | 242 - 243 |
| Avoid empty catch blocks | 257 - 258 |
| Avoid empty catch blocks | 418 - 419 |
| These nested if statements could be combined | 432 - 434 |
| Avoid empty catch blocks | 552 |
| Avoid empty catch blocks | 555 |
| Avoid empty catch blocks | 558 |
| Avoid empty catch blocks | 768 - 769 |
| Avoid empty catch blocks | 961 - 962 |
| Avoid empty catch blocks | 1117 - 1118 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 85 - 86 |
| Avoid empty catch blocks | 189 - 190 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 46 - 47 |
| Avoid empty catch blocks | 61 - 62 |
| Avoid empty catch blocks | 103 - 104 |
| Avoid empty catch blocks | 133 - 134 |
| Avoid empty catch blocks | 189 - 190 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'clu'. | 278 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.util.MessageUtils' | 7 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 41 - 43 |
| Avoid unused private methods such as 'getIncludingLos(String)'. | 122 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 580 - 582 |
| Avoid empty catch blocks | 603 - 605 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 34 - 36 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 44 - 46 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 1108 - 1110 |
| Avoid empty catch blocks | 2563 - 2564 |
| Avoid empty catch blocks | 3125 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 122 - 123 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 144 - 145 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 210 - 211 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 188 - 190 |
| Avoid empty catch blocks | 484 - 485 |
| Avoid empty catch blocks | 564 - 565 |
| Avoid empty catch blocks | 565 - 566 |
| Avoid empty catch blocks | 566 - 567 |
| Avoid unused local variables such as 'credentialClu'. | 704 |
| Avoid empty catch blocks | 705 - 706 |
| Avoid empty catch blocks | 720 - 721 |
| Avoid empty catch blocks | 721 - 722 |
| Avoid empty catch blocks | 722 - 723 |
| Avoid empty catch blocks | 866 - 867 |
| Avoid empty catch blocks | 867 - 868 |
| Avoid empty catch blocks | 868 - 869 |
| Avoid empty catch blocks | 887 - 888 |
| Avoid empty catch blocks | 888 - 889 |
| Avoid empty catch blocks | 889 - 890 |
| Avoid empty catch blocks | 975 - 976 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 137 - 138 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.core.exceptions.InvalidParameterException' | 20 |
| Avoid unused imports such as 'org.kuali.student.core.exceptions.MissingParameterException' | 21 |
| Avoid unused imports such as 'org.kuali.student.core.exceptions.OperationFailedException' | 22 |
| Avoid empty catch blocks | 212 - 213 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'statement'. | 651 |
| Avoid empty catch blocks | 1410 |
| These nested if statements could be combined | 1413 - 1421 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 104 - 105 |
| Avoid empty catch blocks | 186 - 187 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 13 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.major.edit.MajorEditController' | 7 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'id'. | 13 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.ProgramStatus' | 6 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 13 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 13 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 14 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'originalReqCompNL'. | 83 |
| Avoid unused private fields such as 'originalLogicExpression'. | 84 |
| Violation | Line |
|---|---|
| Avoid unnecessary return statements | 101 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 26 - 28 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'majorController'. | 33 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'data'. | 97 - 102 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityConfiguration' | 4 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.MultiplicitySection' | 6 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.Section' | 7 |
| Avoid unused imports such as 'org.kuali.student.core.assembly.data.Metadata' | 12 |
| Avoid unused imports such as 'org.kuali.student.core.assembly.data.QueryPath' | 13 |
| Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.core.assembly.data.ModelDefinition' | 7 |
| Avoid unused private fields such as 'initialized'. | 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'viewContext'. | 51 |
| Avoid unused method parameters such as 'versionaLabel'. | 104 - 112 |
| Avoid unused private methods such as 'createHistoryPanel()'. | 163 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getStatus()'. | 158 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.rpc.AsyncCallback' | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'metadataService'. | 26 |
| Avoid unused private fields such as 'mapper'. | 29 |
| Avoid unused private fields such as 'credPgmMetadata'. | 30 |
| Avoid empty catch blocks | 62 - 64 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 129 - 132 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 128 - 130 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 243 - 250 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 115 - 117 |
| These nested if statements could be combined | 125 - 127 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 52 |
| These nested if statements could be combined | 80 - 82 |
| These nested if statements could be combined | 90 - 92 |
| 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 - 74 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 6 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.core.assembly.data.Data.StringKey' | 4 |
| Avoid unused imports such as 'java.util.Date' | 9 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'metadata'. | 73 - 118 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Date' | 20 |
| Avoid unused imports such as 'com.google.gwt.user.client.rpc.AsyncCallback' | 77 |
| Overriding method merely calls super | 492 - 494 |
| Avoid unused local variables such as 'name'. | 621 |
| Avoid unused local variables such as 'typeKey'. | 622 |
| 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.core.dto.MetaInfo' | 5 |
| Violation | Line |
|---|---|
| Error while parsing /opt/hudson/home/jobs/1-1-site-deploy-perf/workspace/ks-1.1-perf/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-1-site-deploy-perf/workspace/ks-1.1-perf/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-1-site-deploy-perf/workspace/ks-1.1-perf/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 |
|---|---|
| No need to import a type that lives in the same package | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 23 |
| 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'. | 483 - 525 |
| Overriding method merely calls super | 528 - 531 |
| 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.widgets.layout.VerticalFlowPanel' | 7 |
| Avoid unused private fields such as 'searchRpcServiceAsync'. | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.SwapSection' | 11 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.event.SectionUpdateEvent' | 12 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 22 |
| Avoid unused imports such as 'org.kuali.student.lum.program.dto.MajorDisciplineInfo' | 25 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'a'. | 60 |
| These nested if statements could be combined | 114 - 122 |
| These nested if statements could be combined | 131 - 138 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.gwt.user.client.ui.Label' | 103 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.VerticalPanel' | 104 |
| Avoid unused constructor parameters such as 'fieldWidget'. | 1281 - 1286 |
| Avoid unused constructor parameters such as 'modelWidgetBinding'. | 1281 - 1286 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 454 - 456 |
| These nested if statements could be combined | 719 - 721 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'STATEMENTS_PATH'. | 77 |
| Avoid unused private fields such as 'ID_TRANSLATION'. | 78 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 652 - 654 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'lastId1'. | 60 |
| Avoid unused private fields such as 'lastId2'. | 61 |
| Avoid unused method parameters such as 'id1Model'. | 130 - 182 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'idType'. | 170 |
| Avoid unused local variables such as 'viewContextId'. | 171 |
| Avoid unused private methods such as 'getCurrentVersion(ModelRequestCallback,Callback)'. | 308 |
| Avoid unused private methods such as 'getQuitButton()'. | 381 |
| Avoid unused private methods such as 'createActionSubmitSuccessHandler()'. | 413 |
| 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'. | 83 |
| Avoid unused private fields such as 'originalLogicExpression'. | 84 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.util.WindowTitleUtils' | 6 |
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.headers.KSDocumentHeader' | 7 |
| 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.widgets.KSErrorDialog' | 19 |
| Avoid unused private methods such as 'addIfPermitted(PermissionType,String)'. | 77 |
| Avoid unused method parameters such as 'searchType'. | 82 - 102 |
| Avoid empty if statements | 91 - 99 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.widgets.CourseWorkflowActionList' | 13 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 125 - 127 |
| Avoid empty if statements | 131 - 133 |
| Avoid unused method parameters such as 'newState'. | 147 - 155 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'onSubmitSuccessHandler'. | 57 - 66 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'test'. | 119 |
| Avoid empty if statements | 139 - 141 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 143 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 145 |
| Avoid unused local variables such as 'currVerPrevState'. | 166 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 25 |
| Avoid unused imports such as 'java.util.Date' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.headers.KSHeader' | 32 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSFooter' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.controllers.CurriculumHomeController.LUMViews' | 6 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'appUrl'. | 85 |
| Avoid unused private fields such as 'codeServer'. | 90 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getLabel(String)'. | 135 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'container'. | 39 |
| Overriding method merely calls super | 142 - 144 |
| 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'. | 258 - 277 |
| Overriding method merely calls super | 288 - 292 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 325 |
| Overriding method merely calls super | 432 - 434 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 107 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.layout.VerticalFlowPanel' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSErrorDialog' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.student.common.ui.shared.IdAttributes' | 8 |
| Avoid unused imports such as 'org.kuali.student.core.assembly.data.AssemblyException' | 9 |
| Avoid unused imports such as 'org.kuali.student.core.assembly.transform.MetadataFilter' | 15 |
| Avoid unused imports such as 'org.kuali.student.core.assembly.transform.TransformFilter' | 16 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 73 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 30 - 32 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 113 - 115 |
| Overriding method merely calls super | 118 - 120 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.xpath.XPathAPI' | 35 |
| Avoid unused imports such as 'org.w3c.dom.NodeList' | 51 |
| Avoid unused local variables such as 'principal'. | 69 |
| Avoid empty if statements | 92 - 94 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.io.StringWriter' | 5 |
| Avoid unused imports such as 'javax.xml.transform.Transformer' | 18 |
| Avoid unused imports such as 'javax.xml.transform.TransformerFactory' | 19 |
| Avoid unused imports such as 'javax.xml.transform.dom.DOMSource' | 20 |
| Avoid unused imports such as 'javax.xml.transform.stream.StreamResult' | 21 |