Fix #3845 by checking if all items are already selected (#3866)

vnbaaij · dvoituron · web-flow · commit b1fb5becbdf4 · 2025-06-03T23:00:24.000+02:00
Co-authored-by: Denis Voituron &lt;dvoituron@outlook.com&gt;
diff --git a/src/Core/Components/DataGrid/Columns/SelectColumn.cs b/src/Core/Components/DataGrid/Columns/SelectColumn.cs
@@ -528,9 +528,10 @@ internal async Task OnClickAllAsync(MouseEventArgs e)
             await SelectAllChanged.InvokeAsync(SelectAll);
         }
 
+        var count = SelectedItems.Count();
         // SelectedItems
         _selectedItems.Clear();
-        if (SelectAll == true)
+        if (SelectAll == true && count != InternalGridContext.TotalItemCount)
         {
             // Only add selectable items
             _selectedItems.AddRange((InternalGridContext.Grid.Items?.ToList() ?? InternalGridContext.Items)

Original file line number	Diff line number	Diff line change
`@@ -528,9 +528,10 @@ internal async Task OnClickAllAsync(MouseEventArgs e)`
`528`	`528`	`await SelectAllChanged.InvokeAsync(SelectAll);`
`529`	`529`	`}`
`530`	`530`
	`531`	`+ var count = SelectedItems.Count();`
`531`	`532`	`// SelectedItems`
`532`	`533`	`_selectedItems.Clear();`
`533`		`- if (SelectAll == true)`
	`534`	`+ if (SelectAll == true && count != InternalGridContext.TotalItemCount)`
`534`	`535`	`{`
`535`	`536`	`// Only add selectable items`
`536`	`537`	`_selectedItems.AddRange((InternalGridContext.Grid.Items?.ToList() ?? InternalGridContext.Items)`