Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
acquia_search/config/install/search_api.index.acquia_search_index.yml declares a config dependency on field.storage.node.body, but the module itself does not declare a dependency on the node module.
In a clean install scenario, it is possible for Drupal to attempt to install acquia_search before node, which leads to a unmet dependency error.
Patch attached.
Comment | File | Size | Author |
---|---|---|---|
#2 | acquia_search_node_dependency-2833250-2.patch | 338 bytes | timcosgrove |
Comments
Comment #2
timcosgrove CreditAttribution: timcosgrove commentedComment #3
gg4 CreditAttribution: gg4 commentedComment #4
alonaoneill CreditAttribution: alonaoneill at Hook 42 commentedAlso according to Drupal standards, modules should include dependencies in the .info.yml file. Also capitalize the name of the module, because module names are proper nouns.
The same issue was open:
https://www.drupal.org/project/acquia_connector/issues/3002073
Looks like, we should close this one?
Comment #5
Dane Powell CreditAttribution: Dane Powell at Acquia commentedAlthough #3002073: Capitalize the name of the submodule and dependency namespacing in .info.yml file and this both relate to the info.yml file, I think they should be treated separately.
Comment #6
Dane Powell CreditAttribution: Dane Powell at Acquia commentedThis seems related to #2962395: acquia_search Error on module installation under install profile: unmet dependencies
Comment #8
Dane Powell CreditAttribution: Dane Powell at Acquia commented