Exported server definitions don't list storage engine module as a dependency [#1600986]

When you export a search_api server definition to a feature it does not list the module that provides the storage engine as a dependency. This is particularly problematic when the feature is used in an installation profile as it causes the installation to fail. In both circumstances, if the relevant module is not already enabled it will fail with the following error:
SearchApiException: Search server with machine name site_db specifies illegal service class search_api_db_service. in SearchApiServer->ensureProxy() (line 133 of search_api/includes/server_entity.inc).

Continued from #1593312: SearchApiException: Search server with machine name site_db specifies illegal service class search_api_db_service.

Comment	File	Size	Author
#3	search_api-n1600986-3.patch	1.4 KB	DamienMcKenna
#3
#1	search_api-n1600986.patch	1.41 KB	DamienMcKenna
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

DamienMcKenna

NH, USA

CreditAttribution: DamienMcKenna commented 25 May 2012 at 21:04

Status:

Active

» Needs review

File	Size
search_api-n1600986.patch	1.41 KB

This patch resolves the problem. It uses hook_features_export_alter() to run through the list of exported server specifications, loads each one and then works out what module is responsible for generating it - if the module isn't in the dependencies list it is added.