Opened 18 years ago

Closed 18 years ago

#1732 closed enhancement (fixed)

MR branch #2799, user stupidity in models.py leads to mysteriously missing tables list in admin view

Reported by: ross.lazarus@… Owned by: Adrian Holovaty
Component: contrib.admin Version: magic-removal
Severity: minor Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

This is definitely a user error, but the failure is silent which was confusing for a few minutes...
While developing a complex model.py, I introduced a bogus
casePCPid = models.CharacterField('PCP',maxlength=20,blank=True)
instead of
casePCPid = models.CharField('PCP',maxlength=20,blank=True)

Stupidity - all my own fault.

Unfortunately, 0 errors are reported when manage.py runserver checks the models, but none of the application tables showed up in the admin list. Took a while to track down and might be a trap for young players (like me)... I'll take a poke at it but this is probably a fix someone familiar with the model checking code will need to make. This is with the latest MR branch svn - I was using an older one but updated - same problem - models validate ok but the admin is missing ALL the tables.

Change History (2)

comment:1 by Adrian Holovaty, 18 years ago

This is happening because django.db.models.loading.get_apps silently passes on ImportError, so that people can have apps without models in them. But the CharacterField is causing an ImportError, which is getting silently eaten by get_apps(). Hmmmm...

comment:2 by Adrian Holovaty, 18 years ago

Resolution: fixed
Status: newclosed

(In [2995]) Fixed #1732 -- AttributeErrors in models are no longer ignored by the model validator.

Note: See TracTickets for help on using tickets.
Back to Top