Bug #1560

LiveUser\Admin\Admin.php, function:updateUser, Line 472

Details

Submitted	2004-06-04 21:24 UTC
From	waxdipped at yahoo dot co dot uk
Assigned	lsmith
Status	Closed
Package	LiveUser
PHP Version	Irrelevant
OS	Linux (SuSE 9.0)
Roadmaps	(Not assigned)

Comments

[2004-06-04 21:24 UTC] waxdipped at yahoo dot co dot uk

Description:
------------
auth_user_id being set to nothing in table liveuser_perm_users after calling updateUser

LiveUser\Admin\Admin.php, function:updateUser, Line 472 from current CVS version:

$authId has not been initialised or set.

$perm = $this->perm->updateUser($permId, $authId, $this->authContainerName, $type);

should almost certainly be

$perm = $this->perm->updateUser($permId, $authData['auth_user_id'], $this->authContainerName, $type);

Reproduce code:
---------------
just call updateUser

Expected result:
----------------
select * from liveuser_perm_users;
+--------------+--------------+-----------+---------------------+
| perm_user_id | auth_user_id | perm_type | auth_container_name |
+--------------+--------------+-----------+---------------------+
| 17 | 40 | 1 | 0 |
+--------------+--------------+-----------+---------------------+

Actual result:
--------------
select * from liveuser_perm_users;
+--------------+--------------+-----------+---------------------+
| perm_user_id | auth_user_id | perm_type | auth_container_name |
+--------------+--------------+-----------+---------------------+
| 17 | | 1 | 0 |
+--------------+--------------+-----------+---------------------+

[2004-06-04 21:35 UTC] smith at backendmedia dot com

I have a fix pending for this issue.

[2004-06-08 20:42 UTC] smith at backendmedia dot com

This bug has been fixed in CVS.

In case this was a documentation problem, the fix will show up at the
end of next Sunday (CET) on pear.php.net.

In case this was a pear.php.net website problem, the change will show
up on the website in short time.

Thank you for the report, and for helping us make PEAR better.