[keyselectionmodel] Bugfixes in some of the required methods

fjalvingh · Jan 10, 2025 · 2935572 · 2935572
1 parent b1ee919
commit 2935572
Showing 1 changed file with 13 additions and 4 deletions.
diff --git a/to.etc.domui/src/main/java/to/etc/domui/component/tbl/KeySelectionModel.java b/to.etc.domui/src/main/java/to/etc/domui/component/tbl/KeySelectionModel.java
@@ -150,19 +150,23 @@ public List<T> getSelectedInstances() {
 	}
 
 	@Override
-	public void clearSelection(ITableModel<T> model) {
+	public void clearSelection(ITableModel<T> model) throws Exception {
 		if(model instanceof TableKeyModelBase) {
 			TableKeyModelBase<K, T> tm = (TableKeyModelBase<K, T>) model;
 			for(K key : tm.getKeys()) {
 				m_selectedSet.remove(key);
 			}
 		} else {
-			throw new IllegalStateException("Expecting a keyed table model");
+			for(T item : model.getItems(0, model.getRows())) {
+				K key = getKey(item);
+				m_selectedSet.remove(key);
+			}
 		}
+		callSelectionAllChanged();
 	}
 
 	@Override
-	public boolean isCompleteModelSelected(ITableModel<T> model) {
+	public boolean isCompleteModelSelected(ITableModel<T> model) throws Exception {
 		if(model instanceof TableKeyModelBase) {
 			TableKeyModelBase<K, T> tm = (TableKeyModelBase<K, T>) model;
 			for(K key : tm.getKeys()) {
@@ -171,7 +175,12 @@ public boolean isCompleteModelSelected(ITableModel<T> model) {
 			}
 			return true;
 		} else {
-			throw new IllegalStateException("Expecting a keyed table model");
+			for(T item : model.getItems(0, model.getRows())) {
+				K key = getKey(item);
+				if(!m_selectedSet.containsKey(key))
+					return false;
+			}
+			return true;
 		}
 	}