On 1/12/06, Arthur <[EMAIL PROTECTED]> wrote:
> I've had a closer look at ticket #1165 where the wrong SQL column name
> is used for PositiveIntegerFields. The problem is that the data type
> for foreignKeys gets derived from its referenced field with the
> __dict__ of the referenced field. This is correct for fields like
> CharFields, where i.e. maxlength  is needed but wrong for types where
> a column name is needed. At present this is only for
> PositiveIntegerFields and PositiveSmallIntegerField the case.

Hey Arthur,

Thanks for reporting this bug and submitting a patch! I just committed
it -- I think it's as elegant of a solution as we can get at this
point. If any future fields have a constraint, let's cross that bridge
when we come to it.

Adrian

--
Adrian Holovaty
holovaty.com | djangoproject.com | chicagocrime.org

Reply via email to