Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:127856
Precedence: bulk
MIME-Version: 1.0
References: <hzKdzBWWITU1ePjv3qcyK-OzroFAkWst9MQBw2zvKSA8pk1iqruLZZWTTh9gYDo2QX8EY_PiirVYDuwne2Upr3GlqZk570wtLOF49aaP-HQ=@gpb.moe>
In-Reply-To: <hzKdzBWWITU1ePjv3qcyK-OzroFAkWst9MQBw2zvKSA8pk1iqruLZZWTTh9gYDo2QX8EY_PiirVYDuwne2Upr3GlqZk570wtLOF49aaP-HQ=@gpb.moe>
Date: Thu, 3 Jul 2025 12:46:12 +0200
Message-ID: <CAEKnhAGNx_tfhXvsCk9x_XokR82ohAvj12JCri=pDAuWhxkKrg@mail.gmail.com>
Subject: Re: [PHP-DEV] [RFC] Deprecations for PHP 8.5
To: "Gina P. Banyard" <internals@gpb.moe>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000b994ed0639041685"
From: bukka@php.net (Jakub Zelenka)

--000000000000b994ed0639041685
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi,

On Wed, Jul 2, 2025 at 10:00=E2=80=AFPM Gina P. Banyard <internals@gpb.moe>=
 wrote:

> Hello internals,
>
> It is this time of year again where we proposed a list of deprecations to
> add in PHP 8.5:
>
> https://wiki.php.net/rfc/deprecations_php_8_5
>
>
Here are few notes on the ones that I don't agree with:

>  Deprecate backticks as an alias for shell_exec

I think this might be too big BC break that might impact many scripts -
would be good to see also if it impacts OSS projects. I guess it will
impact even more non public code bases.

> Deprecate the __sleep() and __wakeup() magic methods

I'm not sure about this one. I don't think it's worth it. It's just an
unnecessary BC break IMHO. I would also consider more ext/standard thing
rather than language.

> Deprecate the error_prepend_string and error_append_string INI directives

I think it doesn't have a good enough reason to be deprecated and might be
still used. I don't see any problem with keeping them.

> Deprecate the report_memleaks INI directive

I think this should be kept as one might not necessarily have time to fix
it but still need to work reasonably with debug build so it makes sense to
suppress it.

> Deprecate the $http_response_header predefined variable

I think it's too early for this. I would prefer to not deprecate it at this
time because alternative is available only from 8.4...

> Deprecate filter_input(), filter_input_array(), and filter_has_var()

I think those functions might be useful in some cases. I would also like to
see their current usage. The case of modification of super global is not
exactly a usual thing so I wouldn't worry much about it.

Kind regards

Jakub

--000000000000b994ed0639041685
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div>Hi,</div><br><div class=3D"gmail_quote gmail_quote_co=
ntainer"><div dir=3D"ltr" class=3D"gmail_attr">On Wed, Jul 2, 2025 at 10:00=
=E2=80=AFPM Gina P. Banyard &lt;internals@gpb.moe&gt; wrote:<br></div><bloc=
kquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:=
1px solid rgb(204,204,204);padding-left:1ex">Hello internals,<br>
<br>
It is this time of year again where we proposed a list of deprecations to a=
dd in PHP 8.5:<br>
<br>
<a href=3D"https://wiki.php.net/rfc/deprecations_php_8_5" rel=3D"noreferrer=
" target=3D"_blank">https://wiki.php.net/rfc/deprecations_php_8_5</a><br><b=
r></blockquote><div><br></div><div>Here are few notes on the ones that I do=
n&#39;t agree with:</div><div><br></div><div>&gt;=C2=A0=C2=A0Deprecate back=
ticks as an alias for shell_exec</div><div><br></div><div>I think this migh=
t be too big BC break that might impact many scripts - would be good to see=
 also if it impacts OSS projects. I guess it will impact even more non publ=
ic code bases.</div><div><br></div><div>&gt;=C2=A0Deprecate the __sleep() a=
nd __wakeup() magic methods</div><div><br></div><div>I&#39;m not sure about=
 this one. I don&#39;t think it&#39;s worth it. It&#39;s just an unnecessar=
y BC break IMHO. I would also consider more ext/standard thing rather than =
language.</div><div><br></div><div>&gt;=C2=A0Deprecate the error_prepend_st=
ring and error_append_string INI directives</div><div><br></div><div>I thin=
k it doesn&#39;t have a good enough reason to be deprecated and might be st=
ill used. I don&#39;t see any problem with keeping them.</div><div><br></di=
v><div>&gt;=C2=A0Deprecate the report_memleaks INI directive</div><div><br>=
</div><div>I think this should be kept as one might not necessarily have ti=
me to fix it but still need to work reasonably with debug build so it makes=
 sense to suppress it.</div><div><br></div><div>&gt; Deprecate the $http_re=
sponse_header predefined variable</div><div><br></div><div>I think it&#39;s=
 too early for this. I would prefer to not deprecate it at this time becaus=
e alternative is available only from 8.4...</div><div><br></div><div>&gt;=
=C2=A0Deprecate filter_input(), filter_input_array(), and filter_has_var()<=
/div><div><br></div><div>I think those functions might be useful in some ca=
ses. I would also like to see their current usage. The case of modification=
 of super global is not exactly a usual thing so I wouldn&#39;t worry much =
about it.</div><div><br></div><div>Kind regards</div><div><br></div><div>Ja=
kub</div></div></div>

--000000000000b994ed0639041685--