[pve-devel] superseded: [PATCH manager] ui: lxc options: fix rendering default and '0' value for tty count

[Dominik Csapak]

superseded by two patches:
https://lore.proxmox.com/pve-devel/20251117092110.628324-1-d.csapak@proxmox.com/
https://lore.proxmox.com/pve-devel/20251117112334.1400372-1-d.csapak@proxmox.com/

On 11/17/25 10:21 AM, Dominik Csapak wrote:
> if '0' was entered, the options grid would show an empty line. Fix that
> by adding a renderer that checks if the value is a number.
> 
> If the default is set, the edit window would always pre-fill the value
> '2' instead of leaving the field empty. Fix that by removing the
> manually set 'value' property.
> 
> Reported-by: Dietmar Maurer <dietmar at proxmox.com>
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>   www/manager6/lxc/Options.js | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/www/manager6/lxc/Options.js b/www/manager6/lxc/Options.js
> index 6cc2e2b4..0acb88f4 100644
> --- a/www/manager6/lxc/Options.js
> +++ b/www/manager6/lxc/Options.js
> @@ -81,6 +81,7 @@ Ext.define('PVE.lxc.Options', {
>               tty: {
>                   header: gettext('TTY count'),
>                   defaultValue: 2,
> +                renderer: (value) => (Ext.isNumber(value) ? value : '2'),
>                   editor: caps.vms['VM.Config.Options']
>                       ? {
>                             xtype: 'proxmoxWindowEdit',
> @@ -90,7 +91,6 @@ Ext.define('PVE.lxc.Options', {
>                                 name: 'tty',
>                                 minValue: 0,
>                                 maxValue: 6,
> -                              value: 2,
>                                 fieldLabel: gettext('TTY count'),
>                                 emptyText: gettext('Default'),
>                                 deleteEmpty: true,