Problem/Motivation
\Drupal\elasticsearch_connector\Event\SynonymsSubscriber::onAlterSettings has
$settings['analysis']['analyser']['querytime_synonyms'] = [
'type' => 'custom',
'tokenizer' => 'standard',
'filter' => ['lowercase', 'asciifolding', 'synonyms'],
];
As shown in elasticsearch documentation, the setting key here should be analyzer.
Proposed resolution
Fix the spelling of the setting name.
Remaining tasks
Write an 9.0.x patch— merge request !178 created by @fathershawn in #2Fix test— done by @mparker17 in #3Review and feedback— done by @mparker17 in #4RTBC and feedback— done by @mparker17 in #4Commit to 9.0.x— commited by @mparker17 in #5Write an 8.0.x patch— merge request !179 created by @mparker17 in #7Review and feedback— skipping as it is identical to the 9.0.x branchRTBC and feedback— skipping as it is identical to the 9.0.x branchCommit to 8.0.x— done by @mparker17 in #9Release 9.0.x— released in 9.0.0-alpha2Release 8.0.x— released in 8.0.0-alpha6
User interface changes
none.
API changes
none.
Data model changes
none.
Issue fork elasticsearch_connector-3575982
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #4
mparker17This looks good; I updated the test, hopefully that fixes the test-failure. I've also marked this for backport.
Comment #6
mparker17Committed to 9.0.x; will move it to 8.0.x
Comment #8
mparker17Updated the issue summary
Comment #10
mparker17Committed to 8.0.x. I will update this issue when this change becomes part of a release.
Note I've created #3576581: Add a Backend test for Synonyms to help catch problems with synonyms in the future.
Comment #12
mparker17This was released in elasticsearch_connector-9.0.0-alpha2.
Comment #13
mparker17This was released in elasticsearch_connector-8.0.0-alpha6