[pve-devel] [PATCH manager] remove 'VZDump backup file' option from upload web gui

Wolfgang Bumiller w.bumiller at proxmox.com
Tue Feb 2 12:27:05 CET 2016


Looks fine as the API currently only recognizes 'iso' and 'vztmpl',
while 'backup' runs into a `raise_param_exc()`.

On Tue, Feb 02, 2016 at 11:43:24AM +0100, Dominik Csapak wrote:
> since we do not want to upload vzdump backup files (see bug #498),
> it is confusing to expose this option in the gui, since currently
> it can never succeed.
> 
> the correct way would probably allow it based on the users permissions
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>

Acked-by: Wolfgang Bumiller <w.bumiller at proxmox.com>

> ---
>  www/manager/storage/ContentView.js | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/www/manager/storage/ContentView.js b/www/manager/storage/ContentView.js
> index 55b11df..265fab9 100644
> --- a/www/manager/storage/ContentView.js
> +++ b/www/manager/storage/ContentView.js
> @@ -184,7 +184,7 @@ Ext.define('PVE.storage.Upload', {
>  	    items: [
>  		{
>  		    xtype: 'pveContentTypeSelector',
> -		    cts: ['iso', 'backup', 'vztmpl'],
> +		    cts: ['iso', 'vztmpl'],
>  		    fieldLabel: gettext('Content'),
>  		    name: 'content',
>  		    value: 'iso'
> -- 
> 2.1.4
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 




More information about the pve-devel mailing list