The following document contains the results of FindBugs Report
FindBugs Version is 1.3.9
Threshold is medium
Effort is min
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Redundant nullcheck of o, which is known to be non-null in org.kuali.rice.krad.bo.AdHocRouteWorkgroupId.equals(Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 66 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.bo.SessionDocument.getSerializedDocumentForm() may expose internal representation by returning SessionDocument.serializedDocumentForm | MALICIOUS_CODE | EI_EXPOSE_REP | 60 | Medium |
| org.kuali.rice.krad.bo.SessionDocument.setSerializedDocumentForm(byte[]) may expose internal representation by storing an externally mutable object into SessionDocument.serializedDocumentForm | MALICIOUS_CODE | EI_EXPOSE_REP2 | 67 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unused field: org.kuali.rice.krad.bo.authorization.BusinessObjectAuthorizerBase.persistenceStructureService | PERFORMANCE | UUF_UNUSED_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to collectionClasses in org.kuali.rice.krad.datadictionary.DataDictionary.buildReadDescriptor(Class, String) | STYLE | DLS_DEAD_LOCAL_STORE | 581 | Medium |
| Dead store to collectionClasses in org.kuali.rice.krad.datadictionary.DataDictionary.getCollectionElementClass(Class, String) | STYLE | DLS_DEAD_LOCAL_STORE | 522 | Medium |
| Dead store to indexName in org.kuali.rice.krad.datadictionary.DataDictionary.indexSource(String) | STYLE | DLS_DEAD_LOCAL_STORE | 127 | Medium |
| org.kuali.rice.krad.datadictionary.DataDictionary.persistenceStructureService should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Medium | |
| Write to static field org.kuali.rice.krad.datadictionary.DataDictionary.validateEBOs from instance method org.kuali.rice.krad.datadictionary.DataDictionary.validateDD(boolean) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 197 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| String is incompatible with expected argument type AttributeDefinition in org.kuali.rice.krad.datadictionary.DataDictionaryEntryBase.addNestedAttributes(ComplexAttributeDefinition, String) | CORRECTNESS | GC_UNRELATED_TYPES | 353 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.datadictionary.DataObjectEntry.getJstlKey() uses the same code for two branches | STYLE | DB_DUPLICATE_BRANCHES | 69 | Medium |
| Redundant nullcheck of org.kuali.rice.krad.datadictionary.DataObjectEntry.dataObjectClass, which is known to be non-null in org.kuali.rice.krad.datadictionary.DataObjectEntry.getJstlKey() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 69 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Possible null pointer dereference of sourceClass in org.kuali.rice.krad.datadictionary.PrimitiveAttributeDefinition.completeValidation(Class, Class) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 103 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to componentBeans in org.kuali.rice.krad.datadictionary.UifDictionaryIndex.buildViewIndicies() | STYLE | DLS_DEAD_LOCAL_STORE | 167 | Medium |
| Method org.kuali.rice.krad.datadictionary.UifDictionaryIndex.buildTypeIndex(Map) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 241 | Medium |
| Method org.kuali.rice.krad.datadictionary.UifDictionaryIndex.buildTypeIndex(Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 241 | Medium |
| Method org.kuali.rice.krad.datadictionary.UifDictionaryIndex.buildViewIndicies() makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 154 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to regexString in org.kuali.rice.krad.datadictionary.validation.CharacterLevelValidationPattern.getRegexPattern() | STYLE | DLS_DEAD_LOCAL_STORE | 82 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.datadictionary.validation.ValidationUtils.getBoolean(Object) has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 212 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.datadictionary.validation.result.ConstraintValidationResult.getErrorParameters() may expose internal representation by returning ConstraintValidationResult.errorParameters | MALICIOUS_CODE | EI_EXPOSE_REP | 106 | Medium |
| org.kuali.rice.krad.datadictionary.validation.result.ConstraintValidationResult.setErrorParameters(String[]) may expose internal representation by storing an externally mutable object into ConstraintValidationResult.errorParameters | MALICIOUS_CODE | EI_EXPOSE_REP2 | 113 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.rice.krad.datadictionary.validation.result.EntryValidationResult.entryName | PERFORMANCE | URF_UNREAD_FIELD | 32 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| instanceof will always return true in org.kuali.rice.krad.document.MaintenanceDocumentBase.validateBusinessRules(KualiDocumentEvent), since all org.kuali.rice.krad.document.MaintenanceDocumentBase are instances of org.kuali.rice.krad.document.MaintenanceDocument | STYLE | BC_VACUOUS_INSTANCEOF | 709 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.document.authorization.DocumentAuthorizerBase.LOG isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 42 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method call in new org.kuali.rice.krad.exception.ExceptionIncident(Exception, Map) passes null for nonnull parameter of initialize(Exception, Map) | CORRECTNESS | NP_NULL_PARAM_DEREF | 79 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.inquiry.InquirableImpl.SUPER_CLASS_TRANSLATOR_LIST isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 72 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.lookup.LookupUtils.forceUppercase(Class, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 115 | Medium |
| Method org.kuali.rice.krad.lookup.LookupUtils.getExternalizableBusinessObjectFieldValues(String, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 329 | Medium |
| Method org.kuali.rice.krad.lookup.LookupUtils.preprocessDateFields(Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 243 | Medium |
| Method org.kuali.rice.krad.lookup.LookupUtils.removeExternalizableBusinessObjectFieldValues(Class, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 313 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.uif.container.View to org.kuali.rice.krad.uif.container.LookupView in org.kuali.rice.krad.lookup.LookupableImpl.performInitialization(View) | STYLE | BC_UNCONFIRMED_CAST | 120 | Medium |
| Dead store to lookupView in org.kuali.rice.krad.lookup.LookupableImpl.getMaintenanceActionLink(LinkField, Object, String) | STYLE | DLS_DEAD_LOCAL_STORE | 667 | Medium |
| Dead store to searchUsingOnlyPrimaryKeyValues in org.kuali.rice.krad.lookup.LookupableImpl.getSearchResults(LookupForm, Map, boolean) | STYLE | DLS_DEAD_LOCAL_STORE | 169 | Medium |
| Dead store to pkNames in org.kuali.rice.krad.lookup.LookupableImpl.performSearch(LookupForm, Map, boolean) | STYLE | DLS_DEAD_LOCAL_STORE | 151 | Medium |
| Dead store to user in org.kuali.rice.krad.lookup.LookupableImpl.performSearch(LookupForm, Map, boolean) | STYLE | DLS_DEAD_LOCAL_STORE | 152 | Medium |
| Dead store to dataObjectExample in org.kuali.rice.krad.lookup.LookupableImpl.validateSearchParameterWildcardAndOperators(AttributeField, String) | STYLE | DLS_DEAD_LOCAL_STORE | 496 | Medium |
| org.kuali.rice.krad.lookup.LookupableImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 73 | Medium |
| Method org.kuali.rice.krad.lookup.LookupableImpl.getActionUrlHref(LookupForm, Object, String, List) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 716 | Medium |
| Method org.kuali.rice.krad.lookup.LookupableImpl.getReturnUrlParameters(LookupView, LookupForm, Object) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 634 | Medium |
| Method org.kuali.rice.krad.lookup.LookupableImpl.getSearchResultsForEBO(Map, boolean) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 258 | Medium |
| Method org.kuali.rice.krad.lookup.LookupableImpl.processSearchCriteria(LookupForm, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 218 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to docOrBoClass in org.kuali.rice.krad.rules.DocumentRuleBase.isAddHocRoutePersonValid(Document, AdHocRoutePerson) | STYLE | DLS_DEAD_LOCAL_STORE | 397 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Invocation of toString on org.kuali.rice.krad.util.ErrorMessage.getMessageParameters() in org.kuali.rice.krad.rules.MaintenanceDocumentRuleBase.showErrorMap() | CORRECTNESS | DMI_INVOKING_TOSTRING_ON_ARRAY | 1094 | Medium |
| org.kuali.rice.krad.rules.MaintenanceDocumentRuleBase.LOG isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 73 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to org.kuali.rice.krad.web.form.DocumentFormBase in org.kuali.rice.krad.uif.authorization.DocumentAuthorizerBase.getActionFlags(UifFormBase, Person, Set) | STYLE | BC_UNCONFIRMED_CAST | 50 | Medium |
| org.kuali.rice.krad.uif.authorization.DocumentAuthorizerBase.LOG isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 41 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to org.kuali.rice.krad.web.form.DocumentFormBase in org.kuali.rice.krad.uif.authorization.DocumentPresentationControllerBase.getActionFlags(UifFormBase) | STYLE | BC_UNCONFIRMED_CAST | 42 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.uif.container.CollectionGroupBuilder.buildLine(View, Object, CollectionGroup, String, List, boolean, Object, int) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 202 | Medium |
| org.kuali.rice.krad.uif.container.CollectionGroupBuilder.removeNonRenderLineFields(View, Object, CollectionGroup, List, Object, int) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 274 | Medium |
| Method org.kuali.rice.krad.uif.container.CollectionGroupBuilder.removeNonRenderLineFields(View, Object, CollectionGroup, List, Object, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 273 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception is caught when Exception is not thrown in org.kuali.rice.krad.uif.container.InquiryView.populateExportCapabilities() | STYLE | REC_CATCH_EXCEPTION | 115 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.uif.core.MethodInvokerConfig.getArgumentTypes() may expose internal representation by returning MethodInvokerConfig.argumentTypes | MALICIOUS_CODE | EI_EXPOSE_REP | 57 | Medium |
| org.kuali.rice.krad.uif.core.MethodInvokerConfig.setArgumentTypes(Class[]) may expose internal representation by storing an externally mutable object into MethodInvokerConfig.argumentTypes | MALICIOUS_CODE | EI_EXPOSE_REP2 | 66 | Medium |
| Read of unwritten field staticMethod in org.kuali.rice.krad.uif.core.MethodInvokerConfig.getMethodArgumentTypes() | CORRECTNESS | NP_UNWRITTEN_FIELD | 77 | Medium |
| Unwritten field: org.kuali.rice.krad.uif.core.MethodInvokerConfig.staticMethod | CORRECTNESS | UWF_UNWRITTEN_FIELD | 76 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.uif.field.ActionField.performFinalize(View, Object, Component) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 128 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.uif.field.AttributeQuery.getQueryFieldMappingJsString() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 137 | Medium |
| Method org.kuali.rice.krad.uif.field.AttributeQuery.getQueryMethodArgumentFieldsJsString() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 179 | Medium |
| Method org.kuali.rice.krad.uif.field.AttributeQuery.getReturnFieldMappingJsString() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 159 | Medium |
| Class org.kuali.rice.krad.uif.field.AttributeQuery defines non-transient non-serializable instance field queryMethodInvokerConfig | BAD_PRACTICE | SE_BAD_FIELD | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.uif.field.FieldBase.performFinalize(View, Object, Component) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 97 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.rice.krad.uif.field.GeneratedField defines non-transient non-serializable instance field renderingMethodInvoker | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.uif.history.History.getGeneratedBreadcrumbs() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 198 | Medium |
| Method org.kuali.rice.krad.uif.history.History.getGeneratedCurrentBreadcrumb() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 231 | Medium |
| Method org.kuali.rice.krad.uif.history.History.getHistoryParameterString() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 162 | Medium |
| Method org.kuali.rice.krad.uif.history.History.setCurrent(UifFormBase, HttpServletRequest) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 335 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.uif.layout.StackedLayoutManager.buildLineHeaderText(Object) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 184 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.uif.layout.TableLayoutManager.addHeaderField(Field, int) uses the same code for two branches | STYLE | DB_DUPLICATE_BRANCHES | 308 | Medium |
| org.kuali.rice.krad.uif.layout.TableLayoutManager.buildLine(View, Object, CollectionGroup, List, List, String, List, String, Object, int) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 183 | Medium |
| Method org.kuali.rice.krad.uif.layout.TableLayoutManager.buildLine(View, Object, CollectionGroup, List, List, String, List, String, Object, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 172 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.uif.util.ClientValidationUtils.createRule(AttributeField, Constraint, String, View) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 352 | Medium |
| Method org.kuali.rice.krad.uif.util.ClientValidationUtils.generateMessageFromLabelKey(String) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 124 | Medium |
| Method org.kuali.rice.krad.uif.util.ClientValidationUtils.generateValidatorMessagesOption() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 148 | Medium |
| Method org.kuali.rice.krad.uif.util.ClientValidationUtils.getMustOccurStatement(AttributeField, MustOccurConstraint) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 599 | Medium |
| Method org.kuali.rice.krad.uif.util.ClientValidationUtils.processWhenConstraint(AttributeField, View, CaseConstraint, WhenConstraint, String, String, String) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 269 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.uif.util.CloneUtils.deepCloneObject(Object, Map) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 95 | Medium |
| Load of known null value in org.kuali.rice.krad.uif.util.CloneUtils.internalField(Object, String) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 232 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Possible null pointer dereference of origField in org.kuali.rice.krad.uif.util.ComponentFactory.getComponentByIdWithLifecycle(UifFormBase, String) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 140 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.uif.util.ComponentUtils.processIds(Component, Map) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 349 | Medium |
| Method org.kuali.rice.krad.uif.util.ComponentUtils.sort(List, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 417 | Medium |
| Method org.kuali.rice.krad.uif.util.ComponentUtils.updateContextForLine(Component, Object, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 341 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.uif.util.LookupInquiryUtils.getLinkTitleText(String, Class, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 212 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.uif.util.ViewModelUtils.getPropertyType(View, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 63 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.rice.krad.util.KRADUtils.getIntegerValue(String) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 118 | Medium |
| Method org.kuali.rice.krad.util.KRADUtils.buildMapParameterString(Map) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 280 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Call to equals() with null argument in org.kuali.rice.krad.util.ObjectUtils.isNull(Object) | CORRECTNESS | EC_NULL_ARG | 836 | Medium |
| Method org.kuali.rice.krad.util.ObjectUtils.createHybridBusinessObject(BusinessObject, BusinessObject, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 194 | Medium |
| Method org.kuali.rice.krad.util.ObjectUtils.equalByKeys(PersistableBusinessObject, PersistableBusinessObject) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 673 | Medium |
| Method org.kuali.rice.krad.util.ObjectUtils.materializeSubObjectsToDepth(PersistableBusinessObject, int) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 936 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.rice.krad.util.documentserializer.DocumentSerializationState$SerializationPropertyElement be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 34 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.rice.krad.util.documentserializer.SerializationState$SerializationPropertyElement be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 35 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.web.controller.DocumentControllerBase.DOCUMENT_LOAD_COMMANDS should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 69 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to org.kuali.rice.krad.web.form.IncidentReportForm in org.kuali.rice.krad.web.controller.IncidentReportController.submitReport(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 54 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to org.kuali.rice.krad.web.form.InquiryForm in org.kuali.rice.krad.web.controller.InquiryController.export(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 102 | Medium |
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to org.kuali.rice.krad.web.form.InquiryForm in org.kuali.rice.krad.web.controller.InquiryController.start(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 72 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to org.kuali.rice.krad.web.form.LookupForm in org.kuali.rice.krad.web.controller.LookupController.cancel(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 126 | Medium |
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to org.kuali.rice.krad.web.form.LookupForm in org.kuali.rice.krad.web.controller.LookupController.start(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 112 | Medium |
| Method org.kuali.rice.krad.web.controller.LookupController.search(LookupForm, BindingResult, HttpServletRequest, HttpServletResponse) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 181 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.DocumentFormBase to org.kuali.rice.krad.web.form.MaintenanceForm in org.kuali.rice.krad.web.controller.MaintenanceDocumentController.docHandler(DocumentFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 72 | Medium |
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to org.kuali.rice.krad.web.form.MaintenanceForm in org.kuali.rice.krad.web.controller.MaintenanceDocumentController.start(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 121 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.rice.krad.web.form.LookupForm.processBooleanParameter(String) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 160 | Medium |
| org.kuali.rice.krad.web.form.LookupForm.processBooleanParameter(String) has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 162 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.rice.krad.web.form.UifFormBase defines non-transient non-serializable instance field attachmentFile | BAD_PRACTICE | SE_BAD_FIELD | Medium |
