Refactor \Drupal\Core\Extension\ModuleInstaller

Created on 18 May 2015, almost 10 years ago
Updated 11 May 2025, 1 day ago

Problem/Motivation

Subsequent to the addition of the rollback method in the parent ticket the Module Installer is going to be left with a LOT of duplicate code that should be refactored for performance and maintenance reasons. Even if that patch (parent to this ticket) is never accepted, the install and uninstall methods have duplicated code between them and have grown too lengthy to be easily maintained.

Proposed resolution

Pay the technical debt - refactor. There will be no UIX or API changes.

πŸ“Œ Task
Status

Closed: works as designed

Version

11.0 πŸ”₯

Component

extension system

Created by

πŸ‡ΊπŸ‡ΈUnited States Aki Tendo

Live updates comments and jobs are added and updated live.
  • Performance

    It affects performance. It is often combined with the Needs profiling tag.

Sign in to follow issues

Comments & Activities

Not all content is available!

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

  • πŸ‡ΊπŸ‡ΈUnited States nicxvan

    I think this is safe to close, there is no rollback method that is mentioned in the issue summary, a different solution was committed.

Production build 0.71.5 2024