[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