[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