[wp-trac] [WordPress Trac] #53811: Rename `retrieve_widgets` to `sync_registered_widgets`
WordPress Trac
noreply at wordpress.org
Wed Sep 1 19:29:18 UTC 2021
#53811: Rename `retrieve_widgets` to `sync_registered_widgets`
-------------------------------------------------+-------------------------
Reporter: zieladam | Owner:
| hellofromTonya
Type: defect (bug) | Status: reopened
Priority: normal | Milestone: 5.9
Component: Widgets | Version:
Severity: normal | Resolution:
Keywords: has-patch has-unit-tests commit | Focuses:
fixed-major |
-------------------------------------------------+-------------------------
Changes (by azaozz):
* milestone: 5.8.2 => 5.9
Comment:
As far as I know only (major) bug fixes and regression fixes belong in dot
releases. There are two reasons:
1. Dot releases are usually not tested as thoroughly as major releases.
The risks of introducing a bug and deploying it across millions of
installs should be minimized as much as possible.
2. The autoupdate failures rate slightly increases when the update is
larger/has more files. That increase is very small as a percentage but
given the size of WP it translates into thousands or tens of thousands of
installs.
In that terms moving to 5.9 for further consideration.
> The name is misleading and it bit us in
https://github.com/WordPress/gutenberg/issues/33335#issuecomment-879903958
and #53657.
Perhaps a better way to fix this would be to extend and enhance the inline
docs. Would be great to add some inline comments too. True, the new name
is better than the old, but at the same time that function does a lot of
things and the name will never be able to describe it particularly well.
Also renaming and deprecating a function has some downsides like filling
the PHP error logs with deprecation warnings on thousands of servers.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/53811#comment:26>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list