View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
09848Bug reportsConditionspublic2015-08-27 17:222015-09-30 11:01
Reporterpvdemael Assigned ToDenisChenu  
PriorityhighSeveritypartial_block 
Status closedResolutionfixed 
Product Version2.06+ 
Fixed in Version2.06+ 
Summary09848: date_min is badly interpreted in a date question With survey in one page
Description

When a date question has a date_min attribute depending on a previous date question answer, the date_min is wrong when the question has a relevance equation.
This problem does not occur when the date question has no relevance equation

Steps To Reproduce
  1. Create a date question startDate
  2. Create f.i. a Y/N question showDate
  3. Create a date question dateConditional
  4. Set startDate as date_min in dateConditional
  5. Set the relevance equation of dateConditional to showDate == 'Y'
TagsNo tags attached.
Attached Files
limesurvey_survey_924448.lss (15,401 bytes)
Bug heat6
Complete LimeSurvey version number (& build)Versie 2.06+ Build 150825
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMySQL 5.5.37
Server OS (if known)Ubuntu 13.10 (GNU/Linux 3.11.0-26-generic i686)
Webserver software & version (if known)Apache/2.2.22 (Ubuntu)
PHP VersionPHP Version 5.4.6-1ubuntu1.2

Users monitoring this issue

There are no users monitoring this issue.

Activities

mfaber

mfaber

2015-08-27 19:03

reporter   ~33023

This does not seem to be the only problem. Eg, the text "Answer must be greater or equal to .." is missing for the first question. Minimum date appears with "0:00", which should not be. Other funny stuff as well.

No problem in Version 2.05+ Build 150520. (only older version I have available)

Devs: Maybe a change to expression manager between 2.05+ Build 150520 and more recent builds?
DenisChenu

DenisChenu

2015-08-31 08:31

developer   ~33045

More strange : preview group seem OK ....

But we have a bug in first question too (of i don't made error).

  • Preview : Answer must be greater or equal to 2015-08-01 (OK)
  • live : just 2015-08-01 00:00
    for 'not conditionnal. The problem is here :)
DenisChenu

DenisChenu

2015-08-31 08:33

developer   ~33046

Set "show welcome screen" to TRUE : it work.

$_SESSION part (started|not started) issue (after|before) Expression Manager.

@mfaber : for older version : did you test with survey in one page or something like that ?
mfaber

mfaber

2015-09-01 14:33

reporter   ~33051

Denis: Yes, in Version 2.05+ Build 150520, it works "all on one page", "group by group" and "question by question".
mfaber

mfaber

2015-09-05 02:15

reporter   ~33062

Denis: I did a git bisect for the "Answer must be greater or equal to 2015-08-01 (OK)" vs. "2015-08-01 00:00" problem.
This is the result, could you have a look?

a3389cbc439a238e2db603a00d0f8493026bf618 is the first bad commit
commit a3389cbc439a238e2db603a00d0f8493026bf618
Author: Denis Chenu courXXXX@XXXXXXXX.net
Date: Tue Dec 9 19:46:47 2014 +0100

Fixed issue #09395: All in one survey with mandatory array have errormandato

ry without submit
Dev: JumpTo seem better to hack $move even for uneeded showwelcome==N

:040000 040000 9f786bfd673cbb63db513c41bd585716cce1007b 03c473bda775efa1fa1f0bb0
ad40057c1d63fa67 M application
DenisChenu

DenisChenu

2015-09-05 10:41

developer   ~33063

@mfaber : but 2.05 seems to have same fix too ?

But force 2015-08-01 00:00 to be 2015-08-01 must be done (and it must be done for previous button too, no ?
DenisChenu

DenisChenu

2015-09-07 08:24

developer   ~33064

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

DenisChenu

2015-09-07 08:33

developer   ~33065

Using moveForward fix the issue. The issue is still here with JumpTo.

But :

  • EM tips (tracevar) seems OK
  • EM is called only one time
  • No change betwwen force =(true|false)

etc .....
c_schmitz

c_schmitz

2015-09-30 11:01

administrator   ~33229

Version 2.06+ Build 150930 released

Related Changesets

LimeSurvey: master 876fca05

2015-09-07 08:24

DenisChenu


Details Diff		 Fixed issue 09848 : missinterpretation of tips with survey without welcome page
Dev: Use NavigateForwards and not JumpTo
Dev: Some js optimisation on do_date function		 Affected Issues
09848
mod - application/helpers/SurveyRuntimeHelper.php Diff File
mod - application/helpers/qanda_helper.php Diff File
mod - scripts/date.js Diff File

Issue History

Date Modified Username Field Change
2015-08-27 17:22 pvdemael New Issue
2015-08-27 17:22 pvdemael File Added: limesurvey_survey_924448.lss
2015-08-27 19:03 mfaber Note Added: 33023
2015-08-31 08:31 DenisChenu Note Added: 33045
2015-08-31 08:31 DenisChenu Status new => confirmed
2015-08-31 08:33 DenisChenu Note Added: 33046
2015-08-31 08:34 DenisChenu Summary date_min is badly interpreted in a date question with a relevance equation => date_min is badly interpreted in a date question With survey in one page
2015-09-01 14:33 mfaber Note Added: 33051
2015-09-05 02:15 mfaber Note Added: 33062
2015-09-05 02:16 mfaber Assigned To => DenisChenu
2015-09-05 02:16 mfaber Status confirmed => assigned
2015-09-05 10:41 DenisChenu Note Added: 33063
2015-09-07 08:24 DenisChenu Changeset attached => LimeSurvey master 876fca05
2015-09-07 08:24 DenisChenu Note Added: 33064
2015-09-07 08:24 DenisChenu Resolution open => fixed
2015-09-07 08:33 DenisChenu Note Added: 33065
2015-09-07 08:33 DenisChenu Status assigned => resolved
2015-09-07 08:33 DenisChenu Fixed in Version => 2.06+
2015-09-30 11:01 c_schmitz Note Added: 33229
2015-09-30 11:01 c_schmitz Status resolved => closed