Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Comment | File | Size | Author |
---|---|---|---|
#3 | annotation_links_2409195_2_3.interdiff.txt | 2.47 KB | cgalli |
#3 | annotation_links_2409195_3.patch | 8.98 KB | cgalli |
#1 | annotation_links_2409195_2.patch | 7.58 KB | cgalli |
Comments
Comment #1
cgalli CreditAttribution: cgalli commentedChange in D8 Core: https://www.drupal.org/node/2382937
Changed links in the annotation and added links for view (canonical) and list (collection) for future use by core
Comment #2
BerdirThis will become a standard with #2401505: Add an entity collection template for lists , so it makes sense to follow that pattern already.
As shown to @dasjo before, you can simplify this to $entity->link($this->getLabel($entity)), and even the getLabel() could be removed, because the only thing that is doing is a checkPlain(), which we get by default by making it a link. So, just $entity->link() is in fact enough.
Comment #3
cgalli CreditAttribution: cgalli commentedShortened the link in listbuilder.
Added documentation for annotation link pattern
Comment #4
wesleydv CreditAttribution: wesleydv commentedTested and reviewed, seems ok to me
Comment #6
marvil07 CreditAttribution: marvil07 commentedThanks for changes and suggestions.
Comment #9
Berdir