Flush index in all cases

monsieurbiz / SyliusSearchPlugin

A search plugin using Elasticsearch for Sylius

MIT License

45 stars 38 forks source link

Flush index in all cases #204

Closed t-richard closed 3 months ago

t-richard commented 8 months ago

Currently in the indexer, indexByDocuments flushes only when the document is translatable.

With this fix, it flushes in all cases. The only drawback being it now flushes for each locale when it's translatable and no locale was provided which is IMHO acceptable.

Happy to discuss about this !

PS: thanks for the plugin

delyriand commented 3 months ago

Thanks @t-richard! indeed, a final flush was missing in this case