Bug #3813

LiveUser_Admin::adduser() allows duplicate handles

Details

Submitted	2005-03-14 22:44 UTC
From	benpjohnson at gmail dot com
Assigned	lsmith
Status	Closed
Package	LiveUser
PHP Version	5.0.3
OS	Windows XP
Roadmaps	(Not assigned)

Comments

[2005-03-14 22:44 UTC] benpjohnson at gmail dot com

Description:
------------
package version: 0.2.0

The adduser function allows multiple user handles to be created when the 'allowDuplicateHandles' setting is set to false.

Reproduce code:
---------------
$test = $admin->addUser('ben','test');
$test = $admin->addUser('ben','test');

if(PEAR::isError($test)) {
die($test->getMessage());
}

Expected result:
----------------
An error message 'Duplicate Handles are not allowed'

Actual result:
--------------
The user information is added to the database.

[2005-03-15 08:20 UTC] smith at backendmedia dot com

Hmm .. actually this feature is more of a performance optimization for the XML container. For the database containers it probably is more efficient to handle this by setting the handle field unique. Maybe we should even do so by default.

[2005-03-15 08:43 UTC] smith at backendmedia dot com

Ok, the handle field is now unique by default.

[2005-03-15 08:43 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.