View Issue Details

Jump to NotesJump to History
This bug affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
04453Bug reportsSurvey takingpublic2010-06-30 20:082014-02-28 16:42
Reporterjdalegonzalez Assigned Tolemeur  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version1.87+ 
Fixed in Version1.90RC3 
Summary04453: Refreshing a page in Internet Explorer will cause conditional fields to not be displayed.
Description

Fields that should be displayed conditionally are written out with 'style="display: none"'. On page load, a checkconditions script is run and it will change this to .style.display='' for those questions that should display. The problem is that group.php adds javascript to only run these condition checks if document.getElementById('runonce').value =='0'. Once the function has been run, document.getElementById('runonce').value is updated to '1'.

With rational browsers, this works fine. With Internet Explorer, (at least with v 8), everything is hidden on a screen refresh but the value of document.getElementById('runonce') is retained. This means that the checkconditions script bails out having never made visible any elements that were conditionally displayed.

Steps To Reproduce
  1. Create a survey with conditionally displayed questions
  2. Set the conditions so that the question would be displayed on startup
  3. verify that you can see your conditional question.
  4. hit refresh.
  5. notice that the question is no longer displayed.
Additional Information

Setting the value of 'runonce' to 0 in window.ready OR not using a hidden form value but rather a text field with display:none both seem to solve the problem.

TagsNo tags attached.
Bug heat6
Complete LimeSurvey version number (& build)7987
I will donate to the project if issue is resolved
BrowserInternet Explorer v. 8
Database type & versionall
Server OS (if known)all
Webserver software & version (if known)all
PHP Versionall

Relationships

Relationship GraphDependency Graph
parent of 08783 closedDenisChenu The main container form is not 508 compliant 

Users monitoring this issue

There are no users monitoring this issue.

Activities

lemeur

lemeur

2010-07-12 16:49

developer   ~12385

I haven't been able to reproduce the problem, however I've committed a fix to rev 8926.

Thibault
Mazi

Mazi

2010-07-13 01:36

updater   ~12388

jdalegonzalez, once we have released a 1.90 RC3 or stable version, can you check if the problem is solved?!
You can also get the latest files via SVN for testing lemeurs fix right now.
jdalegonzalez

jdalegonzalez

2010-07-13 14:56

reporter   ~12417

Sure. I'll take a look at it on RC3

Issue History

Date Modified Username Field Change
2010-06-30 20:08 jdalegonzalez New Issue
2010-06-30 20:08 jdalegonzalez Status new => assigned
2010-06-30 20:08 jdalegonzalez Assigned To => user372
2010-07-01 22:11 c_schmitz Assigned To user372 => lemeur
2010-07-12 16:49 lemeur Note Added: 12385
2010-07-12 16:49 lemeur Status assigned => resolved
2010-07-12 16:49 lemeur Fixed in Version => 1.90RC3
2010-07-12 16:49 lemeur Resolution open => fixed
2010-07-13 01:36 Mazi Note Added: 12388
2010-07-13 14:56 jdalegonzalez Note Added: 12417
2010-07-20 22:16 c_schmitz Status resolved => closed
2010-10-25 00:18 c_schmitz Category Survey at Runtime => Survey taking
2014-02-28 16:42 DenisChenu Relationship added parent of 08783