[pve-devel] [PATCH manager 2/2] Fix jslint warning: 'sortedList' is already defined

Emmanuel Kasper e.kasper at proxmox.com
Mon Oct 10 14:32:51 CEST 2016


sortedList was defined twice in the function
remove the declaration at the beginning of function as declaring a var
just before using it is more similar to the rest of code
---
 www/manager6/form/ControllerSelector.js | 1 -
 1 file changed, 1 deletion(-)

diff --git a/www/manager6/form/ControllerSelector.js b/www/manager6/form/ControllerSelector.js
index b5c71d9..bf8a53e 100644
--- a/www/manager6/form/ControllerSelector.js
+++ b/www/manager6/form/ControllerSelector.js
@@ -18,7 +18,6 @@ Ext.define('PVE.form.ControllerSelector', {
     vmconfig: {}, // used to check for existing devices
 
     sortByPreviousUsage: function(vmconfig, controllerList) {
-	var sortedList = [];
 
 	var usedControllers = Ext.clone(PVE.form.ControllerSelector.maxIds);
 
-- 
2.1.4





More information about the pve-devel mailing list