Problem/Motivation
D10 deprecated code needs removal in D11.

Proposed resolution
Remove deprecated code, legacy tests, PHPStan baselined errors.

Issue fork drupal-3432326

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

mondrake created an issue. See original summary.

mondrake opened merge request !7105

mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Active » Needs review
mondrake’s picture

mondrake
Location 🇮🇹
commented
Assigned: mondrake » Unassigned
smustgrave’s picture

smustgrave commented
Title: [11.x] Remove deprecated code from the Extension API » Remove deprecated code from the Extension API
Status: Needs review » Reviewed & tested by the community

Removal looks good to me.

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Needs review

One question on the MR.

mondrake’s picture

mondrake
Location 🇮🇹
commented

Filed #3433034: Refactor InstallerNonDefaultDatabaseDriverTest to avoid DatabaseDriverUninstallValidator re-fetch the database connection, that's going to be tricky; will adjust the todo now.

mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Needs review » Reviewed & tested by the community

Just changed to @todo reference to the new follow-up, think it's OK for me to self reRTBC.

  • catch committed d84398bf on 11.x 
    Issue #3432326 by mondrake: Remove deprecated code from the Extension...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Thanks new follow-up looks good.

Committed/pushed to 11.x, thanks!

mondrake closed merge request !7105

Status: Fixed » Closed (fixed)

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