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:2But 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:456And 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
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | screenshot-2025-12-08 10_34_47.png | 15.09 KB | grevil |
Issue fork user_preference_login_redirect-3561742
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
Comment #2
anybodyComment #3
anybodyThis is urgently needed.
Comment #5
grevil commentedDone. 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.
Comment #7
grevil commentedHere are the new adjusted settings:

Comment #8
anybodyVery nice, will be merged once tests are green (#3561734: Beta release review)
Comment #9
anybodyThanks, very nice work :)
Comment #11
anybody