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

  1. Write an 9.0.x patch — merge request !178 created by @fathershawn in #2
  2. Fix test — done by @mparker17 in #3
  3. Review and feedback — done by @mparker17 in #4
  4. RTBC and feedback — done by @mparker17 in #4
  5. Commit to 9.0.x — commited by @mparker17 in #5
  6. Write an 8.0.x patch — merge request !179 created by @mparker17 in #7
  7. Review and feedback — skipping as it is identical to the 9.0.x branch
  8. RTBC and feedback — skipping as it is identical to the 9.0.x branch
  9. Commit to 8.0.x — done by @mparker17 in #9
  10. Release 9.0.x — released in 9.0.0-alpha2
  11. Release 8.0.x — released in 8.0.0-alpha6

User interface changes

none.

API changes

none.

Data model changes

none.

Command icon 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

fathershawn created an issue. See original summary.

mparker17 made their first commit to this issue’s fork.

mparker17’s picture

Status: Active » Needs review
Issue tags: +needs backport to 8.0.x

This looks good; I updated the test, hopefully that fixes the test-failure. I've also marked this for backport.

mparker17’s picture

Version: 9.0.x-dev » 8.0.x-dev
Status: Needs review » Reviewed & tested by the community
Issue tags: -needs backport to 8.0.x

Committed to 9.0.x; will move it to 8.0.x

mparker17’s picture

Issue summary: View changes

Updated the issue summary

mparker17’s picture

Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed 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.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

mparker17’s picture

Issue summary: View changes
mparker17’s picture

Issue summary: View changes

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.