Opened 18 years ago

Closed 18 years ago

Last modified 18 years ago

#2587 closed defect (fixed)

[patch] unnecessary escaping of fields in admin_doc templates

Reported by: Gary Wilson <gary.wilson@…> Owned by: Adrian Holovaty
Component: contrib.admin Version: dev
Severity: normal Keywords:
Cc: gary.wilson@…, treborhudson@… Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The doc.model_detail view is writing html to verbose to provide links to related models, but this is then getting escaped in the template.

Attachments (1)

admin_doc-view_detail.diff​ (439 bytes ) - added by treborhudson@… 18 years ago.
Removes unnecessary escaping from admin_doc/view_detail.html template

Download all attachments as: .zip

Change History (6)

comment:1 by Adrian Holovaty, 18 years ago

Resolution: → fixed
Status: new → closed

(In [3642]) Fixed #2587 -- Removed unnecessary escaping from admin_doc/model_detail.html template. Thanks for reporting, Gary Wilson

comment:2 by treborhudson@…, 18 years ago

Resolution: fixed
Status: closed → reopened
Summary: unnecessary escaping of field.verbose in admin_doc/model_detail.html → unnecessary escaping of fields in admin_doc templates

The same bug applies to: view_detail.html -> summary|escape

This may be another bug, but many of the admin views in the "Home > Documentation > Views" list have a "_checklogin" appended which results in a 404 error. Each is listed as: "View function: django.contrib.admin.views.decorators._checklogin" and the URL goes to: "​http://localhost:8000/admin/doc/views/django.contrib.admin.views.decorators._checklogin/"

comment:3 by Gary Wilson <gary.wilson@…>, 18 years ago

Cc: gary.wilson@… added

by treborhudson@…, 18 years ago

Attachment: admin_doc-view_detail.diff​ added

Removes unnecessary escaping from admin_doc/view_detail.html template

comment:4 by treborhudson@…, 18 years ago

Cc: treborhudson@… added
Summary: unnecessary escaping of fields in admin_doc templates → [patch] unnecessary escaping of fields in admin_doc templates

Adding patch to summary...

comment:5 by Malcolm Tredinnick, 18 years ago

Resolution: → fixed
Status: reopened → closed

(In [3856]) Fixed #2587 -- Removed some escaping from the summary fields.

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