The following document contains the results of PMD 4.2.2.
Violation | Line |
---|---|
Avoid unused imports such as 'org.apache.commons.beanutils.BeanUtils' | 21 |
Avoid empty catch blocks | 286 - 288 |
These nested if statements could be combined | 326 - 333 |
Avoid empty catch blocks | 413 - 414 |
Avoid empty catch blocks | 414 - 415 |
Avoid empty catch blocks | 415 - 416 |
Avoid empty catch blocks | 436 - 437 |
Avoid empty catch blocks | 437 - 438 |
Avoid empty catch blocks | 438 - 439 |
These nested if statements could be combined | 479 - 486 |
These nested if statements could be combined | 651 - 653 |
These nested if statements could be combined | 690 - 692 |
These nested if statements could be combined | 729 - 731 |
These nested if statements could be combined | 770 - 772 |
These nested if statements could be combined | 810 - 812 |
These nested if statements could be combined | 840 - 842 |
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()'. | 1059 |
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 |
---|---|
Avoid unused imports such as 'java.lang.ref.SoftReference' | 11 |
Avoid unused imports such as 'java.util.concurrent.ConcurrentHashMap' | 19 |
These nested if statements could be combined | 381 - 384 |
These nested if statements could be combined | 397 - 405 |
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/workspace/ks-1.1/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/workspace/ks-1.1/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/workspace/ks-1.1/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 | 345 - 351 |
These nested if statements could be combined | 347 - 350 |
These nested if statements could be combined | 356 - 359 |
Avoid unused local variables such as 'and'. | 471 |
These nested if statements could be combined | 602 - 606 |
These nested if statements could be combined | 740 - 743 |
Avoid unused method parameters such as 'attribute'. | 754 - 770 |
Avoid unused method parameters such as 'attribute'. | 772 - 788 |
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.core.assembly.data.MetadataInterrogator' | 21 |
Avoid unused imports such as 'org.kuali.student.core.assembly.data.MetadataInterrogator.ConstraintIds' | 22 |
Violation | Line |
---|---|
These nested if statements could be combined | 366 - 368 |
These nested if statements could be combined | 417 - 419 |
These nested if statements could be combined | 429 - 432 |
These nested if statements could be combined | 436 - 438 |
Violation | Line |
---|---|
Avoid empty if statements | 67 - 69 |
Violation | Line |
---|---|
Do not use if statements that are always true or always false | 155 |
Violation | Line |
---|---|
These nested if statements could be combined | 491 - 495 |
Violation | Line |
---|---|
These nested if statements could be combined | 88 - 91 |
Violation | Line |
---|---|
Avoid unused constructor parameters such as 'style'. | 45 - 48 |
Violation | Line |
---|---|
Avoid unused private fields such as 'fieldMap'. | 65 |
Violation | Line |
---|---|
An empty statement (semicolon) not part of a loop | 354 |
Violation | Line |
---|---|
These nested if statements could be combined | 53 - 56 |
These nested if statements could be combined | 71 - 74 |
Violation | Line |
---|---|
Overriding method merely calls super | 617 - 619 |
Violation | Line |
---|---|
Avoid unused private fields such as 'title'. | 31 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.FieldDescriptor' | 25 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.LayoutController' | 9 |
Avoid unused imports such as 'org.kuali.student.common.ui.client.event.SectionUpdateEvent' | 10 |
Violation | Line |
---|---|
Avoid unused private fields such as 'controllerId'. | 54 |
Violation | Line |
---|---|
Avoid unused imports such as 'com.google.gwt.user.client.ui.Hidden' | 33 |
Avoid unused private fields such as 'timer'. | 224 |
Violation | Line |
---|---|
Avoid empty catch blocks | 34 - 36 |
Violation | Line |
---|---|
Avoid unused imports such as 'com.google.gwt.user.client.ui.SimplePanel' | 48 |
Violation | Line |
---|---|
Avoid unused imports such as 'com.google.gwt.event.dom.client.BlurEvent' | 24 |
Avoid unused imports such as 'com.google.gwt.event.dom.client.FocusEvent' | 26 |
Violation | Line |
---|---|
Avoid unused private fields such as 'PANEL_CONTENT_REMOVE_LINK'. | 35 |
Violation | Line |
---|---|
Avoid unused imports such as 'com.google.gwt.user.client.ui.Composite' | 38 |
Avoid unused imports such as 'com.google.gwt.user.client.ui.FocusWidget' | 42 |
Avoid unused imports such as 'com.google.gwt.user.client.ui.HasWidgets' | 43 |
Violation | Line |
---|---|
Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyDownEvent' | 23 |
Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyDownHandler' | 24 |
Violation | Line |
---|---|
These nested if statements could be combined | 68 - 70 |
Violation | Line |
---|---|
Avoid unused private fields such as 'mouseOver'. | 53 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.application.ApplicationContext' | 24 |
Avoid unused private fields such as 'appUrl'. | 86 |
Violation | Line |
---|---|
Avoid unused private fields such as 'title'. | 16 |
Violation | Line |
---|---|
Avoid unused private fields such as 'title'. | 38 |
Violation | Line |
---|---|
Avoid unused private fields such as 'margin'. | 80 |
Avoid empty if statements | 413 - 415 |
Violation | Line |
---|---|
Avoid unused constructor parameters such as 'fieldTitle'. | 15 - 17 |
Avoid unused constructor parameters such as 'widget'. | 15 - 17 |
Avoid unused constructor parameters such as 'fieldTitle'. | 19 - 21 |
Avoid unused constructor parameters such as 'widget'. | 19 - 21 |
Avoid unused constructor parameters such as 'key'. | 19 - 21 |
Violation | Line |
---|---|
Avoid unused private fields such as 'lineCount'. | 50 |
Violation | Line |
---|---|
Avoid unused private fields such as 'fieldArea'. | 25 |
Violation | Line |
---|---|
Avoid unused private fields such as 'top'. | 23 |
Avoid unused private fields such as 'buttonPanel'. | 24 |
Violation | Line |
---|---|
These nested if statements could be combined | 66 - 68 |
Violation | Line |
---|---|
Avoid unused private fields such as 'allowedKeyPress'. | 57 |
Avoid unused private fields such as 'hasWatermark'. | 58 |
Avoid unused private fields such as 'watermarkShowing'. | 59 |
Avoid unused private fields such as 'watermarkText'. | 60 |
Violation | Line |
---|---|
Overriding method merely calls super | 51 - 53 |
Violation | Line |
---|---|
These nested if statements could be combined | 278 - 282 |
These nested if statements could be combined | 297 - 301 |
Violation | Line |
---|---|
Avoid unused local variables such as 'w'. | 74 |
Violation | Line |
---|---|
These nested if statements could be combined | 142 - 145 |
These nested if statements could be combined | 153 - 156 |
Violation | Line |
---|---|
These nested if statements could be combined | 149 - 152 |
These nested if statements could be combined | 160 - 163 |
Violation | Line |
---|---|
Avoid unused local variables such as 'ids'. | 178 |
Violation | Line |
---|---|
Avoid unused private fields such as 'buttonLabel'. | 42 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.service.SearchRpcServiceAsync' | 32 |
Avoid unused imports such as 'org.kuali.student.common.ui.client.service.SearchServiceFactory' | 33 |
Avoid unused local variables such as 'value'. | 463 |
These nested if statements could be combined | 642 - 652 |
Violation | Line |
---|---|
Avoid unused private fields such as 'key'. | 42 |
Violation | Line |
---|---|
These nested if statements could be combined | 328 - 338 |
These nested if statements could be combined | 373 - 375 |
These nested if statements could be combined | 447 - 452 |
These nested if statements could be combined | 510 - 520 |
An empty statement (semicolon) not part of a loop | 616 |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 794 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.service.CachingSearchService' | 23 |
Violation | Line |
---|---|
The null check here is misplaced; if the variable is null therell be a NullPointerException | 79 |
The null check here is misplaced; if the variable is null therell be a NullPointerException | 79 |
Violation | Line |
---|---|
Avoid unused method parameters such as 'searchId'. | 152 - 167 |
Avoid unused method parameters such as 'query'. | 169 - 184 |
Violation | Line |
---|---|
These nested if statements could be combined | 71 - 73 |
Violation | Line |
---|---|
Avoid unused private fields such as 'focusType'. | 47 |
Violation | Line |
---|---|
Avoid unused private methods such as 'createSummaryTableFromDataModel()'. | 30 |
Violation | Line |
---|---|
These nested if statements could be combined | 196 - 199 |
These nested if statements could be combined | 212 - 215 |
These nested if statements could be combined | 226 - 229 |
These nested if statements could be combined | 240 - 243 |
Avoid unused constructor parameters such as 'style'. | 264 - 279 |
Violation | Line |
---|---|
Overriding method merely calls super | 42 - 44 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.core.exceptions.MissingParameterException' | 21 |
Violation | Line |
---|---|
Avoid duplicate imports such as 'org.kuali.rice.kim.service.IdentityManagementService' | 26 |
Violation | Line |
---|---|
Avoid unused method parameters such as 'request'. | 45 - 224 |
Violation | Line |
---|---|
These nested if statements could be combined | 31 - 33 |
Violation | Line |
---|---|
Avoid unused imports such as 'java.util.Properties' | 35 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.messagebuilder.impl.exceptions.MessageBuilderException' | 22 |
Violation | Line |
---|---|
An empty statement (semicolon) not part of a loop | 42 |
Avoid empty finally blocks | 67 - 68 |
Avoid empty finally blocks | 87 - 88 |
Avoid empty finally blocks | 107 - 108 |
Avoid empty finally blocks | 127 - 128 |
Avoid empty finally blocks | 183 - 184 |
Avoid empty finally blocks | 232 - 233 |
Avoid empty finally blocks | 290 - 291 |
Avoid empty finally blocks | 310 - 311 |
Avoid empty finally blocks | 326 - 327 |
Violation | Line |
---|---|
These nested if statements could be combined | 96 - 97 |
Avoid empty finally blocks | 146 - 147 |
Avoid empty finally blocks | 232 - 233 |
Avoid empty finally blocks | 318 - 319 |
Avoid unused local variables such as 'LP9'. | 338 |
Avoid unused local variables such as 'RP11'. | 339 |
Avoid unused local variables such as 'LP9_tree'. | 344 |
Avoid unused local variables such as 'RP11_tree'. | 345 |
Avoid empty finally blocks | 411 - 412 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.messagebuilder.impl.exceptions.MessageBuilderException' | 24 |
Violation | Line |
---|---|
Avoid empty catch blocks | 90 |
Avoid empty catch blocks | 141 |
Violation | Line |
---|---|
Avoid empty catch blocks | 83 - 84 |
Violation | Line |
---|---|
Avoid empty catch blocks | 100 - 102 |
Violation | Line |
---|---|
Avoid empty catch blocks | 54 - 56 |
Violation | Line |
---|---|
Avoid empty catch blocks | 55 - 57 |
Avoid empty catch blocks | 74 - 76 |
Violation | Line |
---|---|
Avoid empty catch blocks | 54 - 55 |
Violation | Line |
---|---|
Avoid unused imports such as 'java.util.ArrayList' | 19 |
Avoid unused imports such as 'java.util.Date' | 20 |
Avoid unused imports such as 'java.util.HashMap' | 21 |
Avoid unused imports such as 'java.util.List' | 22 |
Avoid unused imports such as 'java.util.Map' | 23 |
Avoid unused imports such as 'javax.xml.bind.annotation.XmlAttribute' | 27 |
Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' | 29 |
Avoid unused imports such as 'org.kuali.student.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 | 90 |
Avoid empty catch blocks | 90 |
Avoid empty catch blocks | 134 |
Avoid empty catch blocks | 134 |
Avoid empty catch blocks | 164 |
Avoid empty catch blocks | 164 |
Violation | Line |
---|---|
Overriding method merely calls super | 41 - 43 |
Violation | Line |
---|---|
Overriding method merely calls super | 45 - 47 |
Violation | Line |
---|---|
An empty statement (semicolon) not part of a loop | 296 |
Violation | Line |
---|---|
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 | 102 - 103 |
Avoid empty catch blocks | 139 - 140 |
Avoid empty catch blocks | 181 - 182 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.rice.kim.service.IdentityService' | 27 |
Violation | Line |
---|---|
Overriding method merely calls super | 43 - 45 |
Violation | Line |
---|---|
Overriding method merely calls super | 36 - 38 |
Violation | Line |
---|---|
Avoid unused private fields such as 'statementTypeId'. | 16 |
Avoid unused private fields such as 'reqComponentTypeId'. | 19 |
Violation | Line |
---|---|
Avoid unused private fields such as 'statementTypeId'. | 16 |
Avoid unused private fields such as 'childStatementTypeId'. | 19 |
Violation | Line |
---|---|
Avoid unused imports such as '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 | 497 - 499 |
Avoid empty catch blocks | 528 - 530 |
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'. | 311 - 359 |
Avoid unused method parameters such as 'deletionParentKeys'. | 311 - 359 |
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 | 168 - 173 |
These nested if statements could be combined | 210 - 216 |
Avoid unused private methods such as 'getDeeperNode(List)'. | 265 |
Avoid unused private methods such as 'findNodeIndex(List,int)'. | 382 |
Avoid unused private methods such as 'hasParenthesis(List)'. | 393 |
Violation | Line |
---|---|
Avoid unused local variables such as 'rf'. | 87 |
Violation | Line |
---|---|
Avoid unused private fields such as '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 - 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 duplicate imports such as 'org.kuali.rice.kim.service.IdentityManagementService' | 23 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.list.KSSelectedList' | 28 |
Violation | Line |
---|---|
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 60 |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 98 |
Violation | Line |
---|---|
Avoid unused imports such as 'java.util.ArrayList' | 19 |
Violation | Line |
---|---|
Avoid unused imports such as 'java.util.Date' | 20 |
Violation | Line |
---|---|
Avoid unused imports such as 'java.util.Date' | 20 |
Violation | Line |
---|---|
Avoid unused imports such as 'java.util.ArrayList' | 20 |
Avoid unused imports such as 'java.util.List' | 21 |
Violation | Line |
---|---|
No need to import a type that lives in the same package | 18 |
Violation | Line |
---|---|
Avoid empty catch blocks | 224 - 225 |
Avoid empty catch blocks | 242 - 243 |
Avoid empty catch blocks | 257 - 258 |
Avoid empty catch blocks | 429 - 430 |
These nested if statements could be combined | 443 - 445 |
Avoid empty catch blocks | 563 |
Avoid empty catch blocks | 566 |
Avoid empty catch blocks | 569 |
Avoid empty catch blocks | 780 - 781 |
Avoid empty catch blocks | 975 - 976 |
Avoid empty catch blocks | 1138 - 1139 |
Violation | Line |
---|---|
Avoid empty catch blocks | 87 - 88 |
Avoid empty catch blocks | 193 - 194 |
Violation | Line |
---|---|
Avoid empty catch blocks | 46 - 47 |
Avoid empty catch blocks | 61 - 62 |
Avoid empty catch blocks | 103 - 104 |
Avoid empty catch blocks | 133 - 134 |
Avoid empty catch blocks | 189 - 190 |
Violation | Line |
---|---|
Avoid unused local variables such as 'clu'. | 284 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.util.MessageUtils' | 7 |
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 | 1116 - 1118 |
Avoid empty catch blocks | 2596 - 2597 |
Avoid empty catch blocks | 3168 |
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'. | 663 |
Avoid empty catch blocks | 1434 |
These nested if statements could be combined | 1437 - 1445 |
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.common.ui.client.widgets.KSErrorDialog' | 7 |
Avoid unused imports such as 'org.kuali.student.core.assembly.data.Metadata' | 8 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 13 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 13 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' | 14 |
Violation | Line |
---|---|
Avoid unused private fields such as 'originalReqCompNL'. | 83 |
Avoid unused private fields such as 'originalLogicExpression'. | 84 |
Violation | Line |
---|---|
Avoid unnecessary return statements | 107 |
Violation | Line |
---|---|
Overriding method merely calls super | 26 - 28 |
Violation | Line |
---|---|
Avoid unused 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'. | 113 - 121 |
Avoid unused private methods such as 'createHistoryPanel()'. | 172 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.lum.program.client.ProgramClientConstants' | 24 |
Avoid unused imports such as 'org.kuali.student.lum.program.dto.CoreProgramInfo' | 26 |
Avoid unused imports such as 'org.kuali.student.lum.program.dto.CredentialProgramInfo' | 27 |
Avoid unused private fields such as 'metadataService'. | 38 |
Avoid unused private fields such as 'programService'. | 39 |
Avoid unused private fields such as 'mapper'. | 41 |
Avoid unused private fields such as 'credPgmMetadata'. | 42 |
Violation | Line |
---|---|
These nested if statements could be combined | 129 - 132 |
Violation | Line |
---|---|
These nested if statements could be combined | 128 - 130 |
Violation | Line |
---|---|
These nested if statements could be combined | 243 - 250 |
Violation | Line |
---|---|
Avoid unused local variables such as 'resolvedOrgShortName'. | 157 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.rice.student.bo.KualiStudentKimAttributes' | 22 |
Avoid unused imports such as 'org.kuali.student.core.search.dto.SearchParam' | 24 |
Avoid unused imports such as 'org.kuali.student.core.search.dto.SearchRequest' | 25 |
Avoid unused imports such as 'org.kuali.student.core.search.dto.SearchResult' | 26 |
Avoid unused imports such as 'org.kuali.student.core.search.dto.SearchResultCell' | 27 |
These nested if statements could be combined | 113 - 115 |
Violation | Line |
---|---|
Avoid unused private fields such as 'LOG'. | 51 |
These nested if statements could be combined | 77 - 79 |
Violation | Line |
---|---|
Avoid unused imports such as 'javax.xml.namespace.QName' | 9 |
Avoid unused imports such as 'org.kuali.rice.core.resourceloader.GlobalResourceLoader' | 12 |
Avoid unused imports such as 'org.kuali.rice.kew.role.QualifierResolver' | 14 |
These nested if statements could be combined | 62 - 73 |
Violation | Line |
---|---|
Avoid unused imports such as 'java.util.HashMap' | 6 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.core.assembly.data.Data.StringKey' | 4 |
Avoid unused imports such as 'java.util.Date' | 9 |
Violation | Line |
---|---|
Avoid unused imports such as 'com.google.gwt.user.client.ui.VerticalPanel' | 39 |
Violation | Line |
---|---|
Overriding method merely calls super | 460 - 462 |
Avoid unused local variables such as 'name'. | 589 |
Avoid unused local variables such as 'typeKey'. | 590 |
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/workspace/ks-1.1/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/workspace/ks-1.1/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/workspace/ks-1.1/ks-lum/ks-lum-ui-common/src/main/java/org/kuali/student/lum/common/client/lo/LoDisplayInfoSortedSet.java: Encountered "(" at line 13, column 14. Was expecting one of: "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... "." ... | 0 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 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 | 294 - 296 |
Violation | Line |
---|---|
Avoid unused imports such as 'java.util.Set' | 7 |
Avoid unused imports such as 'com.google.gwt.user.client.Window' | 51 |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 95 |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 105 |
Avoid unused method parameters such as 'searchParams'. | 490 - 532 |
Overriding method merely calls super | 535 - 538 |
Violation | Line |
---|---|
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 144 |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 151 |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 159 |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 166 |
Violation | Line |
---|---|
These nested if statements could be combined | 35 - 41 |
Violation | Line |
---|---|
Avoid unused method parameters such as 'enabled'. | 126 - 139 |
Violation | Line |
---|---|
Avoid unused method parameters such as 'enabled'. | 126 - 139 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.SwapSection' | 11 |
Avoid unused imports such as 'org.kuali.student.common.ui.client.event.SectionUpdateEvent' | 12 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 22 |
Avoid unused imports such as 'org.kuali.student.lum.program.dto.MajorDisciplineInfo' | 25 |
Violation | Line |
---|---|
Avoid unused local variables such as 'a'. | 60 |
These nested if statements could be combined | 114 - 122 |
These nested if statements could be combined | 131 - 138 |
Violation | Line |
---|---|
Avoid unused constructor parameters such as 'fieldWidget'. | 1232 - 1237 |
Avoid unused constructor parameters such as 'modelWidgetBinding'. | 1232 - 1237 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.progress.KSBlockingProgressIndicator' | 23 |
Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.controllers.CourseProposalController' | 47 |
Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.requirements.CourseRequirementsDataModel' | 48 |
Avoid unused private fields such as 'loadDataTask'. | 93 |
These nested if statements could be combined | 460 - 462 |
These nested if statements could be combined | 717 - 719 |
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 | 681 - 683 |
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'. | 174 |
Avoid unused local variables such as 'viewContextId'. | 175 |
Avoid unused private methods such as 'getCurrentVersion(ModelRequestCallback,Callback)'. | 312 |
Avoid unused private methods such as 'getQuitButton()'. | 385 |
Avoid unused private methods such as 'createActionSubmitSuccessHandler()'. | 417 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.breadcrumb.BreadcrumbManager' | 6 |
Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.history.HistoryManager' | 7 |
Violation | Line |
---|---|
Avoid unused private fields such as 'originalReqCompNL'. | 83 |
Avoid unused private fields such as 'originalLogicExpression'. | 84 |
Violation | Line |
---|---|
Avoid unused constructor parameters such as 'modelId'. | 10 - 12 |
Avoid unused constructor parameters such as 'viewEnum'. | 10 - 12 |
Avoid unused constructor parameters such as 'name'. | 10 - 12 |
Avoid unused constructor parameters such as 'controller'. | 10 - 12 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.widgets.CourseWorkflowActionList' | 13 |
Violation | Line |
---|---|
Avoid empty if statements | 125 - 127 |
Avoid empty if statements | 131 - 133 |
Avoid unused method parameters such as 'newState'. | 147 - 155 |
Violation | Line |
---|---|
Avoid unused constructor parameters such as 'onSubmitSuccessHandler'. | 57 - 66 |
Violation | Line |
---|---|
Avoid empty if statements | 44 - 46 |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 48 |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 50 |
Avoid unused local variables such as 'currVerPrevState'. | 71 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' | 25 |
Avoid unused imports such as 'java.util.Date' | 28 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.application.Application' | 7 |
Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.ApplicationPanel' | 14 |
Avoid unused imports such as 'com.google.gwt.user.client.DOM' | 19 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.controllers.CurriculumHomeController.LUMViews' | 6 |
Violation | Line |
---|---|
Avoid unused imports such as 'org.kuali.student.common.ui.client.application.ApplicationContext' | 24 |
Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.ApplicationPanel' | 32 |
Avoid unused imports such as 'com.google.gwt.user.client.ui.HTML' | 55 |
Avoid unused imports such as 'com.google.gwt.user.client.ui.Hidden' | 56 |
Avoid unused imports such as 'com.google.gwt.user.client.ui.PopupPanel' | 57 |
Avoid unused private fields such as 'versionAnchor'. | 80 |
Avoid unused private fields such as 'appUrl'. | 88 |
Avoid unused private fields such as 'appVersion'. | 92 |
Avoid unused private fields such as 'codeServer'. | 93 |
Violation | Line |
---|---|
Avoid unused private methods such as 'getLabel(String)'. | 135 |
Violation | Line |
---|---|
Avoid unused private fields such as 'container'. | 39 |
Overriding method merely calls super | 133 - 135 |
Violation | Line |
---|---|
Avoid unused local variables such as 'cluRangeFieldEditDescriptor'. | 135 |
Avoid unused private methods such as 'createCluSetSection()'. | 225 |
Avoid unused private methods such as 'editCluSetSection()'. | 311 |
Avoid unused private methods such as 'viewCluSetSection()'. | 352 |
Violation | Line |
---|---|
Avoid unused method parameters such as 'modelDefinition'. | 255 - 274 |
Overriding method merely calls super | 277 - 281 |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 314 |
Avoid unused method parameters such as 'saveActionEvent'. | 326 - 366 |
Overriding method merely calls super | 392 - 394 |
Violation | Line |
---|---|
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 109 |
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 'com.google.gwt.core.client.GWT' | 34 |
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 |