Add changed time tracking to path_alias

Created on 6 October 2021, over 3 years ago

Updated 11 February 2023, about 2 years ago

Problem/Motivation

Unlike nearly every content entity in core path_alias entities don't store any information about their latest change. That's an issue for all contrib content synchronization modules like default_content_deploy → .

Steps to reproduce

Proposed resolution

Let PathAlias implement the EntityChangedInterface and add a "changed" base field.

Remaining tasks

Upgrade path
Upgrade tests
Subsystem review
Code review

User interface changes

No UI changes.

API changes

PathAlias implements the EntityChangedInterface.

Data model changes

PathAlias has a "changed" base field.

Release notes snippet

✨ Feature request

Status

Needs work

Version

10.1 ✨

Component

Path →

Last updated 8 days ago

Maintained by
🇬🇧United Kingdom @catch

Created by

🇩🇪Germany mkalkbrenner 🇩🇪

Live updates comments and jobs are added and updated live.

Contributed project blocker
It denotes an issue that prevents porting of a contributed project to the stable version of Drupal due to missing APIs, regressions, and so on.

Needs upgrade path tests

Needs tests
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Needs Review Queue Initiative
Used to track the progress of issues reviewed by the Drupal Needs Review Queue Initiative.

Needs subsystem maintainer review
It is used to alert the maintainer(s) of a particular core subsystem that an issue significantly impacts their subsystem, and their signoff is needed (see the governance policy draft for more information). Also, if you use this tag, make sure the issue component is set to the correct subsystem. If an issue significantly impacts more than one subsystem, use needs framework manager review instead.

Incomplete comments

Merge Requests

!10954Add changed time tracking to path_alias
Open
🇨🇦Canada joseph.olstad
updated about 2 months ago

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment about 2 years ago →
🇺🇸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.

Moving to NW for the tags in #9, which I agree will need an upgrade path.

Tagging for subsystem review also just in case.
Comment about 2 months ago →
🇨🇦Canada joseph.olstad
joseph.olstad → made their first commit to this issue’s fork.
Merge request !10954Issue #3241056 by mbalkbrenner, joseph.olstad: Add changed time tracking to... → (Open) created by joseph.olstad
Pipeline finished with Failed
about 2 months ago
Total: 126s
#399854
Comment about 2 months ago →
🇨🇦Canada joseph.olstad
Patch is currently the same as the MR
Pipeline finished with Failed
about 2 months ago
Total: 496s
#399859

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024