Skip to content
Snippets Groups Projects
Commit 5709179c authored by Laurent Destailleur's avatar Laurent Destailleur
Browse files

Fix: Box was not added on correct side

parent 0e13ded6
No related branches found
No related tags found
No related merge requests found
...@@ -57,7 +57,7 @@ if ($action == 'add') ...@@ -57,7 +57,7 @@ if ($action == 'add')
$db->begin(); $db->begin();
// Initialize distinctfkuser with all already existing values of fk_user (user that use a personalized view of boxes for pos) // Initialize distinct fkuser with all already existing values of fk_user (user that use a personalized view of boxes for page "pos")
$distinctfkuser=array(); $distinctfkuser=array();
if (! $error) if (! $error)
{ {
...@@ -85,33 +85,31 @@ if ($action == 'add') ...@@ -85,33 +85,31 @@ if ($action == 'add')
} }
} }
$distinctfkuser['0']='0'; // Add entry for fk_user = 0. We must use string as key and val
foreach($distinctfkuser as $fk_user) foreach($distinctfkuser as $fk_user)
{ {
if (! $error && $fk_user != 0) // We will add fk_user = 0 later. if (! $error && $fk_user != '')
{ {
$sql = "INSERT INTO ".MAIN_DB_PREFIX."boxes ("; $nbboxonleft=$nbboxonright=0;
$sql.= "box_id, position, box_order, fk_user, entity"; $sql = "SELECT box_order FROM ".MAIN_DB_PREFIX."boxes WHERE position = ".GETPOST("pos","alpha")." AND fk_user = ".$fk_user." AND entity = ".$conf->entity;
$sql.= ") values (";
$sql.= GETPOST("boxid","int").", ".GETPOST("pos","alpha").", 'A01', ".$fk_user.", ".$conf->entity;
$sql.= ")";
dol_syslog("boxes.php activate box sql=".$sql); dol_syslog("boxes.php activate box sql=".$sql);
$resql = $db->query($sql); $resql = $db->query($sql);
if (! $resql) if ($resql)
{ {
$errmesg=$db->lasterror(); while($obj = $db->fetch_object($resql))
$error++; {
} $boxorder=$obj->box_order;
if (preg_match('/A/',$boxorder)) $nbboxonleft++;
if (preg_match('/B/',$boxorder)) $nbboxonright++;
} }
} }
else dol_print_error($db);
// If value 0 was not included, we add it.
if (! $error)
{
$sql = "INSERT INTO ".MAIN_DB_PREFIX."boxes ("; $sql = "INSERT INTO ".MAIN_DB_PREFIX."boxes (";
$sql.= "box_id, position, box_order, fk_user, entity"; $sql.= "box_id, position, box_order, fk_user, entity";
$sql.= ") values ("; $sql.= ") values (";
$sql.= GETPOST("boxid","int").", ".GETPOST("pos","alpha").", 'A01', 0, ".$conf->entity; $sql.= GETPOST("boxid","int").", ".GETPOST("pos","alpha").", '".(($nbboxonleft > $nbboxonright) ? 'B01' : 'A01')."', ".$fk_user.", ".$conf->entity;
$sql.= ")"; $sql.= ")";
dol_syslog("boxes.php activate box sql=".$sql); dol_syslog("boxes.php activate box sql=".$sql);
...@@ -122,6 +120,7 @@ if ($action == 'add') ...@@ -122,6 +120,7 @@ if ($action == 'add')
$error++; $error++;
} }
} }
}
if (! $error) if (! $error)
{ {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment