[geeklog-devel] GSoC 2010 is on
tuxcanfly at gmail.com
Sun Apr 4 09:44:21 EDT 2010
So, I added a few user groups from the database by setting grp_owner in
gl_groups table. Then I tried submitting a story as admin and was surprised
to see that the user groups turn up in groups drop down. Looking closer, I
found that SEC_getGroupDropdown doesn't really return only system groups.
It calls SEC_getUserGroups without any filter and returns *all* groups.
Shouldn't SEC_getUserGroups be called with the "grp_owner=0" filter to get
only system groups?
On Sat, Apr 3, 2010 at 10:37 PM, Joe Mucchiello <joe at throwingdice.com>wrote:
> At 12:03 PM 4/3/2010, Jakh Daven wrote:
>> Thanks for the detailed reply. However, the same argument is valid for
>> public groups isn't it? We have a unique constraint on (grp_owner,
>> grp_name). So nothing stops both you and me from creating two public groups
>> with the same name, "The New Jersey PHP User Group", for example. Socnet
>> still has to differentiate between the two groups when showing them to a
>> member of both the groups.
> That's an issue for socnet. There is no core code to create a user-owned
> Other than that, I have upgraded to 188.8.131.52 and have been using it. IMHO
>> we can have an option for the admin to set the group owner when
>> creating/editing a group. Of course Socnet will set the user who creates a
>> group as the group owner, but the admin should be able to change the owner
>> by editing the group.
> I left that function out on purpose. socnet might have other features
> attached to a group that are very user specific. Likewise, if a group is
> private, even an Admin should not be able to change who owns that private
> group because it violates the privacy of the original creator.
> Will keep you posted as I delve deeper into the changes :)
> No problem.
> Joe Mucchiello
> Throwing Dice Games
> No virus found in this outgoing message
> Checked by PC Tools AntiVirus (184.108.40.206 - 10.004.176).
> geeklog-devel mailing list
> geeklog-devel at lists.geeklog.net
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the geeklog-devel