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

Fix #17726 : holliday with advanced perms works

Browse Source
This commit is contained in:
lmarcouiller 2021-06-01 16:34:39 +02:00
parent d18c1ea0c0
commit af509742e6
1 changed files with 17 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
htdocs/holiday/card.php
View File

@ -163,19 +163,23 @@ if (empty($reshook)) {
$description = trim(GETPOST('description', 'restricthtml')); $description = trim(GETPOST('description', 'restricthtml'));
// Check that leave is for a user inside the hierarchy or advanced permission for all is set // Check that leave is for a user inside the hierarchy or advanced permission for all is set
if ((empty($conf->global->MAIN_USE_ADVANCED_PERMS) && empty($user->rights->holiday->write)) if (empty($conf->global->MAIN_USE_ADVANCED_PERMS)) {
|| (!empty($conf->global->MAIN_USE_ADVANCED_PERMS) && $user->id == $fuserid && empty($user->rights->holiday->write)) if (empty($user->rights->holiday->write)) {
|| (!empty($conf->global->MAIN_USE_ADVANCED_PERMS) && $user->id != $fuserid && empty($user->rights->holiday->writeall_advance)) $error++;
) { setEventMessages($langs->trans("NotEnoughPermissions"), null, 'errors');
$error++; } elseif (!in_array($fuserid, $childids)) {
setEventMessages($langs->trans("NotEnoughPermissions"), null, 'errors'); $error++;
setEventMessages($langs->trans("UserNotInHierachy"), null, 'errors');
$action = 'create';
}
} else { } else {
if (empty($conf->global->MAIN_USE_ADVANCED_PERMS) || empty($user->rights->holiday->writeall_advance)) { if (empty($user->rights->holiday->write) && empty($user->rights->holiday->writeall_advance)) {
if (!in_array($fuserid, $childids)) { $error++;
$error++; setEventMessages($langs->trans("NotEnoughPermissions"), null, 'errors');
setEventMessages($langs->trans("UserNotInHierachy"), null, 'errors'); } elseif (empty($user->rights->holiday->writeall_advance) && !in_array($fuserid, $childids)) {
$action = 'create'; $error++;
} setEventMessages($langs->trans("UserNotInHierachy"), null, 'errors');
$action = 'create';
} }
} }
@ -1136,7 +1140,7 @@ if ((empty($id) && empty($ref)) || $action == 'create' || $action == 'add') {
} }
// On vérifie si l'utilisateur à le droit de lire cette demande // On vérifie si l'utilisateur à le droit de lire cette demande
if ($cancreate) { if ($canread) {
$head = holiday_prepare_head($object); $head = holiday_prepare_head($object);
if (($action == 'edit' && $object->statut == Holiday::STATUS_DRAFT) || ($action == 'editvalidator')) { if (($action == 'edit' && $object->statut == Holiday::STATUS_DRAFT) || ($action == 'editvalidator')) {