Title search should exclude the current node

It looks good, but I think the patch need to be rerolled!

Rerolled the MR.

Hi, I just tested MR !3 and it is working for me. + 1 for RTBC

I tested this, but I got a funny result. I tried to first create one node with title "Test node 1", then another node "Test node 2". After I try to edit the node "Test node 2", and then the current node ( "Test node 2") is still displayed in the list. However the count is correct (not counting current node), but the current node is still displayed in the list.

I can't seem to reproduce that issue. Did you test this using the last dev-4.0.x version with the latest state of the MR applied as patch?

@@DieterHolvoet
This issue fixes for all conditions now. Before last commit, when the option Contains any word from the title checked in search types config, the condition for ignoring current node not added.
After this, I tested all cases and it is working fine.
RTBC +1

I refactored the code a little bit and fixed an issue where no results were returned when adding a new node ($entity->id() would give NULL).

I tested this manually, and I still see current node in results when adding the exact same title:
https://git.drupalcode.org/project/anti_duplicates/-/merge_requests/3.diff

Are you sure it's the same node? I just tested using the latest dev-4.0.x and only the MR here applied as patch, the current node doesn't appear in the results.

Okay I tested and looked over again now, thanks!

DieterHolvoet → closed merge request !3

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