Django

Code

Ticket #4258 (closed: fixed)

Opened 2 years ago

Last modified 1 year ago

[newforms-admin] Models aren't grouped by application on index page

Reported by: Matthias Pronk <django@masida.nl> Assigned to: adrian
Milestone: Component: django.contrib.admin
Version: newforms-admin Keywords: newforms-admin
Cc: jm.bugtracking@gmail.com Triage Stage: Accepted
Has patch: 1 Needs documentation: 0
Needs tests: 0 Patch needs improvement: 0

Description

The newforms-admin doesn't group the models per application.

Attached patch fixes this.

Attachments

admin-index.diff (1.5 kB) - added by Matthias Pronk <django@masida.nl> on 05/09/07 19:06:31.

Change History

05/09/07 19:06:31 changed by Matthias Pronk <django@masida.nl>

  • attachment admin-index.diff added.

05/16/07 20:51:17 changed by Simon G. <dev@simon.net.nz>

  • needs_better_patch changed.
  • stage changed from Unreviewed to Accepted.
  • version changed from other branch to newforms branch.
  • needs_tests changed.
  • needs_docs changed.

05/18/07 09:55:29 changed by anonymous

  • cc set to jm.bugtracking@gmail.com.

(follow-up: ↓ 4 ) 06/05/07 05:35:56 changed by mtredinnick

There's also an alternative patch in #4477. I haven't made any attempt to work out if one is slightly better than the other; both seem reasonable.

(in reply to: ↑ 3 ) 06/05/07 08:23:11 changed by django@masida.nl

Replying to mtredinnick:

There's also an alternative patch in #4477. I haven't made any attempt to work out if one is slightly better than the other; both seem reasonable.

I didn't know about 'django.utils.datastructures.SortedDict', which seems to be a slightly more elegant way of doing it. So I would suggest using the patch in #4477.

06/07/07 16:54:24 changed by adrian

  • version changed from newforms branch to newforms-admin.

06/07/07 17:12:58 changed by adrian

  • status changed from new to closed.
  • resolution set to fixed.

(In [5441]) newforms-admin: Fixed #4258 and #4477 -- Changed admin index page to group by applications, and alphabetized things. Thanks to Matthias Pronk and Honza Kral for the patches; I ended up using a hybrid of both

Add/Change #4258 ([newforms-admin] Models aren't grouped by application on index page)




Change Properties
Action