LimeSurvey issue tracker
Registration

View Issue Details Jump to Notes ] Related Changesets ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
07895Bug reports[All Projects] Conditionspublic2013-05-31 11:342013-07-08 13:50
ReporterDenisChenu 
Assigned ToDenisChenu 
PrioritynormalSeverityminor 
StatusclosedResolutionfixed 
Product Version2.00+ 
Target Version2.00+Fixed in Version2.00+ 
Summary07895: Quota are not tested for hidden question (relevance to 1)
DescriptionWe can have some question set by expression manager, even with hidding it. Then we can set quota at this question.

But this don't work : user inside quota goes to the end of survey without out by quota.
If we shown question and set it after with expression : it work
Steps To ReproduceImport first survey (question hidden) and look at quota and test it
Import second survey (question shown) : answer the "hors quota" value (no quota) and click next.
Additional InformationReally don't understand why it work with question shown and not with question hidden.
I will donate to the project if issue is resolved within 48 hrsNo
LimeSurvey build number OR git commit ID130531
Browsernot relevant
Database & DB-VersionMysql 5
Operating System (Server)debian/linux
Webserver software & versionapache
PHP VersionPHP Version 5.3
Attached Files? file icon survey_archive_hidden.lsa [^] (3,718 bytes) 2013-05-31 11:34
? file icon survey_archive_nothidden.lsa [^] (3,728 bytes) 2013-05-31 11:35

- Relationships

-  Notes
User avatar (25432)
DenisChenu (developer)
2013-05-31 11:59
edited on: 2013-05-31 12:01

Problem is in frontend_helper/checkQuota
// Lets only continue if any of the quota fields is in the posted page
if (isset($_POST['fieldnames']))
....

Then : 3 solutions:
- Add hidden question to POST value
- Test question step and not POST value set
- Remove this test

User avatar (25436)
DenisChenu (developer)
2013-05-31 12:47

Testing if (isset($_SESSION['survey_'.$surveyid][$checkfield])) seems OK,

But need some test again with other quota system.
User avatar (25454)
DenisChenu (developer)
2013-06-03 08:35

Target version ?
Need option ?
User avatar (25607)
DenisChenu (developer)
2013-06-21 19:46

Using "Terminate survey with warning" allow user to modify is answers.

Then : choose to:
Change this string "Terminate survey with warning" to "Terminate survey but allow user to modify gis answers"

Check if $_POST is set if this setting is used
Don't check if not (enforced/real quota).

Some template issue too :
Have to use navigator.pstpl
User avatar (25613)
DenisChenu (developer)
2013-06-22 11:40

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=12632 [^]
User avatar (25614)
DenisChenu (developer)
2013-06-22 11:41

Todo for 2.05. Ither thing must be done for 2.05 (and tested)
User avatar (25723)
c_schmitz (administrator)
2013-07-08 13:50

2.00+ Build 130708 released.

- Related Changesets
LimeSurvey: master 720aab24
Timestamp: 2013-06-22 09:36:45
Author: DenisChenu
Committer: Denis Chenu
Details ] Diff ]
Fixed issue 07895: Quota are not tested for hidden question (relevance to 1)
Fixed issue : Previous button in quota in citronade are not the same.
Dev: Change sentence of quota_action : reflect reality now
Dev: Don't change behaviour for quota_action=='2'
Dev: remove some $_POST var ($fieldname not in actual page)
Dev: this fieldname is needed for EM javascript : put disabled='disabled' inside.
Dev: DON'T do same think for relevance : else eremoved $_POST[$fieldname] : to be fixed
Dev: Same HTML previous button for quota page: todo : add navigator.pstpl (but allow save and clear all must be removed)
mod - application/helpers/expressions/em_manager_helper.php Diff ] File ]
mod - application/helpers/frontend_helper.php Diff ] File ]
mod - application/views/admin/quotas/editquota_view.php Diff ] File ]
mod - application/views/admin/quotas/newquota_view.php Diff ] File ]
mod - templates/citronade/template.css Diff ] File ]
LimeSurvey: master 3a96fb1b
Timestamp: 2013-06-22 10:01:37
Author: DenisChenu
Committer: Denis Chenu
Details ] Diff ]
Dev: bug 07895 / 720aab24 child : lost one sentence
mod - application/views/admin/quotas/viewquotasrow_view.php Diff ] File ]

- Issue History
Date Modified Username Field Change
2013-05-31 11:34 DenisChenu New Issue
2013-05-31 11:34 DenisChenu File Added: survey_archive_hidden.lsa
2013-05-31 11:35 DenisChenu File Added: survey_archive_nothidden.lsa
2013-05-31 11:35 DenisChenu Summary Quota are not tested with question set by expression => Quota are not tested for hidden question (relevance to 1)
2013-05-31 11:59 DenisChenu Note Added: 25432
2013-05-31 12:01 DenisChenu Note Edited: 25432 View Revisions
2013-05-31 12:47 DenisChenu Note Added: 25436
2013-06-02 23:04 c_schmitz Assigned To => DenisChenu
2013-06-02 23:04 c_schmitz Status new => assigned
2013-06-03 08:35 DenisChenu Note Added: 25454
2013-06-21 19:46 DenisChenu Note Added: 25607
2013-06-21 19:46 DenisChenu Reproducibility have not tried => always
2013-06-21 19:46 DenisChenu Target Version => 2.00+
2013-06-22 11:40 DenisChenu Changeset attached => LimeSurvey master 720aab24
2013-06-22 11:40 DenisChenu Note Added: 25613
2013-06-22 11:40 DenisChenu Resolution open => fixed
2013-06-22 11:41 DenisChenu Note Added: 25614
2013-06-22 11:41 DenisChenu Status assigned => resolved
2013-06-22 11:41 DenisChenu Fixed in Version => 2.00+
2013-06-22 12:05 DenisChenu Changeset attached => LimeSurvey master 3a96fb1b
2013-07-08 13:50 c_schmitz Note Added: 25723
2013-07-08 13:50 c_schmitz Status resolved => closed


Copyright © 2000 - 2014 MantisBT Team
Powered by Mantis Bugtracker