-
Notifications
You must be signed in to change notification settings - Fork 367
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 deprecated errors for php 8.1 compatibility #96
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, since chr(60.6)
= chr(60)
@neitanod Any chance to merge and release this fix for PHP 8.1? 😃 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks. I'd forked this project, you can create a pr if you want
@neitanod Can you please merge and tag this PR? It would help a lot :-) In addition to this PR, the minimum PHP version has to be increased to PHP 7 in the composer.json. intdiv wasn't present in PHP 5 |
Beste relatie,
Ik ben donderdag 5-5-2022 niet op kantoor. Vrijdag ben ik weer beschikbaar. In geval van nood, gelieve te bellen naar 06-24514448.
Met vriendelijke groet,
Danny Neleman
Eye2I Design VOF
06-24514448
|
$cc1 = (chr(intdiv(ord($c1), 64)) | "\xc0"); |
Bedankt voor je e-mail. Ik ben vandaag afwezig en heb beperkte toegang tot e-mail. Indien dringend verzoek ik u een WhatsApp bericht te sturen naar mijn mobiele telefoon (+31624514448).
Excuses voor het ongemak.
Met vriendelijke groet,
Danny Neleman
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use intdiv() for php 8.1 compatibility
No description provided.