[TYPO3-core] RFC #9355: Add an error and exception handler (backported from FLOW3)
Rupert Germann
rupi at gmx.li
Fri Sep 25 17:18:00 CEST 2009
hi,
after further reading I revoke my +1
php manual says for function set_error_handler():
--
If the function returns FALSE then the normal error handler continues.
--
we should do this, too.
concerning the @ singn php-manual says:
Of particular note is that this [error_reporting] value will be 0 if the
statement that caused the error was prepended by the @ error-control
operator.
but then our error handler should not output anything.
the initialisation of the error handler is a bit weird - should not be
done in function setExceptionalErrors()
I changed this in the attached patch,
but I've still no idea how about get around that @ problem
greets
rupert
Rupert Germann schrieb:
> Ernesto Baschny [cron IT] schrieb:
> ....
>> Just remove the line that overwrites the exceptionalErrors. The default
>> is E_ALL&~(E_NOTICE|E_WARNING), which will exclude warnings from the
>> exception handling, because this might lead to stuff like you noticed.
>> :) We are still not "warning-free" in a default TYPO3 installation.
>
> just noticed that the warnings from the exceptionhandler ignore @ signs.
> e.g.: I get a waring when I open a tt_content record for editing:
>
> t3lib_error_ErrorHandler::handleError(2, "in_array() [<a
> href='function.in-array'>function.in-array</a>]: Wrong datatype for
> second argument",
> "/srv/SVN/forge.typo3.org/Core/trunk/t3lib/class.t3lib_tceforms.php",
> 3769, array)
>
> and the mentioned line says:
>
> && (!$wConf['enableByTypeConfig'] ||
> @in_array($wid,$specConf['wizards']['parameters']))
>
> I know that there are quite a lot of @ signs more in the core. That
> would lead to a unusable BackEnd if someone enables E_WARNING in
> exceptionalErrors.
>
> Any Ideas what we can do about this?
>
> greets
> rupert
More information about the TYPO3-team-core
mailing list