When adding a new media image or editing an existing media image all image styles are deleted. This can cause a large performance issue as all the derivative images need to be rebuilt which is an expensive process and if you have many images using the same image style on a page this can lead to a server timeout.
Limit the deletion of the image styles to a path for a specific media item instead of the whole folder of image styles
Current code deleting whole image style folder
/**
* Flush contextuals derivative of the style.
*
* @param \Drupal\image\Entity\ImageStyle $imageStyle
* Image Style.
*/
public function flushStyle(ImageStyle $imageStyle) {
$folder_uri = 'public://contextual/styles/' . $imageStyle->id();
$this->fileSystem->deleteRecursive($folder_uri);
}
Purposed code to just target an individual media image
/**
* Flush contextual derivative of the style.
*
* @param \Drupal\image\Entity\ImageStyle $imageStyle
* Image Style.
* @param string|null $path
* (optional) The original image path or URI. If supplied, only derivatives
* for this specific image will be flushed.
*/
public function flushStyle(ImageStyle $imageStyle, $path = NULL) {
if ($path !== NULL) {
// Only flush derivatives for this specific image path.
// Build the contextual path pattern for this specific image.
$scheme = $this->streamWrapperManager::getScheme($path);
$target = $this->streamWrapperManager::getTarget($path);
$target_as_folder = str_replace('.', '__', $target);
$contextual_folder = 'public://contextual/styles/' . $imageStyle->id() . '/' . $scheme . '/' . $target_as_folder;
// Only delete the specific contextual derivatives for this image.
if (is_dir($contextual_folder)) {
$this->fileSystem->deleteRecursive($contextual_folder);
}
}
}
Active
2.1
Code
Here is a patch of the merge request to the 2.1.x branch
Issue #3533749 use path when flushing image styles