Re: [PATCH proxmox-backup v6 7/8] ui: add move namespace action

[Arthur Bied-Charreton <a.bied-charreton@proxmox.com>]

On Tue, Mar 31, 2026 at 02:34:08PM +0200, Hannes Laimer wrote:
> Add a "Move" action to the namespace action column. Opens a dialog
> where the user selects a new parent namespace and name, then submits
> a PUT to the move_namespace API endpoint.
> 
> The source namespace and its descendants are excluded from the parent
> selector to prevent cycles.
> 
Hey, one comment inline :)
> Signed-off-by: Hannes Laimer <h.laimer@proxmox.com>
> ---
>  www/Makefile                  |  1 +
>  www/datastore/Content.js      | 27 +++++++++++-
>  www/form/NamespaceSelector.js | 11 +++++
>  www/window/NamespaceMove.js   | 79 +++++++++++++++++++++++++++++++++++
>  4 files changed, 117 insertions(+), 1 deletion(-)
>  create mode 100644 www/window/NamespaceMove.js
> 
> diff --git a/www/Makefile b/www/Makefile
> index 7745d9f4..f2011af3 100644
> --- a/www/Makefile
> +++ b/www/Makefile
> @@ -80,6 +80,7 @@ JSSRC=							\
>  	window/CreateDirectory.js			\
>  	window/DataStoreEdit.js				\
>  	window/NamespaceEdit.js				\
> +	window/NamespaceMove.js				\
>  	window/MaintenanceOptions.js			\
>  	window/NotesEdit.js				\
>  	window/RemoteEdit.js				\
> diff --git a/www/datastore/Content.js b/www/datastore/Content.js
> index 585a1a2d..b53f9b67 100644
> --- a/www/datastore/Content.js
> +++ b/www/datastore/Content.js
> @@ -668,6 +668,26 @@ Ext.define('PBS.DataStoreContent', {
>              });
>          },
>  
> +        moveNS: function () {
> +            let me = this;
> +            let view = me.getView();
> +            if (!view.namespace || view.namespace === '') {
> +                return;
> +            }
> +            let win = Ext.create('PBS.window.NamespaceMove', {
> +                datastore: view.datastore,
> +                namespace: view.namespace,
> +                taskDone: (success) => {
> +                    if (success) {
> +                        let newNs = win.getNewNamespace();
> +                        view.down('pbsNamespaceSelector').store?.load();
> +                        me.nsChange(null, newNs);
> +                    }
> +                },
> +            });
> +            win.show();
> +        },
> +
>          moveGroup: function (data) {
>              let me = this;
>              let view = me.getView();
> @@ -686,6 +706,8 @@ Ext.define('PBS.DataStoreContent', {
>              let me = this;
>              if (data.ty === 'group') {
>                  me.moveGroup(data);
> +            } else if (data.ty === 'ns') {
> +                me.moveNS();
>              }
>          },
>  
> @@ -1114,10 +1136,13 @@ Ext.define('PBS.DataStoreContent', {
>                          if (data.ty === 'group') {
>                              return Ext.String.format(gettext("Move group '{0}'"), v);
>                          }
> -                        return '';
> +                        return Ext.String.format(gettext("Move namespace '{0}'"), v);
>                      },
>                      getClass: (v, m, { data }) => {
>                          if (data.ty === 'group') { return 'fa fa-arrows'; }
> +                        if (data.ty === 'ns' && !data.isRootNS && data.ns === undefined) {
> +                            return 'fa fa-arrows';
> +                        }
The move action is only accessible when already inside the namespace
being moved. If I want to move `root/x`, I have to navigate to `x` first.
IMO it would be more intuitive/convenient to have the move button on the
`x` row in the root view as well. 
>                          return 'pmx-hidden';
>                      },
>                      isActionDisabled: (v, r, c, i, { data }) => false,
> diff --git a/www/form/NamespaceSelector.js b/www/form/NamespaceSelector.js
> index ddf68254..d349b568 100644
> --- a/www/form/NamespaceSelector.js
> +++ b/www/form/NamespaceSelector.js
> @@ -90,6 +90,17 @@ Ext.define('PBS.form.NamespaceSelector', {
>              },
>          });
>  
> +        if (me.excludeNs) {
> +            me.store.addFilter(
> +                new Ext.util.Filter({
> +                    filterFn: (rec) => {
> +                        let ns = rec.data.ns;
> +                        return ns !== me.excludeNs && !ns.startsWith(`${me.excludeNs}/`);
> +                    },
> +                }),
> +            );
> +        }
> +
>          me.callParent();
>      },
>  });
> diff --git a/www/window/NamespaceMove.js b/www/window/NamespaceMove.js
> new file mode 100644
> index 00000000..415d509d
> --- /dev/null
> +++ b/www/window/NamespaceMove.js
> @@ -0,0 +1,79 @@
> +Ext.define('PBS.window.NamespaceMove', {
> +    extend: 'Proxmox.window.Edit',
> +    alias: 'widget.pbsNamespaceMove',
> +    mixins: ['Proxmox.Mixin.CBind'],
> +
> +    onlineHelp: 'storage-namespaces',
> +
> +    submitText: gettext('Move'),
> +    isCreate: true,
> +    showTaskViewer: true,
> +
> +    cbind: {
> +        url: '/api2/extjs/admin/datastore/{datastore}/namespace',
> +        title: (get) => Ext.String.format(gettext("Move Namespace '{0}'"), get('namespace')),
> +    },
> +    method: 'PUT',
> +
> +    width: 450,
> +    fieldDefaults: {
> +        labelWidth: 120,
> +    },
> +
> +    cbindData: function (initialConfig) {
> +        let ns = initialConfig.namespace ?? '';
> +        let parts = ns.split('/');
> +        return { nsName: parts[parts.length - 1] };
> +    },
> +
> +    // Returns the new-ns path that was submitted, for use by the caller after success.
> +    getNewNamespace: function () {
> +        let me = this;
> +        let parent = me.down('[name=parent]').getValue() || '';
> +        let name = me.down('[name=name]').getValue();
> +        return parent ? `${parent}/${name}` : name;
> +    },
> +
> +    items: {
> +        xtype: 'inputpanel',
> +        onGetValues: function (values) {
> +            let parent = values.parent || '';
> +            let newNs = parent ? `${parent}/${values.name}` : values.name;
> +            return {
> +                ns: this.up('window').namespace,
> +                'new-ns': newNs,
> +            };
> +        },
> +        items: [
> +            {
> +                xtype: 'displayfield',
> +                fieldLabel: gettext('Namespace'),
> +                cbind: {
> +                    value: '{namespace}',
> +                },
> +            },
> +            {
> +                xtype: 'pbsNamespaceSelector',
> +                name: 'parent',
> +                fieldLabel: gettext('New Parent'),
> +                allowBlank: true,
> +                cbind: {
> +                    datastore: '{datastore}',
> +                    excludeNs: '{namespace}',
> +                },
> +            },
> +            {
> +                xtype: 'proxmoxtextfield',
> +                name: 'name',
> +                fieldLabel: gettext('New Name'),
> +                allowBlank: false,
> +                maxLength: 31,
> +                regex: PBS.Utils.SAFE_ID_RE,
> +                regexText: gettext("Only alpha numerical, '_' and '-' (if not at start) allowed"),
> +                cbind: {
> +                    value: '{nsName}',
> +                },
> +            },
> +        ],
> +    },
> +});
> -- 
> 2.47.3
> 
> 
> 
> 
>