diff --git a/org.adempiere.base/src/org/compiere/model/GridTable.java b/org.adempiere.base/src/org/compiere/model/GridTable.java index bec1206e6f..2cbe2efa1b 100644 --- a/org.adempiere.base/src/org/compiere/model/GridTable.java +++ b/org.adempiere.base/src/org/compiere/model/GridTable.java @@ -101,7 +101,7 @@ public class GridTable extends AbstractTableModel /** * */ - private static final long serialVersionUID = -2181155164268688340L; + private static final long serialVersionUID = 4223765688790104180L; public static final String DATA_REFRESH_MESSAGE = "Refreshed"; @@ -232,6 +232,10 @@ public class GridTable extends AbstractTableModel private final static Integer NEW_ROW_ID = Integer.valueOf(-1); private static final int DEFAULT_FETCH_SIZE = 200; + /** Keep track of last sorted column index and sort direction */ + private int m_lastSortColumnIndex = -1; + private boolean m_lastSortedAscending = true; + /** * Set Table Name * @param newTableName table name @@ -890,6 +894,13 @@ public class GridTable extends AbstractTableModel } if (getRowCount() == 0) return; + + boolean isSameSortEntries = (col == m_lastSortColumnIndex && ascending == m_lastSortedAscending); + if (!isSameSortEntries) + { + m_lastSortColumnIndex = col; + m_lastSortedAscending = ascending; + } //cache changed row Object[] changedRow = m_rowChanged >= 0 ? getDataAtRow(m_rowChanged) : null; @@ -948,10 +959,14 @@ public class GridTable extends AbstractTableModel m_sort.get(i).data = null; } } - // update UI - fireTableDataChanged(); - // Info detected by MTab.dataStatusChanged and current row set to 0 - fireDataStatusIEvent("Sorted", "#" + m_sort.size()); + + if (!isSameSortEntries) + { + // update UI + fireTableDataChanged(); + // Info detected by MTab.dataStatusChanged and current row set to 0 + fireDataStatusIEvent("Sorted", "#" + m_sort.size()); + } } // sort /** @@ -2919,6 +2934,11 @@ public class GridTable extends AbstractTableModel m_changed = false; m_rowChanged = -1; m_inserting = false; + if (m_lastSortColumnIndex >= 0) + { + loadComplete(); + sort(m_lastSortColumnIndex, m_lastSortedAscending); + } fireTableDataChanged(); if (fireStatusEvent) fireDataStatusIEvent(DATA_REFRESH_MESSAGE, ""); @@ -3842,4 +3862,12 @@ public class GridTable extends AbstractTableModel public String get_TrxName() { return m_trxName; } + + /** + * reset the cache sort state ( sort column and sort ascending ) + */ + public void resetCacheSortState() { + m_lastSortColumnIndex = -1; + m_lastSortedAscending = true; + } } diff --git a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/AbstractADWindowContent.java b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/AbstractADWindowContent.java index e95ed650cb..60c8b05694 100644 --- a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/AbstractADWindowContent.java +++ b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/AbstractADWindowContent.java @@ -115,6 +115,7 @@ import org.zkoss.zk.ui.util.Clients; import org.zkoss.zul.Column; import org.zkoss.zul.Columns; import org.zkoss.zul.Div; +import org.zkoss.zul.Grid; import org.zkoss.zul.Hbox; import org.zkoss.zul.Listitem; import org.zkoss.zul.Menuitem; @@ -1656,9 +1657,40 @@ public abstract class AbstractADWindowContent extends AbstractUIPart implements */ public void onRefresh() { + GridTab gridTab = adTabbox.getSelectedGridTab(); + if (gridTab != null && gridTab.getTableModel() != null) + { + gridTab.getTableModel().resetCacheSortState(); + } + Column sortColumn = findCurrentSortColumn(); onRefresh(true, false); + if (sortColumn != null) + { + sortColumn.setSortDirection("natural"); + } } + private Column findCurrentSortColumn() { + IADTabpanel iadtabpanel = getADTab().getSelectedTabpanel(); + if (iadtabpanel instanceof ADTabpanel) { + ADTabpanel adtabpanel = (ADTabpanel) iadtabpanel; + Grid grid = adtabpanel.getGridView().getListbox(); + Columns columns = grid.getColumns(); + List list = columns.getChildren(); + for(int i = 0; i < list.size(); i++) + { + Component c = (Component) list.get(i); + if (c instanceof Column) { + Column column = (Column) c; + if (!"natural".equals(column.getSortDirection())) { + return column; + } + } + } + } + return null; + } + /** * @see ToolbarListener#onHelp() */