Contrib.social

Rename 'update.manager' service and related code

Open on Drupal.org →
Created on 23 April 2025, 17 days ago

Problem/Motivation

Split from 📌 Rename update module back to Update Status Active . update.module no longer "manages" updates, and we're renaming it back to 'Update Status' to make way for a new "Update Manager" via "autoupdates". However, we've got an update.manager service, which is mostly for fetching release histories. It'd be nice to rename this, but there are some BC implications and we don't want to delay the parent ticket about it.

Steps to reproduce

Look in these files:

  1. core/modules/update/src/UpdateManagerInterface.php
  2. core/modules/update/src/UpdateManager.php
  3. core/modules/update/update.services.yml (search for update.manager)

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

📌 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

Comments & Activities

  • Issue created by @dww
  • Comment 17 days ago →
    🇺🇸United States dww

    Drupal\update\UpdateManagerInterface is a pain, and UpdateManagerInterface::NOT_SECURE and friends are sprinkled throughout.

    This might be more trouble than it's worth. From a Slack thread today:
    https://drupal.slack.com/archives/C079NQPQUEN/p1745370644226489

    dww: It's not totally clear to me why this is even a service.
    larowlan: We have a moved class autoloader in core now, search for - you can define a container param with modulename.moved_classes that will load the new name - see core.moved_classes in core.services.yml for an example
    larowlan: You can also add deprecated to service definitions - see library.discovery.collector in core.services.yml
    so I think they should handle the service name and the class names - i.e. I think we have the tools for this now
    dww: Thanks for those details!
    dww: Stepping back a bit -- is this even worth it?
    dww: Maybe "no one" cares that this service / interface have this name, even if it doesn't really "manage" anything?
    larowlan: :shrug: naming is hard
    larowlan: Maybe something to ask the UX team - package manager is working to rename 'Stage' to 'Sandbox' in class names out of feedback from the UX team even though its really only DX
    moshe: i vote for not worth it

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024