backdrop-contrib / search_api

Provides a generic API for modules offering search capabilities
GNU General Public License v2.0
0 stars 6 forks source link

Search api should not set the info for search server modules as `required` #60

Closed argiepiano closed 1 year ago

argiepiano commented 1 year ago

This is a similar issue as the one discovered in Feeds. The problem here is that search_api checks if there is a search server currently in use by indexes, and if it finds any, then it marks the server module's info as required. In Backdrop, this has the effect of completely hiding the module, rather than disabling the checkbox to disable it. This produces WTF moments, as it's impossible to tell what database_search, for example, is enabled but not shown.

argiepiano commented 1 year ago

In Backdrop, the info should be switched to disabled - which disables the checkbox, but still shows the module.

argiepiano commented 1 year ago

@laryn, PR #61 is ready for review.

laryn commented 1 year ago

Thanks @argiepiano!