Opened 8 years ago

Closed 8 years ago

#5251 closed (fixed)

[newforms-admin] "Show in Web" links not working

Reported by: Florian Apolloner Owned by: apollo13
Component: contrib.admin Version: newforms-admin
Severity: Keywords: newforms-admin reverse sprintsept14
Cc: django@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

The "Show in Web" Links on the change pages of the django admin are not working as admin/r/10/1 (just an example) returns "app u'r', model 10 (or something like it ;)) not found"...
This patch moves the checking to contrib/admin/sites to prevent this special url from getting displayed as model_page which obviously fails...

Attachments (2)

show_in_web-6000.diff (1.5 KB) - added by Florian Apolloner 8 years ago.
show_in_web.diff (677 bytes) - added by apollo13 8 years ago.
Shorter and better patch :)

Download all attachments as: .zip

Change History (9)

Changed 8 years ago by Florian Apolloner

comment:1 Changed 8 years ago by Simon G. <dev@…>

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Summary changed from [newforms-admin][patch] "Show in Web" links not working to [newforms-admin] "Show in Web" links not working
  • Triage Stage changed from Unreviewed to Ready for checkin

comment:2 Changed 8 years ago by jacob

  • Patch needs improvement set
  • Triage Stage changed from Ready for checkin to Accepted

The special-casing in this patch isn't right. I think it can be fixed just be reordering the admin urlconf instead of this weird dispatch thing.

Changed 8 years ago by apollo13

Shorter and better patch :)

comment:3 Changed 8 years ago by apollo13

  • Patch needs improvement unset

comment:4 Changed 8 years ago by apollo13

I don't think it can be moved, as admin.site.root tries to call model_page, but my new patch is better :)

comment:5 Changed 8 years ago by apollo13

  • Owner changed from nobody to apollo13
  • Status changed from new to assigned

comment:6 Changed 8 years ago by anonymous

  • Keywords sprintsept14 added; /admin/r/ removed

comment:7 Changed 8 years ago by adrian

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [6207]) newforms-admin: Fixed #5251 -- 'Show in Web' links now work properly again. Thanks, Florian Apolloner

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