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.
Currently the user-link/user-cancel-link/user-edit-link still uses just the UID and constructs the uri for itself, but there is no reason for them to do that,
as the entities are loaded already anyway.
Comment | File | Size | Author |
---|---|---|---|
#10 | 1859884-followup.patch | 776 bytes | tstoeckler |
#7 | drupal-1859884-7.patch | 6.24 KB | dawehner |
#7 | interdiff.txt | 755 bytes | dawehner |
#3 | interdiff.txt | 3.32 KB | dawehner |
#3 | drupal-1859884-3.patch | 6.23 KB | dawehner |
Comments
Comment #1
dawehnerAdapted the parameters so it's way easier to get what is going on.
Comment #3
dawehnerSome improvements.
Comment #5
dawehner#3: drupal-1859884-3.patch queued for re-testing.
Comment #6
tim.plunkettIt's sad to see a stdClass here, but it was actually always there, just not typehinted. So, fine.
Someday we'll write a helper to just get the 'path' part :)
Any reason to not typehint this as well?
Or leave it out altogether, it doesn't seem to be 100% related
Comment #7
dawehnerLet's do that.
Comment #8
tim.plunkettLooks good to me!
Comment #9
catchYep. Committed/pushed to 8.x.
Comment #10
tstoecklerI hope it's OK that I re-open this issue for this, but here's a quick follow-up to fix the wrong indentation in that hunk.
Comment #11
dawehnerSure, this looks perfect.
Comment #12
catchCommitted/pushed the follow-up.