Problem/Motivation

media_install() contains this code

    if (\Drupal::moduleHandler()->moduleExists('media_entity')) {
      $info = \Drupal::service('extension.list.module')->getExtensionInfo('media_entity');
      if (version_compare($info['version'], '8.x-2') < 0) {

This can be safely removed as Media Entity cannot run in Drupal 9 because of a missing core_version_requirement in media_entity.info.yml: https://git.drupalcode.org/project/media_entity/-/blob/8.x-1.x/media_ent...

Steps to reproduce

Proposed resolution

Remove this check.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#3 3254198-3.patch1.17 KBlongwave
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

longwave created an issue. See original summary.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Issue summary: View changes
longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Status: Active » Needs review
FileSize
3254198-3.patch1.17 KB
joachim’s picture

joachim CreditAttribution: joachim at Factorial GmbH commented
Status: Needs review » Reviewed & tested by the community

Good clean-up!

  • catch committed 637ccc1 on 10.0.x 
    Issue #3254198 by longwave: Remove Media Entity compatibility check from...

  • catch committed df9d679 on 9.4.x 
    Issue #3254198 by longwave: Remove Media Entity compatibility check from...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed 637ccc1 and pushed to 10.0.x. Thanks! Cherry-picked to 9.4.x too.

Status: Fixed » Closed (fixed)

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