lainwir3d/dolibarr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

In certain case, holiday_users won't be update (1 delete, 1 add for ex)

Browse Source 
Add a fonction for count the real number of user without CP
This commit is contained in:
Alexis Algoud 2014-02-10 09:39:26 +01:00
parent 408a028320
commit d8e5fd20d2
1 changed files with 22 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
htdocs/holiday/class/holiday.class.php
View File

@ -87,7 +87,7 @@ class Holiday extends CommonObject
$this->updateSoldeCP(); $this->updateSoldeCP();
// Vérifie le nombre d'utilisateur et mets à jour si besoin // Vérifie le nombre d'utilisateur et mets à jour si besoin
$this->verifNbUsers($this->countActiveUsers(),$this->getConfCP('nbUser')); $this->verifNbUsers($this->countActiveUsersWithoutCP(),$this->getConfCP('nbUser'));
return 1; return 1;
} }
@ -1229,7 +1229,23 @@ class Holiday extends CommonObject
$sql = "SELECT count(u.rowid) as compteur"; $sql = "SELECT count(u.rowid) as compteur";
$sql.= " FROM ".MAIN_DB_PREFIX."user as u"; $sql.= " FROM ".MAIN_DB_PREFIX."user as u";
$sql.= " WHERE statut > '0'"; $sql.= " WHERE u.statut > '0'";
$result = $this->db->query($sql);
$objet = $this->db->fetch_object($result);
return $objet->compteur;
}
/**
* Compte le nombre d'utilisateur actifs dans Dolibarr sans CP
*
* @return int retourne le nombre d'utilisateur
*/
function countActiveUsersWithoutCP() {
$sql = "SELECT count(u.rowid) as compteur";
$sql.= " FROM ".MAIN_DB_PREFIX."user as u LEFT OUTER JOIN ".MAIN_DB_PREFIX."holiday_users hu ON (hu.fk_user=u.rowid)";
$sql.= " WHERE u.statut > 0 AND hu.fk_user IS NULL ";
$result = $this->db->query($sql); $result = $this->db->query($sql);
$objet = $this->db->fetch_object($result); $objet = $this->db->fetch_object($result);
@ -1244,13 +1260,13 @@ class Holiday extends CommonObject
* @param int $userCP nombre d'utilisateur actifs dans le module congés payés * @param int $userCP nombre d'utilisateur actifs dans le module congés payés
* @return void * @return void
*/ */
function verifNbUsers($userDolibarr,$userCP) { function verifNbUsers($userDolibarrWithoutCP,$userCP) {
if (empty($userCP)) $userCP=0; if (empty($userCP)) $userCP=0;
dol_syslog(get_class($this).'::verifNbUsers userDolibarr='.$userDolibarr.' userCP='.$userCP); dol_syslog(get_class($this).'::verifNbUsers userDolibarr='.$userDolibarr.' userCP='.$userCP);
// Si il y a plus d'utilisateur Dolibarr que dans le module CP // On vérifie les users Dolibarr sans CP
if ($userDolibarr > $userCP) if ($userDolibarrWithoutCP > 0)
{ {
$this->updateConfCP('nbUser',$userDolibarr); $this->updateConfCP('nbUser',$userDolibarr);
@ -1311,7 +1327,7 @@ class Holiday extends CommonObject
$obj = $this->db->fetch_object($resql); $obj = $this->db->fetch_object($resql);
// On ajoute l'utilisateur // On supprime l'utilisateur
$this->deleteCPuser($obj->fk_user); $this->deleteCPuser($obj->fk_user);
$i++; $i++;