| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 18494 | Bug reports | Expression Manager | public | 2022-11-17 13:42 | 2023-09-25 10:55 |
| Reporter | twilllig | Assigned To | tibor.pacalat | ||
| Priority | none | Severity | partial_block | ||
| Status | closed | Resolution | fixed | ||
| Product Version | 5.4.x | ||||
| Summary | 18494: TypeError during demo survey "ls205_em_tailoring" | ||||
| Description | When you particpate in that survey and reach page 3 "Dynamic Math Display" type error happens. | ||||
| Steps To Reproduce | Steps to reproduce
Expected result
Actual resultat page 3 "Dynamic Math Display" an type error happens See screenshot. | ||||
| Tags | No tags attached. | ||||
| Bug heat | 12 | ||||
| Complete LimeSurvey version number (& build) | 5.4.11 | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | Chrome | ||||
| Database type & version | mySql | ||||
| Server OS (if known) | |||||
| Webserver software & version (if known) | |||||
| PHP Version | 8.0.16 | ||||
 |
Debug set to 2, right ? |
|
|
If yes : need a wrapper (see related) |
 |
Yes, debug = 2 definetely needs a wrapper, and I bet if abs() is solved, the next function will have a type error. This demo survey is full of math functions. |
|
|
Oh ? PHP8 only ? PHP7 : List of function is big … |
|
|
Maybe depends on strict_type setting, |
|
|
yes , see changelog : https://www.php.net/manual/en/function.abs.php |
|
|
partial_block : PHP8 or debug maybe block ? |
|
|
maybe block, but noone else reported it, yet. |
|
|
Debian stable are not on PHP8 |
|
|
google says: Ubuntu 20.04 (which is latest LTS) is on 7.4 So relax until April 2025 when 21.10 will be the new LTS with PHP8? :) |
 |
|
|
|
Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35468 |
|
|
Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35469 |
|
|
@gabrieljenik I see there is a branch for master, but no PR. Are you still working on it, or? |
|
|
@tibor.pacalat Yes, I was pending to review and do the PR |
|
|
Tested and merged fix for master as well. |
|
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35477 |
|
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35476 |
|
|
Fixed in Release 6.2.7+230918 |
|
|
Test Case Updated: https://github.com/LimeSurvey/LimeSurvey/pull/3452 |
|
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35542 |
|
|
Fixed in Release 6.2.9+230925 |
