Bug #9210

imageViewHelper should not throw exception

Added by Felix Oertel over 11 years ago. Updated about 6 years ago.

Status:
Rejected
Priority:
Won't have this time
Assignee:
-
Category:
ViewHelpers
Target version:
-
Start date:
2011-10-15
Due date:
2011-10-15
% Done:

0%

Estimated time:
Has patch:

Description

Hey guys,

i don't like the imageViewHelper to throw exceptions if an image is not found. it should behave like the underlying cObj::getImgResource() and just don't do anything.

regards, foertel


Files

9210.diff (676 Bytes) 9210.diff Felix Oertel, 2010-08-09 12:37

Related issues

Related to TYPO3.Fluid - Feature #9211: Improve ViewHelper exception handlingRejected2010-08-09

Actions
Related to TYPO3 Core - Bug #61861: Frontend throws exception on missing imageClosed2014-09-24

Actions
#1

Updated by Felix Oertel over 11 years ago

#2

Updated by Bastian Waidelich over 11 years ago

  • Project changed from 534 to TYPO3.Fluid
  • Target version deleted (Extbase 1.3.0beta1)
#3

Updated by Bastian Waidelich over 11 years ago

  • Category set to ViewHelpers
  • Branch set to v4
#4

Updated by Felix Oertel over 11 years ago

thanks a bunch ;-)

#5

Updated by Bastian Waidelich over 11 years ago

  • Status changed from New to Rejected
  • Priority changed from Should have to Won't have this time

Currently all ViewHelper exceptions are catched and their message is outputted as string.
The corresponding code snippet in :

try {
    $output = call_user_func_array(array($viewHelper, 'render'), $renderMethodParameters);
} catch (Tx_Fluid_Core_ViewHelper_Exception $exception) {
        // @todo [BW] rethrow exception, log, ignore.. depending on the current context
    $output = $exception->getMessage();
}

As you can see, we're planing to improve the exception handling here. See #9211 for more details.

I'm closing this issue for now, feel free to reopen if you don't agree ;)

#6

Updated by Tizian Schmidlin about 6 years ago

This is still an issue in 6.2.16, could someone take care of it for the next patch set?

Also available in: Atom PDF