Problem/Motivation

Spotted in #3565560: Describe the requirements array as a PHPDoc type, this class exists for a single check:

    if (\Drupal::moduleHandler()->moduleExists('layout_plugin')) {

This module was marked obsolete 9 years ago: https://www.drupal.org/project/layout_plugin

The info.yml file doesn't have core_version_requirement so I doubt this is even installable in modern Drupal.

Steps to reproduce

Proposed resolution

Delete the class.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3582666

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

longwave created an issue. See original summary.

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

adhershmnair opened merge request !15324

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Reviewed & tested by the community

Thank you.

  • amateescu committed 88a393bd on 11.x 
    task: #3582666 Remove Drupal\layout_discovery\Install\Requirements\...

  • amateescu committed 10d11f6a on main 
    task: #3582666 Remove Drupal\layout_discovery\Install\Requirements\...
amateescu’s picture

amateescu commented
Version: main » 11.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 10d11f6ab98 to main and 88a393bda80 to 11.x. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

amateescu closed merge request !15324

Status: Fixed » Closed (fixed)

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