- Issue created by @pcate
- πΊπΈUnited States pcate
From my limited research this error may be because the module's
.info
file does not specify module dependencies. - Merge request !16Add dependency on Drupal core media module to solve service loading order issue. β (Merged) created by pcate
- last update
almost 2 years ago 9 pass - Status changed to Needs review
almost 2 years ago 5:06pm 21 April 2023 - πΊπΈUnited States pcate
Created MR for adding the core media module as a dependency. The MR resolved the issue for me.
- Assigned to maacl
- Status changed to Needs work
almost 2 years ago 8:49am 24 April 2023 - π©πͺGermany maacl
Thanks for the report and the patch, let's see if we can add a test for this.
- πΊπΈUnited States pcate
let's see if we can add a test for this.
Honestly I'm not sure how to test this. Did you have a specific approach you were thinking of?
- Issue was unassigned.
- π©πͺGermany maacl
let's see if we can add a test for this.
Honestly I'm not sure how to test this. Did you have a specific approach you were thinking of?
I think it should be possible to install the module in a test, and then it should only pass if the media and file module are enabled also? And currently they won't be enabled because of the missing dependency.
- πΊπΈUnited States pcate
@maacl not sure how useful a test for this is. It seems like it would be testing Drupal module dependency system rather than this module itself. The module has a clear dependency on the core media module that should be declared.
- First commit to issue fork.
- Status changed to Needs review
about 1 month ago 4:07pm 24 February 2025 Automatically closed - issue fixed for 2 weeks with no activity.