proxmox-mirrors/pve-manager
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-manager synced 2025-07-16 19:30:54 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

ui: bandwidth limit selector: make it possible to allow zero

Browse Source 
The initial value is '' and in getSubmitValue(), previously the branch
   if (v == 0 || v == 0.0) return null;
was taken, because of the lax '==' comparision. Make sure we still return null
for '' by explicitly checking for it.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
This commit is contained in:
Fabian Ebner 2021-03-15 12:57:28 +01:00 committed by Thomas Lamprecht
parent f8d1d5ad9a
commit dcbc3b3ee9
1 changed files with 14 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
www/manager6/form/BandwidthSelector.js
View File

@ -36,12 +36,16 @@ Ext.define('PVE.form.BandwidthField', {
// for KiB set it to 'KiB'
backendUnit: undefined,
// allow setting 0 and using it as a submit value
allowZero: false,
items: [
{
xtype: 'numberfield',
cbind: {
name: '{name}',
emptyText: '{emptyText}',
allowZero: '{allowZero}',
},
minValue: 0,
step: 1,
@ -61,7 +65,9 @@ Ext.define('PVE.form.BandwidthField', {
this._transformed = true;
}
if (v == 0) v = undefined;
if (Number(v) === 0 && !this.allowZero) {
v = undefined;
}
return Ext.form.field.Text.prototype.setValue.call(this, v);
},
@ -69,9 +75,13 @@ Ext.define('PVE.form.BandwidthField', {
let v = this.processRawValue(this.getRawValue());
v = v.replace(this.decimalSeparator, '.');
if (v === undefined) return null;
// FIXME: make it configurable, as this only works if 0 === default
if (v == 0 || v == 0.0) return null;
if (v === undefined || v === '') {
return null;
}
if (Number(v) === 0) {
return this.allowZero ? 0 : null;
}
let fieldct = this.up('pveBandwidthField');
let vm = fieldct.getViewModel();