Problem/Motivation
This issue addresses the same issue as Prepare for the path alias changes in Drupal 8.8 except this patch is for the 8.x-1.x version of the module in Drupal 8.8.x.
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 version.
Proposed resolution
Change references from url_alias to path_alias. Clean up hook alters.
Remaining tasks
UAT for users using this module, Drupal 8.8.x, and the pathauto module, version 8.x-1.6.
Data model changes
- Change references from url_alias to path_alias.
- Remove hook alters to
path_admin_addandpath_alias_deletewhich no longer exist in Drupal 8.8.x. - Change $context from
sourcetopathin views_url_alias_node_pathauto_alias_alter().
| Comment | File | Size | Author |
|---|---|---|---|
| #22 | 3104606-21-drupal-9-fork.patch | 4.46 KB | klimp |
| #21 | 3104606-21.patch | 5.53 KB | el1_1el |
| #15 | 3104606-15.patch | 4.07 KB | esod |
| #14 | 3104606-13.patch | 523 bytes | esod |
| #12 | 3104606-12.patch | 4.07 KB | esod |
Comments
Comment #2
esod commentedComment #3
esod commentedComment #4
esod commentedpatch attached.
Comment #5
esod commentedComment #6
esod commentedComment #7
esod commentedComment #8
esod commentedComment #9
esod commentedOn second thought, because of Path aliases have been converted to revisionable entities, let's leave the alter hooks in for the time being.
Comment #10
esod commentedContinue to include the the 8.8 dependency. D'oh!
Comment #11
esod commentedLeft out the the url_alias to path_alias changes in #10.
Comment #12
esod commentedContinuing on with the Drupal 8.8.x changes for the 8.x-1.0 version of the module:
path.alias_storagetoentityTypeManager.Comment #13
taivu commentedpatch #11 isnt working for me. the table `views_url_alias_node` was empty.
looking at the function `views_url_alias_node_rebuild()`, i was able to populate the table by changing the join() method's parameters from `source` to `path`.
should we also be changing all `source` to `path`? question arises from seeing this:
Comment #14
esod commentedChanging pa.source to pa.path.
Comment #15
esod commentedForget #14. The whole patch, this time with pa.source changed to pa.path in the query.
Comment #16
taivu commentedpatch from #15 works for me! everything seems like it was before the v8.8 update. Thanks @esod!
Comment #17
mxwright commentedComment #18
mxwright commentedPatch at #15 works for me as well, on 8.8.1
Comment #19
mortona2k commented#15 works and looks good to me.
Comment #20
bdanin commentedAny way this can be added to a release, or merged to the dev branch? I want to also apply the Drupal 9 patch (https://www.drupal.org/project/views_url_alias/issues/3142138), and this is blocking us because we need this as well. Patch from #15 has been working well for me for quite some time.
Comment #21
el1_1el commentedI encountered the same issue as #20. Here is a merged patch with 9 changes from 3142138
Comment #22
klimpUpdated the patch from #21 to be compatible with the Drupal 9 issue fork https://git.drupalcode.org/issue/views_url_alias-3142138/-/tree/3142138-... . The fork already includes changes required for Drupal 9 as well as some other fixes from the patch above, so the #21 patch didn't apply against that branch
Comment #23
el1_1el commentedconfirming that patch in 22 works with issue branch views_url_alias-3142138. maybe someday we can go back to normal... doubt it but hoping
Comment #24
_shyComment #25
_shyCouldn't apply a patch to the 8.x-2.x-dev branch.
Comment #26
kpaxman commentedThis ticket specifically says it is for the 8.x-1.x branch, so setting back to that.
The matching issue for 8.x-2.x seems to be #3086817: Prepare for the path alias changes in Drupal 8.8.
FWIW, regardless of this, I was unable to get either patch to apply against either version.
Comment #27
dstorozhukDo we need that task taking into account that branch 8.x-1.x-dev is for Drupal before 8.8, and and this version of the Drupal is not maintained any more?
I propose to close the Issue, and ones that need that fix for their old Drupal 8 sites - can apply patch.
Comment #28
rachel_norfolkHaving had no replies in a long time, I think we can close this as outdated.