ui: Utils: use render functions from widget-toolkit

they live there now, so we can delete them here and use the ones from
widget-toolkit instead

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Dominik Csapak 2021-04-19 13:00:45 +02:00 committed by Thomas Lamprecht
parent b988934559
commit 1bd7bcdbf1
14 changed files with 32 additions and 95 deletions

View File

@ -935,22 +935,6 @@ Ext.define('PVE.Utils', {
return PVE.Utils.log_severity_hash[value] || value; return PVE.Utils.log_severity_hash[value] || value;
}, },
render_cpu: function(value, metaData, record, rowIndex, colIndex, store) {
if (!(record.data.uptime && Ext.isNumeric(value))) {
return '';
}
var maxcpu = record.data.maxcpu || 1;
if (!Ext.isNumeric(maxcpu) && maxcpu >= 1) {
return '';
}
var per = value * 100;
return per.toFixed(1) + '% of ' + maxcpu.toString() + (maxcpu > 1 ? 'CPUs' : 'CPU');
},
calculate_hostcpu: function(data) { calculate_hostcpu: function(data) {
if (!(data.uptime && Ext.isNumeric(data.cpu))) { if (!(data.uptime && Ext.isNumeric(data.cpu))) {
@ -1001,14 +985,6 @@ Ext.define('PVE.Utils', {
return per.toFixed(1) + '% of ' + maxcpu.toString() + (maxcpu > 1 ? 'CPUs' : 'CPU'); return per.toFixed(1) + '% of ' + maxcpu.toString() + (maxcpu > 1 ? 'CPUs' : 'CPU');
}, },
render_size: function(value, metaData, record, rowIndex, colIndex, store) {
if (!Ext.isNumeric(value)) {
return '';
}
return Proxmox.Utils.format_size(value);
},
render_bandwidth: function(value) { render_bandwidth: function(value) {
if (!Ext.isNumeric(value)) { if (!Ext.isNumeric(value)) {
return ''; return '';
@ -1113,7 +1089,7 @@ Ext.define('PVE.Utils', {
return ''; return '';
} }
return PVE.Utils.render_size(value); return Proxmox.Utils.render_size(value);
}, },
calculate_disk_usage: function(data) { calculate_disk_usage: function(data) {
@ -1147,7 +1123,7 @@ Ext.define('PVE.Utils', {
return ''; return '';
} }
return PVE.Utils.render_size(value); return Proxmox.Utils.render_size(value);
}, },
get_object_icon_class: function(type, record) { get_object_icon_class: function(type, record) {
@ -1197,36 +1173,6 @@ Ext.define('PVE.Utils', {
return Proxmox.Utils.format_task_description(type, id); return Proxmox.Utils.format_task_description(type, id);
}, },
/* render functions for new status panel */
render_usage: function(val) {
return (val*100).toFixed(2) + '%';
},
render_cpu_usage: function(val, max) {
return Ext.String.format(gettext('{0}% of {1}') +
' ' + gettext('CPU(s)'), (val*100).toFixed(2), max);
},
render_size_usage: function(val, max) {
if (max === 0) {
return gettext('N/A');
}
return (val*100/max).toFixed(2) + '% '+ '(' +
Ext.String.format(gettext('{0} of {1}'),
PVE.Utils.render_size(val), PVE.Utils.render_size(max)) + ')';
},
/* this is different for nodes */
render_node_cpu_usage: function(value, record) {
return PVE.Utils.render_cpu_usage(value, record.cpus);
},
/* this is different for nodes */
render_node_size_usage: function(record) {
return PVE.Utils.render_size_usage(record.used, record.total);
},
render_optional_url: function(value) { render_optional_url: function(value) {
var match; var match;
if (value && (match = value.match(/^https?:\/\//)) !== null) { if (value && (match = value.match(/^https?:\/\//)) !== null) {

View File

@ -536,7 +536,7 @@ Ext.define('PVE.node.CephOsdTree', {
}, },
render_osd_size: function(value, metaData, rec) { render_osd_size: function(value, metaData, rec) {
return this.render_osd_val(PVE.Utils.render_size(value), metaData, rec); return this.render_osd_val(Proxmox.Utils.render_size(value), metaData, rec);
}, },
control: { control: {

View File

@ -232,9 +232,7 @@ Ext.define('PVE.node.CephPoolList', {
minWidth: 100, minWidth: 100,
flex: 1, flex: 1,
align: 'right', align: 'right',
renderer: function(v, meta, rec) { renderer: (v, meta, rec) => `${v}/${rec.data.min_size}`,
return v + '/' + rec.data.min_size;
},
dataIndex: 'size', dataIndex: 'size',
}, },
{ {
@ -314,11 +312,11 @@ Ext.define('PVE.node.CephPoolList', {
align: 'right', align: 'right',
dataIndex: 'bytes_used', dataIndex: 'bytes_used',
summaryType: 'sum', summaryType: 'sum',
summaryRenderer: PVE.Utils.render_size, summaryRenderer: Proxmox.Utils.render_size,
renderer: function(v, meta, rec) { renderer: function(v, meta, rec) {
let percentage = Ext.util.Format.percent(rec.data.percent_used, '0.00'); let percentage = Ext.util.Format.percent(rec.data.percent_used, '0.00');
let used = PVE.Utils.render_size(v); let used = Proxmox.Utils.render_size(v);
return used + ' (' + percentage + ')'; return `${used} (${percentage})`;
}, },
}, },
], ],

View File

@ -309,8 +309,8 @@ Ext.define('PVE.node.CephStatus', {
let total = pgmap.bytes_total; let total = pgmap.bytes_total;
var text = Ext.String.format(gettext('{0} of {1}'), var text = Ext.String.format(gettext('{0} of {1}'),
PVE.Utils.render_size(used), Proxmox.Utils.render_size(used),
PVE.Utils.render_size(total), Proxmox.Utils.render_size(total),
); );
// update the usage widget // update the usage widget

View File

@ -145,7 +145,7 @@ Ext.define('PVE.data.ResourceStore', {
maxdisk: { maxdisk: {
header: gettext('Disk size'), header: gettext('Disk size'),
type: 'integer', type: 'integer',
renderer: PVE.Utils.render_size, renderer: Proxmox.Utils.render_size,
sortable: true, sortable: true,
hidden: true, hidden: true,
width: 100, width: 100,
@ -170,7 +170,7 @@ Ext.define('PVE.data.ResourceStore', {
maxmem: { maxmem: {
header: gettext('Memory size'), header: gettext('Memory size'),
type: 'integer', type: 'integer',
renderer: PVE.Utils.render_size, renderer: Proxmox.Utils.render_size,
hidden: true, hidden: true,
sortable: true, sortable: true,
width: 100, width: 100,
@ -178,7 +178,7 @@ Ext.define('PVE.data.ResourceStore', {
cpu: { cpu: {
header: gettext('CPU usage'), header: gettext('CPU usage'),
type: 'float', type: 'float',
renderer: PVE.Utils.render_cpu, renderer: Proxmox.Utils.render_cpu,
sortable: true, sortable: true,
width: 100, width: 100,
}, },

View File

@ -209,10 +209,10 @@ Ext.define('PVE.dc.Summary', {
var text = Ext.String.format(gettext('of {0} CPU(s)'), maxcpu); var text = Ext.String.format(gettext('of {0} CPU(s)'), maxcpu);
cpustat.updateValue(cpu/maxcpu, text); cpustat.updateValue(cpu/maxcpu, text);
text = Ext.String.format(gettext('{0} of {1}'), PVE.Utils.render_size(memory), PVE.Utils.render_size(maxmem)); text = Ext.String.format(gettext('{0} of {1}'), Proxmox.Utils.render_size(memory), Proxmox.Utils.render_size(maxmem));
memorystat.updateValue(memory/maxmem, text); memorystat.updateValue(memory/maxmem, text);
text = Ext.String.format(gettext('{0} of {1}'), PVE.Utils.render_size(used), PVE.Utils.render_size(total)); text = Ext.String.format(gettext('{0} of {1}'), Proxmox.Utils.render_size(used), Proxmox.Utils.render_size(total));
storagestat.updateValue(used/total, text); storagestat.updateValue(used/total, text);
gueststatus.updateValues(qemu, lxc, error); gueststatus.updateValues(qemu, lxc, error);

View File

@ -54,7 +54,7 @@ Ext.define('PVE.form.NodeSelector', {
}, },
{ {
header: gettext('CPU usage'), header: gettext('CPU usage'),
renderer: PVE.Utils.render_cpu, renderer: Proxmox.Utils.render_cpu,
sortable: true, sortable: true,
width: 100, width: 100,
dataIndex: 'cpu', dataIndex: 'cpu',

View File

@ -66,7 +66,7 @@ Ext.define('PVE.ha.GroupInputPanel', {
}, },
{ {
header: gettext('CPU usage'), header: gettext('CPU usage'),
renderer: PVE.Utils.render_cpu, renderer: Proxmox.Utils.render_cpu,
sortable: true, sortable: true,
width: 150, width: 150,
dataIndex: 'cpu', dataIndex: 'cpu',

View File

@ -27,7 +27,7 @@ Ext.define('PVE.node.StatusView', {
title: gettext('CPU usage'), title: gettext('CPU usage'),
valueField: 'cpu', valueField: 'cpu',
maxField: 'cpuinfo', maxField: 'cpuinfo',
renderer: PVE.Utils.render_node_cpu_usage, renderer: Proxmox.Utils.render_node_cpu_usage,
}, },
{ {
itemId: 'wait', itemId: 'wait',
@ -54,7 +54,7 @@ Ext.define('PVE.node.StatusView', {
title: gettext('RAM usage'), title: gettext('RAM usage'),
valueField: 'memory', valueField: 'memory',
maxField: 'memory', maxField: 'memory',
renderer: PVE.Utils.render_node_size_usage, renderer: Proxmox.Utils.render_node_size_usage,
}, },
{ {
itemId: 'ksm', itemId: 'ksm',
@ -62,7 +62,7 @@ Ext.define('PVE.node.StatusView', {
title: gettext('KSM sharing'), title: gettext('KSM sharing'),
textField: 'ksm', textField: 'ksm',
renderer: function(record) { renderer: function(record) {
return PVE.Utils.render_size(record.shared); return Proxmox.Utils.render_size(record.shared);
}, },
padding: '0 15 10 15', padding: '0 15 10 15',
}, },
@ -72,7 +72,7 @@ Ext.define('PVE.node.StatusView', {
title: gettext('HD space') + '(root)', title: gettext('HD space') + '(root)',
valueField: 'rootfs', valueField: 'rootfs',
maxField: 'rootfs', maxField: 'rootfs',
renderer: PVE.Utils.render_node_size_usage, renderer: Proxmox.Utils.render_node_size_usage,
}, },
{ {
iconCls: 'fa fa-fw fa-refresh', iconCls: 'fa fa-fw fa-refresh',
@ -81,7 +81,7 @@ Ext.define('PVE.node.StatusView', {
title: gettext('SWAP usage'), title: gettext('SWAP usage'),
valueField: 'swap', valueField: 'swap',
maxField: 'swap', maxField: 'swap',
renderer: PVE.Utils.render_node_size_usage, renderer: Proxmox.Utils.render_node_size_usage,
}, },
{ {
xtype: 'box', xtype: 'box',
@ -94,14 +94,7 @@ Ext.define('PVE.node.StatusView', {
printBar: false, printBar: false,
title: gettext('CPU(s)'), title: gettext('CPU(s)'),
textField: 'cpuinfo', textField: 'cpuinfo',
renderer: function(cpuinfo) { renderer: Proxmox.Utils.render_cpu_model,
return cpuinfo.cpus + " x " + cpuinfo.model + " (" +
cpuinfo.sockets.toString() + " " +
(cpuinfo.sockets > 1
? gettext('Sockets')
: gettext('Socket')
) + ")";
},
value: '', value: '',
}, },
{ {

View File

@ -139,7 +139,7 @@ Ext.define('PVE.node.CreateZFS', {
{ {
text: gettext('Size'), text: gettext('Size'),
dataIndex: 'size', dataIndex: 'size',
renderer: PVE.Utils.render_size, renderer: Proxmox.Utils.render_size,
flex: 1, flex: 1,
}, },
{ {

View File

@ -71,7 +71,7 @@ Ext.define('PVE.panel.GuestStatusView', {
title: gettext('CPU usage'), title: gettext('CPU usage'),
valueField: 'cpu', valueField: 'cpu',
maxField: 'cpus', maxField: 'cpus',
renderer: PVE.Utils.render_cpu_usage, renderer: Proxmox.Utils.render_cpu_usage,
// in this specific api call // in this specific api call
// we already have the correct value for the usage // we already have the correct value for the usage
calculate: Ext.identityFn, calculate: Ext.identityFn,
@ -106,9 +106,9 @@ Ext.define('PVE.panel.GuestStatusView', {
var me = this; var me = this;
me.setPrintBar(used > 0); me.setPrintBar(used > 0);
if (used === 0) { if (used === 0) {
return PVE.Utils.render_size(max); return Proxmox.Utils.render_size(max);
} else { } else {
return PVE.Utils.render_size_usage(used, max); return Proxmox.Utils.render_size_usage(used, max);
} }
}, },
}, },

View File

@ -32,9 +32,9 @@ Ext.define('PVE.panel.StatusView', {
} }
if (!Ext.isNumeric(max) || max === 1) { if (!Ext.isNumeric(max) || max === 1) {
return PVE.Utils.render_usage(val); return Proxmox.Utils.render_usage(val);
} }
return PVE.Utils.render_size_usage(val, max); return Proxmox.Utils.render_size_usage(val, max);
}, },
fieldCalculator: function(used, max) { fieldCalculator: function(used, max) {

View File

@ -45,21 +45,21 @@ Ext.define('PVE.panel.TemplateStatusView', {
iconCls: 'fa fa-fw pve-itype-icon-memory pve-icon', iconCls: 'fa fa-fw pve-itype-icon-memory pve-icon',
title: gettext('Memory'), title: gettext('Memory'),
textField: 'maxmem', textField: 'maxmem',
renderer: PVE.Utils.render_size, renderer: Proxmox.Utils.render_size,
}, },
{ {
itemId: 'swap', itemId: 'swap',
iconCls: 'fa fa-refresh fa-fw', iconCls: 'fa fa-refresh fa-fw',
title: gettext('Swap'), title: gettext('Swap'),
textField: 'maxswap', textField: 'maxswap',
renderer: PVE.Utils.render_size, renderer: Proxmox.Utils.render_size,
}, },
{ {
itemId: 'disk', itemId: 'disk',
iconCls: 'fa fa-hdd-o fa-fw', iconCls: 'fa fa-hdd-o fa-fw',
title: gettext('Bootdisk size'), title: gettext('Bootdisk size'),
textField: 'maxdisk', textField: 'maxdisk',
renderer: PVE.Utils.render_size, renderer: Proxmox.Utils.render_size,
}, },
{ {
xtype: 'box', xtype: 'box',

View File

@ -263,7 +263,7 @@ Ext.define('PVE.window.Migrate', {
}); });
} }
} else { } else {
var size_string = disk.size ? '(' + PVE.Utils.render_size(disk.size) + ')' : ''; var size_string = disk.size ? '(' + Proxmox.Utils.render_size(disk.size) + ')' : '';
migration['with-local-disks'] = 1; migration['with-local-disks'] = 1;
migration.preconditions.push({ migration.preconditions.push({
text: Ext.String.format('Migration with local disk might take long: {0} {1}', text: Ext.String.format('Migration with local disk might take long: {0} {1}',