Home » Authentication » LiveUser » Bug #4322
Allow for longer constant definition names
Details
| Request #4322 | Allow for longer constant definition names |
|---|---|
| Submitted | 2005-05-10 20:41 UTC |
| From | scragz at hotmail dot com |
| Status | Closed |
| Package | LiveUser |
| PHP Version | 4.3.10 |
| OS | irrelevant |
| Roadmaps | (Not assigned) |
Comments
[2005-05-10 20:41 UTC] scragz at hotmail dot com
Description:
------------
32 characters is not enough room to make constants in liveuser_rights->right_define_name and possibly some of the other tables. Just look at LiveUser itself; some of its own constants wouldn't fit in there (LIVEUSER_ERROR_FAILED_INSTANTIATION). At least 64 would be nice.
[2005-05-10 20:43 UTC] smith at backendmedia dot com
yeah this is a limitation in the install.php since we currently hardcode the length of the text fields to 32. you can easily modify this, but we may have to come up with some better solution.
[2005-05-10 20:52 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.
I have added a new optional parameter where you can specify in key-value pairs lengths. The key is a field name and the value is the integer length. If no length is set for the given field it defaults to 32.