oracle inspectdb CharFields missing "max_length"

[Carl Karsten <carl@…>]

forgive the flimsy report - I don't have an oracle db to confirm this. I hacked the oracle code up to work with ceODBC, and inspectdb generated
td_fil = models.CharField(blank=True)

which causes syncdb:
"td_fil": CharFields require a "max_length" attribute.

because

"CharField has an extra required argument, max_length, the maximum length (in characters) of the field. The max_length is enforced at the database level and in Django’s validation." ​http://www.djangoproject.com/documentation/model-api/#charfield

I don't think my hacks caused that. but it would be really nice if someone could confirm it.

[Carl Karsten <carl@…>]

This may be a bug/issue with ceODBC's cursor.description(). or not. here is some relevant code:

core/management/commands/inspectdb.py

for i, row in enumerate(introspection_module.get_table_description(cursor, table_name)):

...

# Add max_length for all CharFields.
if field_type == 'CharField' and row[3]:

extra_paramsmax_length = row[3]

...

field_desc = '%s = models.%s' % (att_name, field_type)
if extra_params:

if not field_desc.endswith('('):

field_desc += ', '

field_desc += ', '.join(% (k, v) for k, v in extra_params.items())

field_desc += ')'
if comment_notes:

field_desc += ' # ' + ' '.join(comment_notes)

yield ' %s' % field_desc

introspection.py

def get_table_description(cursor, table_name):

"Returns a description of the table, with the DB-API cursor.description interface."
sql="SELECT * FROM %s " % quote_name(table_name)
cursor.execute(sql)
return cursor.description

[Carl Karsten <carl@…>]

pretty sure it is a bug with ceODBC's cursor.description()

[Carl Karsten <carl@…>]

100% sure:

odbc:	('cd_fil', 'STRING', 16, 16, 0, 0, 1)
ceODBC:	('cd_fil', <type 'ceODBC.StringVar'>, None, None, 16, 0, True),