Re: [PATCH proxmox-backup v2 5/5] ui: datastore: disable browse action for root ns

[Dominik Csapak <d.csapak@proxmox.com>]

On 5/6/26 2:12 PM, Erik Fastermann wrote:
> Disable the browse action and tooltip for the root namespace.

same as the previous patch, please include a short 'why'
(the 'what' can most often be seen in the code anyway ;) )

> 
> Signed-off-by: Erik Fastermann <e.fastermann@proxmox.com>
> ---
> 
>   changes since v1:
>   * cleaned up boolean logic
> 
>   www/datastore/Content.js | 9 +++++----
>   1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/www/datastore/Content.js b/www/datastore/Content.js
> index 576e70a59..a2047148f 100644
> --- a/www/datastore/Content.js
> +++ b/www/datastore/Content.js
> @@ -1241,13 +1241,14 @@ Ext.define('PBS.DataStoreContent', {
>                           }
>                           return 'pmx-hidden';
>                       },
> -                    isActionDisabled: (v, r, c, i, { data }) =>
> -                        !(
> +                    isActionDisabled: (v, r, c, i, { data }) => {
> +                        const enableFile =
>                               data.ty === 'file' &&
>                               (data.filename.endsWith('.pxar.didx') ||
>                                   data.filename.endsWith('.mpxar.didx')) &&
> -                            data['crypt-mode'] < 3
> -                        ) && data.ty !== 'ns',
> +                            data['crypt-mode'] < 3;
> +                        return !enableFile && (data.ty !== 'ns' || data.root);
> +                    },
>                   },
>               ],
>           },