mirror of
https://git.code.sf.net/p/seeddms/code
synced 2025-05-21 17:11:38 +00:00
propperly returns false if operation fails in addGroup() and delGroup()
This commit is contained in:
parent
f12234b0bd
commit
3a47861a50
|
@ -128,7 +128,7 @@ class SeedDMS_Core_Group {
|
||||||
$queryStr = "INSERT INTO tblGroupMembers (groupID, userID, manager) VALUES (".$this->_id.", ".$user->getID(). ", " . ($asManager?"1":"0") ." )";
|
$queryStr = "INSERT INTO tblGroupMembers (groupID, userID, manager) VALUES (".$this->_id.", ".$user->getID(). ", " . ($asManager?"1":"0") ." )";
|
||||||
$res = $db->getResult($queryStr);
|
$res = $db->getResult($queryStr);
|
||||||
|
|
||||||
if ($res) return false;
|
if (!$res) return false;
|
||||||
|
|
||||||
unset($this->_users);
|
unset($this->_users);
|
||||||
return true;
|
return true;
|
||||||
|
@ -140,7 +140,7 @@ class SeedDMS_Core_Group {
|
||||||
$queryStr = "DELETE FROM tblGroupMembers WHERE groupID = ".$this->_id." AND userID = ".$user->getID();
|
$queryStr = "DELETE FROM tblGroupMembers WHERE groupID = ".$this->_id." AND userID = ".$user->getID();
|
||||||
$res = $db->getResult($queryStr);
|
$res = $db->getResult($queryStr);
|
||||||
|
|
||||||
if ($res) return false;
|
if (!$res) return false;
|
||||||
unset($this->_users);
|
unset($this->_users);
|
||||||
return true;
|
return true;
|
||||||
} /* }}} */
|
} /* }}} */
|
||||||
|
|
Loading…
Reference in New Issue
Block a user