Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#39136 closed enhancement (duplicate)

Remove email length check from rest api

Reported by: milindmore22's profile milindmore22 Owned by:
Milestone: Priority: normal
Severity: normal Version: 4.8
Component: REST API Keywords: has-patch
Focuses: rest-api Cc:

Description

In reference to below tickets -

https://core.trac.wordpress.org/ticket/38506#comment:12

https://core.trac.wordpress.org/ticket/39133

we should also remove email length check from REST API as well.

Attachments (1)

39136.diff (551 bytes) - added by milindmore22 9 years ago.
Removed length check from is_email condition

Download all attachments as: .zip

Change History (4)

@milindmore22
9 years ago

Removed length check from is_email condition

#1 @milindmore22
9 years ago

  • Keywords has-patch added

above patch will remove email length check from rest-api as we are checking length in is_email() function already

#2 @swissspidy
9 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Hey there,

Thanks for the patch! Would you mind uploading it to #39133? We cannot simply remove the length check without adjusting is_email() as well, as is_email() currently only checks for a minimum length of 3 characters.

#3 @swissspidy
9 years ago

  • Type changed from defect (bug) to enhancement
Note: See TracTickets for help on using tickets.