Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:43755
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.220.172 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :cc:content-type:content-transfer-encoding;
        b=XmO4eO6QQUHj728Jzb2qUhbpwPQ5v4Kn4seV5bJVBcVY3OLSEHJHf7VqmlBKF84WMA
         3RZZZH3t6Ru1k8nl0IfPFWiGKhAn0n7OyUZ3OfijixEbogk4HmmnnbyhC4Y+SLxJapZV
         iuqxCjtfuxWvmhmZXjmi1fbwiSgVrRXuE2g8g=
MIME-Version: 1.0
In-Reply-To: <1240357055.6159.9.camel@goldfinger>
References: <2dedb8a0902061106p4333682g56a1ec29196e7f3e@mail.gmail.com>
	 <1240357055.6159.9.camel@goldfinger>
Date: Wed, 22 Apr 2009 08:56:42 +0200
Message-ID: <7f3ed2c30904212356x17fc759dm64a4a793dd3360f@mail.gmail.com>
To: =?ISO-8859-1?Q?Johannes_Schl=FCter?= <johannes@php.net>
Cc: Kalle Sommer Nielsen <kalle@php.net>, Internals <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] Throwing E_DEPRECATED on startup
From: hannes.magnusson@gmail.com (Hannes Magnusson)

2009/4/22 Johannes Schl=FCter <johannes@php.net>:
> On Fri, 2009-02-06 at 20:06 +0100, Kalle Sommer Nielsen wrote:
>> I'd like to propose a change for when PHP starts to throw an
>> E_DEPRECATED notice for each of the deprecated ini directives (such as
>> register_globals, safe_mode ect.) like in HEAD where they just throw a
>> warning (more about that below). I have attached a patch for this in
>> 5.3 (which is the only branch this patch really relates to).
>
> Yeah.
>
> Good thing to notice the user of bad configuration. I'm wondering a bit
> how we can educate people about magic_quotes_gpc ... people who blindly
> disable them probably break stuff which is "secure" by accident. This is
> not only relevant to this patch though. The best idea I have is a clear
> pointer to the docs from the message ...

There are couple of issues with throwing errors at startup:
1) They are thrown so damn early that the user will never see them
2) run-tests.php gets very confused and starts expecting HTTP headers
in the EXPECT section

If we could delay the checks until after php_module_startup()..

-Hannes