TypeError: explode(): Argument #2 ($string) must be of type string, array given in explode()

Raveen Thakur → made their first commit to this issue’s fork.

Still not able to reproduce, what/how happened here!!

Mohd Sahzad → made their first commit to this issue’s fork.

@Mohd, You are checking case only when it comes as string but, the error raised is for -
$libraries = $view_config['display_options']['display_extenders']['library_in_views_display_extender']['attach_library'];
if (is_string($libraries)) {

for second argument, If the error persists, (still i am not able to reprouce) The code won't goes upto your changes. It still shows error.
Could you please let me know more about error, how you reproduced it. I was keeping an eye on it from yesterday.
Can you address the error for me. Please & Thank You.

Can you please share your test results?? Thank You!!

MIstakenly updated/added comment in ticket bio.

Hmm I'm retesting carefully with drupal 10.2 PHP8.2 but I can't reproduce this problem

lazzyvn → closed merge request !3

This occurs on my site as well. You can replicate it by installing version 2.0.3, saving a view with a value in the attach_library field, and then upgrading the module to 2.0.4. That version changed the array lookup where the string is stored (it removed a second `attach_library`) but didn't update any existing information. I suggest adding an update hook to fix the data stored in every view.

@david.muffley Yes, you're right. there are new fixes in the development version, which will make it work, no need to create an update hook to fix all views. please confirm

The new 2.0.5 release fixed this issue where I was seeing it.