Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
Core Configuration Translation fails with following error: InvalidArgumentException: Link templates accepts paths, which have to start with a leading slash. in Drupal\Core\Entity\EntityType->setLinkTemplate()
Proposed resolution
Update annotation (see #5) or implement hook_entity_type_build
Remaining tasks
User interface changes
API changes
Original report by @dmitrii
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 2424441-5.patch | 2.18 KB | dmitrii |
| #3 | 2424441-2.patch | 1.59 KB | dmitrii |
Comments
Comment #1
dmitrii commentedComment #3
dmitrii commentedComment #4
berdirI don't understand what this is supposed to achieve, I added the issue summary template, please fill it out. All those values are set as part of the annotation, there is no reason to duplicate it?
Comment #5
dmitrii commentedProposed annotation change.
Comment #6
dmitrii commentedComment #7
dmitrii commentedComment #10
berdirAh, now I get what you are trying to do. Updating issue title. No idea how that's still working in HEAD?
The test fail is unrelated, and fixed in #2425921: CommentManager::addDefaultField() moved to CommentTestTrait (and other test fixes).
Comment #11
berdirComment #12
drunken monkeyApparently, we're currently not testing that. Although I'm surprised myself that, e.g., the integration test doesn't ever use this …
Adding the "Needs tests" tag.
Anyways, thanks a lot for the patch, seems to work great! Committed.
For others' reference, the Core issue that introduced the change is this one: #2281645: Make entity annotations use link templates instead of route names.
Comment #14
drunken monkey