Bug #401

LiveUser_Admin_Perm_Container_DB_Complex::assignSubgroup()

Details

Submitted	2003-12-10 23:04 UTC
From	kipaten21 at hotmail dot com
Assigned	arnaud
Status	Closed
Package	LiveUser
PHP Version	Irrelevant
OS	ANY
Roadmaps	(Not assigned)

Comments

[2003-12-10 23:04 UTC] kipaten21 at hotmail dot com

Description:
------------
I'm assuming that a group can only have one parent group. Otherwise it would be hard to keep track of the overall group heirarchy.

If a group can only have one parent group, code should be added to LiveUser_Admin_Perm_Container_DB_Complex::assignSubgroup() to check whether or not there is already an entry in LIVEUSER_GROUP_SUBGROUPS for the child group. If there is, than an update should be done to change the parent group_id or a delete should be done before a new insert is done. This will ensure that no groups have more than one parent.

To futher protect against multiple parent groups, a unique constraint should be added to the LIVEUSER_GROUP_SUBGROUPS for (group_id, subgroup_id).