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 |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.application.ViewContext defines compareTo(ViewContext) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 95 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.configurable.mvc.binding.HasDataValueBinding.INSTANCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 63 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.configurable.mvc.binding.HasTextBinding.INSTANCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 37 | High |
| Method call in org.kuali.student.common.ui.client.configurable.mvc.binding.HasTextBinding.setModelValue(HasText, DataModel, String) passes null for nonnull parameter of Integer.parseInt(String) | CORRECTNESS | NP_NULL_PARAM_DEREF | 73 | Medium |
| Exception is caught when Exception is not thrown in org.kuali.student.common.ui.client.configurable.mvc.binding.HasTextBinding.setModelValue(HasText, DataModel, String) | STYLE | REC_CATCH_EXCEPTION | 120 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.configurable.mvc.binding.HasValueBinding.INSTANCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 36 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Suspicious comparison of Integer references in org.kuali.student.common.ui.client.configurable.mvc.binding.ModelWidgetBindingSupport.nullsafeEquals(Data, Data) | CORRECTNESS | RC_REF_COMPARISON | 95 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.configurable.mvc.binding.MultiplicityCompositeBinding.INSTANCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 38 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.configurable.mvc.binding.MultiplicityGroupItemBinding.INSTANCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 34 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.configurable.mvc.binding.MultiplicityItemBinding.INSTANCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 38 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.configurable.mvc.binding.RichTextBinding.INSTANCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 29 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.configurable.mvc.binding.SelectItemWidgetBinding.INSTANCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 41 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from com.google.gwt.user.client.ui.Widget to org.kuali.student.common.ui.client.widgets.list.KSSelectItemWidgetAbstract in org.kuali.student.common.ui.client.configurable.mvc.binding.SelectItemWidgetBinding$1.exec(Widget) | STYLE | BC_UNCONFIRMED_CAST | 109 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Field TabbedSectionLayout.defaultView masks field in superclass org.kuali.student.common.ui.client.configurable.mvc.LayoutController | CORRECTNESS | MF_CLASS_MASKS_FIELD | 49 | Medium |
| Nullcheck of section at line 339 of value previously dereferenced in org.kuali.student.common.ui.client.configurable.mvc.layouts.TabbedSectionLayout.addSection(String[], SectionView) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 326 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityComposite.addItem() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 87 | Medium |
| Possible null pointer dereference of item in org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityComposite.addItem() | CORRECTNESS | NP_NULL_ON_SOME_PATH | 95 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityConfiguration.fieldMap | PERFORMANCE | URF_UNREAD_FIELD | 65 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityGroup.createItem() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 176 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityGroup$ConditionChoices be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 495 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityGroup$SwappableFieldsHelper be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 624 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.configurable.mvc.sections.MultiplicityHeader.title | PERFORMANCE | URF_UNREAD_FIELD | 36 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.common.ui.client.configurable.mvc.sections.RequiredContainer$ShowAllLink be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 210 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.configurable.mvc.sections.SectionBinding.INSTANCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 42 | High |
| Method org.kuali.student.common.ui.client.configurable.mvc.sections.SectionBinding.setModelValue(Section, DataModel, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 110 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.mvc.breadcrumb.BreadcrumbManager.links isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 25 | High |
| Nullcheck of name at line 94 of value previously dereferenced in org.kuali.student.common.ui.client.mvc.breadcrumb.BreadcrumbManager.updateLinks(String) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 94 | Medium |
| Method org.kuali.student.common.ui.client.mvc.breadcrumb.BreadcrumbManager.updateLinks(String) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 78 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.mvc.history.HistoryManager.VIEW_ATR isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 54 | High |
| Method org.kuali.student.common.ui.client.mvc.history.HistoryManager.appendContext(String, ViewContext) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 302 | Medium |
| Method org.kuali.student.common.ui.client.mvc.history.HistoryManager.appendContext(String, ViewContext) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 301 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Boolean is incompatible with expected argument type String in org.kuali.student.common.ui.client.security.SecurityContext.checkPermission(String[], Callback) | CORRECTNESS | GC_UNRELATED_TYPES | 157 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to InvocationException in org.kuali.student.common.ui.client.security.SpringSecurityLoginDialogHandler.isSessionTimeout(Throwable) | STYLE | DLS_DEAD_LOCAL_STORE | 61 | Medium |
| Write to static field org.kuali.student.common.ui.client.security.SpringSecurityLoginDialogHandler.CASrequiresAPageRefreshViaJavascript from instance method org.kuali.student.common.ui.client.security.SpringSecurityLoginDialogHandler.isSessionTimeout(Throwable) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 71 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.util.ExportElement.BOLD isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 19 | High |
| org.kuali.student.common.ui.client.util.ExportElement.DEFAULT isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 18 | High |
| org.kuali.student.common.ui.client.util.ExportElement.LIST isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 22 | High |
| org.kuali.student.common.ui.client.util.ExportElement.LIST_SUBREPORT isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 20 | High |
| org.kuali.student.common.ui.client.util.ExportElement.PARAGRAPH isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 25 | High |
| org.kuali.student.common.ui.client.util.ExportElement.SUBREPORT isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 21 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| instanceof will always return true in org.kuali.student.common.ui.client.util.ExportUtils.getExportElementsFromView(Widget, ArrayList, String, String), since all org.kuali.student.common.ui.client.configurable.mvc.views.VerticalSectionView are instances of org.kuali.student.common.ui.client.configurable.mvc.sections.BaseSection | STYLE | BC_VACUOUS_INSTANCEOF | 439 | Medium |
| Dead store to exportElements in org.kuali.student.common.ui.client.util.ExportUtils.handleExportClickEvent(Controller, String, String) | STYLE | DLS_DEAD_LOCAL_STORE | 215 | Medium |
| org.kuali.student.common.ui.client.util.ExportUtils.getExportItemDetails(ExportElement, Widget, boolean, String, String) invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 101 | Medium |
| org.kuali.student.common.ui.client.util.ExportUtils.getExportItemDetails(ExportElement, Widget, boolean, String, String) invokes inefficient new String() constructor | PERFORMANCE | DM_STRING_VOID_CTOR | 84 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.common.ui.client.util.PrintUtils.print(UIObject) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 23 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.widgets.containers.KSWrapper.appUrl | PERFORMANCE | URF_UNREAD_FIELD | 85 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Nullcheck of result at line 123 of value previously dereferenced in org.kuali.student.common.ui.client.widgets.containers.KSWrapper$1.onSuccess(Map) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 122 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.widgets.dialog.ButtonMessageDialog.title | PERFORMANCE | URF_UNREAD_FIELD | 17 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.widgets.dialog.ConfirmationDialog.title | PERFORMANCE | URF_UNREAD_FIELD | 38 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Comparison of String objects using == or != in org.kuali.student.common.ui.client.widgets.field.layout.element.FieldElement.processValidationResult(ValidationResultInfo) | BAD_PRACTICE | ES_COMPARING_STRINGS_WITH_EQ | 479 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.widgets.field.layout.layouts.Header.updateable | PERFORMANCE | URF_UNREAD_FIELD | 35 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.widgets.field.layout.layouts.HorizontalLayout.fieldArea | PERFORMANCE | URF_UNREAD_FIELD | 25 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.widgets.field.layout.layouts.TableFieldLayout.buttonPanel | PERFORMANCE | URF_UNREAD_FIELD | 24 | Medium |
| Unread field: org.kuali.student.common.ui.client.widgets.field.layout.layouts.TableFieldLayout.top | PERFORMANCE | URF_UNREAD_FIELD | 23 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.widgets.filter.KSFilterOptions.loading | PERFORMANCE | URF_UNREAD_FIELD | 45 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.widgets.impl.KSDatePickerImpl.setValue(Date) may expose internal representation by storing an externally mutable object into KSDatePickerImpl.selectedDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 223 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.widgets.list.KSSelectedList.itemDataHelper isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 83 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.student.common.ui.client.mvc.ModelChangeEvent to org.kuali.student.common.ui.client.mvc.CollectionModelChangeEvent in org.kuali.student.common.ui.client.widgets.list.ModelListItems$1.onModelChange(ModelChangeEvent) | STYLE | BC_UNCONFIRMED_CAST | 82 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to w in org.kuali.student.common.ui.client.widgets.menus.KSListPanel.remove(int) | STYLE | DLS_DEAD_LOCAL_STORE | 75 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.widgets.menus.impl.KSBasicMenuImpl$MenuItemPanel.indent | PERFORMANCE | URF_UNREAD_FIELD | 196 | Medium |
| Unread field: org.kuali.student.common.ui.client.widgets.menus.impl.KSBasicMenuImpl$MenuItemPanel.itemNum | PERFORMANCE | URF_UNREAD_FIELD | 197 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.widgets.menus.impl.KSListMenuImpl$MenuItemPanel.indent | PERFORMANCE | URF_UNREAD_FIELD | 195 | Medium |
| Unread field: org.kuali.student.common.ui.client.widgets.menus.impl.KSListMenuImpl$MenuItemPanel.itemNum | PERFORMANCE | URF_UNREAD_FIELD | 196 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to ids in org.kuali.student.common.ui.client.widgets.pagetable.GenericTableModel.setRows(List) | STYLE | DLS_DEAD_LOCAL_STORE | 178 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.common.ui.client.widgets.progress.BlockingTask.hashCode() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 40 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.common.ui.client.widgets.search.FieldAscendingRowComparator.compare(Row, Row) uses the same code for two branches | STYLE | DB_DUPLICATE_BRANCHES | 299 | Medium |
| org.kuali.student.common.ui.client.widgets.search.FieldAscendingRowComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 284 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.common.ui.client.widgets.search.FieldDescendingRowComparator.compare(Row, Row) uses the same code for two branches | STYLE | DB_DUPLICATE_BRANCHES | 324 | Medium |
| org.kuali.student.common.ui.client.widgets.search.FieldDescendingRowComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 309 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to value in org.kuali.student.common.ui.client.widgets.search.KSPicker$BasicWidget.getDisplayValue() | STYLE | DLS_DEAD_LOCAL_STORE | 474 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from com.google.gwt.user.client.ui.Widget to org.kuali.student.common.ui.client.widgets.list.KSSelectItemWidgetAbstract in org.kuali.student.common.ui.client.widgets.search.KSPicker$BasicWidget$1.exec(Widget) | STYLE | BC_UNCONFIRMED_CAST | 413 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from com.google.gwt.user.client.ui.Widget to org.kuali.student.common.ui.client.widgets.list.KSSelectItemWidgetAbstract in org.kuali.student.common.ui.client.widgets.search.KSPicker$BasicWidget$6.exec(Widget) | STYLE | BC_UNCONFIRMED_CAST | 585 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.common.ui.client.widgets.search.KSPicker$SelectionContainerWidget be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 610 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.common.ui.client.widgets.search.LinkPanel$PanelInfo.content | PERFORMANCE | URF_UNREAD_FIELD | 52 | Medium |
| Unread field: org.kuali.student.common.ui.client.widgets.search.LinkPanel$PanelInfo.key | PERFORMANCE | URF_UNREAD_FIELD | 51 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.widgets.searchtable.ResultRow defines compareTo(ResultRow) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 56 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.widgets.table.scroll.CourseIdAscendingRowComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 103 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.widgets.table.scroll.CourseIdDescendingRowComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 112 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| instanceof will always return true in org.kuali.student.common.ui.client.widgets.table.scroll.Table.updateTableCell(int, int), since all org.kuali.student.common.ui.client.widgets.table.scroll.TableCellWidget are instances of com.google.gwt.event.dom.client.HasChangeHandlers | STYLE | BC_VACUOUS_INSTANCEOF | 416 | Medium |
| instanceof will always return true in org.kuali.student.common.ui.client.widgets.table.scroll.Table.updateTableCell(int, int), since all org.kuali.student.common.ui.client.widgets.table.scroll.TableCellWidget are instances of com.google.gwt.event.dom.client.HasClickHandlers | STYLE | BC_VACUOUS_INSTANCEOF | 408 | Medium |
| Unread field: org.kuali.student.common.ui.client.widgets.table.scroll.Table.focusType | PERFORMANCE | URF_UNREAD_FIELD | 49 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The method name org.kuali.student.common.ui.client.widgets.table.summary.StaticDeletemeLineLogger.AlertLastLine() doesn't start with a lower case letter | BAD_PRACTICE | NM_METHOD_NAMING_CONVENTION | 11 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.client.widgets.table.summary.SummaryTableSection.buildMultiplicityRows(DataModel, DataModel, SummaryTableMultiplicityFieldRow, List, int, Integer) ignores return value of String.replace(CharSequence, CharSequence) | CORRECTNESS | RV_RETURN_VALUE_IGNORED | 257 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.common.ui.server.gwt.DataGwtServlet defines non-transient non-serializable instance field LOG | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Class org.kuali.student.common.ui.server.gwt.DataGwtServlet defines non-transient non-serializable instance field dataService | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.common.ui.server.gwt.ExportDocumentDownload defines non-transient non-serializable instance field LOG | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.common.ui.server.gwt.ExportGwtRpcServlet defines non-transient non-serializable instance field logger | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.common.ui.server.gwt.LogRpcGwtServlet defines non-transient non-serializable instance field logger | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Method org.kuali.student.common.ui.server.gwt.LogRpcGwtServlet.sendLog(Map, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 40 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.common.ui.server.gwt.MessagesRpcGwtServlet defines non-transient non-serializable instance field serviceImpl | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.common.ui.server.gwt.MetadataRpcGwtServlet defines non-transient non-serializable instance field serviceImpl | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.common.ui.server.gwt.SearchDispatchRpcGwtServlet defines non-transient non-serializable instance field idTranslatorFilter | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Class org.kuali.student.common.ui.server.gwt.SearchDispatchRpcGwtServlet defines non-transient non-serializable instance field searchDispatcher | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.common.ui.server.gwt.SearchDispatchRpcGwtServlet$MaxAgeSoftReference be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 154 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.common.ui.server.gwt.SearchDispatchRpcGwtServlet$MaxSizeMap be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 182 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.common.ui.server.gwt.SecurityRpcGwtServlet defines non-transient non-serializable instance field LOG | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Class org.kuali.student.common.ui.server.gwt.SecurityRpcGwtServlet defines non-transient non-serializable instance field permissionService | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.common.ui.server.gwt.ServerPropertiesRpcGwtServlet defines non-transient non-serializable instance field logger | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.server.gwt.old.AbstractBaseDataOrchestrationRpcGwtServlet.isAuthorized(PermissionType, Map) has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 124 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.ui.server.screenreport.jasper.KSCustomDataSource.getFieldValue(JRField) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 68 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.common.ui.theme.standard.client.CommonCssImpl.getCssString() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 32 | Medium |
