Bug #37671

ViewHelper exceptions should not be caught in testing context

Added by Sebastian Kurfuerst about 3 years ago. Updated about 3 years ago.

Status:Resolved Start date:2012-06-01
Priority:Should have Due date:
Assigned To:Sebastian Kurfuerst % Done:

100%
Category:Core
Target version:TYPO3 Flow Base Distribution - 1.1 RC1
Has patch:No Affected Flow version:Git master

Associated revisions

Revision 166e7e54
Added by Sebastian Kurfuerst about 3 years ago

[BUGFIX] ViewHelper exceptions should not be caught in testing context

While it is correct that ViewHelper exceptions are silently logged in
production context, they should be displayed in development and testing
context; and not only in development context.

Fixes: #37671
Releases: 1.1, 1.2
Change-Id: Ic7aab3a3a27f5d8333a98fc07647070d23f0f280

Revision 5a75ed88
Added by Sebastian Kurfuerst about 3 years ago

[BUGFIX] ViewHelper exceptions should not be caught in testing context

While it is correct that ViewHelper exceptions are silently logged in
production context, they should be displayed in development and testing
context; and not only in development context.

Fixes: #37671
Releases: 1.1, 1.2
Change-Id: Ic7aab3a3a27f5d8333a98fc07647070d23f0f280

History

#1 Updated by Gerrit Code Review about 3 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11787

#2 Updated by Gerrit Code Review about 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11787

#3 Updated by Sebastian Kurfuerst about 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#4 Updated by Gerrit Code Review about 3 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch FLOW3-1.1 has been pushed to the review server.
It is available at http://review.typo3.org/11995

#5 Updated by Sebastian Kurfuerst about 3 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF