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.
Problem/Motivation
We want to replace the usages because:
* @deprecated in Drupal 8.0.x-dev and will be removed before Drupal 8.0.0.
* To generate URLs for Drupal routes (that is, most pages generated by
* Drupal), see UrlGeneratorInterface::generateFromRoute() instead. For
* non-routed local URIs relative to the base path (like robots.txt) see
* \Drupal\Core\Utility\UnroutedUrlAssembler.
By using route names consistently, we can avoid needing to change tests if we change route paths
Proposed resolution
Replace path with route names in a Url object when it's pointing to a route
Remaining tasks
User interface changes
API changes
Comment | File | Size | Author |
---|---|---|---|
#6 | remove_calls_to-2425901-6.patch | 16.9 KB | Anonymous (not verified) |
#6 | interdiff-4-6.txt | 12.65 KB | Anonymous (not verified) |
#4 | interdiff-2425901-1-4.txt | 517 bytes | mitrpaka |
#4 | remove_calls_to-2425901-4.patch | 4.25 KB | mitrpaka |
#1 | 2425901-1.patch | 4.93 KB | dawehner |
Comments
Comment #1
dawehnerHere are some of them removed.
Comment #2
klausiwhy? please update the issue summary.
Comment #3
dawehnerComment #4
mitrpaka CreditAttribution: mitrpaka commentedUpdated patch due to #2470980: Use $this->redirect() instead of ResponseRedirect in LocaleController::checkTranslation
Comment #5
pwolanin CreditAttribution: pwolanin at Acquia commentedComment #6
Anonymous (not verified) CreditAttribution: Anonymous at XIO commentedI looked into removing some more calls, but I'm wondering whether this approach is ok or not?
Comment #10
pwolanin CreditAttribution: pwolanin at Acquia commentedI don't think that's the right approach. We should be giving this a route or URL
Comment #11
r_sharma08 CreditAttribution: r_sharma08 at Publicis Sapient for Publicis Sapient commentedComment #12
r_sharma08 CreditAttribution: r_sharma08 at Publicis Sapient for Publicis Sapient commentedComment #13
pwolanin CreditAttribution: pwolanin at Acquia commentedduplicate to #2575869: Remove all remaining usage of deprecated UrlGeneratorInterface::generateFromPath() and delete it
Comment #14
pwolanin CreditAttribution: pwolanin at Acquia commentedComment #15
tim.plunkett