You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
ResponsiveImageStyle:flush() is called when uploading a new file to Drupal's media library. At this point the file is in a temporary folder, no flush is necessary as no resized versions have been created yet. When a new RecursiveDirectoryIterator is created with the $base_style_path equal to "temporary://styles/responsive/temporary/" it provokes an Exception, and the catch below logs the exception needlessly in the error log (no harm has been done, the error is just noise).
The logged error description was also unhelpful, as it says "problem finding responsive image derivatives", perhaps "flushing" would be better than "finding" for when there is a genuine problem?
The text was updated successfully, but these errors were encountered:
ResponsiveImageStyle:flush()
is called when uploading a new file to Drupal's media library. At this point the file is in a temporary folder, no flush is necessary as no resized versions have been created yet. When a newRecursiveDirectoryIterator
is created with the$base_style_path
equal to"temporary://styles/responsive/temporary/"
it provokes an Exception, and the catch below logs the exception needlessly in the error log (no harm has been done, the error is just noise).Suggested solution:
Return early if
$source_scheme
is"temporary"
:The logged error description was also unhelpful, as it says "problem finding responsive image derivatives", perhaps "flushing" would be better than "finding" for when there is a genuine problem?
The text was updated successfully, but these errors were encountered: