Problem/Motivation

Drupal 10 no longer checks for older updates that start with a number lower than minimum supported core. eg Since 2.0.x branch minimum version is 9.1, updates starting with 8NNN are no longer run. That means updates 8001 and 8002 can now be safely removed as they should have run.

However, there is the slight possibility that update 8002 might be missed.

Proposed resolution

- Remove updates 8001 and 8002.
- Let drupal know that they have been removed, but only up to 8001 since 8002 could possibly be missed.
- Add post_update which does the same thing as 8002 but guards against overriding permissions set by admin

Remaining tasks

  • ✅ File an issue
  • ✅ Addition/Change/Update/Fix
  • ✅ Release notes snippet
  • ❌ Release

Release notes snippet

Issue fork cas_server-3465846

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

ELC created an issue. See original summary.

ELC opened merge request !29

  • ELC committed d9f176bd on 3.0.x 
    [#3465846] Remove deprecated updates; provide backup for 8002 update.
elc’s picture

elc commented
Issue summary: View changes
Status: Active » Patch (to be ported)

This needs to be merged into 2.0.x as it suffers the same issue when run on a Drupal 10 site.

ELC opened merge request !30

  • ELC committed 25cbfc82 on 2.0.x 
    [#3465846] Remove deprecated updates; provide backup for 8002 update.
elc’s picture

elc commented
Assigned: elc » Unassigned
Status: Patch (to be ported) » Fixed
elc’s picture

elc commented
Status: Fixed » Needs work

On review, the number set in hook_update_last_removed() should be 8002, as the replacement is no longer an hook_update_NNNN, but rather a post_update.

  • ELC committed e872c15b on 2.0.x 
    [#3465846] Correct last removed update number.

  • ELC committed 9e633a82 on 3.0.x 
    [#3465846] Correct last removed update number.
elc’s picture

elc commented
Status: Needs work » Fixed

Number corrected.

Status: Fixed » Closed (fixed)

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