Problem/Motivation

As of https://docs.google.com/document/d/1BxgNvyIRcxYGzzA1DLkZ6A4-c8gGFixypNvP... we want to make clear which bits of the URL generation
machinery should be used

Proposed resolution

* Mark \Drupal\Core\Routing\UrlGeneratorTrait and \Drupal\Core\Routing\UrlGeneratorTrait::url() as deprecated
* Mark \Drupal\Core\Routing\LinkGeneratorTrait and \Drupal\Core\Routing\LinkGeneratorTrait::l() as deprecated
* It should point to \Drupal\Core\Url | \Drupal\Core\Link instead

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 2606396-4.patch1.76 KBsdstyles

Comments

dawehner created an issue. See original summary.

dawehner’s picture

dawehner
Primary language German
commented
Parent issue: #2606392: mark \Drupal\Core\Utility\LinkGeneratorInterface::generatFromLink as internal » #2491981: There are too many ways to generate URLs and links
dawehner’s picture

dawehner
Primary language German
commented
Component: base system » documentation
sdstyles’s picture

sdstyles commented
Status: Active » Needs review
StatusFileSize
new 2606396-4.patch1.76 KB
dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Thank you!

+1

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 4: 2606396-4.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs work » Reviewed & tested by the community

Seriously.

  • xjm committed 2654e24 on 8.1.x 
    Issue #2606396 by sdstyles, dawehner: mark \Drupal\Core\Routing\...

  • xjm committed f3e1ffc on 
    Issue #2606396 by sdstyles, dawehner: mark \Drupal\Core\Routing\...
xjm’s picture

xjm
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Added the missing word "and" in a couple spots, and then committed and pushed to 8.1.x and 8.0.x. Thanks!

xjm’s picture

xjm
she/her
Primary language English
commented
Component: documentation » routing system

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

tr’s picture

tr commented
Issue summary: View changes

Corrected issue summary.