Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
------ -------------------------------------------------------------------------------
Line src\Controller\ConfigInspectorController.php
------ -------------------------------------------------------------------------------
151 Call to deprecated method l() of class Drupal\Core\Controller\ControllerBase:
in Drupal 8.0.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Link instead.
------ -------------------------------------------------------------------------------
Comment | File | Size | Author |
---|---|---|---|
#15 | 3076794-15.patch | 1.31 KB | idebr |
| |||
#15 | interdiff-12-15.txt | 774 bytes | idebr |
Comments
Comment #3
vuilComment #4
idebr CreditAttribution: idebr at ezCompany commentedThe config inspector overview now generates a an error:
\Drupal::l()
returns a string, whileLink::createFromRoute()
returns an object, so you have to calltoString()
on it to get a string. See https://www.drupal.org/node/2614344 for more information on this deprecation.Comment #6
vuilOh, yes. Thank you @idebr!
Update the issue solution (commit).
Comment #7
idebr CreditAttribution: idebr at ezCompany commentedThis is still broken:
Link::createFromRoute()
takes a route name as its second argument, but it now gets a Url object. This returns a fatal error:Comment #8
vuilComment #10
vuilThank you for the emergency notice!
I committed the new update. I already checked it twice.
Please confirm by your side.
Thank you.
Comment #11
idebr CreditAttribution: idebr at ezCompany commentedNope, still broken:
Comment #12
alan-ps CreditAttribution: alan-ps as a volunteer and at Drupal Ukraine Community commentedHello, I noticed the same issue on my side. This patch should help :)
Comment #13
vuilComment #15
idebr CreditAttribution: idebr at ezCompany commented#12 Thanks, the patch fixes the fatal error.
Attached patch includes a revert of the commit in #9 to fix the automated tests.
Comment #17
vuilCommitted! Thank you, guys.
Comment #19
Kristen PolPer a Slack discussion with Gábor Hojtsy regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing tag cleanup here based on that discussion.
Comment #20
vuil