[wp-trac] [WordPress Trac] #15725: "Skip Confirmation Email" checkbox value is ignored

[WordPress Trac]

#15725: "Skip Confirmation Email" checkbox value is ignored
--------------------------+-------------------------------------------------
 Reporter:  ocean90       |       Owner:           
     Type:  defect (bug)  |      Status:  new      
 Priority:  normal        |   Milestone:  3.1      
Component:  Users         |     Version:  3.1      
 Severity:  blocker       |    Keywords:  has-patch
--------------------------+-------------------------------------------------

Comment(by duck_):

 Replying to [comment:7 duck_]:
 > Replying to [comment:6 wpdavis]:
 > > The isset( $_POST[ 'adduser' ] ) check is needed to make sure the
 right form is being submitted. In MS, there are two forms on that page
 that use the same names for fields — without that check you spawn a ton of
 errors because WordPress tries to submit two forms at once.
 > >
 > > I'll take a look into the problem.
 >
 > In that case I believe we would want `$_POST['createuser']` on line 111
 and I suppose `$_POST['adduser']` above on line 65 (not looked into beyond
 checking the line numbers).

 P.S. I cannot reproduce "a ton of errors" or the double form submit before
 or after removing `isset($_POST['adduser'])`, shouldn't the checks on
 `$_REQUEST['action']` be doing this anyway (lines 34 and 78)?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/15725#comment:9>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software