- 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
I tried replicating on D10 and instead of a fatal error I get a 404. Could someone else confirm?
Moving to PNMI but if still an issue move to needs work as this will need test to show the issue.
- Status changed to Closed: outdated
10 months ago 9:18pm 14 February 2024 - 🇺🇸United States smustgrave
Since there hasn't been a follow up going to close for now. If still a valid bug please reopen
- First commit to issue fork.
- 🇳🇱Netherlands casey
Snapshot of latest state of MR for usage with composer patches on drupal/core:10.3.*
- 🇺🇸United States smustgrave
Will still need test coverage.
Issue summary should probably be updated too. "Add extra check" isn't a complete solution I'd say.
- 🇮🇳India arunkumark Coimbatore
I verified the issue, unable to replicate. Below are the steps I followed to replicate the issue.
Scenario 1: Attached Entity removed
1. Added Comment to Article content type.
2. Created node, and added Comment to Node.
3. Added child comments to the parent comment.
4. Deleted Node programmatically using below script,$nodeStorage = \Drupal::service('entity_type.manager')->getStorage('node'); $node = $nodeStorage->load(NID); $node->delete();
5. Accessed the Comment Edit and Delete URLs.
6. Getting page not found page instead of Error.Scenario 2: Attached Parent comment removed
1. Added Comment to Article content type.
2. Created node, and added Comment to Node.
3. Added child comments to the parent comment.
4. Deleted Parent Comment programmatically using the below script,$commentStorage = \Drupal::service('entity_type.manager')->getStorage('comment'); $comment = $commentStorage->load(CID); $comment->delete();
5. Accessed the Child Comment Edit and Delete URLs.
6. Getting page not found page instead of Error.