Bug #365

Duplicate code in LiveUser_Admin_Perm_DB_Common::removeUser()

Details

Submitted	2003-12-05 23:48 UTC
From	kipaten21 at hotmail dot com
Assigned	lsmith
Status	Closed
Package	LiveUser
PHP Version	Irrelevant
OS	ANY
Roadmaps	(Not assigned)

Comments

[2003-12-05 23:48 UTC] kipaten21 at hotmail dot com

Description:
------------
There is code to remove a user's rights upon removing a user that is duplicated twice in the LiveUser_Admin_Perm_DB_Common::removeUser() method.

function removeUser($authId)
{
<snip>
// Delete user's rights (Perm/DB)
$query = '
DELETE FROM
' . $this->prefix . 'userrights
WHERE
perm_user_id = ' . $permId;

$result = $this->dbc->query($query);

if (DB::isError($result)) {
return $result;
}
<snip>
// Delete right assignments
$query = 'DELETE FROM
' . $this->prefix . 'userrights
WHERE
perm_user_id = ' . $permId;

$result = $this->dbc->query($query);

if (DB::isError($result)) {
return $result;
};
<snip>
}