[FOLLOWUP][BUGFIX] ImageViewHelper should catch exceptions

We also need to catch RuntimeException and InvalidArgumentException
InvalidArgumentException might be thrown if a storage does not exist.
RuntimeException if a file is outside a storage.

Change-Id: I994d8e7ef8b515f23ccc01847cefac0202f14b6f
Releases: master, 6.2
Resolves: #71686
Reviewed-on: https://review.typo3.org/45166


Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>

typo3/sysext/fluid/Classes/ViewHelpers/ImageViewHelper.php

@@ -140,7 +140,13 @@ class ImageViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\AbstractTagBasedV
                 $this->tag->addAttribute('title', $title);
             }
         } catch (ResourceDoesNotExistException $e) {
+            // thrown if file does not exist
         } catch (\UnexpectedValueException $e) {
+            // thrown if a file has been replaced with a folder
+        } catch (\RuntimeException $e) {
+            // RuntimeException thrown if a file is outside of a storage
+        } catch (\InvalidArgumentException $e) {
+            // thrown if file storage does not exist
         }
 
         return $this->tag->render();

typo3/sysext/fluid/Classes/ViewHelpers/Uri/ImageViewHelper.php

@@ -144,7 +144,13 @@ class ImageViewHelper extends AbstractViewHelper implements CompilableInterface
             $processedImage = $imageService->applyProcessingInstructions($image, $processingInstructions);
             return $imageService->getImageUri($processedImage, $absolute);
         } catch (ResourceDoesNotExistException $e) {
+            // thrown if file does not exist
         } catch (\UnexpectedValueException $e) {
+            // thrown if a file has been replaced with a folder
+        } catch (\RuntimeException $e) {
+            // RuntimeException thrown if a file is outside of a storage
+        } catch (\InvalidArgumentException $e) {
+            // thrown if file storage does not exist
         }
         return '';
     }