[wp-trac] [WordPress Trac] #48772: Missing post ID as parameter for get_comments_number_text()

WordPress Trac noreply at wordpress.org
Fri Jan 24 21:44:56 UTC 2020


#48772: Missing post ID as parameter for get_comments_number_text()
-------------------------------------+-----------------------
 Reporter:  Hinjiriyo                |       Owner:  donmhico
     Type:  enhancement              |      Status:  assigned
 Priority:  normal                   |   Milestone:  5.4
Component:  Comments                 |     Version:  4.0
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:
-------------------------------------+-----------------------
Changes (by garrett-eclipse):

 * keywords:  dev-feedback has-patch needs-testing => has-patch needs-
               testing


Comment:

 Thanks @Hinjiriyo
 I assume @donmhico preserved the deprecated parameter as it appears that
 is the standard convention for this type of thing. For instance wp_install
 has a 5th param that's deprecated but also then has a 6th and 7th. By not
 removing the deprecated param we preserve the call to
 `_deprecated_argument` which flags to users they're misusing the function.
 Adding an addition instead of replacing in this case is primarily for
 back-compat reason.
 * @donmhico correct me if I'm wrong

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


More information about the wp-trac mailing list