AO3-7119 Use standard analyzer for tags on bookmark search

[Cubostar]

Pull Request Checklist

• Have you read "How to write the perfect pull request"?
• Have you read the contributing guidelines?
• Have you added tests for any changed functionality?
• Have you added the Jira issue number
  as the first thing in your pull request title (e.g. AO3-1234 Fix thing)
• Do you have fewer than 5 pull requests already open? If not, please wait
  until they are reviewed and merged before creating new pull requests.

Issue

https://otwarchive.atlassian.net/browse/AO3-7119

Purpose

This PR fixes a bookmark indexing issue by changing the Elasticsearch bookmark analyzer from simple to standard.

Credit

Cubostar, he/him

[anmazz]

(Not an official volunteer, so please defer to them if they disagree with me!)

I’d recommend adding a Cucumber .feature test here so the functionality is verified. You can probably just adapt the testing steps outlined in the ticket.

[anmazz]

You may be able to remove this line, the standard analyzer should be the default, but I would wait until an official volunteer confirms before removing. Looking through the code there are some explicit instances of analyzer: "standard", so might be totally fine as is!

[Cubostar]

Sounds good, I'll wait until there's confirmation. Thanks for reviewing my PR!

[Bilka2]

This is fine to keep explicit. Generally, I would say that we have no set style preference for this situation.

Personally, I like when a decision like this has an easy to follow git blame for why it is the way it is, so keeping this line makes it easier to git blame -> find this pr -> find the related test and jira issue

[Bilka2]

Thank you!

[Cubostar]

No problem, thanks for reviewing!