From 682b5dafe780f6596a5990b7a67265e3ff568fe0 Mon Sep 17 00:00:00 2001 From: Uwe Steinmann Date: Thu, 27 Oct 2016 17:05:12 +0200 Subject: [PATCH] do not check for enableOwnerRevApp anymore a user who has been once added as a reviewer/approver may have become the owner of the document later. Such a user may still be allowed to review/approve. enableOwnerRevApp will only affect the list of users when the reviewers/approvers are set. --- views/bootstrap/class.ViewDocument.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/views/bootstrap/class.ViewDocument.php b/views/bootstrap/class.ViewDocument.php index 4619d9273..b15d18265 100644 --- a/views/bootstrap/class.ViewDocument.php +++ b/views/bootstrap/class.ViewDocument.php @@ -577,7 +577,7 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Bootstrap_Style { } else { $reqName = htmlspecialchars($required->getFullName()." (".$required->getLogin().")"); - if($required->getId() == $user->getId() && ($user->getId() != $owner->getId() || $enableownerrevapp == 1)) + if($required->getId() == $user->getId()/* && ($user->getId() != $owner->getId() || $enableownerrevapp == 1)*/) $is_reviewer = true; } break; @@ -588,7 +588,7 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Bootstrap_Style { } else { $reqName = "".htmlspecialchars($required->getName()).""; - if($required->isMember($user) && ($user->getId() != $owner->getId() || $enableownerrevapp == 1)) + if($required->isMember($user)/* && ($user->getId() != $owner->getId() || $enableownerrevapp == 1)*/) $is_reviewer = true; } break; @@ -657,7 +657,7 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Bootstrap_Style { } else { $reqName = "".htmlspecialchars($required->getName()).""; - if($required->isMember($user) && ($user->getId() != $owner->getId() || $enableownerrevapp == 1)) + if($required->isMember($user)/* && ($user->getId() != $owner->getId() || $enableownerrevapp == 1)*/) $is_approver = true; } break;