search

localgovdrupal / localgov_search

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

Issue #29: Solr & DB support #34

Closed ekes closed 2 years ago

ekes commented 2 years ago

Open question about where to put the configuration for the index specific to the server https://github.com/localgovdrupal/localgov_search/issues/29#issuecomment-1175221402

ekes commented 2 years ago

Handy lando configuration at the project level https://github.com/localgovdrupal/localgov_project/tree/feature/2.x/localgov_search-29

finnlewis commented 2 years ago

There are integration tests on this.

Currently the tests fail due to missing dependency on search_api_solr

Can we add search_api_solr to require-dev of the project composer, then change our testing install script to include requiire-dev?

finnlewis commented 2 years ago

@ekes will take a look, but interested to know if @stephen-cox has any thoughts!

andybroomfield commented 2 years ago

Seeing the following error when trying to view the the search sitewide server.

Symfony\Component\Routing\Exception\RouteNotFoundException: Route "solr_configset.config_zip" does not exist. in Drupal\Core\Routing\RouteProvider->getRouteByName() (line 206 of core/lib/Drupal/Core/Routing/RouteProvider.php).

andybroomfield commented 2 years ago

I'm also seeing the following when checking this out after enabling localgov_demo and then localgov_search_solr

TypeError: Solarium\Core\Client\Client::construct(): Argument #2 ($eventDispatcher) must be of type Psr\EventDispatcher\EventDispatcherInterface, Drupal\Component\EventDispatcher\ContainerAwareEventDispatcher given, called in /app/web/modules/contrib/search_api_solr/src/SolrConnector/SolrConnectorPluginBase.php on line 367 in Solarium\Core\Client\Client->construct() (line 293 of /app/vendor/solarium/solarium/src/Core/Client/Client.php).

This is in the UI and not via Drush which seems fine.