Problem/Motivation

I sadly for got that we'll need (named) route parameters.

See https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Url.php/f...
We don't need options.

For example we need a node:123 parameter for entity.node.canonical route.

Tour module does it like this:

entity.node.canonical
- node:2

But I'd rather like to keep one route per line and the tour implementation also is no standard, but yml and I don't think it would be nice to change the logic here.

So here's my suggestion:
Let's split the route parameters by slash (/) and each key-value pair by ":", e.g.:

entity.node.canonical/node:123
entity.node.canonical/node:123/another_param:456

And hope there won't be any conflicts.

Code-wise we'll then have to explode by slash first and each result colon then!

Also please add a description to the field with explanation and these examples.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#7 screenshot-2025-12-08 10_34_47.png15.09 KBgrevil
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

anybody created an issue. See original summary.

anybody’s picture

Issue summary: View changes
anybody’s picture

Priority: Normal » Major

This is urgently needed.

grevil made their first commit to this issue’s fork.

grevil’s picture

Assigned: grevil » Unassigned
Status: Active » Needs review

Done. Please review!

Works as expected! On Login I get redirected to "https://standard-vscode-drupal.ddev.site/test/bla?another_param=456&chec...", when adding and using "entity.node.canonical/node:1/another_param:456|Test" in configuration.

grevil’s picture

StatusFileSize
new15.09 KB

Here are the new adjusted settings:
screenshot

anybody’s picture

Status: Needs review » Needs work

Very nice, will be merged once tests are green (#3561734: Beta release review)

anybody’s picture

Status: Needs work » Reviewed & tested by the community

Thanks, very nice work :)

  • anybody committed 2311a470 on 1.x authored by grevil
    feat: #3561742 Allow route parameters, e.g. for entity.node.canonical...
anybody’s picture

Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.