Synonyms don't work because the analyzer isn't used

I confirming that: Renaming "querytime_synonyms" to "default" fix the problem

diff --git a/src/Event/SynonymsSubscriber.php b/src/Event/SynonymsSubscriber.php
index 829bc4e..56e4713 100644
--- a/src/Event/SynonymsSubscriber.php
+++ b/src/Event/SynonymsSubscriber.php
@@ -26,7 +26,7 @@ class SynonymsSubscriber implements EventSubscriberInterface {
         'lenient' => TRUE,
         'synonyms' => array_map('trim', $synonyms),
       ];
-      $settings['analysis']['analyzer']['querytime_synonyms'] = [
+      $settings['analysis']['analyzer']['default'] = [
         'type' => 'custom',
         'tokenizer' => 'standard',
         'filter' => ['lowercase', 'asciifolding', 'synonyms'],

The solution in #2 did not work for me, but what did work was to disable fuzziness on the server configuration. We may need to find a way to combine fuzziness with synonyms.

The other issue I found is that the AlterSettingsEvent is dispatched when the index is saved, not when the server is saved, yet the synonyms configuration is on the server. It might make more sense to move the synonyms to the index, but failing that we should probably update all indexes for a server when the server is updated.

kim.pepper → made their first commit to this issue’s fork.

kim.pepper → changed the visibility of the branch 3.x to hidden.

Yes, the AlterSettingsEvent should probably be called AlterIndexSettingsEvent.

We don't currently have anything to specifically update the server settings.

The solution from #2 is working for me. I have to rebuild the index after changing the synonyms, but after that it works as expected.