Opened 4 years ago

Closed 4 years ago

Last modified 3 years ago

#15593 closed Bug (fixed)

Docs: result of reverse() is already urlquoted

Reported by: guettli Owned by: nobody
Component: Documentation Version: 1.2
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The documentation of reverse() could be improved:

http://docs.djangoproject.com/en/dev/topics/http/urls/#reverse

Add a note, that the result is already urlquoted.

Patch is attched.

Attachments (1)

reverse_doc_add_urlquote_info.patch (1.0 KB) - added by guettli 4 years ago.

Download all attachments as: .zip

Change History (10)

Changed 4 years ago by guettli

comment:1 Changed 4 years ago by guettli

  • Cc hv@… added
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 4 years ago by guettli

  • Has patch set

comment:3 Changed 4 years ago by gabrielhurley

  • Triage Stage changed from Unreviewed to Accepted

comment:4 Changed 4 years ago by lasko

  • Triage Stage changed from Accepted to Ready for checkin

comment:5 Changed 4 years ago by lukeplant

  • Type set to Bug

comment:6 Changed 4 years ago by lukeplant

  • Severity set to Normal

comment:7 Changed 4 years ago by gabrielhurley

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

In [16054]:

Fixed #15593 -- Added a note that the output of reverse is urlquoted. Thanks to guettli for the report and draft patch.

comment:8 Changed 4 years ago by gabrielhurley

In [16055]:

[1.3.X] Fixed #15593 -- Added a note that the output of reverse is urlquoted. Thanks to guettli for the report and draft patch.

Backport of [16054] from trunk.

comment:9 Changed 3 years ago by guettli

  • Cc hv@… removed
  • Easy pickings unset
  • UI/UX unset
Note: See TracTickets for help on using tickets.
Back to Top