Problem/Motivation

The module is already deprecated and marked as obsolete. Uninstall it from 11.x sites and remove the code for Drupal 12

Proposed resolution

main - Remove the core/modules/sdc folder.
11.x - Add a post-update function to uninstall SDC. The module is already marked as obsolete.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3410389

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:

Comments

e0ipso created an issue. See original summary.

e0ipso’s picture

Title: Remove the sdc module from core on 12.x » [PP-1] Remove the sdc module from core on 12.x
Status: Active » Postponed

Postponing until we have a 12.x branch.

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.

nicxvan’s picture

Title: [PP-1] Remove the sdc module from core on 12.x » Remove the sdc module from core on 12.x
Status: Postponed » Needs work

This can be done now right?

nicxvan’s picture

Title: Remove the sdc module from core on 12.x » Remove the sdc module from core

I guess one question is we usually move to contrib but we're not doing that here.

Is there anything special we have to do?

nicxvan’s picture

Status: Needs work » Needs review
penyaskito’s picture

Category: Feature request » Task

AFAIK we never had a hook_update uninstalling it, so we should land that in 11.x if we want this to happen in 12.x

smustgrave’s picture

Update hook in this ticket or follow up? Imagine it would be small enough probably good all do here?

nicxvan’s picture

Well it will take special consideration.

penyaskito’s picture

@smustgrave You can't do both sdc.info.yml removal and the hook in the same MR (or even release)

smustgrave’s picture

Could do an MR to remove it like we got and an MR to 11.x to uninstall. Just meant the same ticket

dcam made their first commit to this issue’s fork.

dcam’s picture

I made an MR for 11.x. I copied the post-update function that was put in to uninstall migrate_drupal. As far as I could tell that other function didn't have tests, but I didn't check thoroughly because that MR affected 853 files.

dcam’s picture

Issue summary: View changes
nicxvan’s picture

The problem is that someone can update from 11.4.0 which won't have this.

I think we can remove it in one go though with an update hook, we just need to drop the schema from key value and update the config right?

smustgrave’s picture

Status: Needs review » Reviewed & tested by the community

Think this is small enough to still make 12 maybe.

longwave’s picture

We already should have uninstalled the module via system_post_update_sdc_uninstall(), so I don't think we need anything in 11.x here and we can just delete it in 12.

pdureau’s picture

Assigned: Unassigned » pdureau

I can commit this

pdureau’s picture

Doing this today.