- Issue created by @danflanagan8
- Status changed to Needs review
over 1 year ago 1:41pm 27 April 2023 - last update
over 1 year ago 1 fail - last update
over 1 year ago 1 pass - πΊπΈUnited States danflanagan8 St. Louis, US
Here's a fail test and a fix.
The approach in the IS didn't quite pan out because `searchIndex` is protected. Instead I followed the code in the Node class very closely. Seems to work!
This would not clear out data for previously deleted nodes though. That could happen in an update hook, but I don't have the energy for that at the moment.
Also note that the new test coverage here conflicts with the new test coverage in another issue I worked on: π Changing exclude criteria and reindexing does not remove newly excluded nodes from search index Needs review
I think that issue is more important so I'd like to see that fixed first. Then this patch could be rerolled.
The last submitted patch, 2: search_exclude-node-delete-3356664-2-FAIL.patch, failed testing. View results β