Re: [RFC PATCH proxmox-backup 4/4] ui: tape restore: add selector for additional drives

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

when changing to 'drives' instead, this patch
would become just the change from `name: 'drive'` to `name: 'drives'`
and add the multiselect, no?

On 11/6/26 2:38 PM, Jakob-Tobias Winter wrote:
> Allow choosing up to three more drives for a tape restore, to read
> several media in parallel.
> 
> Signed-off-by: Jakob-Tobias Winter <tobias@stuttgart.de>
> ---
>   www/tape/window/TapeRestore.js | 15 +++++++++++++++
>   1 file changed, 15 insertions(+)
> 
> diff --git a/www/tape/window/TapeRestore.js b/www/tape/window/TapeRestore.js
> index 8ee4f17a9..d28d2f242 100644
> --- a/www/tape/window/TapeRestore.js
> +++ b/www/tape/window/TapeRestore.js
> @@ -374,6 +374,11 @@ Ext.define('PBS.TapeManagement.TapeRestoreWindow', {
>                           let controller = me.up('window').getController();
>                           let vm = controller.getViewModel();
>                           let datastores = [];
> +
> +                        let additionalDrives = values['additional-drives'];
> +                        if (!additionalDrives || additionalDrives.length === 0) {
> +                            delete values['additional-drives'];
> +                        }
>                           if (values.store.toString() !== '') {
>                               let target = values.store.toString();
>                               delete values.store;
> @@ -479,6 +484,16 @@ Ext.define('PBS.TapeManagement.TapeRestoreWindow', {
>                               fieldLabel: gettext('Drive'),
>                               labelWidth: 120,
>                           },
> +                        {
> +                            xtype: 'pbsDriveSelector',
> +                            name: 'additional-drives',
> +                            fieldLabel: gettext('Additional Drives'),
> +                            labelWidth: 120,
> +                            multiSelect: true,
> +                            allowBlank: true,
> +                            skipEmptyText: true,
> +                            emptyText: gettext('None'),
> +                        },
>                           {
>                               xtype: 'pbsDataStoreSelector',
>                               name: 'store',