- Issue created by @ericdsd
- π©πͺGermany yoruvo Stuttgart
Looks like the lazy image loading stuff was only added in 10.1.x, so my mistake was applying the fix from #3322231 without specifying the different version required.
-
yoruvo β
authored bfa1bc35 on 8.x-1.x
Issue #3342428 by yoruvo: Warning rying to access array offset on value...
-
yoruvo β
authored bfa1bc35 on 8.x-1.x
- π©πͺGermany yoruvo Stuttgart
Could you check out the newest dev branch and report if that fixes it? If so, I can push the next release.
- π§πͺBelgium adiatis Belgium
dev branch solves the problem. Can you push to next release?
Thx
- πΊπΈUnited States glynster
The dev branch did not solve this issue for us on Drupal 9.5. We had to revert back to 1.2.
Here are the logs:
Message Deprecated function: pathinfo(): Passing null to parameter #1 ($path) of type string is deprecated in Drupal\image\Entity\ImageStyle->addExtension() (line 505 of /sites/dev/cms/releases/20230222200847/web/core/modules/image/src/Entity/ImageStyle.php)
Deprecated function: pathinfo(): Passing null to parameter #1 ($path) of type string is deprecated in _responsive_image_build_source_attributes() (line 380 of /sites/dev/cms/releases/20230222200847/web/core/modules/responsive_image/responsive_image.module)
Could not determine image width for '' using image style with ID: 1024. This image style can not be used for a responsive image style mapping using the 'sizes' attribute.
- πΊπΈUnited States jsutta United States
The dev branch worked for us on Drupal 9.5.3. Just to be completely sure, after I installed the dev version of the module, I rebuilt caches via drush and rechecked the page where I saw the warning. The warning didn't appear, and I didn't see anything in the logs or status report page.
- Status changed to Fixed
over 1 year ago 9:00pm 25 February 2023 - π©πͺGermany yoruvo Stuttgart
I suppose I'll consider this fixed, since #7 looks unrelated.
- πΊπΈUnited States glynster
@yoruvo I am still trying to debug this. Something has changed in the output which causes issues in Drupal 9.5.3.
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
over 1 year ago 2:11pm 19 June 2023 - π«π·France ericdsd France
Hi Yoruvo,
the issue by itself is indeed fixed, but it would be nice to provide a setting in media_responsive_thumbnail formatter to manage loading attribute.
Should i create a feature request for that ?