Opened 9 years ago

Closed 9 years ago

#6356 closed (fixed)

EasyInstance url() not marked safe

Reported by: tangerine Owned by: nobody
Component: contrib.databrowse Version: master
Severity: Keywords: databrowse
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

While using the databrowse app I noticed that the url method of EasyInstance is not marking the returned string as safe. I've attached a patch which marks this safe as well as a change to the __repr__ method of EasyModel to match the use of smart_str() as on other other classes.

Attachments (1)

datastructures.diff (993 bytes) - added by tangerine 9 years ago.
Patch

Download all attachments as: .zip

Change History (3)

Changed 9 years ago by tangerine

Attachment: datastructures.diff added

Patch

comment:1 Changed 9 years ago by Simon Greenhill <dev@…>

Has patch: set
Triage Stage: UnreviewedReady for checkin

comment:2 Changed 9 years ago by Malcolm Tredinnick

Resolution: fixed
Status: newclosed

(In [7061]) Fixed #6356 -- Added a mark_safe() call that was missed in a url() method. Thanks, tangerine.

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