From a6378ab0bb430f146434b1ea564e7e8d67da6a28 Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@destailleur.fr>
Date: Sun, 24 Feb 2013 18:08:27 +0100
Subject: [PATCH] Cleaner HTML code

---
 htdocs/core/lib/ajax.lib.php | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/htdocs/core/lib/ajax.lib.php b/htdocs/core/lib/ajax.lib.php
index 4f7beb5356b..64b1339a944 100644
--- a/htdocs/core/lib/ajax.lib.php
+++ b/htdocs/core/lib/ajax.lib.php
@@ -364,7 +364,8 @@ function ajax_constantonoff($code, $input=array(), $entity=false)
 
 	$entity = ((isset($entity) && is_numeric($entity) && $entity >= 0) ? $entity : $conf->entity);
 
-	$out= '<script type="text/javascript">
+	$out= "\n<!-- Ajax code to switch constant ".$code." -->".'
+	<script type="text/javascript">
 		$(function() {
 			var input = '.json_encode($input).';
 			var url = \''.DOL_URL_ROOT.'/core/ajax/constantonoff.php\';
@@ -376,11 +377,8 @@ function ajax_constantonoff($code, $input=array(), $entity=false)
 			// Set constant
 			$("#set_" + code).click(function() {
 				if (input.alert && input.alert.set) {
-					// Posibility to force label of buttons
-					if (input.alert.set.yesButton)
-						yesButton = input.alert.set.yesButton;
-					if (input.alert.set.noButton)
-						noButton = input.alert.set.noButton;
+					if (input.alert.set.yesButton) yesButton = input.alert.set.yesButton;
+					if (input.alert.set.noButton)  noButton = input.alert.set.noButton;
 					confirmConstantAction("set", url, code, input, input.alert.set, entity, yesButton, noButton);
 				} else {
 					setConstant(url, code, input, entity);
@@ -390,22 +388,20 @@ function ajax_constantonoff($code, $input=array(), $entity=false)
 			// Del constant
 			$("#del_" + code).click(function() {
 				if (input.alert && input.alert.del) {
-					// Posibility to force label of buttons
-					if (input.alert.del.yesButton)
-						yesButton = input.alert.del.yesButton;
-					if (input.alert.del.noButton)
-						noButton = input.alert.del.noButton;
+					if (input.alert.del.yesButton) yesButton = input.alert.del.yesButton;
+					if (input.alert.del.noButton)  noButton = input.alert.del.noButton;
 					confirmConstantAction("del", url, code, input, input.alert.del, entity, yesButton, noButton);
 				} else {
 					delConstant(url, code, input, entity);
 				}
 			});
 		});
-	</script>';
+	</script>'."\n";
 
 	$out.= '<div id="confirm_'.$code.'" title="" style="display: none;"></div>';
 	$out.= '<span id="set_'.$code.'" class="linkobject '.(! empty($conf->global->$code)?'hideobject':'').'">'.img_picto($langs->trans("Disabled"),'switch_off').'</span>';
 	$out.= '<span id="del_'.$code.'" class="linkobject '.(! empty($conf->global->$code)?'':'hideobject').'">'.img_picto($langs->trans("Enabled"),'switch_on').'</span>';
+	$out.="\n";
 
 	return $out;
 }