addGrpRecepient() and addIndRecepient() return the log id not 0

This commit is contained in:
Uwe Steinmann 2023-10-15 19:24:17 +02:00
parent ee568bb916
commit 8b4c01e339

View File

@ -124,7 +124,19 @@ foreach ($pIndRev as $p) {
$uml = $accessIndex["i"][$p]->getEmail(); $uml = $accessIndex["i"][$p]->getEmail();
switch ($res) { switch ($res) {
case 0: case -1:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("internal_error"));
break;
case -2:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("access_denied"));
break;
case -3:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("recipient_already_assigned"));
break;
case -4:
// email error
break;
default:
// Send an email notification to the new recipient. // Send an email notification to the new recipient.
if($settings->_enableNotificationAppRev) { if($settings->_enableNotificationAppRev) {
if ($notifier) { if ($notifier) {
@ -145,18 +157,6 @@ foreach ($pIndRev as $p) {
} }
} }
break; break;
case -1:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("internal_error"));
break;
case -2:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("access_denied"));
break;
case -3:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("recipient_already_assigned"));
break;
case -4:
// email error
break;
} }
} }
else { else {
@ -231,7 +231,19 @@ foreach ($pGrpRev as $p) {
$res = $content->addGrpRecipient($accessIndex["g"][$p], $user); $res = $content->addGrpRecipient($accessIndex["g"][$p], $user);
$gnm = $accessIndex["g"][$p]->getName(); $gnm = $accessIndex["g"][$p]->getName();
switch ($res) { switch ($res) {
case 0: case -1:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("internal_error"));
break;
case -2:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("access_denied"));
break;
case -3:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("recipient_already_assigned"));
break;
case -4:
// email error
break;
default:
// Send an email notification to the new recipient. // Send an email notification to the new recipient.
if($settings->_enableNotificationAppRev) { if($settings->_enableNotificationAppRev) {
if ($notifier) { if ($notifier) {
@ -252,18 +264,6 @@ foreach ($pGrpRev as $p) {
} }
} }
break; break;
case -1:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("internal_error"));
break;
case -2:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("access_denied"));
break;
case -3:
UI::exitError(getMLText("document_title", array("documentname" => $document->getName())),getMLText("recipient_already_assigned"));
break;
case -4:
// email error
break;
} }
} }
else { else {