- 🇨🇦Canada andrew.wang
I came here from Google - seems this is still the case in Drupal 10.3.10.
I found https://www.drupal.org/node/3013865 → and https://www.drupal.org/project/drupal/issues/1154180 → , but path alias still doesn't seem to show up when comparing revisions using the diff module. - First commit to issue fork.
- Merge request !105273321805 - Path alias Entity made revisionable → (Open) created by Unnamed author
- 🇮🇳India ramprassad
I have created an MR. Path alias is now made revisionable and revisions get created for the updates made. Please check
- 🇺🇸United States smustgrave
Issue summary is incomplete. but briefly looking at the change seems you're making user a hard dependency for path_alias now and not sure that's the route we want to take
- 🇮🇳India ramprassad
@smustgrave, I have removed some unwanted user dependencies in the tests, but this will be required for few tests as revision_uid field will be included for revisionable Path alias. Updated the issue summary and the MR. Please check.
- 🇺🇸United States smustgrave
Don’t think this is the right direction as it’s making user a dependency of path alias now
- Status changed to Needs work
about 1 month ago 5:35pm 3 March 2025 - 🇬🇧United Kingdom catch
This was partially done in #3007669: Add publishing status to path aliases → but I don't think we had all the various interfaces at that point.
@smustgrave can you explain why you think the user dependency is a problem? User is a required module so it's not really making the problem worse. If we really wanted to decouple revisions from the user module we'd need to find a way to handle revision_uid differently, but I think that could be done generically in its own ticket.
Left a comment on the MR - update path doesn't look right to me.
- 🇮🇳India ramprassad
@catch, Moving this to review as the comments in the MR are addressed. Please check.