Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#6069 closed (wontfix)

Fix auto-escaping Password-Change-Link in Admin

Reported by: anonymous Owned by: anonymous
Component: contrib.admin Version: dev
Severity: Keywords: admin, escaping, sprintdec01
Cc: thomas-django@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Auto-escaping broke Password-Change link in Admin.

Attachments (2)

6777-escaping.patch (1.6 KB ) - added by Thomas Kerpe 17 years ago.
6777-escaping.2.patch (1.6 KB ) - added by Thomas Kerpe 17 years ago.
New Patch with suggestion from comment:3

Download all attachments as: .zip

Change History (15)

by Thomas Kerpe, 17 years ago

Attachment: 6777-escaping.patch added

comment:1 by Thomas Kerpe, 17 years ago

Cc: thomas-django@… added
Has patch: set
Owner: changed from nobody to anonymous
Status: newassigned

comment:2 by anonymous, 17 years ago

_(mark_safe('string')) looks wrong to me - I think it should be mark_safe(_('string'))

comment:3 by anonymous, 17 years ago

Is it a dupe from #6061? But [6781] Does not fix this issue.

comment:4 by anonymous, 17 years ago

Owner: changed from anonymous to Thomas Kerpe
Status: assignednew

by Thomas Kerpe, 17 years ago

Attachment: 6777-escaping.2.patch added

New Patch with suggestion from comment:3

comment:5 by anonymous, 17 years ago

Summary: Fix auto-esacaping Password-Change-Link in AdminFix auto-escaping Password-Change-Link in Admin
Triage Stage: UnreviewedAccepted

comment:6 by Thomas Kerpe, 17 years ago

Status: newassigned

comment:7 by Thomas Kerpe, 17 years ago

Keywords: sprint1dec added

comment:8 by Thomas Kerpe, 17 years ago

Keywords: sprintdec1 added; sprint1dec removed

comment:9 by anonymous, 17 years ago

Owner: changed from Thomas Kerpe to anonymous
Status: assignednew

comment:10 by Honza Král, 17 years ago

Keywords: sprintdec01 added; sprintdec1 removed

changed sprint keyword to match the others

comment:11 by Malcolm Tredinnick, 17 years ago

Resolution: wontfix
Status: newclosed

This isn't necessary. Help text should never be auto-escaped. Is this still an issue after [6781]?

If this is still a bug on trunk, there should be some kind of template fix marking the help_text output as "safe", not a patch required to the model text.

comment:12 by Karen Tracey <kmtracey@…>, 17 years ago

I can't replicate the problem with trunk r6805. Help text for the password form shows up fine, with a working link.

comment:13 by anonymous, 17 years ago

It is indeed fixed by r6781. The Report and Fix was against r6777.

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