Fix Workspaces' hidden dependency on the path_alias module

Created on 10 April 2024, 2 months ago
Updated 26 April 2024, about 2 months ago

Problem/Motivation

As "planned" ~5 years ago in #3092090-13: Remove legacy Path Alias subsystem β†’ , we should remove the hidden dependency of workspaces on the path_alias module.

Steps to reproduce

Check any kernel test from the workspaces module and observe that path_alias needs to be installed even though the test doesn't have any assertions related to path aliases.

Proposed resolution

Decorate the path_alias.manager service optionally, so we can add the current workspace ID to its cache key.

Remaining tasks

Review.

User interface changes

Nope.

API changes

Not really.

Two constructor parameters are removed from \Drupal\workspaces\EventSubscriber\WorkspaceRequestSubscriber, but that's an event subscriber class so it's considered internal.

Data model changes

Nope.

Release notes snippet

Nope.

πŸ“Œ Task
Status

Fixed

Version

10.3 ✨

Component
WorkspacesΒ  β†’

Last updated 7 days ago

No maintainer
Created by

πŸ‡·πŸ‡΄Romania amateescu

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.69.0 2024