gnh1201/seeddms-code
1
0
Fork 0
mirror of https://git.code.sf.net/p/seeddms/code synced 2024-10-04 23:22:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

setDefaultAccess() cleanly removes als notifiers without read access

Browse Source 
This failed because the first call of removeNotify() has invalidated
the list of notifiers
This commit is contained in:
Uwe Steinmann 2015-06-19 12:47:30 +02:00
parent a0d0790a34
commit 8ee6d74613
1 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
SeedDMS_Core/Core/inc.ClassFolder.php
View File

@ -333,6 +333,14 @@ class SeedDMS_Core_Folder extends SeedDMS_Core_Object {
return $this->_defaultAccess;
} /* }}} */
/**
* Set default access mode
*
* This method sets the default access mode and also removes all notifiers which
* will not have read access anymore.
*
* @param integer $mode access mode
*/
function setDefaultAccess($mode) { /* {{{ */
$db = $this->_dms->getDB();
@ -346,12 +354,18 @@ class SeedDMS_Core_Folder extends SeedDMS_Core_Object {
// remove their subscription.
if (empty($this->_notifyList))
$this->getNotifyList();
foreach ($this->_notifyList["users"] as $u) {
/* Make a copy of both notifier lists because removeNotify will empty
* $this->_notifyList and the second foreach will not work anymore.
*/
$nusers = $this->_notifyList["users"];
$ngroups = $this->_notifyList["groups"];
foreach ($nusers as $u) {
if ($this->getAccessMode($u) < M_READ) {
$this->removeNotify($u->getID(), true);
}
}
foreach ($this->_notifyList["groups"] as $g) {
foreach ($ngroups as $g) {
if ($this->getGroupAccessMode($g) < M_READ) {
$this->removeNotify($g->getID(), false);
}