Missing media library dependency in info.yml

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dishabhadra created an issue. See original summary.

dishabhadra’s picture

Status: Active » Needs review
FileSize
267 bytes

Added the patch.

dishabhadra’s picture

Assigned: dishabhadra » Unassigned
jaapjan’s picture

Status: Needs review » Reviewed & tested by the community

Ran into the same issue when trying to install this module.

drush en media_entity_link
  Configuration objects provided by <em class="placeholder">media_entity_link</em> have unmet dependencies: <em class="placeholder">core.entity_form_display.media.link.media_library (core.entity_form_mode.media.media_library), core.entity_view_display.media.link.media_library (core.entity_view_mode.media.med  
  ia_library)</em>                                                                                                                                                                                                                                                                                                     

I think could either remove the configuration file or indeed add the dependency. Adding the dependency works :-)

Johan den Hollander’s picture

The patch did not apply when using this via composer. This is because of: https://www.drupal.org/project/drupalorg/issues/3066468

I updated the patch by putting the dependencies in a different order. It applies correctly now.

Johan den Hollander’s picture

Status: Reviewed & tested by the community » Needs review
dishabhadra’s picture

Status: Needs review » Reviewed & tested by the community

#5 looks good to me.

dishabhadra’s picture

Status: Reviewed & tested by the community » Fixed
dishabhadra’s picture

Status: Fixed » Closed (fixed)