On Tue, Oct 14, 2008 at 10:39:28PM -0400, Justin Sherrill wrote: > Originally I was toying with the notion of simply deleting all child > channels (assuming you have permissions to) of a parent when you called > channel.software.delete() on the parent. Since we don't allow this in > the web ui I decided to simply throw an error telling you to delete the > child channels first. (Maybe we could add a 'recursiveDelete()' call > to do this, as I could see it being useful). > > So in my initial work you actually needed all the child channel objects > in order to delete them. But now since I am just needing to know if > there are any child channels or not (in order to throw an error), I > don't really need to load them all. I guess I just left it there since > I thought it might be useful in some other place and I would have had to > change the query and method from my previous work. :} > > Anyways, I can switch it to just query for emptiness if you feel that > strongly, although it's not loading that much data on average (probably > less than 5 channels).
That's fair enough. You can leave it alone for now. I'd look at the query it generates to make sure it's not too heinous, if it's not too bad, leave it alone. Thanks for the thorough explanation. jesus -- jesus m. rodriguez | [EMAIL PROTECTED] sr. software engineer | irc: zeus rhn satellite & spacewalk | 919.754.4413 (w) rhce # 805008586930012 | 919.623.0080 (c) +-------------------------------------------+ | "Those who cannot learn from history | | are doomed to repeat it." | | -- George Santayana | +-------------------------------------------+ _______________________________________________ Spacewalk-devel mailing list [email protected] https://www.redhat.com/mailman/listinfo/spacewalk-devel
