Re: [PATCH proxmox-backup v8 12/13] ui: add move namespace action

["Michael Köppl" <m.koeppl@proxmox.com>]

On Wed Apr 22, 2026 at 3:39 PM CEST, Hannes Laimer wrote:

[snip]

> +    items: {
> +        xtype: 'inputpanel',
> +        onGetValues: function (values) {
> +            let win = this.up('window');
> +            let result = {
> +                ns: win.namespace,
> +                'target-ns': win.getTargetNs(),
> +            };
> +            if (values['delete-source'] !== undefined) {
> +                result['delete-source'] = values['delete-source'];
> +            }
> +            if (values['merge-groups'] !== undefined) {
> +                result['merge-groups'] = values['merge-groups'];
> +            }

You probably noticed this already, but wanted to mention it nonetheless:
these are also not sent to the backend if the checkbox is unchecked. So
in this case it would e.g. delete the source even if you unchecked.

> +            if (values['max-depth'] !== undefined && values['max-depth'] !== '') {
> +                result['max-depth'] = values['max-depth'];
> +            }
> +            return result;
> +        },
> +        items: [
> +            {

[snip]