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

Merge pull request #18337 from atm-maxime/fix_holidays_users

Browse Source 
Fix : we only want employee users for holidays
This commit is contained in:
Laurent Destailleur 2021-08-17 18:37:38 +02:00 committed by GitHub
commit a19f8cd87d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -1630,6 +1630,7 @@ class Holiday extends CommonObject
$sql .= " WHERE u.entity IN (".getEntity('user').")";
}
$sql .= " AND u.statut > 0";
$sql .= " AND u.employee = 1"; // We only want employee users for holidays
if ($filters) $sql .= $filters;
$resql = $this->db->query($sql);
@ -1720,6 +1721,7 @@ class Holiday extends CommonObject
}
$sql .= " AND u.statut > 0";
$sql .= " AND u.employee = 1"; // We only want employee users for holidays
if ($filters) $sql .= $filters;
$resql = $this->db->query($sql);