Parent Project
Modules
Project Documentation
The following document contains the results of PMD 4.2.2.
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 19 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 41 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 29 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 118 - 124 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 61 |
| Avoid unused local variables such as 'memberPrefix'. | 81 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 33 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlRootElement' | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'fetchLimit'. | 63 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Date' | 18 |
| Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.jaxb.SqlDateAdapter' | 22 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.jaxb.SqlDateAdapter' | 22 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 131 - 132 |
| Violation | Line |
|---|---|
| Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. | 54 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 78 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'keys'. | 81 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 326 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 125 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'CONFIG_CODED_DEFAULTS'. | 55 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 85 - 87 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 19 |
| Avoid unused imports such as 'java.util.Map' | 21 |
| Avoid unused imports such as 'org.kuali.rice.core.exception.RiceRuntimeException' | 29 |
| Avoid unused imports such as 'org.kuali.rice.core.util.ClassLoaderUtils' | 33 |
| Avoid unused private fields such as 'LOG'. | 53 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 230 - 238 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'baseProperties'. | 40 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.SQLException' | 19 |
| Avoid unused imports such as 'java.util.Map' | 20 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 79 - 80 |
| Avoid empty catch blocks | 85 - 86 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 92 - 93 |
| Avoid empty catch blocks | 98 - 99 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'bd'. | 289 |
| Avoid unused method parameters such as 'allowWildcards'. | 388 - 399 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'bindParamCount'. | 60 |
| Avoid unused method parameters such as 'parentCriteria'. | 347 - 370 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 19 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 74 |
| Avoid empty catch blocks | 139 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 77 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 79 |
| Avoid empty catch blocks | 82 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 21 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 21 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 126 - 128 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 20 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 21 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 20 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 22 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 22 |
| Overriding method merely calls super | 45 - 47 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 25 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'suppressedServices'. | 68 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 220 - 222 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'name'. | 54 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 33 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 33 - 36 |
| These nested if statements could be combined | 47 - 53 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 131 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.config.Config' | 31 |
| Avoid unused imports such as 'org.kuali.rice.core.exception.RiceRuntimeException' | 33 |
| Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! | 109 |
| Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! | 126 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'supportedVersion'. | 125 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 211 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.io.IOException' | 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.xml.sax.XMLFilter' | 18 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 174 - 187 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.xml.sax.SAXException' | 19 |
| Avoid unused imports such as 'org.xml.sax.SAXParseException' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedList' | 18 |
| Avoid unused imports such as 'java.util.List' | 19 |
| Avoid unused imports such as 'org.kuali.rice.core.lifecycle.Lifecycle' | 22 |
| Avoid unused imports such as 'org.kuali.rice.core.ojb.BaseOjbConfigurer' | 23 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'mode'. | 73 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 19 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 89 - 91 |
| Avoid empty if statements | 106 - 108 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.namespace.QName' | 27 |
| Avoid unused imports such as 'org.kuali.rice.core.config.ConfigContext' | 30 |
| Avoid unused imports such as 'org.kuali.rice.core.resourceloader.GlobalResourceLoader' | 31 |
| Avoid unused imports such as 'org.kuali.rice.ken.service.KENAPIService' | 38 |
| Avoid unused imports such as 'org.kuali.rice.ken.service.KENServiceConstants' | 39 |
| Avoid unused private fields such as 'KEW_CHANNEL'. | 53 |
| Avoid unused local variables such as 'error'. | 146 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 249 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.CascadeType' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.JoinColumns' | 24 |
| Avoid unused private fields such as 'notification'. | 49 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notification'. | 46 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.beans.factory.BeanInitializationException' | 24 |
| Avoid unused private fields such as 'LOG'. | 33 |
| Avoid unused private fields such as 'dataSource'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'NOTIFICATION_MESSAGE_DELIVERY_DISPATCH_SERVICE'. | 47 |
| Avoid unused private fields such as 'NOTIFICATION_MESSAGE_DELIVERER_REGISTRY_SERVICE'. | 52 |
| Avoid unused private fields such as 'NOTIFICATION_EMAIL_SERVICE'. | 55 |
| Avoid unused private fields such as 'NOTIFICATION_CONFIG'. | 56 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 18 |
| Avoid unused imports such as 'java.util.LinkedHashMap' | 19 |
| Avoid unused imports such as 'org.kuali.rice.ken.exception.ErrorList' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 20 |
| Avoid unused imports such as 'java.util.LinkedHashMap' | 21 |
| Avoid unused imports such as 'org.kuali.rice.ken.exception.ErrorList' | 28 |
| Avoid empty if statements | 135 - 138 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.NetworkIdDTO' | 38 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' | 21 |
| Avoid unused imports such as 'org.xml.sax.SAXException' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 18 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'prefsService'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Avoid unused private fields such as 'businessObjectDao'. | 37 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.ojb.broker.query.QueryByCriteria' | 23 |
| Avoid unused imports such as 'org.apache.ojb.broker.query.QueryFactory' | 24 |
| Avoid unused imports such as 'org.kuali.rice.kew.doctype.bo.DocumentType' | 29 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'content'. | 368 - 384 |
| Avoid unused method parameters such as 'notification'. | 368 - 384 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'firstMessageDelivery'. | 70 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'userPreferenceService'. | 56 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notificationWorkflowDocumentService'. | 52 |
| These nested if statements could be combined | 141 - 145 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notificationChannelService'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 245 - 251 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 369 - 447 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 396 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'response'. | 303 - 335 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.web.servlet.view.RedirectView' | 40 |
| Avoid unused method parameters such as 'response'. | 291 - 346 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 602 - 604 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ken.deliverer.NotificationMessageDeliverer' | 31 |
| Avoid unused imports such as 'org.kuali.rice.ken.exception.ErrorList' | 32 |
| An empty statement (semicolon) not part of a loop | 149 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 178 - 180 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 577 |
| Avoid empty catch blocks | 582 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'ROUTEHEADERID_PROP'. | 88 |
| Avoid unused private fields such as 'DOCTITLE_PROP'. | 91 |
| Avoid unused private fields such as 'ACTIONITEM_ROUTEHEADERID_INVALID_ERRKEY'. | 96 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 377 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 386 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 389 |
| Avoid unused local variables such as 'index'. | 617 |
| Avoid unused method parameters such as 'request'. | 772 - 774 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 23 |
| Avoid unused method parameters such as 'request'. | 183 - 185 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.action.ActionErrors' | 50 |
| Avoid empty catch blocks | 219 - 221 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'groupName'. | 178 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.service.KIMServiceLocator' | 19 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 383 - 386 |
| Avoid unused local variables such as 'allDeactivated'. | 458 |
| Avoid unused local variables such as 'matchedArs'. | 519 |
| Avoid unused local variables such as 'arGroups'. | 536 |
| These nested if statements could be combined | 699 - 701 |
| These nested if statements could be combined | 815 - 819 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.engine.RouteContext' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 28 |
| These nested if statements could be combined | 141 - 143 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 28 |
| These nested if statements could be combined | 144 - 146 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 27 |
| These nested if statements could be combined | 130 - 132 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 27 |
| These nested if statements could be combined | 139 - 141 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 200 - 205 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'actionTaken'. | 80 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 19 |
| These nested if statements could be combined | 103 - 114 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 19 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 148 - 156 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 21 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'successful'. | 195 - 215 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'parameters'. | 91 |
| An empty statement (semicolon) not part of a loop | 133 |
| Avoid unused local variables such as 'pkNames'. | 183 |
| Avoid empty catch blocks | 278 - 280 |
| Avoid empty if statements | 391 - 393 |
| An empty statement (semicolon) not part of a loop | 886 |
| Avoid empty catch blocks | 1102 - 1105 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'returnKeys'. | 135 |
| Avoid unused local variables such as 'pkNames'. | 136 |
| Avoid unused local variables such as 'user'. | 137 |
| Avoid empty try blocks | 188 - 191 |
| Avoid empty catch blocks | 191 - 194 |
| Avoid empty if statements | 256 - 258 |
| An empty statement (semicolon) not part of a loop | 427 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 233 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 583 - 585 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 72 - 75 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 18 |
| Avoid unused imports such as 'java.util.Map' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kew.util.Utilities' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashSet' | 20 |
| Avoid unused imports such as 'java.util.List' | 21 |
| Avoid unused imports such as 'java.util.Map' | 22 |
| Avoid unused imports such as 'java.util.HashMap' | 23 |
| Avoid unused imports such as 'java.util.Set' | 24 |
| Avoid unused imports such as 'org.kuali.rice.kew.engine.node.KeyValuePair' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 21 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'searchCriteriaDisplayed'. | 36 - 38 |
| Avoid unused constructor parameters such as 'headerBarDisplayed'. | 36 - 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'CREATE_DATE_FIELD_STRING'. | 84 |
| Avoid unused private fields such as 'APPROVE_DATE_FIELD_STRING'. | 85 |
| Avoid unused private fields such as 'FINALIZATION_DATE_FIELD_STRING'. | 86 |
| Avoid unused private fields such as 'LAST_STATUS_UPDATE_DATE'. | 87 |
| Avoid unused private fields such as 'STATUS_TRANSITION_DATE_FIELD_STRING'. | 88 |
| Avoid unused private methods such as 'parseAndConvertDateToRange(String)'. | 295 |
| Avoid unused local variables such as 'queryTableColumnName'. | 648 |
| These nested if statements could be combined | 1735 - 1737 |
| Avoid unused private methods such as 'tokenizeCriteria(String)'. | 1833 |
| Avoid unused private methods such as 'validateNumber(List)'. | 1840 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 49 |
| Avoid unnecessary return statements | 268 |
| These nested if statements could be combined | 290 - 307 |
| These nested if statements could be combined | 291 - 306 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 339 |
| Violation | Line |
|---|---|
| Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. | 92 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'documentSecurityService'. | 67 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 256 - 262 |
| These nested if statements could be combined | 284 - 290 |
| These nested if statements could be combined | 312 - 318 |
| These nested if statements could be combined | 340 - 346 |
| Avoid unused private methods such as 'validateNetworkId(List)'. | 351 |
| Avoid unused local variables such as 'pList'. | 380 |
| Avoid unused private methods such as 'getValidDocumentType(String)'. | 734 |
| Avoid unused private methods such as 'getValidDocumentTypeOld(String)'. | 761 |
| Avoid unused private methods such as 'tokenizeCriteria(String)'. | 848 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.action.ActionForm' | 19 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 280 - 282 |
| These nested if statements could be combined | 394 - 396 |
| Avoid unused method parameters such as 'dataType'. | 419 - 425 |
| Avoid unused method parameters such as 'quickfinderService'. | 443 - 472 |
| Avoid unused private methods such as 'getBooleanWithPotentialOverrides(String,NamedNodeMap,NamedNodeMap,NamedNodeMap)'. | 532 |
| These nested if statements could be combined | 609 - 611 |
| Avoid unused local variables such as 'rangeDefinitionAttributes'. | 729 |
| Avoid empty if statements | 771 - 781 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.GeneratedValue' | 22 |
| Avoid unused imports such as 'javax.persistence.SequenceGenerator' | 27 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.FetchType' | 23 |
| Avoid unused imports such as 'javax.persistence.JoinColumn' | 26 |
| Avoid unused imports such as 'javax.persistence.ManyToOne' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kew.doctype.bo.DocumentType' | 32 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 155 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.io.IOException' | 19 |
| Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' | 26 |
| Avoid unused imports such as 'org.kuali.rice.core.reflect.ObjectDefinition' | 31 |
| Avoid unused imports such as 'org.kuali.rice.core.resourceloader.GlobalResourceLoader' | 32 |
| Avoid unused imports such as 'org.xml.sax.SAXException' | 48 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 88 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 91 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 100 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 103 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 113 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 116 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1241 - 1243 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'docType'. | 62 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.util.Utilities' | 33 |
| These nested if statements could be combined | 138 - 140 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 94 - 96 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.rule.RuleDelegation' | 28 |
| These nested if statements could be combined | 167 - 170 |
| These nested if statements could be combined | 177 - 180 |
| These nested if statements could be combined | 221 - 225 |
| Avoid unused local variables such as 'extensions'. | 243 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.doctype.bo.DocumentType' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.notes.Note' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.text.SimpleDateFormat' | 22 |
| Avoid empty if statements | 631 - 633 |
| Violation | Line |
|---|---|
| This usage of the Collection.toArray() method will throw a ClassCastException. | 1319 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 191 - 193 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 97 - 100 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 96 - 105 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kew.user.AuthenticationUserId' | 23 |
| Avoid duplicate imports such as 'org.kuali.rice.kim.bo.entity.KimPrincipal' | 24 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 189 |
| Avoid empty if statements | 193 - 195 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'currentPage'. | 57 - 65 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'edlContext'. | 54 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 41 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 82 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.NetworkIdDTO' | 20 |
| Avoid unused local variables such as 'requestParser'. | 92 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 25 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'nextNodeInstance'. | 325 - 333 |
| Avoid unused method parameters such as 'nodeInstance'. | 557 - 579 |
| Avoid unused method parameters such as 'nodeInstanceId'. | 593 - 614 |
| Avoid unused method parameters such as 'nodeInstanceId'. | 656 - 677 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'routeHeaderId'. | 52 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 23 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 122 - 124 |
| Overriding method merely calls super | 154 - 156 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 76 - 79 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.actionrequest.ActionRequestValue' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 28 |
| These nested if statements could be combined | 129 - 132 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 32 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 294 - 296 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.PrePersist' | 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.engine.node.Branch' | 37 |
| Avoid unused local variables such as 'i'. | 117 |
| Avoid unused local variables such as 'documentType'. | 158 |
| Avoid empty if statements | 164 - 189 |
| Avoid unused method parameters such as 'helper'. | 229 - 278 |
| Avoid unused method parameters such as 'helper'. | 390 - 410 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 205 - 207 |
| Avoid empty catch blocks | 337 - 339 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 107 - 110 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'doc'. | 54 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 20 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'constructSubject(FeedbackForm)'. | 135 |
| Avoid unused private methods such as 'constructEmailBody(FeedbackForm)'. | 143 |
| Avoid unused method parameters such as 'request'. | 163 - 165 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'errors'. | 107 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.util.RiceConstants' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kns.exception.ValidationException' | 37 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.action.ActionForm' | 19 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'logger'. | 49 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 19 |
| Avoid unused imports such as 'java.util.List' | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 64 - 66 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 34 |
| No need to import a type that lives in the same package | 35 |
| No need to import a type that lives in the same package | 36 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 232 - 234 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.CascadeType' | 21 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'bytesWritten'. | 96 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 207 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 236 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 254 |
| Avoid unused method parameters such as 'request'. | 320 - 322 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.util.KEWConstants' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kew.util.Utilities' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 24 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 80 - 83 |
| Avoid empty catch blocks | 141 - 143 |
| Avoid empty catch blocks | 155 - 157 |
| Avoid empty catch blocks | 168 - 170 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.config.Config' | 29 |
| Avoid unused imports such as 'org.kuali.rice.core.config.ConfigContext' | 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.KimPrincipal' | 20 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_COLOR'. | 74 |
| Avoid unused private fields such as 'DEFAULT_ACTION_LIST_SIZE'. | 75 |
| Avoid unused private fields such as 'DEFAULT_REFRESH_RATE'. | 76 |
| Avoid unused local variables such as 'stagingDirectory'. | 98 |
| Avoid empty if statements | 190 - 192 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'request'. | 103 - 105 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowServiceErrorImpl' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kew.util.KEWConstants' | 23 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 42 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 81 - 82 |
| Avoid empty catch blocks | 88 - 89 |
| Avoid empty catch blocks | 95 - 96 |
| Avoid empty catch blocks | 102 - 103 |
| Avoid empty catch blocks | 169 - 170 |
| Avoid empty catch blocks | 175 - 176 |
| Avoid empty catch blocks | 214 - 215 |
| Avoid empty catch blocks | 220 - 221 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.action.ActionMessages' | 28 |
| Avoid unused imports such as 'org.kuali.rice.kew.web.KeyValue' | 32 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.GlobalVariables' | 34 |
| Avoid unused method parameters such as 'request'. | 86 - 88 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'removeReplaceDocument'. | 49 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'removeReplaceDocument'. | 49 |
| Violation | Line |
|---|---|
| Do not use if statements that are always true or always false | 135 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'INVALID_REPLACEMENT_USER_ID_MSG'. | 64 |
| Avoid unused method parameters such as 'request'. | 490 - 492 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.web.WorkflowRoutingForm' | 27 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 38 |
| 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.rice.kew.exception.ResourceUnavailableException' | 45 |
| Avoid unused private fields such as 'searchableAttributeValues'. | 178 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kew.user.UserUtils' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kew.web.session.UserSession' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.dto.KimEntityDefaultInfo' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.jpa.annotations.Sequence' | 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.IdClass' | 24 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'routeContext'. | 76 - 93 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.ojb.broker.query.Criteria' | 32 |
| Avoid unused imports such as 'org.apache.ojb.broker.query.QueryFactory' | 33 |
| Avoid unused imports such as 'org.apache.ojb.broker.query.ReportQueryByCriteria' | 34 |
| Avoid empty if statements | 152 - 154 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 133 - 134 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'principal'. | 451 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAttributes' | 63 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.types.dto.AttributeSet' | 64 |
| Avoid unused imports such as 'org.kuali.rice.kim.service.KIMServiceLocator' | 65 |
| Avoid unused imports such as 'org.kuali.rice.kim.util.KimConstants' | 66 |
| Avoid unused imports such as 'org.kuali.rice.kns.exception.AuthorizationException' | 67 |
| These nested if statements could be combined | 300 - 303 |
| Avoid unused method parameters such as 'request'. | 312 - 314 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 26 |
| Avoid unused private fields such as 'LOG'. | 56 |
| Avoid empty if statements | 123 - 125 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'ruleTemplateName'. | 80 - 82 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'makeActionRequests(ActionRequestFactory,RouteContext,RuleBaseValues,DocumentRouteHeaderValue,ActionRequestValue,RuleDelegation)'. | 238 |
| Avoid empty catch blocks | 355 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'parseParameters(String)'. | 57 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 442 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 445 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 448 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 460 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.ManyToOne' | 27 |
| Avoid unused imports such as 'javax.persistence.Transient' | 30 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 98 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.GeneratedValue' | 19 |
| Avoid unused imports such as 'javax.persistence.GenerationType' | 20 |
| Avoid unused imports such as 'javax.persistence.SequenceGenerator' | 26 |
| Avoid unused imports such as 'javax.persistence.Transient' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.Guid' | 42 |
| Ensure you override both equals() and hashCode() | 190 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 135 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'documentContent'. | 92 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.util.KEWPropertyConstants' | 25 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'ROLE_REVIEWER_PROPERTY_NAME'. | 88 |
| Avoid unused private fields such as 'DOC_TYP_LOOKUPABLE'. | 93 |
| Avoid unused private fields such as 'RULE_TEMPLATE_LOOKUPABLE'. | 94 |
| Avoid unused private fields such as 'WORKGROUP_LOOKUPABLE'. | 95 |
| Avoid unused private fields such as 'PERSON_LOOKUPABLE'. | 96 |
| Avoid unused private fields such as 'WORKGROUP_ID_PROPERTY_NAME'. | 98 |
| Avoid unused local variables such as 'i'. | 123 |
| Avoid unused local variables such as 'errors'. | 192 |
| Avoid unused local variables such as 'attributeLabel'. | 263 |
| Avoid unused local variables such as 'attributeLabel'. | 447 |
| Avoid unused local variables such as 'attributeLabel'. | 455 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'RULE_TEMPLATE_ID_PROPERTY_NAME'. | 77 |
| Avoid unused private fields such as 'DOC_TYP_LOOKUPABLE'. | 88 |
| Avoid unused private fields such as 'RULE_TEMPLATE_LOOKUPABLE'. | 89 |
| Avoid unused private fields such as 'WORKGROUP_LOOKUPABLE'. | 90 |
| Avoid unused private fields such as 'PERSON_LOOKUPABLE'. | 91 |
| Avoid unused local variables such as 'i'. | 116 |
| Avoid unused local variables such as 'delegationWizard'. | 200 |
| Avoid unused local variables such as 'isDelegateRule'. | 207 |
| Avoid unused local variables such as 'attributeLabel'. | 242 |
| Avoid unused local variables such as 'attributeLabel'. | 410 |
| Avoid unused local variables such as 'attributeLabel'. | 419 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.Transient' | 30 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 266 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 42 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 121 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 118 |
| These nested if statements could be combined | 219 - 221 |
| Avoid unused private methods such as 'getWorkgroupOrCriteria(Collection)'. | 386 |
| Avoid unused private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. | 409 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. | 143 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.jpa.criteria.Criteria' | 22 |
| Avoid unused imports such as 'org.kuali.rice.core.jpa.criteria.QueryByCriteria' | 23 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 130 - 132 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 920 - 923 |
| Avoid empty if statements | 950 - 952 |
| Avoid empty if statements | 1000 - 1002 |
| These nested if statements could be combined | 1329 - 1332 |
| These nested if statements could be combined | 1418 - 1428 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 78 - 81 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'initiatorPrincipal'. | 332 |
| Avoid unused method parameters such as 'isDelegate'. | 373 - 434 |
| Avoid unused private methods such as 'makeLookupPathParam(ActionMapping,HttpServletRequest)'. | 448 |
| Avoid unused method parameters such as 'request'. | 461 - 463 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Properties' | 23 |
| Avoid unused imports such as 'org.apache.struts.action.ActionForm' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 39 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.UrlFactory' | 40 |
| These nested if statements could be combined | 96 - 102 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'o'. | 146 |
| An empty statement (semicolon) not part of a loop | 317 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'quickfinderService'. | 141 |
| Avoid empty if statements | 153 - 156 |
| These nested if statements could be combined | 460 - 462 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 26 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 81 - 83 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 56 - 60 |
| Violation | Line |
|---|---|
| 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. | 154 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 869 - 880 |
| These nested if statements could be combined | 942 - 944 |
| Avoid unused local variables such as 'principalId'. | 1179 |
| Violation | Line |
|---|---|
| Avoid unnecessary temporaries when converting primitives to Strings | 57 |
| Avoid unnecessary temporaries when converting primitives to Strings | 72 |
| Avoid unnecessary temporaries when converting primitives to Strings | 91 |
| Avoid unnecessary temporaries when converting primitives to Strings | 125 |
| Avoid unnecessary temporaries when converting primitives to Strings | 149 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 104 - 106 |
| Avoid empty catch blocks | 130 - 132 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'request'. | 254 - 260 |
| These nested if statements could be combined | 402 - 404 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.KimEntityPrivacyPreferences' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.ObjectUtils' | 27 |
| Avoid unused private fields such as 'RESTRICTED_DATA_MASK'. | 41 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 71 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 60 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.config.SimpleConfig' | 33 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 46 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.user.UserUtils' | 31 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'pk'. | 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 20 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'nodeTypeName'. | 1154 - 1307 |
| These nested if statements could be combined | 1211 - 1214 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 106 - 108 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'attributes'. | 200 - 210 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_GROUP_DESCRIPTION'. | 53 |
| Avoid unused private methods such as 'isBlank(Object)'. | 276 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_GROUP_DESCRIPTION'. | 58 |
| Avoid unused private fields such as 'groupAttributes'. | 62 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 322 - 324 |
| Avoid empty if statements | 392 - 394 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'checkForDuplicateRuleDelegations(List)'. | 157 |
| Avoid unused local variables such as 'delegations'. | 379 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'attributes'. | 144 - 154 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 60 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 337 - 340 |
| Avoid unused method parameters such as 'nodeType'. | 422 - 424 |
| Avoid unused method parameters such as 'nodeType'. | 429 - 431 |
| Violation | Line |
|---|---|
| Avoid unnecessary temporaries when converting primitives to Strings | 78 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 78 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.util.KEWConstants' | 30 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'exportRuleDelegations(Element,RuleBaseValues)'. | 183 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.jdom.output.Format.TextMode' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kns.exception.ExportNotSupportedException' | 33 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'dateMask()'. | 372 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.util.KimCommonUtils' | 34 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.reference.EmploymentStatus' | 30 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.reference.EmploymentType' | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.dto.KimEntityDefaultInfo' | 31 |
| Avoid unused imports such as 'org.kuali.rice.kim.util.KimCommonUtils' | 35 |
| Violation | Line |
|---|---|
| The null check here is misplaced; if the variable is null therell be a NullPointerException | 185 |
| These nested if statements could be combined | 205 - 211 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.reference.dto.AddressTypeInfo' | 27 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 21 |
| Avoid unused imports such as 'org.apache.commons.lang.builder.EqualsBuilder' | 28 |
| Avoid unused imports such as 'org.apache.commons.lang.builder.HashCodeBuilder' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.Role' | 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAbstractMemberImpl' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Date' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' | 26 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 33 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.FetchType' | 23 |
| Avoid unused imports such as 'javax.persistence.JoinColumn' | 25 |
| Avoid unused imports such as 'javax.persistence.ManyToOne' | 26 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.reference.CitizenshipStatus' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.reference.impl.CitizenshipStatusImpl' | 30 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.PersistableBusinessObjectBase' | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.PersistableBusinessObjectBase' | 33 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Date' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.PersistableBusinessObjectBase' | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.Id' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.service.impl.UiDocumentServiceImpl' | 35 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.dto.KimPrincipalInfo' | 26 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 33 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 82 - 86 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.logging.Log' | 24 |
| Avoid unused imports such as 'org.apache.commons.logging.LogFactory' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedList' | 19 |
| Avoid unused imports such as 'java.util.List' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.lifecycle.Lifecycle' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.impl.KimPrincipalImpl' | 30 |
| Violation | Line |
|---|---|
| Avoid empty try blocks | 628 - 630 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.ojb.broker.PersistenceBroker' | 25 |
| Avoid unused imports such as 'org.apache.ojb.broker.PersistenceBrokerException' | 26 |
| Avoid unused imports such as 'org.kuali.rice.kim.util.KimCommonUtils' | 48 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'spacesPattern'. | 99 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 92 - 99 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 298 - 307 |
| These nested if statements could be combined | 453 - 464 |
| Avoid unused local variables such as 'i'. | 707 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 155 - 159 |
| These nested if statements could be combined | 372 - 385 |
| These nested if statements could be combined | 373 - 384 |
| These nested if statements could be combined | 530 - 543 |
| These nested if statements could be combined | 531 - 542 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 75 - 77 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 77 - 79 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.lookup.Lookupable' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collections.sort' | 18 |
| Avoid unused imports such as 'java.util.ListIterator' | 28 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.group.dto.GroupInfo' | 39 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.ui.KimAttributeDataComparator' | 43 |
| Avoid unused local variables such as 'skipPropTypeCheck'. | 290 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.GenericPermission' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.GroupImpl' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.types.impl.KimTypeImpl' | 23 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 58 |
| Avoid unused local variables such as 'kimTypeId'. | 110 |
| Avoid unused private methods such as 'setupAttributeRows()'. | 143 |
| Avoid unused local variables such as 'attributeRows'. | 253 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 136 - 149 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.rule.ui.AddMemberRule' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.rule.ui.AddDelegationMemberRule' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.rule.ui.AddPersonDocumentRoleQualifierRule' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.ui.PersonDocumentGroup' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kns.document.Document' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'java.util.List' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 36 |
| Avoid unused local variables such as 'i'. | 81 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'document'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 18 |
| Avoid unused imports such as 'java.util.Map' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAttributes' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kim.document.IdentityManagementRoleDocument' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kim.util.KimConstants' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.ui.KimDocumentRoleMember' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.ui.RoleDocumentDelegation' | 24 |
| Avoid unused local variables such as 'i'. | 71 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 136 - 138 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 46 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'EXEC_INTERVAL_SECS'. | 69 |
| Avoid unused private fields such as 'MAX_WRITE_QUEUE_SIZE'. | 70 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 77 - 79 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 63 |
| These nested if statements could be combined | 224 - 226 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 423 - 439 |
| These nested if statements could be combined | 603 - 605 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.role.impl.ResponsibilityAttributeDataImpl' | 39 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.role.impl.RoleMemberAttributeDataImpl' | 40 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.types.dto.KimTypeAttributeInfo' | 44 |
| Avoid unused imports such as 'org.kuali.rice.kim.service.ResponsibilityUpdateService' | 48 |
| Avoid unused imports such as 'org.kuali.rice.kns.service.BusinessObjectService' | 54 |
| Avoid unused imports such as 'org.kuali.rice.kns.service.SequenceAccessorService' | 56 |
| Avoid unused imports such as 'org.kuali.rice.ksb.cache.RiceCacheAdministrator' | 58 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.exception.RiceRemoteServiceConnectionException' | 32 |
| Avoid unused imports such as 'org.kuali.rice.core.resourceloader.GlobalResourceLoader' | 33 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 415 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 206 - 209 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 60 |
| These nested if statements could be combined | 684 - 700 |
| These nested if statements could be combined | 980 - 982 |
| These nested if statements could be combined | 992 - 994 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getPrincipalImpl(String)'. | 623 |
| Avoid empty if statements | 1679 - 1681 |
| Avoid empty if statements | 1687 - 1689 |
| Avoid empty if statements | 1695 - 1697 |
| These nested if statements could be combined | 2307 - 2309 |
| These nested if statements could be combined | 2316 - 2318 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.role.dto.DelegateInfo' | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 239 - 251 |
| Avoid empty catch blocks | 261 - 263 |
| These nested if statements could be combined | 385 - 387 |
| These nested if statements could be combined | 394 - 397 |
| These nested if statements could be combined | 404 - 408 |
| Avoid empty catch blocks | 478 - 480 |
| Avoid empty catch blocks | 492 - 494 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LAST_ACCCESS_TIMEOUT_FIELD'. | 28 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'kimTypeServiceCache'. | 61 |
| These nested if statements could be combined | 322 - 328 |
| These nested if statements could be combined | 338 - 344 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.question.ConfirmationQuestion' | 37 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.RoleImpl' | 36 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 45 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.GroupImpl' | 37 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 57 |
| Avoid empty if statements | 548 - 551 |
| These nested if statements could be combined | 576 |
| These nested if statements could be combined | 585 - 587 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 20 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 24 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'persistenceStructureService'. | 43 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 19 |
| Avoid unused imports such as 'java.util.Set' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashSet' | 19 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 25 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 270 - 278 |
| These nested if statements could be combined | 274 - 276 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.Column' | 18 |
| Avoid unused imports such as 'javax.persistence.Id' | 19 |
| Avoid unused imports such as 'javax.persistence.CascadeType' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.Version' | 21 |
| Avoid unused imports such as 'javax.persistence.CascadeType' | 24 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 20 |
| No need to import a type that lives in the same package | 21 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 20 |
| No need to import a type that lives in the same package | 21 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 178 - 180 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 205 - 207 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.Version' | 21 |
| Avoid unused imports such as 'javax.persistence.TemporalType' | 22 |
| Avoid unused imports such as 'javax.persistence.Temporal' | 23 |
| Avoid unused imports such as 'javax.persistence.Id' | 25 |
| Avoid unused imports such as 'javax.persistence.CascadeType' | 26 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'attachmentLink'. | 74 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.Version' | 19 |
| Avoid unused imports such as 'javax.persistence.CascadeType' | 22 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 171 - 178 |
| These nested if statements could be combined | 190 - 197 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.ExternalizableBusinessObject' | 21 |
| No need to import a type that lives in the same package | 21 |
| No need to import a type that lives in the same package | 22 |
| No need to import a type that lives in the same package | 23 |
| Avoid unused private fields such as 'buildingCode'. | 34 |
| Avoid unused private fields such as 'buildingRoomNumber'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.Version' | 21 |
| Avoid unused imports such as 'javax.persistence.TemporalType' | 22 |
| Avoid unused imports such as 'javax.persistence.Temporal' | 23 |
| Avoid unused imports such as 'javax.persistence.Id' | 25 |
| Avoid unused imports such as 'javax.persistence.CascadeType' | 26 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 148 - 151 |
| Overriding method merely calls super | 159 - 163 |
| Overriding method merely calls super | 171 - 175 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 21 |
| No need to import a type that lives in the same package | 22 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 33 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 30 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 42 |
| Avoid unused private fields such as 'ojbCollectionHelper'. | 49 |
| Avoid empty catch blocks | 71 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 19 |
| Avoid unused imports such as 'org.apache.commons.beanutils.BeanUtils' | 26 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.Group' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kim.service.KIMServiceLocator' | 30 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.AdHocRoutePerson' | 31 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.AdHocRouteWorkgroup' | 32 |
| Avoid unused private fields such as 'LOG'. | 45 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.Group' | 26 |
| Avoid unused imports such as 'org.kuali.rice.kim.service.KIMServiceLocator' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.AdHocRoutePerson' | 28 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.AdHocRouteWorkgroup' | 29 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Date' | 20 |
| Avoid unused imports such as 'org.apache.ojb.broker.query.QueryFactory' | 35 |
| Avoid empty catch blocks | 227 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Date' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.DocumentHeader' | 27 |
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 37 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.EntityManager' | 23 |
| Avoid unused imports such as 'javax.persistence.PersistenceContext' | 24 |
| Avoid empty catch blocks | 120 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 33 |
| Avoid unused private fields such as 'businessObjectDao'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 41 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 43 |
| Avoid unused private fields such as 'DOCUMENT_AD_HOC_SERVICE_NAME'. | 51 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.config.ConfigurationException' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kns.dao.impl.PersistenceDaoJpa' | 27 |
| Avoid unused local variables such as 'entityManager'. | 55 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 18 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 20 |
| Avoid unused imports such as 'org.apache.commons.logging.LogFactory' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kew.attribute.Attribute' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collections' | 25 |
| Avoid unused imports such as 'java.util.HashSet' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.exception.CompletionException' | 40 |
| Avoid unused imports such as 'org.kuali.rice.kns.service.ModuleService' | 42 |
| Avoid unused local variables such as 'indexName'. | 122 |
| These nested if statements could be combined | 506 - 534 |
| Avoid empty catch blocks | 583 - 585 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 117 - 118 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 270 - 272 |
| These nested if statements could be combined | 276 - 278 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 49 - 53 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 24 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 123 - 125 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 193 - 195 |
| These nested if statements could be combined | 199 - 201 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 23 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 38 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 419 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 19 |
| Avoid unused imports such as 'java.util.List' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.mask.MaskFormatterLiteral' | 27 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'regexString'. | 81 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kns.exception.ValidationException' | 30 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 327 - 331 |
| These nested if statements could be combined | 596 - 620 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 710 - 718 |
| Avoid empty if statements | 737 - 744 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 27 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 127 - 132 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 171 - 173 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.PersonImpl' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 18 |
| Avoid unused imports such as 'java.util.Map' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'java.util.List' | 20 |
| Avoid unused private fields such as 'LOG'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 18 |
| Avoid unused imports such as 'java.util.Map' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAttributes' | 24 |
| Avoid unused imports such as 'org.kuali.rice.kim.util.KimConstants' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 21 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 359 - 368 |
| These nested if statements could be combined | 959 - 961 |
| These nested if statements could be combined | 1009 - 1016 |
| Avoid unused local variables such as 'fieldsForLookup'. | 1085 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.SelectedObjectIds' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kns.exception.AuthorizationException' | 32 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getBusinessObjectAuthorizationService()'. | 206 |
| Avoid unused local variables such as 'elementClass'. | 214 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.SelectedObjectIds' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.PersistableBusinessObject' | 30 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSPropertyConstants' | 37 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'kualiConfigurationService'. | 69 |
| Avoid empty catch blocks | 142 - 144 |
| These nested if statements could be combined | 495 - 514 |
| These nested if statements could be combined | 567 - 569 |
| Avoid unused local variables such as 'attributeRefName'. | 624 |
| Avoid unused method parameters such as 'nestedObjectPrefix'. | 692 - 729 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.RiceApplicationConfigurationService' | 28 |
| Avoid unused private fields such as 'parameterService'. | 33 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'parameterService'. | 52 |
| These nested if statements could be combined | 98 - 100 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.SelectedObjectIds' | 26 |
| Avoid unused imports such as 'org.kuali.rice.kns.exception.AuthorizationException' | 27 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'parameterNamesapce'. | 40 - 45 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 160 - 162 |
| These nested if statements could be combined | 168 - 170 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 409 - 412 |
| These nested if statements could be combined | 426 - 428 |
| Avoid empty if statements | 452 - 455 |
| Avoid unused local variables such as 'cachedValues'. | 710 |
| These nested if statements could be combined | 1321 - 1356 |
| Avoid unused local variables such as 'catchme'. | 1332 |
| Avoid unused local variables such as 'catchme'. | 1399 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 134 - 136 |
| Avoid unused local variables such as 'fieldValues'. | 306 |
| These nested if statements could be combined | 1076 - 1078 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.util.ObjectUtils' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.util.ObjectUtils' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.util.ObjectUtils' | 22 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'logEvent()'. | 46 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.document.Document' | 18 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'docOrBoClass'. | 380 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 222 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.types.dto.AttributeSet' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.mask.Mask' | 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.exception.ClassNotPersistableException' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.exception.ClassNotPersistableException' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 18 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.dao.NoteDao' | 33 |
| These nested if statements could be combined | 243 - 245 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 227 - 251 |
| These nested if statements could be combined | 290 - 300 |
| Avoid empty catch blocks | 545 - 548 |
| Avoid unused local variables such as 'namespaceCode'. | 619 |
| Avoid unused local variables such as 'componentName'. | 620 |
| Avoid unused local variables such as 'propertyName'. | 621 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 207 - 209 |
| These nested if statements could be combined | 224 - 226 |
| These nested if statements could be combined | 240 - 242 |
| These nested if statements could be combined | 257 - 259 |
| These nested if statements could be combined | 274 - 277 |
| These nested if statements could be combined | 342 - 344 |
| Avoid empty if statements | 505 - 512 |
| These nested if statements could be combined | 522 - 536 |
| These nested if statements could be combined | 526 - 535 |
| These nested if statements could be combined | 559 - 577 |
| These nested if statements could be combined | 561 - 576 |
| These nested if statements could be combined | 569 - 573 |
| Avoid empty catch blocks | 587 - 591 |
| These nested if statements could be combined | 660 - 662 |
| These nested if statements could be combined | 730 - 732 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'attributeRefName'. | 431 |
| These nested if statements could be combined | 506 - 509 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 146 |
| Avoid empty catch blocks | 288 - 291 |
| Avoid unused method parameters such as 'refFieldName'. | 435 - 450 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 54 - 56 |
| These nested if statements could be combined | 89 - 91 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 240 - 242 |
| These nested if statements could be combined | 689 - 692 |
| These nested if statements could be combined | 703 - 707 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 267 - 278 |
| These nested if statements could be combined | 273 - 275 |
| Avoid empty catch blocks | 462 - 464 |
| Avoid empty catch blocks | 477 - 479 |
| These nested if statements could be combined | 782 - 784 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| This usage of the Collection.toArray() method will throw a ClassCastException. | 159 |
| Avoid unused method parameters such as 'documentHeaderId'. | 598 - 604 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.util.Utilities' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kim.service.KIMServiceLocator' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.mask.Mask' | 32 |
| Avoid unused private fields such as 'persistenceService'. | 54 |
| Avoid unused local variables such as 'formatter'. | 126 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 37 - 39 |
| These nested if statements could be combined | 46 - 48 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 28 |
| Avoid unused private methods such as 'checkParameterArgument(Parameter,String)'. | 97 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Avoid unused private fields such as 'EMPTY_COLLECTION'. | 40 |
| Avoid unused private fields such as 'dataDictionaryService'. | 44 |
| Avoid unused private fields such as 'persistenceStructureService'. | 45 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Arrays' | 18 |
| Avoid unused imports such as 'java.util.Collections' | 19 |
| Avoid unused imports such as 'java.util.Iterator' | 20 |
| Avoid unused imports such as 'java.util.Set' | 21 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 578 - 581 |
| Avoid unused method parameters such as 'businessObject'. | 654 - 671 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 63 - 65 |
| These nested if statements could be combined | 73 - 75 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 52 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.log4j.Logger' | 23 |
| Avoid unused local variables such as 'key'. | 426 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.log4j.Logger' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.log4j.Logger' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 27 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 36 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 131 - 133 |
| Avoid unused local variables such as 'refPkNames'. | 310 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KualiModuleService' | 48 |
| These nested if statements could be combined | 408 - 410 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.CascadeType' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' | 37 |
| These nested if statements could be combined | 554 - 556 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 582 - 584 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Arrays' | 19 |
| Avoid unused imports such as 'java.util.Collections' | 20 |
| Avoid unused imports such as 'org.apache.commons.collections.ListUtils' | 23 |
| Avoid unused private fields such as 'dateTimeService'. | 57 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 198 - 200 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 48 - 52 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'difference'. | 123 |
| Avoid unused local variables such as 'difference'. | 155 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 184 - 187 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kns.service.KualiModuleService' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kns.service.ModuleService' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kns.web.format.Formatter' | 67 |
| Avoid unused private methods such as 'containsIntermediateNull(Object,String)'. | 692 |
| These nested if statements could be combined | 748 - 754 |
| Avoid empty if statements | 963 - 966 |
| These nested if statements could be combined | 1015 - 1017 |
| Avoid unused local variables such as 'propertyName'. | 1154 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedList' | 19 |
| Avoid unused imports such as 'java.util.List' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kew.engine.RouteContext' | 23 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 78 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 19 |
| Avoid unused imports such as 'java.util.Iterator' | 20 |
| Avoid unused imports such as 'java.util.List' | 21 |
| Avoid unused imports such as 'java.util.ListIterator' | 22 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 30 |
| Avoid unused local variables such as 'entryKey'. | 336 |
| These nested if statements could be combined | 737 - 741 |
| These nested if statements could be combined | 738 - 740 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 248 - 250 |
| Avoid empty catch blocks | 250 - 252 |
| Avoid empty catch blocks | 304 - 306 |
| Avoid empty catch blocks | 309 - 311 |
| An empty statement (semicolon) not part of a loop | 611 |
| These nested if statements could be combined | 853 - 855 |
| These nested if statements could be combined | 912 - 914 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'S_TRUE'. | 28 |
| Avoid unused private fields such as 'S_FALSE'. | 29 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 67 - 69 |
| Avoid empty catch blocks | 109 - 111 |
| Avoid unused method parameters such as 'origSource'. | 122 - 134 |
| Violation | Line |
|---|---|
| Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal | 29 |
| An operation on an Immutable object (BigDecimal or BigInteger) won't change the object itself | 65 |
| Violation | Line |
|---|---|
| Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal | 28 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 126 - 128 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'params'. | 346 |
| Avoid unused local variables such as 'files'. | 364 |
| Avoid unused private methods such as 'getMultipartHandler(HttpServletRequest,ActionForm)'. | 408 |
| Avoid unused method parameters such as 'request'. | 408 - 417 |
| Avoid empty if statements | 410 - 412 |
| These nested if statements could be combined | 458 - 461 |
| These nested if statements could be combined | 506 - 508 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 73 - 75 |
| Avoid unused local variables such as 'copySize'. | 87 |
| Avoid empty catch blocks | 108 - 110 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'prop'. | 74 - 76 |
| Avoid unused constructor parameters such as 'broker'. | 74 - 76 |
| Avoid unused local variables such as 'cache'. | 83 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 22 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 66 - 68 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 108 - 110 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'java.util.List' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.exception.PropertiesException' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.exception.PropertiesException' | 20 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 72 - 74 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.transaction.annotation.Transactional' | 22 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 37 - 53 |
| These nested if statements could be combined | 66 - 82 |
| These nested if statements could be combined | 94 - 110 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.config.SimpleConfig' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 18 |
| No need to import a type that lives in the same package | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 79 - 81 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 133 - 135 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Arrays' | 21 |
| Avoid unused imports such as 'java.util.List' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.RiceKeyConstants' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.servlet.http.HttpServletRequest' | 34 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 39 - 41 |
| Overriding method merely calls super | 47 - 49 |
| Overriding method merely calls super | 51 - 53 |
| Overriding method merely calls super | 55 - 57 |
| Overriding method merely calls super | 59 - 61 |
| Overriding method merely calls super | 63 - 65 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 37 - 41 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 50 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'form'. | 75 - 82 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'response'. | 254 - 265 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 542 |
| These nested if statements could be combined | 628 - 630 |
| Avoid unused method parameters such as 'response'. | 782 - 815 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.exception.ValidationException' | 68 |
| Avoid unused imports such as 'org.kuali.rice.kns.rule.event.SaveDocumentEvent' | 75 |
| Avoid unnecessary temporaries when converting primitives to Strings | 947 |
| These nested if statements could be combined | 1162 - 1178 |
| Avoid empty catch blocks | 1304 - 1305 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 29 |
| Avoid unused local variables such as 'label'. | 313 |
| Avoid unused local variables such as 'objectDescription'. | 314 |
| Avoid unused local variables such as 'helpForm'. | 458 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'kualiInquirable'. | 297 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.BusinessObjectEntry' | 38 |
| Avoid empty if statements | 121 - 123 |
| These nested if statements could be combined | 133 - 135 |
| Avoid unused method parameters such as 'request'. | 164 - 168 |
| Avoid unused local variables such as 'refreshCaller'. | 241 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 232 - 249 |
| Avoid empty catch blocks | 247 |
| Avoid empty catch blocks | 788 - 790 |
| Avoid empty catch blocks | 791 - 793 |
| Avoid empty catch blocks | 794 - 796 |
| These nested if statements could be combined | 829 - 832 |
| Avoid unused local variables such as 'documentAuthorizer'. | 952 |
| Avoid unused local variables such as 'newBo'. | 958 |
| Avoid unused local variables such as 'maintenanceDocumentAuthorizer'. | 994 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'refreshCaller'. | 137 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Properties' | 20 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 214 - 219 |
| These nested if statements could be combined | 215 - 218 |
| These nested if statements could be combined | 537 - 541 |
| These nested if statements could be combined | 558 - 562 |
| Avoid unused method parameters such as 'response'. | 704 - 715 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 19 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 22 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'targetName'. | 37 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.DataDictionaryService' | 30 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'inactiveRecordDisplay'. | 79 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 348 - 350 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 428 - 430 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 117 - 151 |
| These nested if statements could be combined | 189 - 192 |
| These nested if statements could be combined | 195 - 198 |
| Avoid unused local variables such as 'docTypeName'. | 453 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'ddService'. | 147 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 223 |
| These nested if statements could be combined | 322 - 325 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.web.format.EncryptionFormatter' | 46 |
| Avoid empty catch blocks | 178 - 180 |
| Avoid unused method parameters such as 'requestParams'. | 234 - 252 |
| Avoid unused private methods such as 'getAllParametersForMultipartRequest(HttpServletRequest,MultipartRequestHandler)'. | 278 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 35 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.ExternalizableBusinessObjectUtils' | 36 |
| Avoid empty catch blocks | 369 - 370 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.taglib.TagUtils' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.taglib.TagUtils' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.taglib.TagUtils' | 22 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 205 - 209 |
| These nested if statements could be combined | 263 - 265 |
| These nested if statements could be combined | 285 - 287 |
| These nested if statements could be combined | 506 - 508 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 192 - 194 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getBusinessObjectAuthorizationService()'. | 63 |
| Avoid unused local variables such as 'row'. | 101 |
| These nested if statements could be combined | 354 - 673 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 182 - 184 |
| Avoid unused local variables such as 'boEntry'. | 436 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'docContent'. | 96 |
| Avoid unused local variables such as 'name'. | 118 |
| These nested if statements could be combined | 163 - 204 |
| Avoid unused private methods such as 'getLastXPath(Element)'. | 459 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 50 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.UserIdDTO' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kew.exception.InvalidActionTakenException' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.InvalidActionTakenException' | 32 |
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 33 |
| These nested if statements could be combined | 291 - 293 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 456 - 459 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.InvalidActionTakenException' | 24 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 141 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 62 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Properties' | 19 |
| Avoid unused imports such as 'org.apache.log4j.Logger' | 21 |
| Avoid unused imports such as 'org.apache.ws.security.components.crypto.Crypto' | 22 |
| Avoid unused imports such as 'org.apache.ws.security.components.crypto.Merlin' | 23 |
| Avoid unused imports such as 'org.apache.ws.security.handler.RequestData' | 24 |
| Avoid unused imports such as 'org.apache.ws.security.handler.WSHandlerConstants' | 25 |
| Avoid unused imports such as 'org.kuali.rice.core.config.ConfigContext' | 31 |
| Avoid unused imports such as 'org.kuali.rice.core.exception.RiceRuntimeException' | 32 |
| Avoid unused imports such as 'org.kuali.rice.core.util.ClassLoaderUtils' | 33 |
| Avoid unused imports such as 'org.kuali.rice.ksb.config.wss4j.CryptoPasswordCallbackHandler' | 34 |
| Avoid unused imports such as 'org.kuali.rice.ksb.messaging.ServiceInfo' | 35 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Properties' | 19 |
| Avoid unused imports such as 'org.apache.log4j.Logger' | 21 |
| Avoid unused imports such as 'org.apache.ws.security.components.crypto.Crypto' | 22 |
| Avoid unused imports such as 'org.apache.ws.security.components.crypto.Merlin' | 23 |
| Avoid unused imports such as 'org.apache.ws.security.handler.RequestData' | 24 |
| Avoid unused imports such as 'org.apache.ws.security.handler.WSHandlerConstants' | 25 |
| Avoid unused imports such as 'org.kuali.rice.core.config.ConfigContext' | 30 |
| Avoid unused imports such as 'org.kuali.rice.core.exception.RiceRuntimeException' | 31 |
| Avoid unused imports such as 'org.kuali.rice.core.util.ClassLoaderUtils' | 32 |
| Avoid unused imports such as 'org.kuali.rice.ksb.config.wss4j.CryptoPasswordCallbackHandler' | 33 |
| Avoid unused imports such as 'org.kuali.rice.ksb.messaging.ServiceInfo' | 34 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 91 - 93 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 72 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 58 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 73 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 75 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 91 - 93 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 80 - 82 |
| These nested if statements could be combined | 157 - 159 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.net.URLEncoder' | 21 |
| Avoid unused local variables such as 'i'. | 115 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 289 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 172 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.util.OrmUtils' | 29 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 140 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'transactionManager'. | 37 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 64 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 41 |
| These nested if statements could be combined | 61 - 63 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 44 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 43 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 52 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 83 - 85 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 136 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'serviceInfo'. | 43 |
| Avoid unused private fields such as 'serviceLocator'. | 44 |
| Avoid unused local variables such as 'server'. | 102 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.config.Config' | 15 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 35 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 63 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'serviceInfo'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Avoid unused private fields such as 'urlPathHelper'. | 37 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 46 - 48 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' | 33 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAttributes' | 34 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'routeQueueId'. | 77 |
| These nested if statements could be combined | 390 - 393 |
| Violation | Line |
|---|---|
| Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) | 201 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 120 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 57 - 59 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 95 - 97 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 97 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 69 - 71 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 159 |
| Avoid empty catch blocks | 162 - 164 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 61 - 64 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 91 - 93 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.CascadeType' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'uk.ltd.getahead.dwr.util.LoggingOutput' | 27 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'rows'. | 34 |