Re: Patch on #6776

2008-04-29 Thread Brian Rosner
On Apr 29, 2008, at 8:07 AM, Michael wrote: > Thanks for the work, Brian; > > I assume the patch that goes to /dev/null should be going to contrib/ > admin/loading.py. Don't really get a chance to work with git all too > often. > > One thing that I am noticing real quick, while applying this

Re: Patch on #6776

2008-04-29 Thread Michael
After a few more sips of coffee, I thought I should also mention that this does fix all the problems that I had with strange hidden AlreadyRegistered errors. Specifically with mod_python doing these multiple imports. This patch fixes #6718 at least with my initial tests (it is a hard ticket to dupl

Re: Patch on #6776

2008-04-29 Thread Michael
Thanks for the work, Brian; I assume the patch that goes to /dev/null should be going to contrib/admin/loading.py. Don't really get a chance to work with git all too often. One thing that I am noticing real quick, while applying this patch. Any model that is being imported, whether it is in the i

Patch on #6776

2008-04-28 Thread Brian Rosner
I have attached a patch to #6776 [1] that solves the AlreadyRegistered [2] problem according to what seemed to be the consensus. Which basically is to throw the AlreadyRegistered exception on registration of a different ModelAdmin that may have been previously registered with a model. I am