On Wednesday 26 September 2012 20:26:11 Andre Terra wrote:
> On Mon, Sep 24, 2012 at 9:41 PM, Shai Berger <[email protected]> wrote:
> > While preparing the patch, I ran into the inspectdb regression test
> > suite; this suite includes, among other things, a model with a database
> > column named
> > 'prc(%) x', which breaks test database creation on Oracle on current
> > trunk. I
> > intend to include a fix for this (in a separate commit, of course) in the
> > first
> > part as well (doubling percent signs in quote_name()).
>
> For the record, I believe the relevant ticket and discussion are located
> at:
>
> https://code.djangoproject.com/ticket/18843
>
No and maybe: I think it's the "other side" of the same issue. The ticket is
about problems in inspection; the problem I ran into is in creating a table,
where the column name is given in the model. My code does nothing to fix the
inspetion (and indeed, there are failures in the test suite with it); but
without some fix to the problem I solved, the test suite cannot be run at all.
It may be the problem Anssi refers to in his comment there, I'm not sure.
I can open a ticket for this if core devs like, but it feels like unnecessary
procedure when it is included with a fix that has already been discussed.
Thanks,
Shai.
--
You received this message because you are subscribed to the Google Groups
"Django developers" 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/django-developers?hl=en.
