[wp-trac] [WordPress Trac] #18039: Allow apostrophes in email addresses when accounts are added via Dashboard

WordPress Trac noreply at wordpress.org
Thu Dec 12 21:43:33 UTC 2013


#18039: Allow apostrophes in email addresses when accounts are added via Dashboard
-----------------------------------------------+---------------------------
 Reporter:  boonebgorges                       |       Owner:
     Type:  defect (bug)                       |      Status:  new
 Priority:  normal                             |   Milestone:  Future
Component:  Users                              |  Release
 Severity:  normal                             |     Version:  3.2
 Keywords:  has-patch needs-testing 3.9-early  |  Resolution:
-----------------------------------------------+---------------------------

Comment (by boonebgorges):

 Replying to [comment:12 nacin]:
 > An esc_* function should be used for output. (esc_url_raw() is an
 exception; it was done to as not to deviate much from esc_url()). We
 already have sanitize_email() — would that work here?

 Yup, good call. I think I'd introduced an esc_* function because
 esc_html() was already in use here. Bonus: `sanitize_email()` already
 supports apostrophes.

 18039.04.patch adds the stripslashes(), swaps the esc_html() for
 sanitize_email(), and adds a passing apostrophe unit test for
 sanitize_email().

--
Ticket URL: <http://core.trac.wordpress.org/ticket/18039#comment:13>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list