Upload error in PHP 8.1

[seunex17]

I am having problem uploading image in PHP 8.1 it works on older version. The main issue is i can not event see the actual error this is what i am getting.

[junefw]

I have same issue when upgrade php 8.2.0. hope fix it soon

[junefw]

@seunex17 Hey, you can follow fixed this issue https://github.com/trippo/ResponsiveFilemanager/issues/673 reply by faisalrehmanlums

I have two step to fixed:

1. At this file: /filemanager/config/config.php Replace mb_http_input('UTF-8') with mb_http_input();

2. At saveImage function in this file: filemanager/include/php_image_magician.php Change: if ( ! is_resource($this->imageResized)) To: if ( ! is_resource($this->imageResized) && !($this->imageResized instanceof \GdImage))

*Note: enable GD extention extension=gd in php.ini Done.

[seunex17]

@junefw thanks

[mcsaygili]

Same issue for PHP 8.2.5 Any updates ?

[mcsaygili]

Fixed.

• dialog.php change str_replace(['[', ']'], ['\[', '\]'], $field_id'') to str_replace(['[', ']'], ['\[', '\]'], $field_id ?? '')

• php_image_magician.php change $cropStartX = $cropArray['x']; $cropStartY = $cropArray['y']; to $cropStartX = (int)$cropArray['x']; $cropStartY = (int)$cropArray['y'];

• UploadHandler.php add protected $response;

[satriags]

it's works !! thank youu

[slaweknaczynski]

Fixed.

• dialog.php change str_replace(['[', ']'], ['\[', '\]'], $field_id'') to str_replace(['[', ']'], ['\[', '\]'], $field_id ?? '')
• php_image_magician.php change $cropStartX = $cropArray['x']; $cropStartY = $cropArray['y']; to $cropStartX = (int)$cropArray['x']; $cropStartY = (int)$cropArray['y'];
• UploadHandler.php add protected $response;

Why don't you do pull request with those changes so that owner can merge it into master for others to use instead of applying manually? I can see owner merged some pull requests this year? Thanks

[dleffler]

Not yet fixed in the code here. Though there's fixes above the error is in /include/php_image_magician.php whereby the floating point parameters ($cropStartX & $cropStartY are not recommended for the imagecopyresampled() call. I simply added an '(int)' type cast to each parameter and the 'warning' no longer spits out which causes the above error. (See #718)