[wp-trac] [WordPress Trac] #18749: Remove "the hackiest hack that ever did hack" from add_user()
WordPress Trac
wp-trac at lists.automattic.com
Thu Jan 5 20:57:33 UTC 2012
#18749: Remove "the hackiest hack that ever did hack" from add_user()
-----------------------------------+------------------
Reporter: duck_ | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 3.4
Component: Users | Version:
Severity: normal | Resolution:
Keywords: has-patch 2nd-opinion |
-----------------------------------+------------------
Comment (by nacin):
I agree.
Grain of salt, but I prefer `! $user_id = edit_user()` to `! ( $user_id =
edit_user() )`, the latter looks like a situation where you're trying to
invert what's inside. The former looks like a deliberate assignment and a
check against the value of the assignment.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/18749#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list