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: PHP 8.1 "Serialization of 'ReflectionMethod' is not allowed" #6136

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

j-bonex
Copy link

@j-bonex j-bonex commented Jun 10, 2022

With 0925b50 there have been proper fixes for CI_Migration and CI_Xmlrpcs classes.
But for CI_Codeigniter the former is_callable check has been removed without any substitution.

This causes "Serialization of 'ReflectionMethod' is not allowed" errors in several cases, especially when running PHPUnit integration tests using @runInSeparateProcess Feature.

Please accept the same fix for CI_Codeigniter main class as well.

@j-bonex j-bonex changed the title Update CodeIgniter.php bugfix "Serialization of 'ReflectionMethod' is not allowed" Jun 10, 2022
@j-bonex j-bonex changed the title bugfix "Serialization of 'ReflectionMethod' is not allowed" fix: PHP 8.1 "Serialization of 'ReflectionMethod' is not allowed" Jun 10, 2022
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.

1 participant