From: Dominik Csapak <d.csapak@proxmox.com>
Date: Mon, 21 Jan 2019 08:44:37 +0000 (+0100)
Subject: refactor format_content_types
X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=6b82734a72572674499f9e49065f494c84d953f5;p=pve-manager-legacy.git

refactor format_content_types

much shorter and more generic

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---

diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js
index a190a7ab..cf9cbdc9 100644
--- a/www/manager6/Utils.js
+++ b/www/manager6/Utils.js
@@ -435,11 +435,13 @@ Ext.define('PVE.Utils', { utilities: {
 	return days.toFixed(1) + 'd';
     },
 
-    imagesText: gettext('Disk image'),
-    backupFileText: gettext('VZDump backup file'),
-    vztmplText: gettext('Container template'),
-    isoImageText: gettext('ISO image'),
-    containersText: gettext('Container'),
+    contentTypes: {
+	'images': gettext('Disk image'),
+	'backup': gettext('VZDump backup file'),
+	'vztmpl': gettext('Container template'),
+	'iso': gettext('ISO image'),
+	'rootdir': gettext('Container')
+    },
 
     storageSchema: {
 	dir: {
@@ -549,23 +551,9 @@ Ext.define('PVE.Utils', { utilities: {
     },
 
     format_content_types: function(value) {
-	var cta = [];
-
-	Ext.each(value.split(',').sort(), function(ct) {
-	    if (ct === 'images') {
-		cta.push(PVE.Utils.imagesText);
-	    } else if (ct === 'backup') {
-		cta.push(PVE.Utils.backupFileText);
-	    } else if (ct === 'vztmpl') {
-		cta.push(PVE.Utils.vztmplText);
-	    } else if (ct === 'iso') {
-		cta.push(PVE.Utils.isoImageText);
-	    } else if (ct === 'rootdir') {
-		cta.push(PVE.Utils.containersText);
-	    }
-	});
-
-	return cta.join(', ');
+	return value.split(',').sort().map(function(ct) {
+	    return PVE.Utils.contentTypes[ct] || ct;
+	}).join(', ');
     },
 
     render_storage_content: function(value, metaData, record) {