Reverted back to previous object map storage, however added check for modified groups in RefreshDB and CheckDB this should now pass the current Unit Tests
This commit is contained in:
parent
9b0d4b4d84
commit
ce4850d2de
|
@ -83,7 +83,7 @@ abstract class GroupMembershipResolver
|
|||
|
||||
Benchmark::measure('Rechecking all objects');
|
||||
$this->recheckAllObjects($this->getAppliedGroups());
|
||||
if (empty($this->objects)) {
|
||||
if (empty($this->objects) && empty($this->groups)) {
|
||||
Benchmark::measure('Nothing to check, got no qualified object');
|
||||
return $this;
|
||||
}
|
||||
|
@ -106,7 +106,7 @@ abstract class GroupMembershipResolver
|
|||
if ($force || ! $this->isDeferred()) {
|
||||
$this->checkDb();
|
||||
|
||||
if (empty($this->objects)) {
|
||||
if (empty($this->objects) && empty($this->groups)) {
|
||||
Benchmark::measure('Nothing to check, got no qualified object');
|
||||
|
||||
return $this;
|
||||
|
@ -508,11 +508,11 @@ abstract class GroupMembershipResolver
|
|||
$staticGroups = [];
|
||||
|
||||
if ($this->objects === null) {
|
||||
$this->objects = $this->fetchAllObjects();
|
||||
$objects = $this->fetchAllObjects();
|
||||
} else {
|
||||
$objects = & $this->objects;
|
||||
}
|
||||
|
||||
$objects = & $this->objects;
|
||||
|
||||
$times = array();
|
||||
|
||||
foreach ($objects as $object) {
|
||||
|
|
Loading…
Reference in New Issue