Adding support for PHP 8.1?

[cdgo]

Are there any plans/roadmap to add support for PHP 8.1?

As a first step from PHP 8.0 to PHP 8.2 I restarted testing with PHP 8.1.

[burt]

What part(s) of Phoenix fail in 8.2 ?

[cdgo]

Hi Burt,

I think it's more the add-ons than core, but I'll make a list if any for the core, and add them to this thread.

[swguy]

The basic operations seem solid for PHP 8.2. Nicely done, Burt and team!

[burt]

I am in a php8.2 sandbox, running the upcoming bootstrap 5 theme.
So far, everything seems solid.

If I find anything, I will also post back to this thread.

[swguy]

Possibly the same question @cdgo intended: when will the home page for PhoenixCart show that it has been tested higher than 8.0 (what it shows now)? Seems like it might be reasonable to bump that number to at least 8.1, if not 8.2.

[ecartz]

Possibly the same question @cdgo intended: when will the home page for PhoenixCart show that it has been tested higher than 8.0 (what it shows now)? Seems like it might be reasonable to bump that number to at least 8.1, if not 8.2.

Last time I bumped it (to 8.0), it was after I, myself, had it running on 8.0. For various reasons, it's unlikely that I will run on either 8.1 or 8.2 any time soon. Perhaps Burt will bump it this time.

[swguy]

What part(s) of Phoenix fail in 8.2 ?

I'm finding a few warning logs in PHP 8.1. I'm opening issues for them and would be happy to PR.

[cdgo]

One of the issues which ofcourse is not directly caused by the core, but I think should be handled by the core is the failling of translations:

PHP Fatal error: Uncaught Error: Undefined constant "TITLE" in /home/user/domains/domain/public_html/includes/classes/application.php:65

The adding of all the nessecary translations is part of the added plugin, but there should be a fallback to prevent the whole shop to seize working when only a translations is missing.
Most likely it happens in the additional languages so a fallback to the default language should be working for those cases.
And if it is in the default language, a default output could be used like "mising translation" or even just one space.

[cdgo]

PHP Deprecated: Function strftime() is deprecated in /home/user/domains/domain/public_html/includes/modules/content/index/templates/tpl_cm_i_card_products.php

Function strftime() should be replaced by:
$formatter = new IntlDateFormatter('nl_NL', IntlDateFormatter::LONG, IntlDateFormatter::NONE);
echo $formatter->format(time());