[PATCH pve-manager v2 1/2] ui: dc summary: update node grid fields in a single set() call

[Erik Fastermann <e.fastermann@proxmox.com>]

Each set() call fires its own update event, so the grid redrew the row
once per field. Passing all fields as an object fires a single event
instead.

The commit() call only served to clear the dirty flag, which the
documented dirty option does directly, without emitting another update
event per row.

This is a refactor and should have no user-visible effect.

Signed-off-by: Erik Fastermann <e.fastermann@proxmox.com>
---
 www/manager6/dc/Summary.js | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/www/manager6/dc/Summary.js b/www/manager6/dc/Summary.js
index bf7e99c6..cd4749be 100644
--- a/www/manager6/dc/Summary.js
+++ b/www/manager6/dc/Summary.js
@@ -164,12 +164,16 @@ Ext.define('PVE.dc.Summary', {
 
                         if (gridstore.getById(data.id)) {
                             let griditem = gridstore.getById(data.id);
-                            griditem.set('cpuusage', data.cpu);
                             let max = data.maxmem || 1;
                             let val = data.mem || 0;
-                            griditem.set('memoryusage', val / max);
-                            griditem.set('uptime', data.uptime);
-                            griditem.commit(); // else the store marks the field as dirty
+                            griditem.set(
+                                {
+                                    cpuusage: data.cpu,
+                                    memoryusage: val / max,
+                                    uptime: data.uptime,
+                                },
+                                { dirty: false },
+                            );
                         }
                         break;
                     case 'storage': {
-- 
2.47.3