[wp-trac] [WordPress Trac] #41760: wp_list_comments callback params

WordPress Trac noreply at wordpress.org
Thu Aug 31 09:05:54 UTC 2017


#41760: wp_list_comments callback params
--------------------------+------------------------------
 Reporter:  milana_cap    |       Owner:
     Type:  enhancement   |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Comments      |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  dev-feedback  |     Focuses:  template
--------------------------+------------------------------

Comment (by ayeshrajans):

 It certainly makes sense to have semantic parameter order I totally agree.
 However, I'm afraid changing the callback arguments will introduce a lot
 of backwards incompatibility issues, and I doubt the implementors will be
 happy we changed the parameter order purely for the semantic purposes.

 https://github.com/search?l=PHP&q=wp_list_comments+callback%3D&type=Code&utf8=✓
 here is a fuzzy GitHub search for the function call with callback
 parameter.

 Both class methods you mentioned are protected methods, which means if we
 change the parameter order of those two methods, we would only need to
 change the current class and it's subclasses.

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


More information about the wp-trac mailing list