Problem/Motivation

Themes now support post update functions. See https://www.drupal.org/node/3259199

However when visiting /update.php to process the theme's update function, it refers to the theme as a module.

I expect the update function to refer to the theme as a theme.

Steps to reproduce

Proposed resolution

Change DbUpdateController to track updated by extension type, to not always assume that the extension is a module.

Remaining tasks

Commit

User interface changes

Before

After screenshot

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mherchel created an issue. See original summary.

alexpott’s picture

Status: Active » Needs review
FileSize
6.55 KB

Nice catch - here's a fix for this screen and the results screen.

mherchel’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
71.41 KB

Works great!

Version: 9.4.x-dev » 9.5.x-dev

Drupal 9.4.0-alpha1 was released on May 6, 2022, which means new developments and disruptive changes should now be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

quietone’s picture

Issue summary: View changes
Issue tags: +Bug Smash Initiative

What a nice change.

Thanks!

  • catch committed 47f2a07 on 10.0.x
    Issue #3280398 by alexpott, mherchel, quietone: Theme's post updates...

  • catch committed 829d166 on 9.4.x
    Issue #3280398 by alexpott, mherchel, quietone: Theme's post updates...
  • catch committed 4649b00 on 9.5.x
    Issue #3280398 by alexpott, mherchel, quietone: Theme's post updates...
catch’s picture

Version: 9.5.x-dev » 9.4.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 10.0.x, cherry-picked to 9.5.x and 9.4.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.