View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
10346Bug reportsSurvey takingpublic2016-02-08 18:032016-02-10 21:03
ReporterDenisChenu Assigned ToLouisGac 
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.50.x 
Summary10346: Multiple text with comment : coment are filled with Y
Description

With multiple text with comment : if multiple is checked : comments are filled with Y

Steps To Reproduce

Import lss
Launch
Using default value : SQ1 is checked and comment are set to Y (no default for commments)
Put "anythinkg" in SQ1 and SQ2 comment, try to submit : see the Y

TagsNo tags attached.
Attached Files
limesurvey_survey_multicomments.lss (14,564 bytes)
Bug heat6
Complete LimeSurvey version number (& build)160208
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database type & versionnot relevant
Server OS (if known)not relevant
Webserver software & version (if known)not relevant
PHP Versionnot relevant

Users monitoring this issue

There are no users monitoring this issue.

Activities

LouisGac

LouisGac

2016-02-08 18:44

developer   ~34726

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

DenisChenu

2016-02-08 18:49

developer   ~34730

Last edited: 2016-02-08 18:49

No

This time: comments is always empty.

$SESSION['survey'.Yii::app()->getConfig('surveyID')][$myfname] is not only for default value, but for filled value too.

See thge reproduce part:
Put "anythinkg" in SQ1 and SQ2 comment, try to submit : see the Y
LouisGac

LouisGac

2016-02-08 18:55

developer   ~34732

" $SESSION['survey'.Yii::app()->getConfig('surveyID')][$myfname] is not only for default value, but for filled value too."
ok
LouisGac

LouisGac

2016-02-08 19:04

developer   ~34733

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

c_schmitz

2016-02-10 21:03

administrator   ~34868

2.5 Build 160210 released

Related Changesets

LimeSurvey: master df67f0fd

2016-02-08 17:44:47

LouisGac

Details Diff 		Fixed issue 10346: removed default comment values for multiple text with comments Affected Issues
10346
mod - application/helpers/qanda_helper.php Diff File

LimeSurvey: master 173fbf01

2016-02-08 18:03:55

LouisGac

Details Diff 		Fixed issue 10346: used $myfname2 insteas of $myfname Affected Issues
10346
mod - application/helpers/qanda_helper.php Diff File

Issue History

Date Modified Username Field Change
2016-02-08 18:03 DenisChenu New Issue
2016-02-08 18:03 DenisChenu File Added: limesurvey_survey_multicomments.lss
2016-02-08 18:44 LouisGac Changeset attached => LimeSurvey master df67f0fd
2016-02-08 18:44 LouisGac Note Added: 34726
2016-02-08 18:44 LouisGac Assigned To => LouisGac
2016-02-08 18:44 LouisGac Resolution open => fixed
2016-02-08 18:46 LouisGac Status new => resolved
2016-02-08 18:49 DenisChenu Note Added: 34730
2016-02-08 18:49 DenisChenu Status resolved => assigned
2016-02-08 18:49 DenisChenu Note Edited: 34730
2016-02-08 18:55 LouisGac Note Added: 34732
2016-02-08 19:04 LouisGac Changeset attached => LimeSurvey master 173fbf01
2016-02-08 19:04 LouisGac Note Added: 34733
2016-02-08 19:04 LouisGac Status assigned => resolved
2016-02-10 21:03 c_schmitz Note Added: 34868
2016-02-10 21:03 c_schmitz Status resolved => closed