Opened 15 years ago

Closed 14 years ago

Last modified 12 years ago

#9764 closed (fixed)

IDN (Internationalized Domain Names) support for EmailField and URLField

Reported by: Ulrich Petri Owned by: Jannis Leidel
Component: Forms Version: 1.0
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Currently EmailField and URLField don't accept values that contain IDNs.

The submitted patch adds this capability.

Attachments (2)

idn_support.diff (3.6 KB) - added by Ulrich Petri 15 years ago.
EmailField, URLField IDN support
idn_support_02.diff (7.6 KB) - added by Ulrich Petri 14 years ago.
Updated IDN patch to trunk (r12454); Added docs

Download all attachments as: .zip

Change History (14)

Changed 15 years ago by Ulrich Petri

Attachment: idn_support.diff added

EmailField, URLField IDN support

comment:1 Changed 15 years ago by Jacob

Triage Stage: UnreviewedDesign decision needed

comment:2 Changed 15 years ago by Emil Stenström

I just had to add verify_exists = False on a URLField because users complained that they couldn't use IDN fields. This patch seem very well architected, and solves a common annoyance (for me).

+1

comment:3 Changed 14 years ago by Jannis Leidel

milestone: 1.2
Owner: changed from nobody to Jannis Leidel
Status: newassigned
Triage Stage: Design decision neededAccepted

comment:4 Changed 14 years ago by Jannis Leidel

Needs documentation: set

comment:5 Changed 14 years ago by rlaager@…

What documentation is needed here? Just the fact that IDN is supported as of Django version 1.2?

comment:6 Changed 14 years ago by Jannis Leidel

Yes, please add a versionadded: 1.2 section.

Changed 14 years ago by Ulrich Petri

Attachment: idn_support_02.diff added

Updated IDN patch to trunk (r12454); Added docs

comment:7 Changed 14 years ago by Ulrich Petri

Needs documentation: unset

I've updated the patch for current trunk (r12454) and added a short note on the EmailFiled/URLField documentation.

comment:8 Changed 14 years ago by Jannis Leidel

Resolution: fixed
Status: assignedclosed

(In [12474]) Fixed #9764 - Updated EmailField and URLField to support IDN (Internationalized Domain Names). Thanks, UloPe.

comment:9 Changed 14 years ago by bjunix

Are there plans to backport this patch to 1.1.X ?

comment:10 in reply to:  9 ; Changed 14 years ago by Karen Tracey

Replying to bjunix:

Are there plans to backport this patch to 1.1.X ?

No. The doc contains version changed flags indicating the new behavior in 1.2, that implies it won't be added to 1.1.X.

comment:11 in reply to:  10 Changed 14 years ago by bjunix

Replying to kmtracey:

No. The doc contains version changed flags indicating the new behavior in 1.2, that implies it won't be added to 1.1.X.

Ok, I see. Thanks for your answer. Hoped this one would fall under bugfixes.

comment:12 Changed 12 years ago by Jacob

milestone: 1.2

Milestone 1.2 deleted

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