[wp-trac] [WordPress Trac] #53811: Rename `retrieve_widgets` to `sync_registered_widgets` (was: Rename `retrieve_widgets` to `recover_lost_widgets`)

WordPress Trac noreply at wordpress.org
Tue Aug 31 19:25:56 UTC 2021


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

 * keywords:  has-patch has-unit-tests 2nd-opinion => has-patch has-unit-
     tests commit
 * owner:  (none) => hellofromTonya
 * status:  new => reviewing
 * milestone:  Awaiting Review => 5.8.1


Comment:

 A new name has been selected: `sync_registered_widgets`. I agree this new
 name is more readable and understandable as to what it's purpose is.
 Removing the `2nd-opinion`.

 Moving this PR into the milestone to match its epic ticket #53816.

 Timothy has already reviewed [https://github.com/WordPress/wordpress-
 develop/pull/1524 PR 1524]. I'm doing a pre-commit review and then get it
 committed.

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


More information about the wp-trac mailing list