Home » Authentication » LiveUser_Admin » Bug #3975
LiveUser_Admin_Perm_Complex _updateImpliedStatus() bug
Details
| Submitted | 2005-03-28 09:10 UTC |
|---|---|
| From | ivanchoo at singnet dot com dot sg |
| Status | No Feedback |
| Package | LiveUser_Admin |
| PHP Version | 4.3.10 |
| OS | OS X Darwin |
| Roadmaps | (Not assigned) |
Comments
[2005-03-28 09:10 UTC] ivanchoo at singnet dot com dot sg
Description:
------------
_updateImpliedStatus() sets boolean field has_implied to
1 or 0 where the LiveUser package recognizes Y or N.
This happens for the is_active column in the
liveuser_users and liveuser_groups table
Reproduce code:
---------------
Fix: change the line 563 _updateImpliedStatus()
//$data = array('has_implied' => (bool)$count);
to
$data = array('has_implied' => (bool)$count ? 'Y' : 'N'); // BUGFIX:: boolean field to Y for yes
[2005-03-28 20:40 UTC] smith at backendmedia dot com
I think what you are encountering is a BC break we failed to mention. I presume you are using the DB container? We previously converted everything to Y/N in the DB containers. We no longer do so. Instead its upto the storage container to handle these kinds of things and we have decided to better model the storage containers according to the backend. This means that in DB we simply pass the value to quoteSmart() and do no conversion on fetch. Should work fine for most databases.
So let us know if my assumptions were right. Sorry for the inconvinience and the lack of information about BC breaks in this period of mass refactoring.
[2005-04-18 20:41 UTC] smith at backendmedia dot com
No feedback was provided. The bug is being suspended because
we assume that you are no longer experiencing the problem.
If this is not the case and you are able to provide the
information that was requested earlier, please do so and
change the status of the bug back to "Open". Thank you.