View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
09328Bug reportsSurvey takingpublic2014-12-09 19:43
ReporterResearchOnBlogs Assigned ToDenisChenu  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.05+ 
Target Version2.05+Fixed in Version2.05+ 
Summary09328: Language changer causes warning for mandatory questions
Description

If you change the language of a survey and have mandatory questions in it, you get a warning that you have to fill out the questions. But if I can't read the language, I have to change the lang at first :-)!

Additional Information

I can't find the calling function.
Is it
" function mandatory_popup($ia, $notanswered=null)" ?

Isn't it possible to read the causing site-object for the page refresh and if the it's the lang changer avoid this warnings?

TagsNo tags attached.
Attached Files
Bug heat6
Complete LimeSurvey version number (& build)141020
I will donate to the project if issue is resolvedNo
Browser
Database type & versionxx
Server OS (if known)xx
Webserver software & version (if known)x
PHP Versionx

Relationships

related to 09198 closedDenisChenu complete group's navigation can stop before clicked group 
parent of 09389 closedDenisChenu Problem with tableau if the question is not mandatory and if I do not respond another question 
parent of 09395 closedDenisChenu All in one survey with mandatory array show like an error 

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2014-10-26 10:30

developer   ~30858

I'm sure we don't have this bug before ....

Surely something change elsewhere.

You mean : the language question type or the global language changer ?

ResearchOnBlogs

ResearchOnBlogs

2014-10-26 22:33

developer   ~30869

the global langchanger

DenisChenu

DenisChenu

2014-10-27 08:53

developer   ~30871

Confirmed, something broken somewhere ....

DenisChenu

DenisChenu

2014-10-28 17:06

developer   ~30880

Last edited: 2014-10-28 17:33

Don't understand .....

For change lang: we do JumpTo 2 times, second is OK. But first :
$surveyid != LimeExpressionManager::getLEMsurveyId()

It's because setDirtyFlag when update lang ....

DenisChenu

DenisChenu

2014-11-01 17:57

developer   ~30899

@ResearchOnBlogs

Can you do some test with https://github.com/LimeSurvey/LimeSurvey/pull/227 ,

Maybe with save survey, save survey with token etc ... or maybe i don't test survey more.

I update EM, then test is needed :)

ResearchOnBlogs

ResearchOnBlogs

2014-11-07 08:52

developer   ~30925

I'm in a big project right now, will test es soon as possible

DenisChenu

DenisChenu

2014-11-13 16:33

developer   ~30974

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

c_schmitz

c_schmitz

2014-11-13 17:11

administrator   ~30982

Version 2.05 Build 141113 released

Related Changesets

LimeSurvey: master be983708

2014-11-01 16:47:55

DenisChenu

Details Diff
Fixed issue 09328: Language changer causes warning for mandatory questions
Fixed issue 09285: Index complet + mandatory question :undefined notice
Fixed issue 09198: complete group's navigation can stop before clicked group
Fixed issue 08806: questionindex to 2 allow invalid survey submitted
Dev: use $force in _validateGroup and _validateQuestion
Dev: this allow save value in DB for equation and some other, but never return error.
Dev: update EM, then need testing
Affected Issues
08806, 09198, 09285, 09328
mod - application/helpers/SurveyRuntimeHelper.php Diff File
mod - application/helpers/expressions/em_manager_helper.php Diff File

LimeSurvey: master e4ebdb8e

2014-12-05 17:35:55

DenisChenu

Details Diff
Fixed issue : question help text are not updated if update lang by url param Affected Issues
09328
mod - application/helpers/expressions/em_manager_helper.php Diff File

LimeSurvey: 2.06 e3a99cd2

2014-12-05 17:38:07

DenisChenu

Details Diff
Fixed issue : question help text are not updated if update lang by url param
Dev : now this don't break change language
Affected Issues
09328
mod - application/helpers/expressions/em_manager_helper.php Diff File

Issue History

Date Modified Username Field Change
2014-10-24 21:23 ResearchOnBlogs New Issue
2014-10-24 21:23 ResearchOnBlogs File Added: limesurvey_survey_21226.lss.xml
2014-10-26 10:30 DenisChenu Note Added: 30858
2014-10-26 22:33 ResearchOnBlogs Note Added: 30869
2014-10-27 08:53 DenisChenu Note Added: 30871
2014-10-27 08:53 DenisChenu Assigned To => DenisChenu
2014-10-27 08:53 DenisChenu Status new => assigned
2014-10-28 17:06 DenisChenu Note Added: 30880
2014-10-28 17:33 DenisChenu Note Edited: 30880
2014-11-01 17:48 DenisChenu Relationship added related to 09198
2014-11-01 17:57 DenisChenu Note Added: 30899
2014-11-07 08:52 ResearchOnBlogs Note Added: 30925
2014-11-13 16:33 DenisChenu Changeset attached => LimeSurvey master be983708
2014-11-13 16:33 DenisChenu Note Added: 30974
2014-11-13 16:33 DenisChenu Resolution open => fixed
2014-11-13 16:37 DenisChenu Status assigned => resolved
2014-11-13 16:37 DenisChenu Fixed in Version => 2.05+
2014-11-13 17:11 c_schmitz Note Added: 30982
2014-11-13 17:11 c_schmitz Status resolved => closed
2014-12-05 16:03 DenisChenu Relationship added parent of 09389
2014-12-05 18:39 DenisChenu Changeset attached => LimeSurvey 2.06 e3a99cd2
2014-12-05 18:39 DenisChenu Changeset attached => LimeSurvey master e4ebdb8e
2014-12-09 19:43 DenisChenu Relationship added parent of 09395