From bcddfd878a3e7532018684f6023658a70ee61c91 Mon Sep 17 00:00:00 2001
From: Regis Houssin <regis@dolibarr.fr>
Date: Mon, 13 Feb 2012 09:44:24 +0100
Subject: [PATCH 1/3] Fix: bad entity with ajax constant on/off

---
 htdocs/core/ajax/constantonoff.php | 4 ++--
 htdocs/core/lib/ajax.lib.php       | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/htdocs/core/ajax/constantonoff.php b/htdocs/core/ajax/constantonoff.php
index ac2f244969a..3ed30274972 100644
--- a/htdocs/core/ajax/constantonoff.php
+++ b/htdocs/core/ajax/constantonoff.php
@@ -47,10 +47,10 @@ print '<!-- Ajax page called with url '.$_SERVER["PHP_SELF"].'?'.$_SERVER["QUERY
 // Registering the location of boxes
 if ((isset($_GET['action']) && ! empty($_GET['action'])) && (isset($_GET['name']) && ! empty($_GET['name'])) )
 {
-	$entity = (GETPOST('entity','int') >= 0 ? GETPOST('entity','int') : $conf->entity);
+	$entity = GETPOST('entity','int');
 	$action = GETPOST('action', 'alpha');
 	$name = GETPOST('name', 'alpha');
-	
+
 	if ($user->admin)
 	{
 		if ($action == 'set')
diff --git a/htdocs/core/lib/ajax.lib.php b/htdocs/core/lib/ajax.lib.php
index ec500d7ae46..937c187d1e5 100644
--- a/htdocs/core/lib/ajax.lib.php
+++ b/htdocs/core/lib/ajax.lib.php
@@ -1,6 +1,6 @@
 <?php
 /* Copyright (C) 2007-2010 Laurent Destailleur  <eldy@users.sourceforge.net>
- * Copyright (C) 2007-2011 Regis Houssin        <regis@dolibarr.fr>
+ * Copyright (C) 2007-2012 Regis Houssin        <regis@dolibarr.fr>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -229,7 +229,7 @@ function ajax_constantonoff($code,$input=array(),$entity=false)
 {
 	global $conf, $langs;
 
-	$entity = ((! empty($entity) && is_numeric($entity) && $entity > 0) || $entity == 0 ? $entity : $conf->entity);
+	$entity = ((isset($entity) && is_numeric($entity) && $entity >= 0) ? $entity : $conf->entity);
 
 	$out= '<script type="text/javascript">
 		$(function() {

From fdb2122070ad76ba8014d25d193f1ce0f8379d35 Mon Sep 17 00:00:00 2001
From: Regis Houssin <regis@dolibarr.fr>
Date: Mon, 13 Feb 2012 09:45:03 +0100
Subject: [PATCH 2/3] Info: ldap sid is for active directory

---
 htdocs/user/fiche.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/htdocs/user/fiche.php b/htdocs/user/fiche.php
index 9854c298be6..eda9775cc52 100644
--- a/htdocs/user/fiche.php
+++ b/htdocs/user/fiche.php
@@ -669,7 +669,7 @@ if (($action == 'create') || ($action == 'adduserldap'))
     print '</td></tr>';
 
     $generated_password='';
-    if (! $ldap_sid)    // ldap_sid ?
+    if (! $ldap_sid)    // ldap_sid is for activedirectory
     {
         require_once(DOL_DOCUMENT_ROOT."/core/lib/security2.lib.php");
         $generated_password=getRandomPassword('');

From c80646671f586deb2d8ada176640b419134555ec Mon Sep 17 00:00:00 2001
From: Regis Houssin <regis@dolibarr.fr>
Date: Mon, 13 Feb 2012 09:45:38 +0100
Subject: [PATCH 3/3] Fix: add ajax constant on/off

---
 htdocs/admin/workflow.php | 42 ++++++++++++++++++++++++---------------
 1 file changed, 26 insertions(+), 16 deletions(-)

diff --git a/htdocs/admin/workflow.php b/htdocs/admin/workflow.php
index 2f9b43c57db..36389fabeea 100755
--- a/htdocs/admin/workflow.php
+++ b/htdocs/admin/workflow.php
@@ -1,8 +1,8 @@
 <?php
 /* Copyright (C) 2004      Rodolphe Quiedeville <rodolphe@quiedeville.org>
  * Copyright (C) 2004      Eric Seigne          <eric.seigne@ryxeo.com>
- * Copyright (C) 2005-2011 Laurent Destailleur  <eldy@users.sourceforge.net>
- * Copyright (C) 2005-2010 Regis Houssin        <regis@dolibarr.fr>
+ * Copyright (C) 2005-2012 Laurent Destailleur  <eldy@users.sourceforge.net>
+ * Copyright (C) 2005-2012 Regis Houssin        <regis@dolibarr.fr>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -30,13 +30,14 @@ require_once(DOL_DOCUMENT_ROOT."/core/lib/admin.lib.php");
 $langs->load("admin");
 $langs->load("workflow");
 
-if (!$user->admin)
-accessforbidden();
+if (! $user->admin) accessforbidden();
+
+$action = GETPOST('action', 'alpha');
 
 /*
  * Actions
  */
-if (preg_match('/set(.*)/',$_GET["action"],$reg))
+if (preg_match('/set(.*)/',$action,$reg))
 {
     $code=$reg[1];
     if (dolibarr_set_const($db, $code, 1, 'chaine', 0, '', $conf->entity) > 0)
@@ -50,7 +51,7 @@ if (preg_match('/set(.*)/',$_GET["action"],$reg))
     }
 }
 
-if (preg_match('/del(.*)/',$_GET["action"],$reg))
+if (preg_match('/del(.*)/',$action,$reg))
 {
     $code=$reg[1];
     if (dolibarr_del_const($db, $code, $conf->entity) > 0)
@@ -82,8 +83,8 @@ print "<br>";
 
 // Choix du module de gestion des codes clients / fournisseurs
 
-print "<table class=\"noborder\" width=\"100%\">\n";
-print "<tr class=\"liste_titre\">\n";
+print '<table class="noborder" width="100%">'."\n";
+print '<tr class="liste_titre">'."\n";
 print '  <td>'.$langs->trans("Description").'</td>';
 print '  <td align="center">'.$langs->trans("Status").'</td>';
 //print '  <td align="center" width="80">'.$langs->trans("Infos").'</td>';
@@ -104,18 +105,27 @@ if (count($workflowcodes) > 0)
     	$var = !$var;
     	print "<tr ".$bc[$var].">\n";
     	print "<td>".$langs->trans('desc'.$code)."</td>\n";
-    	if (! empty($conf->global->$code))
+    	print '<td align="center">';
+    	if ($conf->use_javascript_ajax)
     	{
-            print '<td align="center"><a href="'.$_SERVER['PHP_SELF'].'?action=del'.$code.'">';
-            print img_picto($langs->trans("Activated"),'switch_on');
-            print '</a></td>';
+    		print ajax_constantonoff($code);
     	}
     	else
     	{
-    		print '<td align="center"><a href="'.$_SERVER['PHP_SELF'].'?action=set'.$code.'">';
-    		print img_picto($langs->trans("Disabled"),'switch_off');
-    		print '</a></td>';
+    		if (! empty($conf->global->$code))
+    		{
+    			print '<a href="'.$_SERVER['PHP_SELF'].'?action=del'.$code.'">';
+    			print img_picto($langs->trans("Activated"),'switch_on');
+    			print '</a>';
+    		}
+    		else
+    		{
+    			print '<a href="'.$_SERVER['PHP_SELF'].'?action=set'.$code.'">';
+    			print img_picto($langs->trans("Disabled"),'switch_off');
+    			print '</a>';
+    		}
     	}
+    	print '</td>';
 
     	//print '<td align="center">';
     	//$s=$modCodeTiers->getToolTip($langs,$soc,-1);
@@ -131,8 +141,8 @@ else
 }
 print '</table>';
 
+llxFooter();
 
 $db->close();
 
-llxFooter();
 ?>