[wp-trac] [WordPress Trac] #47398: Remove wp_favicon_request

WordPress Trac noreply at wordpress.org
Tue May 28 14:57:46 UTC 2019


#47398: Remove wp_favicon_request
----------------------------+------------------------------
 Reporter:  jonoaldersonwp  |       Owner:  SergeyBiryukov
     Type:  enhancement     |      Status:  reviewing
 Priority:  normal          |   Milestone:  Awaiting Review
Component:  Bootstrap/Load  |     Version:
 Severity:  normal          |  Resolution:
 Keywords:  has-patch seo   |     Focuses:
----------------------------+------------------------------

Comment (by birgire):

 In general (whether that's the outcome here or not, that's to be decided),
 I would think it would be better to deprecate a function, instead of
 removing the function's definition directly as in [attachment:"wp-favicon-
 request.diff"].

 Specifically if the function is not marked as private, i.e. not prefixed
 with underscore {{{_}}}.

 So one could look into the file {{{/wp-includes/deprecated.php}}} for
 storing the deprecated function's definition, use the
 {{{_deprecated_function()}}} function and the {{{@deprecated}}} tag for
 the inline docs.

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


More information about the wp-trac mailing list