Opened 15 years ago

Closed 13 years ago

Last modified 13 years ago

#9189 closed Bug (fixed)

urlize filter doesn't do .us domains without http and www

Reported by: Greg Allard Owned by: Alexander Herrmann
Component: Documentation Version: 1.0
Severity: Normal Keywords: urlize, filter
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

Try tinyid.us as an example. This passes the url field test, but doesn't get urlized.

Attachments (2)

9189.diff (499 bytes ) - added by Eric Holscher 15 years ago.
9189-r16351.diff (573 bytes ) - added by Alexander Herrmann 13 years ago.
path for current source

Download all attachments as: .zip

Change History (9)

comment:1 by Eric Holscher, 15 years ago

Triage Stage: UnreviewedAccepted

The docstring for urlize says that it only works on .com .org and .net domains. I think this is meant to urlize the simple cases of common urls.

I'm accepting this ticket, because it either needs to be documented that it only works for those three TLD's, or it needs to be fixed so that it works for any "URL looking string". That has problems (such as marking non-urls as urls), so I'm assuming fixing the docs is the correct result.

Current docs: http://docs.djangoproject.com/en/dev/ref/templates/builtins/?from=olddocs#urlize

by Eric Holscher, 15 years ago

Attachment: 9189.diff added

comment:2 by Luke Plant, 13 years ago

Severity: Normal
Type: Bug

comment:3 by Alexander Herrmann, 13 years ago

Easy pickings: unset
Owner: changed from nobody to Alexander Herrmann
Status: newassigned
UI/UX: unset

by Alexander Herrmann, 13 years ago

Attachment: 9189-r16351.diff added

path for current source

comment:4 by Alexander Herrmann, 13 years ago

Has patch: set

i updated the patch to the current source

comment:5 by Jannis Leidel, 13 years ago

Component: Template systemDocumentation
Triage Stage: AcceptedReady for checkin

comment:6 by Jannis Leidel, 13 years ago

Resolution: fixed
Status: assignedclosed

In [16372]:

Fixed #9189 -- Added not about urlize only covering .com, .net, and .org domains. Thanks, dArignac.

comment:7 by Chris Beaven, 13 years ago

In [16470]:

More accurate documentation for how the urlize template works. Refs #9189.

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