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

mb_convert_encoding deprecation with PHP 8.3 #46

Closed
lee-peuker opened this issue Dec 5, 2023 · 4 comments
Closed

mb_convert_encoding deprecation with PHP 8.3 #46

lee-peuker opened this issue Dec 5, 2023 · 4 comments

Comments

@lee-peuker
Copy link

lee-peuker commented Dec 5, 2023

I have updated to php 8.3 and have an issue with this package on the latest version (3.0.1):

Deprecated: mb_convert_encoding(): Handling HTML entities via mbstring is deprecated; use htmlspecialchars, htmlentities, or mb_encode_numericentity/mb_decode_numericentity instead in /var/www/projects/vgl-event-worker/vendor/wa72/htmlpagedom/src/Helpers.php on line 97

Seems like it could be related to this #44

Any plans or ideas how to fix this?

@wasinger
Copy link
Owner

wasinger commented Dec 5, 2023

You don't use latest master or 3.x version, I guess. Just added PHP 8.3 to the unit tests workflow and all tests do pass.

@wasinger
Copy link
Owner

wasinger commented Dec 5, 2023

Just realized that you might need to use version 2.x with PHP 8.3. Indeed, in 2.x, the mb_convert_encoding problem is still present. I'll look for it in the coming days. PRs welcome :-)

@lee-peuker
Copy link
Author

lee-peuker commented Dec 6, 2023

Hm, after downgrading to 2.0.3 and upgrading again to the latest version (3.0.2) it seems to work fine. Not sure why 3.0.1 did not work before, but I guess this were issues on my machine.

Thanks for the fast answer and your awesome work <3

@wasinger
Copy link
Owner

wasinger commented Dec 7, 2023

Release v2.0.4 now fixes this issue for the 2.x branch, too.

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

No branches or pull requests

2 participants