Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Search index needs flushing when merging people #746

Open
CHTJonas opened this issue Nov 7, 2019 · 2 comments
Labels

Comments

@CHTJonas
Copy link
Member

@CHTJonas CHTJonas commented Nov 7, 2019

Low priority bug. It appears that the search cache is not flushed for the affected entities when two people are merged together. This can result in stale entries in the search dropdown.

Support Ticket: 2206

@GKFX

This comment has been minimized.

Copy link
Member

@GKFX GKFX commented Nov 8, 2019

I don't think this is normal behaviour. I'm sure I usually check after doing a merge that the old search entry has gone, and also I can't reproduce this on a fresh checkout of master. Elastica is listening for the postFlush Doctrine event, which is called by

$this->entityManager->flush();

On my machine looking at the Symfony profiler this all works correctly and a request to delete is sent to Elasticsearch and then carried out ok.

@CHTJonas

This comment has been minimized.

Copy link
Member Author

@CHTJonas CHTJonas commented Nov 8, 2019

@GKFX have you tested all 4 permutations of the radio buttons on the merge page? I seem to recall this was the first time a performed a merge using a URL rather than a name. Not sure why that would have any affect though...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.