[prev in list] [next in list] [prev in thread] [next in thread] 

List:       bricolage-devel
Subject:    Re: [Bricolage-Devel] Re: [PATCH] remove groups from Category.pm
From:       David Wheeler <david () wheeler ! net>
Date:       2002-07-16 23:49:36
[Download RAW message or body]

On 7/16/02 6:56 PM, "Sam Tregar" <sam@tregar.com> claimed:

> And done.  I also commented out this region of Bric::Biz::Asset
> get_grp_ids():

Shit! I should have known that there was another to be another reason for
the groups...

These lines of code are how we determine all the groups that assets are in,
including in categories. It's what allows one to set permissions on stories
as grouped by category. But, looking at that code, I'm not sure what I was
thinking. A better choice would have been to use the asset group object
that's part of every category. I think we can still do that, but writing an
upgrade script is going to be a bitch. I'll have to take a look at that
tomorrow.

Thanks for committing this, Sam! And good work, Japhy!

David

-- 
David Wheeler                                     AIM: dwTheory
david@wheeler.net                                 ICQ: 15726394
http://david.wheeler.net/                      Yahoo!: dew7e
                                               Jabber: Theory@jabber.org




-------------------------------------------------------
This sf.net email is sponsored by: Jabber - The world's fastest growing 
real-time communications platform! Don't just IM. Build it in! 
http://www.jabber.com/osdn/xim
_______________________________________________
Bricolage-Devel mailing list
Bricolage-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bricolage-devel
[prev in list] [next in list] [prev in thread] [next in thread]