Deprecate _update_cron_notify() and remove update.fetch.inc

Created on 5 April 2020, over 5 years ago
Updated 4 August 2025, 2 months ago

Problem/Motivation

Once upon a time, update.fetch.inc included all the code that runs while fetching new available update data and related support methods. Over time, folks have OO-ified this into src/UpdateFetcher.php, src/UpdateProcessor.php and others. The once huge update.fetch.inc now contains a single, lonely function, _update_cron_notify().

Proposed resolution

  • Move the brains from _update_cron_notify() into a more appropriate place (TBD).
  • Deprecate _update_cron_notify().
  • Remove update.fetch.inc

Remaining tasks

  1. Decide where the logic from _update_cron_notify() should really live.
  2. Implement the proposed resolution.
  3. Reviews.
  4. RTBC.
  5. Commit.

User interface changes

N/A

API changes

TBD. Nothing public, since _update_cron_notify() is prefixed with an underscore and therefore officially considered internal.

Data model changes

N/A

Release notes snippet

TBD, probably none.

πŸ“Œ Task
Status

Active

Version

11.0 πŸ”₯

Component

update.module

Created by

πŸ‡ΊπŸ‡ΈUnited States dww

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024