- Issue created by @fago
- 🇷🇸Serbia petar_basic
Here's a patch to take into account the optional path parameter.
- Status changed to Needs review
7 months ago 2:56pm 10 April 2024 - Status changed to RTBC
7 months ago 1:25am 23 April 2024 - 🇳🇿New Zealand ericgsmith
Found this issue while investigating a performance issue on a site when using views bulk operations to bulk publish media.
Thank you @fago - 🐛 Uploading a file to media library flushes theme registry Active and this helped pinpoint some of what I was seeing 😱
Tested patch - code change looks good and fixes the issue.
- 🇮🇹Italy mondrake 🇮🇹
Thank you. Maybe it's not strictly necessary, but let's wait for minimum Drupal core to be 10.2 as set by 📌 [Wait for D10.2 as min version] Fix D10.2 deprecations Postponed before committing this.
- 🇮🇹Italy mondrake 🇮🇹
string $path = NULL
is an implicit nullable, deprecated in PHP 8.4, see https://php.watch/versions/8.4/implicitly-marking-parameter-type-nullabl...Fix on commit if not fixed before.
-
mondrake →
committed 08c53e4a on 8.x-3.x
Issue #3439990 by mondrake, petar_basic, fago, ericgsmith: With 10.2...
-
mondrake →
committed 08c53e4a on 8.x-3.x
- Status changed to Fixed
7 months ago 5:30pm 29 April 2024 Automatically closed - issue fixed for 2 weeks with no activity.