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 Deprecated #97

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

fix PHP Deprecated #97

wants to merge 1 commit into from

Conversation

neokdev
Copy link

@neokdev neokdev commented Mar 7, 2022

fix PHP Deprecated: Implicit conversion from float to int loses precision
adding floor() on lines 201, 210, 219, 224, 232
for compat PHP 8.1

fix PHP Deprecated:  Implicit conversion from float to int loses precision
adding floor() on lines 201, 210, 219, 224, 232
for compat PHP 8.1
@DavidGoodwin
Copy link

If this project isn't dead, it'd be easier for someone to approve/merge this PR if the diff only contained the actual fixes mentioned ... (and not e.g reindentation, as welcome as it may be!).

@BafS
Copy link

BafS commented Jun 2, 2022

@neitanod is there any hope about merging some fixes? Would add maintainers be an options to let the project die? Thanks in advance

@andreamk
Copy link

@neitanod if it was possible could you update the library with fixes for PHP 8.1 ?
thanks

@DavidGoodwin
Copy link

@andreamk I'd suggest moving to https://packagist.org/packages/fylax/forceutf8

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.

5 participants