Problem/Motivation

There are/will be some deprecated modules in D8.

- block_place (#3084472: Remove Place Blocks module)
- entityreference (properly deprecate, prepare for complete removal in 10.x) #3111645: Uninstall entity_reference module and prevent it being enabled again, remove deprecated code.
- field_layout -> replaced by layout_builder (#3007167: [policy] Deprecate field_layout module and move it to contrib)
- simpletest -> Move to contrib (#3075490: Move simpletest module to contrib)

Proposed resolution

Criteria:

Remaining tasks

User interface changes

None

API changes

None as these APIs are already deprecated.

Data model changes

None.

Comments

jibran created an issue. See original summary.

jibran’s picture

jibran’s picture

Title: [META] Remove the deprecated module before Drupal 9 » [META] Remove the deprecated module(s) before Drupal 9
andypost’s picture

effulgentsia’s picture

I don't see it as a duplicate at all. That issue, as currently written, proposes to remove features from core. Whereas the items in this issue's current summary aren't about removing features.

jibran’s picture

Issue summary: View changes

simpletest is another one which can be removed. I was unable to find the original issue.

catch’s picture

There's an issue to have the simpletest UI generate a string to run phpunit tests from the command line. If that feature existed, it's one thing we might want to keep from simpletest module. #2566767: Don't allow running phpunit-based tests via the UI

damienmckenna’s picture

There needs to be a UI in Drupal to run tests, whether that's in Simpletest or something else it doesn't matter so long as one exists. Right now Simpletest cannot be removed because a replacement isn't available.

damienmckenna’s picture

mile23’s picture

Regarding the hook stuff in the IS... See https://www.drupal.org/node/2881531

gábor hojtsy’s picture

Title: [META] Remove the deprecated module(s) before Drupal 9 » [META] Remove deprecated modules on the Drupal 9 branch
Issue tags: +Drupal 9
Parent issue: #2608062: [META] Requirements for tagging Drupal 9.0.0-alpha1 » #3015625: [META] Prepare patches to be committed to the Drupal 9 branch once it is open

Clarifying title. They would be removed before the Drupal 9 release but not in Drupal 8 :) Also re-parenting to the right issue.

gábor hojtsy’s picture

Version: 8.7.x-dev » 9.x-dev
Priority: Normal » Critical

The overall task of this issue is a must for Drupal 9. Therefore raising to critical. The concrete list of them is not agreed on.

andypost’s picture

mile23’s picture

The language is a bit unclear what the proposed resolution is.

jibran’s picture

I'll let @larowlan explain it.

e0ipso’s picture

In #13 @Gábor Hojtsy said

The concrete list of them is not agreed on.

Should we have a separate issue for the most contentious items? For instance, action ➡️ action_ui should not be blocked on toolbar/settings_tray.

e0ipso’s picture

I also want to propose to deprecate the HAL module in D8.8 so we can move it into contrib in D9. I have a very basic proposal in #3049856: [policy] Mark HAL module as deprecated in D9 so it can be removed in D10. Maybe we can get some eyes on it.

pancho’s picture

For completeness, some more coremodules considered to be moved into main Core: #2202925: Move simple field types into Core/Field

e0ipso’s picture

gábor hojtsy’s picture

Issue summary: View changes

Added migrate multilingual issue.

gábor hojtsy’s picture

Issue summary: View changes
rivimey’s picture

I am obviously out of the loop but please can we have some explanations here as to why these changes are being made? I'm not averse to making changes but I am not happy about changes "just because".

Could we have a principle here that a separate issue is created for each proposed change, and then linked back here, which gives space for questions about particular modules to be addressed outside of this issue, keeping the conversation clean. This also gives space for the necessary explanation of how to work around the removal for sites that currently use that functionality.

gábor hojtsy’s picture

@rivimey: there will be no modules removed in this issue, this is a Plan/META issue to help track the list of issues. Each module will definitely have its own issue. For modules that are already deprecated at least there will be an issue to remove them against the Drupal 9.0.x. branch (once it is open). For modules that are not even deprecated yet, there will also be an issue against 8.y.x to deprecate it first.

rivimey’s picture

Hey Gábor, thanks for that update. It is good to know.

I admit I'm getting a little nervous about all this deprecating, considering the promise that D9 will be an easy upgrade. I'm with you all on things that have been deprecated for many versions (e.g. since 8.2.x) but deprecating in 8.8 for removal in 9 seems to me a step too far - I hope this is not happening.

damienmckenna’s picture

Issue summary: View changes

Added #2083649, #3007167 and criteria to the IS.

Removed #2966859 because it was postponed until D9.

andypost’s picture

rpsu’s picture

mile23’s picture

fgm’s picture

Issue summary: View changes

Version: 9.x-dev » 9.0.x-dev

The 9.0.x branch will open for development soon, and the placeholder 9.x branch should no longer be used. Only issues that require a new major version should be filed against 9.0.x (for example, removing deprecated code or updating dependency major versions). New developments and disruptive changes that are allowed in a minor version should be filed against 8.9.x, and significant new features will be moved to 9.1.x at committer discretion. For more information see the Allowed changes during the Drupal 8 and 9 release cycles and the Drupal 9.0.0 release plan.

rpsu’s picture

gábor hojtsy’s picture

catch’s picture

catch’s picture

Issue summary: View changes

#2083649: Rename action module to action_ui is not actually deprecated yet, removing it from the issue summary.

- settings_tray/toolbar - settings_tray is in core as stable, so again nothing is deprecated. Also removing this.

Block -> block_ui has not happened at all, removing this too.

We could open another meta for module mergers and renames, but this should only deal with actually-deprecated and/or experimental modules that are never going to be stable.

Updated the issue references for simpletest and entity_reference to the current ones.

catch’s picture

Issue summary: View changes
catch’s picture

Status: Active » Fixed

This has been done to the extent that it can be, all individual modules have their own tracking isses.

Status: Fixed » Closed (fixed)

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

gábor hojtsy’s picture