[wp-trac] [WordPress Trac] #51742: Core auto-updates UI: disable checkbox when filter or constant takes precedent on the option
WordPress Trac
noreply at wordpress.org
Fri Nov 13 11:21:28 UTC 2020
#51742: Core auto-updates UI: disable checkbox when filter or constant takes
precedent on the option
---------------------------------------+-----------------------
Reporter: audrasjb | Owner: audrasjb
Type: defect (bug) | Status: reopened
Priority: normal | Milestone: 5.6
Component: Upgrade/Install | Version: trunk
Severity: normal | Resolution:
Keywords: has-patch has-screenshots | Focuses: ui
---------------------------------------+-----------------------
Comment (by SergeyBiryukov):
Replying to [comment:42 audrasjb]:
> `51742.10.diff` fixes this issue.
Just a quick note that in [49592] I changed the value passed to the filter
from `null` to `false`, for consistency with the corresponding
[source:tags/5.5.3/src/wp-admin/includes/class-wp-site-health-auto-
updates.php?marks=152,153#L144 Site Health check].
I know we pass `null` to the `auto_update_{$type}` filter to detect
whether nothing has hooked into it, as discussed in #50848, but in this
case it looks like we just need to detect whether the updater is disabled
or not, so passing `false` seems appropriate.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/51742#comment:47>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list