[wp-trac] [WordPress Trac] #27006: New user "Send this password to the new user by email" toggle option value not remembered

[WordPress Trac]

#27006: New user "Send this password to the new user by email" toggle option value
not remembered
--------------------------------------+------------------------------
 Reporter:  setsailmedia              |       Owner:  chriseverson
     Type:  defect (bug)              |      Status:  assigned
 Priority:  normal                    |   Milestone:  Awaiting Review
Component:  Users                     |     Version:  3.8.1
 Severity:  normal                    |  Resolution:
 Keywords:  good-first-bug has-patch  |     Focuses:  administration
--------------------------------------+------------------------------
Changes (by nacin):

 * owner:   => chriseverson
 * status:  new => assigned


Comment:

 Replying to [comment:3 chriseverson]:
 > Just submitted a patch. Next step will be to look into the root of the
 issue in the {{{checked()}}}  function.

 Thanks for the patch, chriseverson!

 The issue is that checked() is explicit — it wants true === true, not just
 'on' == true. (Or more accurately, it converts both to a string and then
 does a comparison. 'on' does not equal '1'.) checked() isn't the problem
 here.

 Rather, our ugly code that sets this $new_user_send_password variable
 simply sets it to the value of `$_POST['send_password']` if it's set, or
 false if it's not. Checkboxes without values in the HTML end up receiving
 'on' as a value.

 This fix is a bit cryptic; simply casting the variable to a boolean would
 probably solve it in a slightly better way. So, `checked( (bool)
 $new_user_send_password )`. Make sense?

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