- Status changed to Needs review
about 2 years ago 5:41pm 24 January 2023 - π«π·France dydave
Hi everyone,
Thank you very much for raising this issue.
We encountered the same problem running with:
- Drupal core: 9.5.2
- PWA: 8.x-1.5
- PHP: 8.1.14
and without any value forURLs to cache on install
on the admin settings page at/admin/config/pwa/settings
.Based on a quick search around we found several other modules having the same issue, in particular, see:
#3321432-2: Deprecated function: explode(): Passing null to parameter #2 ($string) of type string is deprecated on ProxySubscriber.php βTherefore attaching a patch with a preferred syntax to fix the problem:
File attached as: 3335176-6-php81-fix-warning-explode-null-deprecated.patch β$list = $string ? explode("\n", $string) : [];
The changes from the MR !30 above should work as well, so feel free to pick your preferred solution.
Any feedback, comments, suggestions or reviews would be greatly appreciated.
Thanks in advance. -
AlexBorsody β
committed e8180d4b on 8.x-1.x authored by
DYdave β
Issue #3335176 by _pratik_, DYdave, batkor: PHP 8.1: explode(): Passing...
-
AlexBorsody β
committed e8180d4b on 8.x-1.x authored by
DYdave β
- Status changed to Fixed
about 2 years ago 4:27pm 24 February 2023 - Status changed to Fixed
about 2 years ago 4:28pm 24 February 2023 - π©πͺGermany Anybody Porta Westfalica
Follow-up: π Remove patch file from the repo Fixed