Commit d416cff0 authored by Georg Ringer's avatar Georg Ringer Committed by Oliver Hader
Browse files

[BUGFIX] Avoid timeout issues through SvgFilesSanitization

Skip a real check of the file system for SVG files in advance
in the SvgFilesSanitization wizard to avoid timeouts in e.g. the reports
module.

Resolves: #94801
Releases: master, 11.3, 10.4, 9.5
Change-Id: I4ed52d357effec4a8e698d5b117f024150a01beb
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70448


Tested-by: core-ci's avatarcore-ci <typo3@b13.com>
Tested-by: Wouter Wolters's avatarWouter Wolters <typo3@wouterwolters.nl>
Tested-by: Oliver Hader's avatarOliver Hader <oliver.hader@typo3.org>
Reviewed-by: Wouter Wolters's avatarWouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Benni Mack's avatarBenni Mack <benni@typo3.org>
Reviewed-by: Oliver Hader's avatarOliver Hader <oliver.hader@typo3.org>
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70534
parent 6c634fc7
......@@ -84,26 +84,13 @@ class SvgFilesSanitization implements UpgradeWizardInterface, ConfirmableInterfa
}
/**
* Is an update necessary?
*
* Is used to determine whether a wizard needs to be run.
* Check if data for migration exists.
* To avoid timeout issues, no check is performed in advance
*
* @return bool
*/
public function updateNecessary(): bool
{
foreach ($this->resolveLocalStorages() as $storage) {
try {
$svgFiles = $this->resolveSvgFiles($storage);
} catch (InsufficientFolderAccessPermissionsException $exception) {
continue;
}
if (count($svgFiles) > 0) {
return true;
}
}
return false;
return true;
}
/**
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment