Pull Request for Issue #33490

Summary of Changes

Reason behind this issue

I debugged the array of id(s) which were queried for deletion and their query result:

Array of ids to be deleted: [5,4,10,14,13,12,11,15,7,6,9,8,3,2]
Status:
5 - done
4 - done
10 - failed

Changes Made:

The condition that checks if the key is in the table is now shifted outside the loop so when an element is deleted, that doesn't generate any error and return prematurely. Incase any key is not present, then it would be checked before deletion and the execution would stop then and there

Testing Instructions

Before testing, Please take a backup of #_finder_taxonomy table to avoid the loss of any data

1. Visit Backend
2. Components -> Smart Search -> Content Map
3. Ensure that you have sufficient data in this table with multiple parent-level and child-level maps (Installting Blog Sample Data will also work)
4. Click on the select all checkbox and proceed to delete them.

Before the PR, only the first parent and its children would get deleted and then the function would return false skipping the remaining elements.
After the PR, you should see the Empty State, ie. All items deleted successfully.

Actual result BEFORE applying this Pull Request

Please refer to the video in #33490

Expected result AFTER applying this Pull Request

Documentation Changes Required

None