- First commit to issue fork.
- Merge request !5849Resolve #3245249 "Replace fileuploadhandlerloadbyuri with FileRepositoryInterface::loadByUri" β (Closed) created by dimitriskr
- Status changed to Needs review
about 1 year ago 8:59pm 17 December 2023 - π¬π·Greece dimitriskr
I don't think we eventually need a CR, right? This method is protected
- Status changed to Needs work
about 1 year ago 2:42pm 18 December 2023 - πΊπΈUnited States smustgrave
See a few more instances of ->loadByUri in core. Also issue summary needs to updated to include that the proposed solution is to deprecate and API changes.
There only appear to be about 10 instances in core. Wonder if this counts as deprecating in D11.
- Status changed to Needs review
about 1 year ago 8:58pm 21 December 2023 - Status changed to RTBC
about 1 year ago 5:05pm 22 December 2023 - πΊπΈUnited States smustgrave
Based on slack conversation think this is good.
-
longwave β
committed 52948158 on 11.x
Issue #3245249 by dimitriskr, smustgrave, kim.pepper: Replace...
-
longwave β
committed 52948158 on 11.x
- Status changed to Fixed
12 months ago 2:02pm 18 February 2024 - π¬π§United Kingdom longwave UK
Yes, whoops, done that as well - thanks!
Automatically closed - issue fixed for 2 weeks with no activity.