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

Compatible with php 8.3 #122

Open
wants to merge 4 commits into
base: nextgen
Choose a base branch
from
Open

Compatible with php 8.3 #122

wants to merge 4 commits into from

Conversation

Alex300
Copy link

@Alex300 Alex300 commented Sep 28, 2024

Fix Deprecated: Implicit conversion from float to int loses precision
Fix dinamic creation object property is deprecated

@peterdd
Copy link

peterdd commented Sep 28, 2024

return type definitions are PHP 7+, securimage is PHP5.4+

(assuming there are still PHP5.6 installs at least)

https://wiki.php.net/rfc/return_types

@Alex300
Copy link
Author

Alex300 commented Sep 30, 2024

I removed return type definitions. I didn't even know that someone could use PHP 5.x

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