[wp-trac] [WordPress Trac] #22187: Add docblocks to wp-signup.php

WordPress Trac noreply at wordpress.org
Wed May 15 17:53:44 UTC 2013


#22187: Add docblocks to wp-signup.php
------------------------------+-----------------------------
 Reporter:  DrewAPicture      |       Owner:  SergeyBiryukov
     Type:  enhancement       |      Status:  closed
 Priority:  normal            |   Milestone:  3.6
Component:  Inline Docs       |     Version:  3.4
 Severity:  minor             |  Resolution:  fixed
 Keywords:  has-patch commit  |
------------------------------+-----------------------------

Comment (by SergeyBiryukov):

 As noted in [comment:ticket:24210:11], `$meta` value passed to
 `confirm_blog_signup()` and `confirm_another_blog_signup()` is an array,
 not a string (although it's unused in both functions):
 [source:tags/3.5.1/wp-signup.php#L209], [source:tags/3.5.1/wp-
 signup.php#L343].

 Also, there's an inconsistency in optional arguments. In
 `confirm_blog_signup()`, both `$user_name` and `$user_email` are optional.
 In `confirm_another_blog_signup()`, `$user_name` is required. Neither of
 them make sense as optional there, but that's out of scope for this
 ticket.

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


More information about the wp-trac mailing list