From 5e0b5db23bc033fea7531ffda669dd99c69606d2 Mon Sep 17 00:00:00 2001
From: gRoussac <gregory.roussac@prestashop.com>
Date: Wed, 31 Jul 2013 15:16:42 +0200
Subject: [PATCH] [*] BO : refact for modules exeptions regarding
 https://github.com/PrestaShop/PrestaShop/pull/614

---
 .../controllers/modules_positions/form.tpl    | 60 +++++++++----------
 1 file changed, 29 insertions(+), 31 deletions(-)

diff --git a/admin-dev/themes/default/template/controllers/modules_positions/form.tpl b/admin-dev/themes/default/template/controllers/modules_positions/form.tpl
index dbce5cbc7..39eca9ff9 100644
--- a/admin-dev/themes/default/template/controllers/modules_positions/form.tpl
+++ b/admin-dev/themes/default/template/controllers/modules_positions/form.tpl
@@ -73,39 +73,37 @@
 	</fieldset>
 </form>
 <script type="text/javascript">
-	//<![CDATA	
-	$(document).ready(function()
+	//<![CDATA
+	function position_exception_textchange()
 	{
-		function position_exception_textchange(obj)
-		{
-			var shopID = obj.attr('id').replace(/\D/g, '');
-			var list = obj.parent().find('#em_list_' + shopID);
-			var values = obj.val().split(',');
-			var len = values.length;
-	
-			list.find('option').prop('selected', false);
-			for (var i = 0; i < len; i++)
-				list.find('option[value="' + $.trim(values[i]) + '"]').prop('selected', true);			
-		}
-
-		function position_exception_listchange(obj)
-		{
-			var shopID = obj.attr('id').replace(/\D/g, '');
-			var str = '';
-			obj.find('option:selected').each(function(){
-				str += obj.val()+', ';
-			});
-	
-			if (str.length > 2)
-				str = str.substr(0, str.length - 2);
-			obj.parent().find('#em_text_' + shopID).val(str);
-		}
+		// TODO : Add & Remove automatically the "custom pages" in the "em_list_x"
+		var obj = $(this);
+		var shopID = obj.attr('id').replace(/\D/g, '');
+		var list = obj.parent().find('#em_list_' + shopID);
+		var values = obj.val().split(',');
+		var len = values.length;
 		
-		$('form[id="hook_module_form"] input[id^="em_text_"]').on('change', function(){
-				position_exception_textchange($(this));
-			}).change();	
-		$('form[id="hook_module_form"] select[id^="em_list_"]').on('change', function(){
-				position_exception_listchange($(this));
+		list.find('option').prop('selected', false);
+		for (var i = 0; i < len; i++)
+			list.find('option[value="' + $.trim(values[i]) + '"]').prop('selected', true);
+	}
+
+	function position_exception_listchange()
+	{
+		var obj = $(this);
+		var shopID = obj.attr('id').replace(/\D/g, '');
+		var str = obj.val().join(', ');
+		
+		obj.parent().find('#em_text_' + shopID).val(str);
+	}
+	
+	$(document).ready(function(){
+		$('form[id="hook_module_form"] input[id^="em_text_"]').each(function(){
+			$(this).change(position_exception_textchange).change();
+		});
+
+		$('form[id="hook_module_form"] select[id^="em_list_"]').each(function(){
+			$(this).change(position_exception_listchange);
 		});
 	});
 	//]]>