We use LinkGeneratorInterface::generate in the LinksWidget. We should instead be using the \Drupal\Core\Link value object.
The widget should be updated and that means the test case can be simplified as well. This is only needed for the linksWidget.

CommentFileSizeAuthor
#2 remove_usage_of-2651952-2.patch4.39 KBborisson_
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

borisson_ created an issue. See original summary.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ for Acquia commented
Status: Active » Needs review
FileSize
remove_usage_of-2651952-2.patch4.39 KB
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ for Acquia commented
Status: Needs review » Fixed

  • borisson_ committed 9e4b957 on 8.x-1.x 
    Issue #2651952 by borisson_: Remove usage of LinkGeneratorInterface::...

Status: Fixed » Needs work

The last submitted patch, 2: remove_usage_of-2651952-2.patch, failed testing.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ for Acquia commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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