- Issue created by @pcate
- πΊπΈUnited States itmaybejj
Aye yeah; I haven't read up on node delete hooks yet. I'm sure it won't be too complicated once I have the time.
At the moment there are two ways to remove deleted pages from the reports:
- If you click through to any page from the report and land on the 404 page instead, either because the page was deleted or the alias changed, the stale entry is automatically removed.
- If you click through to the report for a specific page, there is a manual "Reset results for this page" link below the table.
It's going to be a few weeks before I can take a look at this; I sure would like to add the feature though.
- Status changed to Postponed
almost 2 years ago 12:15pm 16 March 2023 - πΊπΈUnited States pcate
Aye yeah; I haven't read up on node delete hooks yet.
hook_entity_delete
may be helpful. - Status changed to Active
over 1 year ago 2:44pm 27 March 2023 - πΊπΈUnited States itmaybejj
Going to take another look at this -- it looks like the existing automated removals aren't working.
- πΊπΈUnited States itmaybejj
2.0.11 β restores the automatic removal of stale entries when clicking from the dashboard.
I'll work on my delete hook branch again later -- I spent a day on it and hit a wall because the information I am storing is all about the currently active route object when an editor is viewing a page, including language codes, and I have not been able to recreate that from the information available in the entity delete hook.
I'm going to have to modify my tables to store some canonical information. Not difficult, just time consuming. Flipping this back into postponed state as I won't be getting back to this in the next few weeks.
- Assigned to itmaybejj
- Status changed to Postponed
over 1 year ago 7:17pm 24 April 2023 - First commit to issue fork.
- Open on Drupal.org βCore: 10.0.7 + Environment: PHP 8.1 & MySQL 5.7last update
over 1 year ago Not currently mergeable. - @oheller opened merge request.
- πΊπΈUnited States oheller
Started the merge request branch during MidCamp. I will continue to work on this today.
- Status changed to Active
over 1 year ago 5:05pm 28 April 2023 - last update
over 1 year ago 2 pass - πΊπΈUnited States oheller
First bullet point ready for review. I've updated the schema with entity_id and implemented the update_hook() to add the field to the tables.
- last update
about 1 year ago 4 fail - last update
about 1 year ago 2 pass - Status changed to Needs review
about 1 year ago 9:07pm 25 September 2023 -
itmaybejj β
committed 99f02ee5 on 2.0.x authored by
oheller β
Issue #3348190: Remove entry when page is deleted.
-
itmaybejj β
committed 99f02ee5 on 2.0.x authored by
oheller β
-
itmaybejj β
committed 29fa56a9 on 2.0.x
Issue #3348190 by itmaybejj: Fix invalid schema.
-
itmaybejj β
committed 29fa56a9 on 2.0.x
-
itmaybejj β
committed 357f0dc6 on 2.0.x
Issue #3348190 by itmaybejj: Add entity ID to API and node delete hooks.
-
itmaybejj β
committed 357f0dc6 on 2.0.x
- Status changed to Fixed
about 1 year ago 7:05pm 4 October 2023 Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
3 months ago 3:20pm 18 September 2024