Skip to content

Update deprecated php type casts#3739

Open
colemanw wants to merge 1 commit intoStudio-42:masterfrom
colemanw:patch-1
Open

Update deprecated php type casts#3739
colemanw wants to merge 1 commit intoStudio-42:masterfrom
colemanw:patch-1

Conversation

@colemanw
Copy link

In PHP 8.5, non-canonical scalar type casts (boolean|double|integer|binary) deprecated.

See https://php.watch/versions/8.5/boolean-double-integer-binary-casts-deprecated.

In PHP 8.5, non-canonical scalar type casts (boolean|double|integer|binary) deprecated.

See https://php.watch/versions/8.5/boolean-double-integer-binary-casts-deprecated.
Copy link

@escopecz escopecz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are also getting the deprecation. This is the suggested solution:

vendor/studio-42/elfinder/php/elFinder.class.php:4392
Non-canonical cast (double) is deprecated, use the (float) cast instead

@escopecz escopecz mentioned this pull request Jan 12, 2026
7 tasks
@escopecz
Copy link

@troex do you think this could be merged and released? It's blocking support for PHP 8.5 for Mautic. All other dependencies are supporting PHP 8.5 now.

@escopecz
Copy link

Oh man, it would be ridiculous to fork this Composer Package just because of this one line change, wouldn't it?

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