Problem/Motivation

Once path aliases are converted to entities (#2336597: Convert path aliases to full featured entities), we should deprecate the existing hooks hook_path_insert(), hook_path_update and hook_path_delete() in favor of generic entity hooks.

Proposed resolution

Use \Drupal\Core\Extension\ModuleHandlerInterface::invokeAllDeprecated() in \Drupal\Core\Path\PathAliasStorage::invokeHook() and replace all usages of these hooks.

Remaining tasks

All the above.

User interface changes

Nope.

API changes

Nope.

Data model changes

Nope.

CommentFileSizeAuthor
#3 3011003-3-combined.patch90.44 KBamateescu
#2 3011003-combined.patch90.74 KBamateescu
#2 3011003-for-review.patch10.17 KBamateescu

Comments

amateescu created an issue. See original summary.

amateescu’s picture

amateescu commented
Status: Active » Needs review
StatusFileSize
new 3011003-for-review.patch10.17 KB
new 3011003-combined.patch90.74 KB

This should do it.

amateescu’s picture

amateescu commented
Status: Needs review » Postponed
StatusFileSize
new 3011003-3-combined.patch90.44 KB

Rerolled the combined patch, the "for review" one is the same.

amateescu’s picture

amateescu commented
Status: Postponed » Closed (outdated)

This patch was folded into #2336597-92: Convert path aliases to full featured entities.

amateescu’s picture

amateescu commented
Parent issue: #3007661: Modernize the path alias system »