Home » Authentication » LiveUser » Bug #1957

[2004-07-22 18:17 UTC] michael dot caplan at lechateau dot ca

Description:
------------
LiveUser_Admin_Perm_Container_MDB2_Complex::getRights() at line 630 incorrectly defines the DB column types for the query, which results in:

- rights_id being forced to a string (its and int)
- area_id being forced to a bool (its an int)
- application_id being forced to a bool (its an int)
- defined_name being forced to a bool (its a string)

While I am unclear if this is the desired result, I assume it is not, as this breaks outputRightsConstants(array('naming' => 3)) resulting in constants being defined without the right constant name (eg: XXX_XXX_)

I'm thinking this woudl be the correct column type definition:

$types = array('integer', 'integer', 'integer', 'text', 'text', 'text');

Thanks,

Michael

[2004-07-23 08:58 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.

[2004-10-20 11:48 UTC] michael dot caplan at lechateau dot ca

LiveUser_Admin_Perm_Container_MDB2_Complex::getRights() incorrectly sets the name column as a boolean, when it shoudl be text. Line 437 should be:

$types = array('integer', 'integer', 'integer', 'text', 'boolean', 'boolean', 'text', 'text', 'text');