Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for PHP 8.4 #227

Merged
merged 1 commit into from
Jul 10, 2024
Merged

Fix for PHP 8.4 #227

merged 1 commit into from
Jul 10, 2024

Conversation

Alkarex
Copy link
Contributor

@Alkarex Alkarex commented Jul 8, 2024

Deprecated: Gt\CssXPath\Translator::preg_match_collated(): Implicitly marking parameter $transform as nullable is deprecated, the explicit nullable type must be used instead in ./phpgt/cssxpath/src/Translator.php on line 279

https://wiki.php.net/rfc/deprecate-implicitly-nullable-types

> Deprecated: Gt\CssXPath\Translator::preg_match_collated(): Implicitly marking parameter $transform as nullable is deprecated, the explicit nullable type must be used instead in ./phpgt/cssxpath/src/Translator.php on line 279
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request Jul 8, 2024
Sole fix needed so far seems to be related to https://wiki.php.net/rfc/deprecate-implicitly-nullable-types

See also upstream PR PhpGt/CssXPath#227

We are also hitting was seems to be a PHP bug php/php-src#14873
Alkarex added a commit to FreshRSS/FreshRSS that referenced this pull request Jul 9, 2024
* Initial support for PHP 8.4
Sole fix needed so far seems to be related to https://wiki.php.net/rfc/deprecate-implicitly-nullable-types

See also upstream PR PhpGt/CssXPath#227

We are also hitting was seems to be a PHP bug php/php-src#14873

* Fix return type

* Disable OPCache while waiting for PHP fix
@g105b
Copy link
Member

g105b commented Jul 10, 2024

Thank you @Alkarex

@g105b g105b merged commit 45f3ac1 into PhpGt:master Jul 10, 2024
13 checks passed
@Alkarex Alkarex deleted the fix-php-8.4 branch July 10, 2024 13:52
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request Jul 12, 2024
Includes PhpGt/CssXPath#227
Follow-up of PhpGt/CssXPath#227 for PHP 8.4
Requires PHP 8.0+
Full diff PhpGt/CssXPath@d99d35f...45f3ac1
Alkarex added a commit to FreshRSS/FreshRSS that referenced this pull request Sep 6, 2024
Includes PhpGt/CssXPath#227
Follow-up of PhpGt/CssXPath#227 for PHP 8.4
Requires PHP 8.0+
Full diff PhpGt/CssXPath@d99d35f...45f3ac1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants