diff --git a/CHANGELOG b/CHANGELOG index 43c66a3c6..c8c127d04 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,3 +1,11 @@ +-------------------------------------------------------------------------------- + Changes in version 4.1.3 +-------------------------------------------------------------------------------- +- fixed setting document link (no more PHP fatal error) +- fixed typo in variable name which cause a PHP warning when sending the + notification mail after updating a folder comment +- fixed code to determine required approver and reviewer + -------------------------------------------------------------------------------- Changes in version 4.1.2 -------------------------------------------------------------------------------- diff --git a/inc/inc.Utils.php b/inc/inc.Utils.php index 2492d4174..f5340f320 100644 --- a/inc/inc.Utils.php +++ b/inc/inc.Utils.php @@ -326,7 +326,7 @@ function _add_log_line($msg="") { /* {{{ */ function filterDocumentLinks($user, $links) { /* {{{ */ $tmp = array(); foreach ($links as $link) - if ($link->isPublic() || ($link->getUser()->getID == $user->getID()) || $user->isAdmin()) + if ($link->isPublic() || ($link->getUser()->getID() == $user->getID()) || $user->isAdmin()) array_push($tmp, $link); return $tmp; } /* }}} */ diff --git a/views/blue/class.ViewDocument.php b/views/blue/class.ViewDocument.php index a157f64f3..649edf329 100644 --- a/views/blue/class.ViewDocument.php +++ b/views/blue/class.ViewDocument.php @@ -299,7 +299,7 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Blue_Style { case 0: // Approver is an individual. $required = $dms->getUser($a["required"]); if (!is_object($required)) { - $reqName = getMLText("unknown_user")." '".$r["required"]."'"; + $reqName = getMLText("unknown_user")." '".$a["required"]."'"; } else { $reqName = htmlspecialchars($required->getFullName()); @@ -310,7 +310,7 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Blue_Style { case 1: // Approver is a group. $required = $dms->getGroup($a["required"]); if (!is_object($required)) { - $reqName = getMLText("unknown_group")." '".$r["required"]."'"; + $reqName = getMLText("unknown_group")." '".$a["required"]."'"; } else { $reqName = "".htmlspecialchars($required->getName()).""; diff --git a/views/bootstrap/class.ViewDocument.php b/views/bootstrap/class.ViewDocument.php index b9faf82a9..33ad75eaa 100644 --- a/views/bootstrap/class.ViewDocument.php +++ b/views/bootstrap/class.ViewDocument.php @@ -493,7 +493,7 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Bootstrap_Style { case 0: // Approver is an individual. $required = $dms->getUser($a["required"]); if (!is_object($required)) { - $reqName = getMLText("unknown_user")." '".$r["required"]."'"; + $reqName = getMLText("unknown_user")." '".$a["required"]."'"; } else { $reqName = htmlspecialchars($required->getFullName()); @@ -504,7 +504,7 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Bootstrap_Style { case 1: // Approver is a group. $required = $dms->getGroup($a["required"]); if (!is_object($required)) { - $reqName = getMLText("unknown_group")." '".$r["required"]."'"; + $reqName = getMLText("unknown_group")." '".$a["required"]."'"; } else { $reqName = "".htmlspecialchars($required->getName())."";