Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:117700
MIME-Version: 1.0
Date: Mon, 9 May 2022 09:17:49 +0200
Message-ID: <CAH5C8xXynfGa67gqXoCtUVvwJs8ptMcwYkuRhXWpJpe_VB-UMw@mail.gmail.com>
To: PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000216c0d05de8f02ab"
Subject: Declaring tidyNode properties as readonly?
From: kocsismate90@gmail.com (=?UTF-8?B?TcOhdMOpIEtvY3Npcw==?=)

--000000000000216c0d05de8f02ab
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Internals,

When I was working on making the properties of the tidy and tidyNode
classes declared (https://github.com/php/php-src/pull/8515), I noticed that
it doesn't make sense to write the properties of tidyNode, as the property
changes won't have any effect on the parsed HTML node. Christoph provided
an example to highlight the issue with this behavior:
https://github.com/php/php-src/pull/8515#issuecomment-1120400605.

In my opinion, the properties in question should be declared as readonly in
order to make
the behavior of tidyNode more predictable. Of course, this would be a BC
break, but I believe
the use-case is so niche and misleading that we can afford to prevent it
right away in PHP 8.2.

Please let me know what you think about this change, especially if you are
aware of any valid
use-cases regarding the modification of the properties of tidyNode.

Regards,
M=C3=A1t=C3=A9

--000000000000216c0d05de8f02ab--