[PP-1] drupal_image() should include the cache tags for the image style

Added MR !20 to resolve for 3.x.

Also including a backported patch for 8.x-2.x, for those who may need it.

This totally makes sense, I just came across the same idea when working in ImageViewBuilder.php and seeing these lines I asked myself, if the style wasn't considered for caching...

CacheableMetadata::createFromRenderArray($build)
      ->addCacheableDependency($access)
      ->addCacheableDependency($file)
      ->applyTo($build);

Thanks for the MR and tests!

We should postpone this on 🐛 drupal_image() needs #width / #height values for fully working image cache scale, lazy-loading and img attributes RTBC for now, which fixes imagecache handling first.

I think 🐛 drupal_image() needs #width / #height values for fully working image cache scale, lazy-loading and img attributes RTBC is not relevant as it has nothing to do with caching.

I wonder if this issue is specific to S3FS?

Chandreshgiri Gauswami → made their first commit to this issue’s fork.