[wp-trac] [WordPress Trac] #44672: REST-API: invalid email on lowercase/uppercase change

WordPress Trac noreply at wordpress.org
Thu Jan 17 18:23:25 UTC 2019


#44672: REST-API: invalid email on lowercase/uppercase change
--------------------------------------+-----------------------
 Reporter:  fuchsws                   |       Owner:  desrosj
     Type:  defect (bug)              |      Status:  assigned
 Priority:  normal                    |   Milestone:  5.1
Component:  REST API                  |     Version:  4.9.7
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+-----------------------

Comment (by kadamwhite):

 Looks pretty good, although the assignment in this conditional

 {{{#!php
 if ( ( $owner_id = email_exists( $request['email'] ) ) && $owner_id !==
 $id ) {
 }}}


 does not thrill me. I'd like to propose adding a comment to make it clear
 we're borrowing verbatim, or find a way to clarify the logic flow.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/44672#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list