View Issue Details

Jump to NotesJump to History
This bug affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
04375Bug reportsData Entry (non public)public2010-05-28 01:042010-06-08 21:51
Reporterledut Assigned Tolemeur  
PrioritylowSeverityminor 
Status closedResolutionfixed 
Product Version1.90b 
Fixed in Version1.90RC1 
Summary04375: Display subquestion2 even if subquestion2 is NULL in "wieving answers" from admin section.
Description

I didn't check it but, exploring the source code, I notice a weird controle in limesurvey/admin/browse.php line 159 :

if (isset($field['subquestion1']) && isset($field['subquestion1']))
{
$question .=' ('.$field['subquestion1'].':'.$field['subquestion2'].')';
}

something should missing.

Steps To Reproduce

go to
/limesurvey/admin/admin.php?action=browse&sid=12345&subaction=id&id=1
or go to the admin section, select a survey, display answers.

TagsNo tags attached.
Bug heat6
Complete LimeSurvey version number (& build)8745
I will donate to the project if issue is resolved
BrowserMozilla/5.0 (Windows; U; Windows NT 6.1; fr; rv:1.9.2) Gecko/20100115 Firefox/3.6
Database type & versionMysql 5.1.36
Server OS (if known)wamp on Windows seven
Webserver software & version (if known)localhost
PHP Version5.2.11

Users monitoring this issue

There are no users monitoring this issue.

Activities

Mazi

Mazi

2010-05-29 15:45

updater   ~12023

The code really looks weird. Thibault, what do you think?
lemeur

lemeur

2010-05-29 15:50

developer   ~12027

yes the code looks strange... should mosty likeley be subquestion1 and 2.

However, can you explain what is what is the pb exactly ?
can you explain (attache a screenshot).
ledut

ledut

2010-05-29 19:01

reporter   ~12030

Last edited: 2010-05-30 07:19

I have no problem.
But I don't use subquestions feature for my surveys.
I have just noticed that strange line.

I have chosen the bug title "Display subquestion2 even if subquestion2 is NULL in "viewing answers" from admin section." : it's presumptuous because I haven't checked it. It is FHO what that line should be responsible for.
Unfortunatly, I won't have the time to test it.
lemeur

lemeur

2010-06-03 21:07

developer   ~12121

k,
thx ledut for feedback
lemeur

lemeur

2010-06-03 21:10

developer   ~12123

Fixed in rev8785.
Thanks.
ledut

ledut

2010-06-04 05:33

reporter   ~12138

you 're welcome

Issue History

Date Modified Username Field Change
2010-05-28 01:04 ledut New Issue
2010-05-28 01:04 ledut Status new => assigned
2010-05-28 01:04 ledut Assigned To => user372
2010-05-29 15:45 Mazi Note Added: 12023
2010-05-29 15:45 Mazi Assigned To user372 => lemeur
2010-05-29 15:50 lemeur Note Added: 12027
2010-05-29 19:01 ledut Note Added: 12030
2010-05-30 07:19 ledut Note Edited: 12030
2010-06-03 21:07 lemeur Note Added: 12121
2010-06-03 21:10 lemeur Note Added: 12123
2010-06-03 21:10 lemeur Status assigned => resolved
2010-06-03 21:10 lemeur Fixed in Version => 1.90RC1
2010-06-03 21:10 lemeur Resolution open => fixed
2010-06-04 05:33 ledut Note Added: 12138
2010-06-08 21:51 c_schmitz Status resolved => closed