Skip to content
Snippets Groups Projects
Commit 9066917a authored by Tim Steiner's avatar Tim Steiner
Browse files

Bug fix in results of Auth_GroupModel::findByUser

parent aa03bde1
No related branches found
No related tags found
No related merge requests found
......@@ -6,7 +6,7 @@ class Auth_GroupModel extends Unl_Model
{
$db = Zend_Registry::get('db');
$select = new Zend_Db_Select($db);
// get userId as a string or array from the user object(s)
if (Unl_Util::isArray($user)) {
$userId = array();
......@@ -16,7 +16,7 @@ class Auth_GroupModel extends Unl_Model
} else {
$userId = $user->getId();
}
// find the group(s) that the user(s) is a "primary" member of
$select = new Zend_Db_Select($db);
$select->from(array('u' => 'creqUsers'), array('userId'));
......@@ -26,15 +26,22 @@ class Auth_GroupModel extends Unl_Model
} else {
$select->where('u.userId = ?', $userId);
}
$records = $select->query()->fetchAll();
$objects = $objects = new Unl_Model_Collection('Auth_GroupModel');
$objects = array();
if (Unl_Util::isArray($user)) {
foreach ($user as $aUser) {
$objects[$aUser->getId()] = new Unl_Model_Collection(__CLASS__);
}
} else {
$objects[$user->getId()] = new Unl_Model_Collection(__CLASS__);
}
foreach ($records as $record) {
$recordUserId = $record['userId'];
unset($record['userId']);
$objects[$recordUserId][] = Unl_Model_Registry::getInstance()->getOrAdd(new self($record));
}
// find any groups that the user(s) is a "non-primary" member of
$select = new Zend_Db_Select($db);
$select->from(array('u' => 'creqUsers'), array('userId'));
......@@ -45,14 +52,14 @@ class Auth_GroupModel extends Unl_Model
} else {
$select->where('u.userId = ?', $userId);
}
$records = $select->query()->fetchAll();
foreach ($records as $record) {
$recordUserId = $record['userId'];
unset($record['userId']);
$objects[$recordUserId][] = Unl_Model_Registry::getInstance()->getOrAdd(new self($record));
}
// if we were passed an array of users, return an array for each user
if (Unl_Util::isArray($userId)) {
return $objects;
......@@ -61,4 +68,9 @@ class Auth_GroupModel extends Unl_Model
return $objects[$userId];
}
}
public function getId()
{
return $this->_data['groupId'];
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment