New: Amlioration des fonctions addright et delright pour ne plus se baser sur une rgle de gestion par rapport aux id.
This commit is contained in:
Laurent Destailleur
parent
8ae1ca21fc
commit
bc1201e4d6
@ -171,142 +171,192 @@ class User
|
|||||||
/**
|
/**
|
||||||
* \brief Ajoute un droit a l'utilisateur
|
* \brief Ajoute un droit a l'utilisateur
|
||||||
* \param rid id du droit à ajouter
|
* \param rid id du droit à ajouter
|
||||||
|
* \param allmodule Ajouter tous les droits du module allmodule
|
||||||
|
* \param allperms Ajouter tous les droits du module allmodule, perms allperms
|
||||||
* \return int > 0 si ok, < 0 si erreur
|
* \return int > 0 si ok, < 0 si erreur
|
||||||
*/
|
*/
|
||||||
|
|
||||||
function addrights($rid)
|
function addrights($rid,$allmodule='',$allperms='')
|
||||||
{
|
{
|
||||||
if (strlen($rid) == 2)
|
$err=0;
|
||||||
{
|
$whereforadd='';
|
||||||
$topid = substr($rid,0,1);
|
|
||||||
$lowid = substr($rid,1,1);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (strlen($rid) == 3)
|
$this->db->begin();
|
||||||
{
|
|
||||||
$topid = substr($rid,0,2);
|
|
||||||
$lowid = substr($rid,2,1);
|
|
||||||
}
|
|
||||||
|
|
||||||
if ($lowid == 1)
|
if ($rid)
|
||||||
{
|
{
|
||||||
$sql = "DELETE FROM ".MAIN_DB_PREFIX."user_rights WHERE fk_user = $this->id AND fk_id=$rid";
|
// Si on a demandé ajout d'un droit en particulier, on récupère
|
||||||
$this->db->query($sql);
|
// les caractéristiques (module, perms et subperms) de ce droit.
|
||||||
$sql = "INSERT INTO ".MAIN_DB_PREFIX."user_rights (fk_user, fk_id) VALUES ($this->id, $rid)";
|
$sql = "SELECT module, perms, subperms";
|
||||||
if ($this->db->query($sql))
|
$sql.= " FROM ".MAIN_DB_PREFIX."rights_def";
|
||||||
{
|
$sql.= " WHERE ";
|
||||||
|
$sql.=" id = '".$rid."'";
|
||||||
|
|
||||||
|
$result=$this->db->query($sql);
|
||||||
|
if ($result) {
|
||||||
|
$obj = $this->db->fetch_object($result);
|
||||||
|
$module=$obj->module;
|
||||||
|
$perms=$obj->perms;
|
||||||
|
$subperms=$obj->subperms;
|
||||||
}
|
}
|
||||||
}
|
else {
|
||||||
|
$err++;
|
||||||
if ($lowid > 1)
|
dolibarr_print_error($this->db);
|
||||||
{
|
|
||||||
|
|
||||||
$sql = "DELETE FROM ".MAIN_DB_PREFIX."user_rights WHERE fk_user = $this->id AND fk_id=$rid";
|
|
||||||
$this->db->query($sql);
|
|
||||||
$sql = "INSERT INTO ".MAIN_DB_PREFIX."user_rights (fk_user, fk_id) VALUES ($this->id, $rid)";
|
|
||||||
if ($this->db->query($sql))
|
|
||||||
{
|
|
||||||
}
|
}
|
||||||
|
|
||||||
$nid = $topid . "1";
|
// Where pour la liste des droits à ajouter
|
||||||
$sql = "DELETE FROM ".MAIN_DB_PREFIX."user_rights WHERE fk_user = $this->id AND fk_id=$nid";
|
$whereforadd="id=".$rid;
|
||||||
$this->db->query($sql);
|
// Ajout des droits induits
|
||||||
$sql = "INSERT INTO ".MAIN_DB_PREFIX."user_rights (fk_user, fk_id) VALUES ($this->id, $nid)";
|
if ($subperms) $whereforadd.=" OR (module='$module' AND perms='$perms' AND subperms='lire')";
|
||||||
if ($this->db->query($sql))
|
if ($perms) $whereforadd.=" OR (module='$module' AND perms='lire' AND subperms IS NULL)";
|
||||||
{
|
|
||||||
|
|
||||||
|
// Pour compatibilité, si lowid = 0, on est en mode ajout de tout
|
||||||
|
// \todo A virer quand sera géré par l'appelant
|
||||||
|
if (substr($rid,-1,1) == 0) $whereforadd="module='$module'";
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
// Where pour la liste des droits à ajouter
|
||||||
|
if ($allmodule) $whereforadd="module='$allmodule'";
|
||||||
|
if ($allperms) $whereforadd=" AND perms='$allperms'";
|
||||||
|
}
|
||||||
|
|
||||||
|
// Ajout des droits de la liste whereforadd
|
||||||
|
if ($whereforadd)
|
||||||
|
{
|
||||||
|
//print "$module-$perms-$subperms";
|
||||||
|
$sql = "SELECT id";
|
||||||
|
$sql.= " FROM ".MAIN_DB_PREFIX."rights_def";
|
||||||
|
$sql.= " WHERE $whereforadd";
|
||||||
|
|
||||||
|
$result=$this->db->query($sql);
|
||||||
|
if ($result)
|
||||||
|
{
|
||||||
|
$num = $this->db->num_rows($result);
|
||||||
|
$i = 0;
|
||||||
|
while ($i < $num)
|
||||||
|
{
|
||||||
|
$obj = $this->db->fetch_object($result);
|
||||||
|
$nid = $obj->id;
|
||||||
|
|
||||||
|
$sql = "DELETE FROM ".MAIN_DB_PREFIX."user_rights WHERE fk_user = $this->id AND fk_id=$nid";
|
||||||
|
if (! $this->db->query($sql)) $err++;
|
||||||
|
$sql = "INSERT INTO ".MAIN_DB_PREFIX."user_rights (fk_user, fk_id) VALUES ($this->id, $nid)";
|
||||||
|
if (! $this->db->query($sql)) $err++;
|
||||||
|
|
||||||
|
$i++;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
|
$err++;
|
||||||
dolibarr_print_error($this->db);
|
dolibarr_print_error($this->db);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if ($lowid == 0)
|
if ($err) {
|
||||||
{
|
$this->db->rollback();
|
||||||
for ($i = 1 ; $i < 10 ; $i++)
|
return -$err;
|
||||||
{
|
}
|
||||||
$nid = $topid . "$i";
|
else {
|
||||||
|
$this->db->commit();
|
||||||
$sql = "DELETE FROM ".MAIN_DB_PREFIX."user_rights WHERE fk_user = $this->id AND fk_id=$nid";
|
return 1;
|
||||||
$this->db->query($sql);
|
|
||||||
$sql = "INSERT INTO ".MAIN_DB_PREFIX."user_rights (fk_user, fk_id) VALUES ($this->id, $nid)";
|
|
||||||
if ($this->db->query($sql))
|
|
||||||
{
|
|
||||||
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
dolibarr_print_error($this->db);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
return 1;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* \brief Retire un droit a l'utilisateur
|
* \brief Retire un droit a l'utilisateur
|
||||||
* \param rid id du droit à retirer
|
* \param rid id du droit à retirer
|
||||||
|
* \param allmodule Retirer tous les droits du module allmodule
|
||||||
|
* \param allperms Retirer tous les droits du module allmodule, perms allperms
|
||||||
* \return int > 0 si ok, < 0 si erreur
|
* \return int > 0 si ok, < 0 si erreur
|
||||||
*/
|
*/
|
||||||
|
|
||||||
function delrights($rid)
|
function delrights($rid,$allmodule='',$allperms='')
|
||||||
{
|
{
|
||||||
if (strlen($rid) == 2)
|
$err=0;
|
||||||
{
|
$wherefordel='';
|
||||||
$topid = substr($rid,0,1);
|
|
||||||
$lowid = substr($rid,1,1);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (strlen($rid) == 3)
|
$this->db->begin();
|
||||||
{
|
|
||||||
$topid = substr($rid,0,2);
|
|
||||||
$lowid = substr($rid,2,1);
|
|
||||||
}
|
|
||||||
|
|
||||||
if ($lowid > 1)
|
if ($rid)
|
||||||
{
|
{
|
||||||
$sql = "DELETE FROM ".MAIN_DB_PREFIX."user_rights WHERE fk_user = $this->id AND fk_id=$rid";
|
// Si on a demandé supression d'un droit en particulier, on récupère
|
||||||
if ($this->db->query($sql))
|
// les caractéristiques module, perms et subperms de ce droit.
|
||||||
{
|
$sql = "SELECT module, perms, subperms";
|
||||||
|
$sql.= " FROM ".MAIN_DB_PREFIX."rights_def";
|
||||||
|
$sql.= " WHERE ";
|
||||||
|
$sql.=" id = '".$rid."'";
|
||||||
|
|
||||||
|
$result=$this->db->query($sql);
|
||||||
|
if ($result) {
|
||||||
|
$obj = $this->db->fetch_object($result);
|
||||||
|
$module=$obj->module;
|
||||||
|
$perms=$obj->perms;
|
||||||
|
$subperms=$obj->subperms;
|
||||||
}
|
}
|
||||||
|
else {
|
||||||
|
$err++;
|
||||||
|
dolibarr_print_error($this->db);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Where pour la liste des droits à supprimer
|
||||||
|
$wherefordel="id=".$rid;
|
||||||
|
// Suppression des droits induits
|
||||||
|
if ($subperms=='lire') $wherefordel.=" OR (module='$module' AND perms='$perms' AND subperms IS NOT NULL)";
|
||||||
|
if ($perms=='lire') $wherefordel.=" OR (module='$module')";
|
||||||
|
|
||||||
|
// Pour compatibilité, si lowid = 0, on est en mode suppression de tout
|
||||||
|
// \todo A virer quand sera géré par l'appelant
|
||||||
|
if (substr($rid,-1,1) == 0) $wherefordel="module='$module'";
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
// Where pour la liste des droits à supprimer
|
||||||
|
if ($allmodule) $wherefordel="module='$allmodule'";
|
||||||
|
if ($allperms) $wherefordel=" AND perms='$allperms'";
|
||||||
}
|
}
|
||||||
|
|
||||||
if ($lowid == 1)
|
// Suppression des droits de la liste wherefordel
|
||||||
|
if ($wherefordel)
|
||||||
{
|
{
|
||||||
$fid = $topid . "0";
|
//print "$module-$perms-$subperms";
|
||||||
$lid = $topid . "9";
|
$sql = "SELECT id";
|
||||||
$sql = "DELETE FROM ".MAIN_DB_PREFIX."user_rights WHERE fk_user = $this->id AND fk_id >= $fid AND fk_id <= $lid";
|
$sql.= " FROM ".MAIN_DB_PREFIX."rights_def";
|
||||||
if ($this->db->query($sql))
|
$sql.= " WHERE $wherefordel";
|
||||||
{
|
|
||||||
|
|
||||||
|
$result=$this->db->query($sql);
|
||||||
|
if ($result)
|
||||||
|
{
|
||||||
|
$num = $this->db->num_rows($result);
|
||||||
|
$i = 0;
|
||||||
|
while ($i < $num)
|
||||||
|
{
|
||||||
|
$obj = $this->db->fetch_object($result);
|
||||||
|
$nid = $obj->id;
|
||||||
|
|
||||||
|
$sql = "DELETE FROM ".MAIN_DB_PREFIX."user_rights WHERE fk_user = $this->id AND fk_id=$nid";
|
||||||
|
if (! $this->db->query($sql)) $err++;
|
||||||
|
|
||||||
|
$i++;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
|
$err++;
|
||||||
dolibarr_print_error($this->db);
|
dolibarr_print_error($this->db);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if ($lowid == 0)
|
if ($err) {
|
||||||
{
|
$this->db->rollback();
|
||||||
for ($i = 1 ; $i < 10 ; $i++)
|
return -$err;
|
||||||
{
|
}
|
||||||
$nid = $topid . "$i";
|
else {
|
||||||
$sql = "DELETE FROM ".MAIN_DB_PREFIX."user_rights WHERE fk_user = $this->id AND fk_id=$nid";
|
$this->db->commit();
|
||||||
if ($this->db->query($sql))
|
return 1;
|
||||||
{
|
|
||||||
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
dolibarr_print_error($this->db);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
return 1;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user