- Issue created by @andypost
- Status changed to Needs review
almost 2 years ago 6:48pm 19 February 2023 - π«π·France andypost
Filed CR and patch, not sure if deprecation needs separate method as
generate()
also should become internal - Status changed to Needs work
almost 2 years ago 9:40pm 19 February 2023 - π¬π§United Kingdom catch
I think this needs to point to Link::fromTextAndUrl() for the alternative so it's not @internal. But otherwise looks good.
https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Link.php/...
- Status changed to Needs review
almost 2 years ago 8:15am 20 February 2023 - π«π·France andypost
It can't point to
Link::fromTextAndUrl()
because this method returnsGeneratedLink
object butLink
is renderable element - Status changed to RTBC
almost 2 years ago 8:59am 20 February 2023 - π¬π§United Kingdom catch
Oh now I see... this is what gets called by Link::toString().
So all we're doing is removing an internal wrapper for another internal method, it's slightly odd that the deprecation points there, but in practice people should already not be using this, so they won't see it, and if they are, then worst thing is they use ::generate() instead which will be no worse than they're already doing.
- Status changed to Needs review
almost 2 years ago 3:02am 23 February 2023 - π³πΏNew Zealand quietone
The deprecation message for a method β does not start with 'The'. And I was getting a phpcs error on the deprecation message in the interface.
I updated the patch.
- Status changed to RTBC
almost 2 years ago 7:49am 23 February 2023 - π¬π§United Kingdom longwave UK
Committed and pushed to 10.1.x, thanks!
- Status changed to Fixed
almost 2 years ago 2:05pm 23 February 2023 -
longwave β
committed 88834467 on 10.1.x
Issue #3342991 by quietone, andypost, catch: Deprecate...
-
longwave β
committed 88834467 on 10.1.x
Automatically closed - issue fixed for 2 weeks with no activity.