View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
20127Bug reportsResponse browsingpublic2025-06-04 17:19
Reportergabrieljenik Assigned Totibor.pacalat  
PrioritynoneSeveritycrash 
Status resolvedResolutionfixed 
Summary20127: Dump when browsing responses, with DEBUG enabled.
Description

See attachment

TagsNo tags attached.
Attached Files
PHP warning.pdf (264,301 bytes)
Bug heat6
Complete LimeSurvey version number (& build)6.4
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMysql
Server OS (if known)
Webserver software & version (if known)
PHP Version6.13.3

Users monitoring this issue

There are no users monitoring this issue.

Activities

gabrieljenik

gabrieljenik

2025-05-28 21:05

manager   ~82787

@DenisChenu Unsure what to do here.

This seems to be failing because there is a corruption in DB.
Maybe linked to #20054.

Question here is if this method should return null if no language is found for that answer.
(or if it should return an "N/A", or a "?" or a "", or just the answer for the first language).
I incline for null.

This only happens with DEBUG, so maybe this is expected: When DEBUG is set and there are inconsistencies, errors should pop up?

DenisChenu

DenisChenu

2025-05-30 09:49

developer   ~82792

This only happens with DEBUG, so maybe this is expected: When DEBUG is set and there are inconsistencies, errors should pop up?

My opinion about debug : it's a developer setting ! NOt a super admin or Survey administrator

Then :

  1. DB issue must not throw 500 error
  2. DB issue must have a way to be logged every time as error or warning (using Yii::log with specific categorie)
  3. Bonus : a warning must be shown to survey admin about an issue in DB
  4. Super bonus : a Survey option are allowed to Survey admin (not super admn) to automatically fix such DB issue ( i think i already report such feature : checkintegrity restriced to single survey)

Here : 1+2 are OK in my opinon.

Question here is if this method should return null if no language is found for that answer.
(or if it should return an "N/A", or a "?" or a "", or just the answer for the first language).

If exist In primary language but not in this language : code ? But NULL is OK too. fifty/fifty here .
If it does not exist in any language : NULL : 100%

gabrieljenik

gabrieljenik

2025-06-03 15:20

manager   ~82806

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

guest

guest

2025-06-04 17:19

viewer   ~82823

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

Related Changesets

LimeSurvey: master 6e562888

2025-06-04 17:19

Gabriel Jenik

Committer: GitHub


Details Diff
Fixed issue 20127: Dump when browsing responses, with DEBUG enabled (04307)

Co-authored-by: lapiudevgit <devgit@lapiu.biz>
Affected Issues
20127
mod - application/models/Answer.php Diff File

Issue History

Date Modified Username Field Change
2025-05-28 17:01 gabrieljenik New Issue
2025-05-28 17:01 gabrieljenik File Added: PHP warning.pdf
2025-05-28 17:01 gabrieljenik Assigned To => gabrieljenik
2025-05-28 17:01 gabrieljenik Status new => assigned
2025-05-28 21:05 gabrieljenik Note Added: 82787
2025-05-28 21:05 gabrieljenik Bug heat 0 => 2
2025-05-30 09:49 DenisChenu Note Added: 82792
2025-05-30 09:49 DenisChenu Bug heat 2 => 4
2025-06-03 15:20 gabrieljenik Assigned To gabrieljenik => DenisChenu
2025-06-03 15:20 gabrieljenik Status assigned => ready for code review
2025-06-03 15:20 gabrieljenik Note Added: 82806
2025-06-03 15:59 DenisChenu Assigned To DenisChenu => tibor.pacalat
2025-06-03 15:59 DenisChenu Status ready for code review => ready for testing
2025-06-04 17:19 Changeset attached => LimeSurvey master 6e562888
2025-06-04 17:19 guest Note Added: 82823
2025-06-04 17:19 guest Bug heat 4 => 6
2025-06-04 17:19 tibor.pacalat Status ready for testing => resolved
2025-06-04 17:19 tibor.pacalat Resolution open => fixed