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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

timcosgrove created an issue. See original summary.

timcosgrove’s picture

gg4’s picture

Status: Active » Needs review
alonaoneill’s picture

Status: Needs review » Needs work

Also 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?

Dane Powell’s picture

Status: Needs work » Needs review

Although #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.

  • Dane Powell authored e810357 on 8.x-1.x
    Fixes #2962395, #2833250, #2726395: Make acquia_search_index config...
Dane Powell’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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