View Issue Details

IDProjectCategoryView StatusLast Update
14560Bug reports[All Projects] Survey designpublic2019-07-15 11:41
ReporterDenisChenuAssigned Tocdorin 
PrioritynoneSeveritymajor 
Status assignedResolutionopen 
Product Version4.0.0dev 
Target VersionFixed in Version 
Summary14560: Mandatory soft must be related to Question not Page
Description

Currently : seems mandatory soft is related to page then :

  • Group with mandatory hard + mandatory soft can be submitted without mandatory hard validation (3rd group in lss)
  • Group with mandatory soft + validation : popup shown but can not submit.
Steps To Reproduce

Import include lss and test it (in develop)

Additional Information

If mandatory soft is related to question :

  1. We can really fix this 2 issues
  2. We can show the system inside question answer for mandatory soft
  3. core theme can still shown a popup, button can validate all soft mandatory in the page
  4. another theme can show it differently

The fix must be set in expression_manager for starting
https://github.com/LimeSurvey/LimeSurvey/commit/a4ec47355e05ce198b32fa307e71a34cebb6b413#diff-796aba4d06254b9caea9d461e4ea80c9R6542

Not test a global var, but test (for exemple) if the SGQA is in an array of posted var.

Else
Another way to show a mansoft system : checkBoxForText test survey (real used system since more then 2 years).

TagsNo tags attached.
Complete LimeSurvey version number (& build)4.0.0 github
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database & DB-Versionnot relevant
Server OS (if known)not relevant
Webserver software & version (if known)not relevant
PHP Versionnot relevant

Activities

DenisChenu

DenisChenu

2019-02-21 09:51

developer  

survey_archive_manSoftTest.lsa (3,979 bytes)
DenisChenu

DenisChenu

2019-02-21 09:54

developer  

simplescreenrecorder-2019-02-21_09.53.18.webm (1,132,332 bytes)
DenisChenu

DenisChenu

2019-02-21 10:03

developer   ~50658

I think

  1. Fix Expression manager to test (for exemple) :
    1. global mansoft (for whole survey)
    2. 1 array of mansoft for Question (then can have way to disable for question
    3. 1 array of mansoft for SGQ (for subquestion, multiple text, but array of radio too)
    4. 1array of mansoft for Group
  2. Move the function to test POSTED value in public static function ProcessCurrentResponses() https://github.com/LimeSurvey/LimeSurvey/blob/a4ec47355e05ce198b32fa307e71a34cebb6b413/application/helpers/expressions/em_manager_helper.php#L8855 (because it's checked ONLY when need to process post)
  3. Fill information in a twig variable : (i don't remind if we have moveResult in aSurveyInfo)
DenisChenu

DenisChenu

2019-02-21 12:58

developer   ~50659

Last edited: 2019-02-21 13:00

View 2 revisions

Idea of updating #1

  1. In EM/ ProcessCurrentResponses : set EM->manSoft[qid][] with all array of qid checked
  2. In EM/validateQuestion : set mandatiory validation OK if EM->manSoft[qid][$qid] is set
  3. Add in qanda_helper, if question have mansoft a checkbox hidden with name mansoft['qid'][] width value current qid. Hide it for javascript
  4. The current button check all mansoft['qid'][] checkbox

Issue History

Date Modified Username Field Change
2019-02-21 09:51 DenisChenu New Issue
2019-02-21 09:51 DenisChenu Status new => assigned
2019-02-21 09:51 DenisChenu Assigned To => dominikvitt
2019-02-21 09:51 DenisChenu File Added: Capture d’écran du 2019-02-21 09-43-24.png
2019-02-21 09:51 DenisChenu File Added: Capture d’écran du 2019-02-21 09-49-41.png
2019-02-21 09:51 DenisChenu File Added: survey_archive_manSoftTest.lsa
2019-02-21 09:51 DenisChenu Additional Information Updated View Revisions
2019-02-21 09:52 DenisChenu Severity minor => major
2019-02-21 09:54 DenisChenu File Added: simplescreenrecorder-2019-02-21_09.53.18.webm
2019-02-21 10:03 DenisChenu Note Added: 50658
2019-02-21 12:58 DenisChenu Note Added: 50659
2019-02-21 13:00 DenisChenu Note Edited: 50659 View Revisions
2019-07-15 11:41 dominikvitt Assigned To dominikvitt => cdorin