[pve-devel] applied: [PATCH manager v2] fix #5302: ui: vm cpu affinity displayed in hardware overview

[Thomas Lamprecht]

Am 15/10/2024 um 11:17 schrieb Timothy Nicholson:
> Signed-off-by: Timothy Nicholson <t.nicholson at proxmox.com>
> ---
> changes since v1:
> * fix javascript indentation
> 
>  www/manager6/qemu/HardwareView.js | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
>

applied, thanks!

FWIW, and definitively pre-exsiting, this rendered could be cleaned up and
condensed a bit, e.g. by doing something like:

----8<----
diff --git a/www/manager6/qemu/HardwareView.js b/www/manager6/qemu/HardwareView.js
index 59e670db9..0b83a23f7 100644
--- a/www/manager6/qemu/HardwareView.js
+++ b/www/manager6/qemu/HardwareView.js
@@ -104,37 +104,27 @@ Ext.define('PVE.qemu.HardwareView', {
 		group: 3,
 		defaultValue: '1',
 		multiKey: ['sockets', 'cpu', 'cores', 'numa', 'vcpus', 'cpulimit', 'cpuunits', 'affinity'],
-		renderer: function(value, metaData, record, rowIndex, colIndex, store, pending) {
-		    var sockets = me.getObjectValue('sockets', 1, pending);
-		    var model = me.getObjectValue('cpu', undefined, pending);
-		    var cores = me.getObjectValue('cores', 1, pending);
-		    var numa = me.getObjectValue('numa', undefined, pending);
-		    var vcpus = me.getObjectValue('vcpus', undefined, pending);
-		    var cpulimit = me.getObjectValue('cpulimit', undefined, pending);
-		    var cpuunits = me.getObjectValue('cpuunits', undefined, pending);
-		    var cpuaffinity = me.getObjectValue('affinity', undefined, pending);
+		renderer: function(_value, metaData, record, rowIndex, colIndex, store, pending) {
+		    let sockets = me.getObjectValue('sockets', 1, pending);
+		    let cores = me.getObjectValue('cores', 1, pending);
 
 		    let res = Ext.String.format(
 		        '{0} ({1} sockets, {2} cores)', sockets * cores, sockets, cores);
 
-		    if (model) {
-			res += ' [' + model + ']';
-		    }
-		    if (numa) {
-			res += ' [numa=' + numa +']';
-		    }
-		    if (vcpus) {
-			res += ' [vcpus=' + vcpus +']';
-		    }
-		    if (cpulimit) {
-			res += ' [cpulimit=' + cpulimit +']';
-		    }
-		    if (cpuunits) {
-			res += ' [cpuunits=' + cpuunits +']';
-		    }
-		    if (cpuaffinity) {
-			res += ' [cpuaffinity=' + cpuaffinity + ']';
-		    }
+		    let renderKeyIfSet = (key, label, valueOnly) => {
+			let value = me.getObjectValue(key, undefined, pending);
+			if (value) {
+			    label ??= key;
+			    res += valueOnly ? ` [${value}]` : ` [${label}=${value}]`;
+			}
+		    };
+
+		    renderKeyIfSet('cpu', 'model', true);
+		    renderKeyIfSet('numa');
+		    renderKeyIfSet('vcpus');
+		    renderKeyIfSet('cpulimit');
+		    renderKeyIfSet('cpuunits');
+		    renderKeyIfSet('affinity');
 
 		    return res;
 		},