View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
19543Bug reportsFile managerpublic2024-05-08 16:20
Reportertibor.pacalat Assigned Togabrieljenik  
PrioritynoneSeveritypartial_block 
Status resolvedResolutionfixed 
Product Version6.5.x 
Summary19543: KCFinder needs to be updated when using with PHP 8.1
Description

I am getting this error when I try to upload file in email templates for example and using debug=2.

<p>strlen(): Passing null to parameter #1 ($string) of type string is deprecated (/var/www/ls-ce-6x/public/vendor/kcfinder/lib/class_fastImage.php:207)</p>
<pre>#0 /var/www/ls-ce-6x/public/vendor/yiisoft/yii/framework/base/CErrorHandler.php(133): CErrorHandler->handleError()
#1 /var/www/ls-ce-6x/public/vendor/yiisoft/yii/framework/base/CApplication.php(832): CErrorHandler->handle()
#2 unknown(0): LSYii_Application->handleError()
#3 /var/www/ls-ce-6x/public/vendor/kcfinder/lib/class_fastImage.php(207): strlen()
#4 /var/www/ls-ce-6x/public/vendor/kcfinder/lib/class_fastImage.php(78): kcfinder\fastImage->getChars()
#5 /var/www/ls-ce-6x/public/vendor/kcfinder/core/class/browser.php(754): kcfinder\fastImage->getType()
#6 /var/www/ls-ce-6x/public/vendor/kcfinder/core/class/browser.php(152): kcfinder\browser->getFiles()
#7 /var/www/ls-ce-6x/public/vendor/kcfinder/core/class/browser.php(132): kcfinder\browser->act_init()
#8 /var/www/ls-ce-6x/public/vendor/kcfinder/browse.php(18): kcfinder\browser->action()
</pre>

With debug<2 works without issues.

Steps To Reproduce

Steps to reproduce

  1. Set debug=2 in your config
  2. Go to Survey settings->Email templates->Add file and try to upload a file
  3. You get an error

Expected result

(Write here what you expected to happen)

Actual result

(Write here what happened instead)

TagsNo tags attached.
Attached Files
Bug heat6
Complete LimeSurvey version number (& build)6.5.4+240422
I will donate to the project if issue is resolvedNo
Browser
Database type & version.
Server OS (if known)
Webserver software & version (if known)
PHP Version8.1.4

Users monitoring this issue

There are no users monitoring this issue.

Activities

gabrieljenik

gabrieljenik

2024-05-01 15:19

manager   ~80059

https://github.com/LimeSurvey/LimeSurvey/pull/3837

DenisChenu

DenisChenu

2024-05-03 18:29

developer   ~80075

Last edited: 2024-05-03 18:29

@c_schmitz maybe create a repository for our kcfinder here ?

guest

guest

2024-05-08 16:20

viewer   ~80088

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&amp;id=36350

Related Changesets

LimeSurvey: master 632e9e71

2024-05-08 16:08:15

Gabriel Jenik


Committer: GitHub Details Diff
Fixed issue 19543: KCFinder needs to be updated when using with PHP 8.1 (03837)

Co-authored-by: lapiudevgit <devgit@lapiu.biz>
Affected Issues
19543
mod - vendor/kcfinder/core/bootstrap.php Diff File
mod - vendor/kcfinder/core/class/browser.php Diff File
mod - vendor/kcfinder/core/class/minifier.php Diff File
mod - vendor/kcfinder/core/class/uploader.php Diff File
mod - vendor/kcfinder/core/types/type_mime.php Diff File
mod - vendor/kcfinder/integration/drupal.php Diff File
mod - vendor/kcfinder/js_localize.php Diff File
mod - vendor/kcfinder/lib/class_fastImage.php Diff File
mod - vendor/kcfinder/lib/class_image.php Diff File
mod - vendor/kcfinder/lib/class_image_gd.php Diff File
mod - vendor/kcfinder/lib/class_image_gmagick.php Diff File
mod - vendor/kcfinder/lib/class_image_imagick.php Diff File
mod - vendor/kcfinder/lib/helper_file.php Diff File
mod - vendor/kcfinder/lib/helper_httpCache.php Diff File
mod - vendor/kcfinder/lib/helper_path.php Diff File

Issue History

Date Modified Username Field Change
2024-04-26 16:22 tibor.pacalat New Issue
2024-04-26 16:22 tibor.pacalat File Added: Screenshot 2024-04-26 at 16.21.10.png
2024-05-01 15:19 gabrieljenik Assigned To => DenisChenu
2024-05-01 15:19 gabrieljenik Status new => ready for code review
2024-05-01 15:19 gabrieljenik Note Added: 80059
2024-05-01 15:19 gabrieljenik Bug heat 0 => 2
2024-05-03 18:29 DenisChenu Status ready for code review => in code review
2024-05-03 18:29 DenisChenu Note Added: 80075
2024-05-03 18:29 DenisChenu Bug heat 2 => 4
2024-05-03 18:29 DenisChenu Assigned To DenisChenu => c_schmitz
2024-05-03 18:29 DenisChenu Note Edited: 80075
2024-05-08 16:08 tibor.pacalat Status in code review => resolved
2024-05-08 16:08 tibor.pacalat Resolution open => fixed
2024-05-08 16:08 tibor.pacalat Assigned To c_schmitz => gabrieljenik
2024-05-08 16:20 Changeset attached => LimeSurvey master 632e9e71
2024-05-08 16:20 guest Note Added: 80088
2024-05-08 16:20 guest Bug heat 4 => 6