From 7823b35e66503432bdf090f88cbfc68319812865 Mon Sep 17 00:00:00 2001 From: Michal Fox Date: Fri, 2 Oct 2026 16:50:56 +0200 Subject: [PATCH] fix #6109: node network view: fix sorting by active state The API only sets 'active' for active interfaces, so it is either 1 or missing. As 1 compares neither greater nor less than undefined, sorting by the 'Active' column kept the previous order. The same applies to the 'Autostart' and 'VLAN aware' columns. Declare those fields as booleans, so that missing values get converted to false and the interfaces can be sorted by them. The shown values do not change, as missing values were already rendered as 'No'. Signed-off-by: Michal Fox --- src/node/NetworkView.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/node/NetworkView.js b/src/node/NetworkView.js index 29b3fe6..5b0c618 100644 --- a/src/node/NetworkView.js +++ b/src/node/NetworkView.js @@ -1,12 +1,13 @@ Ext.define('proxmox-networks', { extend: 'Ext.data.Model', fields: [ - 'active', + { name: 'active', type: 'boolean' }, 'address', 'address6', 'altnames', - 'autostart', + { name: 'autostart', type: 'boolean' }, 'bridge_ports', + { name: 'bridge_vlan_aware', type: 'boolean' }, 'cidr', 'cidr6', 'comments',