#10635: list_editable: mention Meta class in ordering exception
---------------------------------------+------------------------------------
Reporter: mkb...@gmail.com | Owner: nobody
Status: reopened | Milestone:
Component: Documentation | Version: 1.1-beta-1
Resolution: | Keywords:
Stage: Unreviewed | Has_patch: 0
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 0 |
---------------------------------------+------------------------------------
Changes (by anonymous):
* status: closed => reopened
* resolution: invalid =>
* component: django.contrib.admin => Documentation
* summary: list_editable: logic error when checking for ordering
property => list_editable: mention Meta class
in ordering exception
Comment:
Reopening, and reassigning to docs with a changed summary.
I had an ordering property on the model object, not the
interior Meta class for the model object. This was what
caused that conditional to fail for me; then I moved the
property to the admin object and somehow thought it was
still failing eh voila, my "patch" worked.
I suggest that the error message explicitly
mention the Meta class; maybe something like this:
Please define ordering on either %s or %s's Meta class.
If I had seen Meta in the message, I would have wondered what
it was, searched the docs, and figured out that on the model
object (unlike the admin object), the ordering property should
be an attribute of the interior Meta class, not the model class
itself.
I think it's worth documenting to draw attention to the
different place this property lives on the admin and
model object.
--
Ticket URL: <http://code.djangoproject.com/ticket/10635#comment:2>
Django <http://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
