[META] Make Drupal 7 core compatible with PHP 8.3

D7 default should now be set to PHP 8.2
AND
PHP 8.3 automated tests should be added here:

https://www.drupal.org/node/3060/qa →

Thanks for creating the issue. Let's try how PHP 8.3 test suite runs on D7 core currently. Adding a noop patch.

Sorry, the patch was somehow UTF-16..

This looks good. There seems to be only one problem with unserialization in D7 core tests (there could be more, but not catched by tests).

I have created an issue with the fix for the unserialize failure: 🐛 [D7 PHP 8.3] unserialize(): Extra data starting at offset Fixed

The child issue is fixed. Let's check with another noop patch if we are green now.

Seems like other failure/s appeared in PHP8.3-RC tests, so we need to check these as well (probably something must have changed between PHP8.3 beta and RC, because we were already green after #3380876).

Seems like the TextSummaryTestCase failures could be related to similar failures in D10: 📌 TextSummaryTest:testLength() fails on some libxml versions Closed: outdated . So we need to create an issue for this in D7 as well.

Created new child issue 📌 [D7 PHP 8.3] TextSummaryTestCase::testLength() fails on some libxml versions Active .

queued last patch on 8.3.0RC5 looks the only broken test!

Nice, only 2 fails! Subscribing.

PHP 8.3 is scheduled to release on November 23, 2023

Release candidate 6 is in a few days from now, perhaps we get lucky with rc6 passing more tests than rc5?

or is this a documented/expected fail?

I do not think that the failures will disappear, as it seems like that they are caused by some changes in libxml library, not directly by PHP 8.3 (see the child issue).

Release package to CI images

Failed test for session needs new issue and still active 📌 [D7 PHP 8.3] TextSummaryTestCase::testLength() fails on some libxml versions Active

I have created another child issue for the session failure.

testEmptySessionId() failure: 🐛 [D7 PHP 8.3] Fix SessionHttpsTestCase->testEmptySessionId() failure Needs review

testLength() failure: 📌 [D7 PHP 8.3] TextSummaryTestCase::testLength() fails on some libxml versions Active

Created also a draft MR here with PHP 8.3 testing (https://git.drupalcode.org/project/drupal/-/merge_requests/8020) and after applying fixes from both issues, tests are green, see: https://git.drupalcode.org/project/drupal/-/pipelines/170105 (sorry for multiple commits, but I had/have issues with replicating the session failure locally).

Hiding the old patch as we have now a MR here for testing.

@poker10, great work!
So if I understand correctly, you've put all the fixes into the MR 8020, then split off the actual code that does the fixes into those two issues 3446569 and 3397882 ?
Right now I see 100% green in the MR 8020

Yes - both fixes are in MR 8020 to see the final effect (but this MR will not be committed) and then there are two separate issues with MRs to commit.

I have done it because the MR workflow is more flexible and also because I was not able to simulate the session test failure locally, so I needed to experiment a bit.