[geeklog-devel] Draft of schema for GL2

Tony Bibbs tony at tonybibbs.com
Thu Dec 16 14:35:07 EST 2004

Vincent Furia wrote:

>Yes.  If you look in the geeklog code for calls to SEC_inGroup() it is
>really only used for the  Root group.  Most of the other instance
>where it is used it is used against a group id variable.  The
>remainder I would argue are redundant if we better respected
>privileges in 1.3.x.  I'd recommend a "isRoot" function, but I think
>"inGroup" is really not needed except when actually administering
>group membership.
Ok, I say we yank it...adding that column in if deemed appropriate isn't 
too hard.

>Better names for those columns wouldn't hurt, I just couldn't come up
>with anything better when I was implementing it in 1.3.x.  I looked
>around at other algorithms when I was first looking into improving the
>performance of comments and didn't find anything better.  If you do
>find something let me know.  This was, by far, the best I found.  The
>biggest downside is that I had to lock the table during inserts, but
>that is really OBE since we can use transactions.
Was this an original implementation or was it one you borrowed from 
someplace?  WTF does slashcode use...you'd think they'd have to have 
this issue nailed down by now.


More information about the geeklog-devel mailing list