localgovdrupal / localgov_search

Default sitewide search implementation for LocalGov Drupal.
GNU General Public License v2.0
0 stars 1 forks source link

Configuration objects (search_api.index.localgov_sitewide_search, views.view.localgov_sitewide_search) provided by localgov_search already exist in active configuration #58

Open finnlewis opened 1 year ago

finnlewis commented 1 year ago

While testing installing and uninstalling I came across this.

Error message
Unable to install LocalGov Search, search_api.index.localgov_sitewide_search, views.view.localgov_sitewide_search already exist in active configuration. 

Steps to reproduce:

Fresh install of localgov drupal. Enable localgov_search. Uninstall localgov_search Enable localgov_search.

For example:

lando drush si localgov -y
lando drush en localgov_search -y
lando drush pm-uninstall localgov_search -y
lando drush pm-uninstall localgov_search -y

Gives me:

Configuration objects (search_api.index.localgov_sitewide_search, views.view.localgov_sitewide_search) provided by localgov_search already exist in active configuration  

So I think we need to add dependencies to the configuration objects to force them to be uninstalled with the module.

ekes commented 1 year ago

My initial feeling, and I have an itch that this has been discussed: search should be a starter configuration so you should be able to uninstall it without loosing your configuration.

finnlewis commented 1 year ago

Interesting.... let's review this and look at other cases where this works in the same way.

Possibly need to clarify the intended use in the README