Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
When I am running drush image:flush --all. We are receiving below errors;
[warning] Undefined array key 2 LibraryStream.php:36
[warning] The following theme is missing from the file system: styles ExtensionPathResolver.php:63
[warning] The following theme is missing from the file system: styles ExtensionPathResolver.php:63
[warning] Array to string conversion LocalStream.php:122
I found web/modules/contrib/patternkit/src/StreamWrapper/LibraryStream.php file is causing this warning message. This below line of code is expecting the $uri should contain minimum two "/" however it received one "/" for which it's triggering these warning messages.
[$extension, $library, $type_level] = explode('/', $uri);
Steps to reproduce
Run drush image:flush --all in your Drupal setup
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | Retest.png | 42.45 KB | minsharm |
| #3 | image.png | 129.89 KB | minsharm |
| #2 | LibraryStream.php_.patch | 1.14 KB | pravat231 |
| Screenshot 2023-12-05 at 3.13.57 PM.png | 333.64 KB | pravat231 |
Issue fork patternkit-3407141
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
pravat231 CreditAttribution: pravat231 for Red Hat commentedComment #3
minsharm CreditAttribution: minsharm at Red Hat commentedI tried to reproduce the issue and getting the same warning message
Comment #6
sluceroI've reviewed the code change here and it looks good to me. I was able to reproduce the issue locally, and applying the patch seems to resolve it without any adverse issues I ran into. I've applied the patch onto an MR for testing before getting this merged in.
Thanks for the work @pravat231!
Comment #7
minsharm CreditAttribution: minsharm at Red Hat commentedRetested the issue and result looks good to me.
Result : Not getting the warning message anymore.
Comment #9
sluceroMerged for inclusion in the Beta 9 release. Thanks for the contribution!