[wp-trac] [WordPress Trac] #57716: Remove server-side redirection in site editor

WordPress Trac noreply at wordpress.org
Mon Feb 27 19:23:12 UTC 2023


#57716: Remove server-side redirection in site editor
-----------------------------------------+-----------------------------
 Reporter:  ntsekouras                   |       Owner:  hellofromTonya
     Type:  task (blessed)               |      Status:  reopened
 Priority:  normal                       |   Milestone:  6.2
Component:  Editor                       |     Version:
 Severity:  normal                       |  Resolution:
 Keywords:  gutenberg-merge needs-patch  |     Focuses:
-----------------------------------------+-----------------------------
Changes (by hellofromTonya):

 * keywords:  gutenberg-merge has-patch commit => gutenberg-merge needs-
               patch
 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 Replying to [comment:9 johnbillion]:
 > @hellofromTonya @ntsekouras Can and should the
 `_resolve_home_block_template()` function be removed or deprecated? It's
 no longer used since [55338].

 Good catch @johnbillion! The code in this ticket was added in [53093]
 including `_resolve_home_block_template()`.

 Can it be removed or deprecated?
 Yes, I think so.

 * No longer used in Core or Gutenberg.
 * No instances in wordpress.org's
 [https://wpdirectory.net/search/01GTA3NZAHHCQY5HPQ1R3PJG3X plugin
 directory].
 * No instances in wordpress.org's
 [https://wpdirectory.net/search/01GTA3VEH6QTD4A72CCQZVBYCP theme
 directory].
 * It's marked private (internal Core usage).

 Here's its doc page
 https://developer.wordpress.org/reference/functions/_resolve_home_block_template/.

 Deprecate or remove?
 I'd like to remove it as it's not used (see above), likely low BC risk
 given its specific use case, and not wired as a callback. But "likely low
 BC risk" concerns me. Leaning towards deprecating.

 What do you think @johnbillion?

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


More information about the wp-trac mailing list