PEAR is archived and read-only

This mirror preserves historical PEAR package releases and metadata so existing references remain available.

Home » Authentication » LiveUser_Admin » Bug #3975

LiveUser_Admin_Perm_Complex _updateImpliedStatus() bug

Details

Submitted2005-03-28 09:10 UTC
Fromivanchoo at singnet dot com dot sg
StatusNo Feedback
PackageLiveUser_Admin
PHP Version4.3.10
OSOS 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.