ResponsiveFilemanager icon indicating copy to clipboard operation
ResponsiveFilemanager copied to clipboard
verified publisher icon trippo

Error Upload file PHP 8.2.4

Open bbabolinia69 opened this issue 2 years ago • 3 comments

image

I have done the steps given in the previous issue, but the problem still occurs in PHP 8.2.4

bbabolinia69 avatar Jul 17 '23 11:07 bbabolinia69

Do you know how to review your PHP error logs? I've faced an issue with the php_image_magician.php file where there is a number conversion error (float to int or the other way). I've resolved this by adding a "floor" or "ceil" call depending on your preference so that the value passed is always an INT.

jshster avatar Jul 20 '23 01:07 jshster

thanks for the help, but I haven't solved the problem yet. I will learn more about this issue

bbabolinia69 avatar Jul 20 '23 10:07 bbabolinia69

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

dleffler avatar Feb 01 '24 13:02 dleffler