[TYPO3-core] RFC: error logging for the Core
Martin Kutschker
Martin.Kutschker at blackbox.net
Tue Jan 3 09:23:23 CET 2006
Holzinger Franz <franz at fholzinger.com> writes on
Mon, 02 Jan 2006 22:46:00 +0100 (MET):
> Hello Martin,
>
>
> >>+ if (!$file) {
> >>+ die('Cannot open TYPO3 system log: '.$msg);
> >>+ }
> >>I think this should not die here but only return and maybe send an
> >>email to the shop admin.
> >>
> >>
> >
> >
> >A bit fixed on shops, aren't you? ;-) If we send an email we need
> > another config variable, which I want to avoid. But I prefer the
> > error_log-version anyway.
> >
> >
> For me TYPO3 is mainly a shop system ;-) .
> I thought we could use both variants, one with DevLog and one with
> error_log --> additional parameter. Some people want this the others
> that.
Huh? devLog and sysLog are two distinct functions.
> >Buuuut, which version do you prefer? The one with the fwrite() or the
> > one with the error_log().
> >
> >And what is your general impression of the patch? +1/-1/neutral...
> >
> I want to have both in the same function, then I will give +1.
Why? error_log does also write to a file and we don't need the custom file locking (when typ3 is 3).
See http://www.php.net/manual/en/function.error-log.php
I realized this only after having written variant 1. So it's either or, fwrite and error_log() simply makes no sense.
Masi
More information about the TYPO3-team-core
mailing list