WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#25046 closed enhancement (fixed)

is_email_address_unsafe should be case insensitve

Reported by: jkudish's profile jkudish Owned by: nacin's profile nacin
Milestone: 3.6.1 Priority: normal
Severity: normal Version:
Component: Multisite Keywords: has-patch
Focuses: Cc:

Description

is_email_address_unsafe should be case insensitive

For example: if I add superhacker.com to the list of banned domains, SuperHacker.com should not be allowed either.

Attached patch fixes the problem.

Attachments (2)

25046.diff​ (607 bytes) - added by jkudish 12 years ago.
25046-tests.diff​ (1.3 KB) - added by jkudish 12 years ago.
Adding some tests to go along with the fix

Download all attachments as: .zip

Change History (8)

@jkudish
12 years ago

#1 @helen
12 years ago

  • Keywords needs-unit-tests added

#2 @nacin
12 years ago

  • Milestone changed from Awaiting Review to 3.7

Well that's kind of pathetic.

#3 @jkudish
12 years ago

  • Keywords needs-unit-tests removed

@jkudish
12 years ago

Adding some tests to go along with the fix

#4 @nacin
12 years ago

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

In 25197:

Case insensitivity for is_email_address_unsafe().

props jkudish.
fixes #25046.

#5 @nacin
12 years ago

In 25198:

Case sensitivity for is_email_address_unsafe().

Merges [25197] to the 3.6 branch.

props jkudish.
fixes #25046.

#6 @nacin
12 years ago

  • Milestone changed from 3.7 to 3.6.1
Note: See TracTickets for help on using tickets.