Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
Path aliases have been converted to entities in Drupal 8.8: https://www.drupal.org/node/3013865
Since, the path alias forms will also be converted to entity forms in #3007832: Convert custom path alias forms to entity forms, this module will need a few adjustments to in order to keep it working with the latest Drupal minor version.
Proposed resolution
Figure out what changes are needed, preferably in a way that keeps the module compatible with Drupal 8.7 and below as well.
Remaining tasks
TBD.
User interface changes
Nope.
API changes
TBD.
Data model changes
TBD.
Comments
Comment #2
manuel.adanThanks for notifying!
I've been following #2336597: Convert path aliases to full featured entities, I tested the module against the latest D8.8 dev right once it was committed. As the fixed_path_alias module operates by decorating the path alias manager, the changes do not affect its functionality.
About #3007832: Convert custom path alias forms to entity forms, I still didn't any test, I prefer to wait for the CR to be published.
Comment #3
manuel.adan#3007832: Convert custom path alias forms to entity forms was committed just a few minutes after I sent my previous comment. This is a quick minimal first approach that makes the module aware of the path aliases changes introduced in the Drupal 8.8 branch, without breaking compatibility with previous versions. An entire module review/refactoring is pending, but the module usage is still low and, as every body, I run short of time.
Comment #4
manuel.adanIntroduced changes makes the module compatible only with D8.8+ versions. A 2.x branch should be needed only if a new release of the module were launched within the D8.7 maintenance period.
Comment #5
manuel.adanFixing malformed patch #4.
Comment #6
ralphvdhoudt CreditAttribution: ralphvdhoudt at iO commentedPatch that can be used against beta1 and issue #3013926
Comment #7
ralphvdhoudt CreditAttribution: ralphvdhoudt at iO commentedPatch that can be used against beta1 zip file and issue #3013926
Comment #8
manuel.adanNow than D8.7 is officially unsupported we can safely remove any code that ensures compatibility with D < 8.8, gaining compatibility with D9 by the way. Old path alias hook implementations has been replaced by their entity based equivalent. Interdiff fails due changes in base code.
Comment #10
manuel.adan