[wp-trac] [WordPress Trac] #53811: Rename `retrieve_widgets` to `sync_registered_widgets`

WordPress Trac noreply at wordpress.org
Wed Sep 1 16:19:05 UTC 2021


#53811: Rename `retrieve_widgets` to `sync_registered_widgets`
-------------------------------------------------+-------------------------
 Reporter:  zieladam                             |       Owner:
                                                 |  hellofromTonya
     Type:  defect (bug)                         |      Status:  reopened
 Priority:  normal                               |   Milestone:  5.8.2
Component:  Widgets                              |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests commit      |     Focuses:
  fixed-major                                    |
-------------------------------------------------+-------------------------

Comment (by hellofromTonya):

 For more context,
 [https://wordpress.slack.com/archives/C02RQBWTW/p1630505312078700 here is
 the link] to discussion thread in Make WordPress slack #core channel.

 Decisions needed:

 - Soft vs hard deprecation?
    - For core: a hard deprecation helps to further resolve the function
 purpose confusion (which is the purpose of this ticket)
    - For extenders who do WP cross-version automated testing: a soft
 deprecation helps them avoid code churn; however, each hard deprecation
 introduces this same churn.

 - Include in 5.8.x or wait for 5.9?
    - Including in point release helps to avoid continued code churn and
 delays with ongoing development of and issue fixing with the block
 widgets. As noted in the description, the original name caused these
 problems

 - If soft deprecation, should the original function stay in its file or be
 moved to deprecated file?

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


More information about the wp-trac mailing list