Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#51743 closed task (blessed) (fixed)

Core auto-updates UI: Consider setting it by default for new installations

Reported by: audrasjb's profile audrasjb Owned by:
Milestone: 5.6 Priority: normal
Severity: normal Version: 5.6
Component: Upgrade/Install Keywords: has-patch has-screenshots
Focuses: Cc:

Description

During the last dev chat, setting major auto-updates by default for new installations was considered by the core team. The project leads also wanted to explore this for WP 5.6.

Attachments (5)

51743.diff (482 bytes) - added by audrasjb 4 years ago.
Upgrade/Install: set auto_update_core_major option to true for new installations
Capture d’écran 2020-11-10 à 01.35.57.png (228.0 KB) - added by audrasjb 4 years ago.
Patch works fine.
51743.2.diff (1.3 KB) - added by azaozz 4 years ago.
51743.3.diff (1.5 KB) - added by azaozz 4 years ago.
51743.4.diff (1.3 KB) - added by helen 4 years ago.

Download all attachments as: .zip

Change History (23)

@audrasjb
4 years ago

Upgrade/Install: set auto_update_core_major option to true for new installations

#1 @audrasjb
4 years ago

  • Keywords has-patch has-screenshots dev-feedback added

#2 @mukesh27
4 years ago

Patch 51743.diff working fine for me on WordPress 5.6-beta3-49552 with the fresh installation.

This ticket was mentioned in Slack in #core-auto-updates by audrasjb. View the logs.


4 years ago

This ticket was mentioned in Slack in #core-auto-updates by audrasjb. View the logs.


4 years ago

@azaozz
4 years ago

#5 @azaozz
4 years ago

  • Keywords needs-testing added

In 51743.2.diff: Set the default for the auto_update_core_major option to enabled for new installs and override it to unset on upgrading from a WP version earlier than 5.6.

Please test!

@azaozz
4 years ago

#6 @azaozz
4 years ago

In 51743.3.diff:

  • Populate the three new auto-update options: auto_update_core_dev, auto_update_core_minor, and auto_update_core_major.
  • Add some more inline comments.

These pre-populated options apply only to single site installs despite that they are set/read with update_site_option() and get_site_option(). To do the same for multisite/network installs it seems the code will have to be in upgrade_network() and (perhaps) in populate_network() in schema.php but not sure if that's needed/a good idea. Pinging @jeremyfelt as the resident multisite/network expert for a quick review :)

Still todo:

  • Test on existing and new single-site installs.
  • TBD: should new network installs be set to do major core auto-updates by default, same as single sites?
Version 0, edited 4 years ago by azaozz (next)

This ticket was mentioned in Slack in #core-auto-updates by hellofromtonya. View the logs.


4 years ago

This ticket was mentioned in Slack in #core by helen. View the logs.


4 years ago

@helen
4 years ago

#9 @helen
4 years ago

In 49581:

Upgrade/Install: Enable all core autoupdates for new installs.

Likely needs more work for multisite installs.

Reference: https://make.wordpress.org/core/2020/11/10/wp5-6-auto-update-implementation-change/

Props audrasjb, azaozz.
See #51743.

This ticket was mentioned in Slack in #core-auto-updates by pbiron. View the logs.


4 years ago

#11 @helen
4 years ago

Does this ticket need anything else? Do we need to address multisite at all? My view is that by nature multisite is still really meant for a more hands-on admin so I don't know that autoupdates on by default makes sense right now. That said, is it weird if somebody has explicitly turned major autoupdates on in the UI and then converts to multisite and it's off again? Is it okay to call that a multisite conversion quirk for now? I am not strongly opinionated, hoping somebody else feels strongly enough about this to advocate for a direction before RC :)

This ticket was mentioned in Slack in #core-auto-updates by hellofromtonya. View the logs.


4 years ago

This ticket was mentioned in Slack in #core-auto-updates by hellofromtonya. View the logs.


4 years ago

#14 @audrasjb
4 years ago

  • Keywords dev-feedback needs-testing removed

FWIW, I'm fine with the current implementation _for now_. Multisites can be handled in the next release. Also, they still have the possibility to opt-in to auto-updates once the multisite is set up.

#15 @pbiron
4 years ago

I concur with @audrasjb.

Ultimately, when a single site is converted to a multisite, it would be good if the multisite "inherited" the current setting for the single site during conversion, but I seriously doubt we can get a working/tested patch before RC1 (in a few hours). I know I certainly don't have time.

This "quirk" should be made very explicit in the dev-note so that MS admins are aware.

#16 @audrasjb
4 years ago

This "quirk" should be made very explicit in the dev-note so that MS admins are aware.

Absolutely. FYI I'm working on the dev note right now, I'll let you know when it's ready for review.

This ticket was mentioned in Slack in #core-auto-updates by hellofromtonya. View the logs.


4 years ago

#18 @audrasjb
4 years ago

  • Resolution set to fixed
  • Status changed from new to closed

Closing as fixed. We'll open a new ticket to handle next steps for multisites.

Note: See TracTickets for help on using tickets.