Gabriel Jacobo wrote:
I´ve sent them to the list a few days ago. It is a modification to
ActiveMapper (mostly to the process_relationships function and a
bit of change in ActiveMapperMeta) to allow for multiple
relationships between classes as we discussed earlier in this list.
It mostly separates tables creation from relationships processing,
while shuffling these in the proper order so there are no conflicts
when they are created. I can re-send my original message if it has
not arrived. I´ve tested this modification in a real application
case with 20 tables with multiple references to each other and it
seems to be working ok...
Gabriel,
I haven't had a chance to review the patch yet, but from a cursory
glance things look fine, if not a little hard to follow. I might
clean things up slightly, but if it passes the unit tests, and works
for you, I don't have any huge issues applying a patch and cleaning
it up later. Better to have something that works than something that
doesn't ;)
Please create a patch instead of a wholesale swapout of the code, and
send it to me. I will test it out tomorrow, and if things look good,
I will apply the patch to SVN as soon as I get a chance.
Thanks for putting in the time that I don't have right now to fix
this nagging bug, I greatly appreciate it!
--
Jonathan LaCour
http://cleverdevil.org
-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid0944&bid$1720&dat1642
_______________________________________________
Sqlalchemy-users mailing list
Sqlalchemy-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sqlalchemy-users
