On Apr 18, 4:44 pm, "Jorge Vargas" <[EMAIL PROTECTED]> wrote:
> > I opened the ticket #1360 and I added the proper patch
> > see it at http://trac.turbogears.org/ticket/1360
>
> Could someone using SA check that out it seems a bit hackish to me :)

I've tested this change and it doesn't break the "normal" case.  I
mean "normal" case as when not using column_prefix with the mappers.

I've never actually used column_prefix, but if I add a column_prefix
to my assign_mapper() call and test this change it actually fails.
colname is still being passed through as the table name (ie: not
prefixed).  Without the patch is actually works.  It is possible that
I am not using the column_prefix correctly.

I'd suggest showing an example that breaks without the patch so we can
more clearly see what case it is fixing.

Cheers,
Chris


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"TurboGears" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/turbogears?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to