Motivation

A requirement for experience builder outlined in #3484255: [PP-1] Support adding additional routes for view modes other than 'full' is to have alternative canonical routes, where the routes are tied to configured view modes. The view modes would have configured paths, just like entities have a canonical path.

This issue exists to add a system for custom paths for alternate canonical routes.

View mode variants will be added in #3484255: [PP-1] Support adding additional routes for view modes other than 'full'.

Contrib impact

  • Pathauto would be able to hook in with their own patterns.
  • Subpathauto would be able to register patterns in a performant way. E.g [entity:title]/customPath. (Computation of "subpaths" can happen beforehand, instead of at runtime)

I suggest this issue as a predecessor to #3484255: [PP-1] Support adding additional routes for view modes other than 'full', since I dont think it makes sense to tie internals of Path/PathAlias, especialy PathAlias entity, PathItem, PathFieldItemList, and PathWidget, with concepts like Entity View Modes. Especially since there should be contrib hookability. And be able to integrate with Core concepts like Entity Form Modes in the future.

Doing all this before #3484255: [PP-1] Support adding additional routes for view modes other than 'full' allows its view mode integration to stay very small. I've created a MR in that issue that stacks on the MR here @ !10349.

What this will not do?

Proposed resolution

  • Implements an event system for determining valid variants for an entity bundle. This equates to entity view modes as they are bundle specific.
  • Implements an event system that determines the internal paths and variants for an entity. And can do so whether the entity has been saved or not.
  • Utilizes enums, so variants can be defined by Enums, in addition to strings.
  • PathItem/PathFieldItemList already calls out to the container a bunch, so the changes here to a different service are not much different.

Remaining tasks

Architectural and technical review
Add coverage for new systems
Test passing

User interface changes

A path fieldset is displayed PER variant.

Introduced terminology

  • Path variants

API changes

  • API surface additions just for variants.
  • New events dispatched
  • Existing path_alias.repository and its interface had a change to \Drupal\path_alias\AliasRepository::lookupBySystemPath method signature to allow optional variant lookups.

Data model changes

  • Database schema changed: Introduces a `variant` field (database column) to PathAlias. All existing pathalias have a NULL value, which equates to the "default" variant.
  • No config schema changes.

Release notes snippet

CommentFileSizeAuthor
#3 mess.png150.42 KBdpi

Issue fork drupal-3489872

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

dpi created an issue. See original summary.

dpi’s picture

StatusFileSize
new150.42 KB

Im currently looking for architectural review, particularly from XB stakeholders, before tests are added.

PHPStan and linting has been much improved, to a level above our formal required minimum level 1.

The change to make path fields multi cardinality impacts serialization and JSONAPI especially, and their related tests.

Some tests are updated, a bunch of random test failures still come up recently..

Some of @larowlans work from #3484255: [PP-1] Support adding additional routes for view modes other than 'full' contained changes to path UI. I couldnt tell which direction he was going as the wip commit was a bit messed up:

screenshot

So I went with a details/fieldsets per path variant instead of a major change to make the existing one fieldset to contain a tabular layout. Something to discuss further anyway. Or even improve in a follow up.

Theres something up with the path.module postupdate from #3478240: Improve performance for path_alias queries in a workspace. Its not updating the index properly with the variant hook_update at the same time. So I've moved it to a hook_update just for the purposes of getting tests going. I'd appreciate advice there.

dpi’s picture

Issue summary: View changes
dpi’s picture

Issue summary: View changes
dpi’s picture

From the contrib issue @ #3481362: Add path/pathauto integration, we might be able to do link templates + associated toUrl bits here.

dpi’s picture

Assigned: dpi » Unassigned
Status: Active » Needs review
dpi’s picture

Issue tags: +Singapore2024
larowlan’s picture

damienmckenna’s picture

Would we need safeguards to avoid someone adding "edit" as a view mode, which would lead to two possible route matches for "node/$nid/edit"?

smustgrave’s picture

Status: Needs review » Needs work

Appears to need a rebase

larowlan’s picture

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.