Some precision for this bug.
the problem is the saving operation of reponses when use navigation index.
The first time, reponses are save but after that they are not save anymore.
I put a new version off lss file.
To repeat the bug, start the survey, you are on the first item. put a value on the field, clic on the next button. Your answer are save.
clic on the navigation link to go to the first item of the navigation,, change your response and clic to the next button. the reponse are not save

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

I've set it up so that it does save values on the page. If they are not valid it will not save them.

At the moment this means that if you change a value there is no way to undo that change, if you navigate using the index it is saved; if you navigate using the next button it is saved.

hi i've pull the last version with the fixe (de20493149ee0f309abd22a00657aeac7650ec6e) and test it. The result are the same.
If i put an other value on the field, it's not saved if i navigate with the index or if i click on the next or previous button

To resolve the problem i've done two things :
In the file application/helpers/SurveyRunTimeHelper.php :
line 339, i have replace :
$moveResult = LimeExpressionManager::NavigateForwards();
by
$next = (int) $_SESSION[$LEMsessid]['step']+1;
$moveResult = LimeExpressionManager::JumpTo($next, false, true, true);

And in the file application/helpers/expressions/em_manager_helper.php
i had in the line 5194 this line :
$LEM->bProcessPost=$processPOST;

With this modifications, the group's tab reponses are save when i click on a item of the navigation and when i click on the next button.

Do you have positive results with your described solution?

the 5194 line in application/helpers/expressions/em_manager_helper.php

does this result in the code as follow:

static function JumpTo($seq,$preview=false,$processPOST=true,$force=false,$changeLang=false) {
$now = microtime(true);
$LEM =& LimeExpressionManager::singleton();
$LEM->bProcessPost=$processPOST;
if(!$preview)
$preview=$LEM->sPreviewMode;
if(!$processPOST || $preview)
$LEM->bProcessPost=false;

?

It would be great to explain more precisely where to put $LEM->bProcessPost=$processPOST; the replacement instruction for SurveyRunTimeHelper.php is clear enough though.

Thanks!

@sammousa

Seems that philg's remarks did help the issue. The code should be inserted as interpreted by me.

It's maybe worth looking at, since the erraneous behaviour could be reproduced in 131209.

For me this works when i try it with your supplied survey.
Steps:

1. Import the survey.
2. Activate the survey.
3. Start the survey.
4. First question, enter 3, click on the question index on question 2.
5. Second question, enter 3, click on submit.

The response is correctly saved.
Any alternative flows where i only use the question index for navigation also work.

Our current session management is pretty much spaghetti.
For now you should consider a work around, any of the following might apply:

• Switch to incremental question index.
• Disable saving / resuming later
• Don't use tokens.

In the upcoming version we will be focusing on cleaning up a lot of our code, this should help us in closing in on this issue.

Honestly I have no idea what other side effects can be caused and I dont have a lot of time to invest in figuring that out right now.

Please stop sending reminders, they cause mail spam; also it makes each of your comments private which means i have to make them public.

hi all, i'm not in my compagny this week but in paris for formation.
i have find a solution to solve the problem. In fact, when you save a group question, some fields of other
group question are add in the update query with null value.
If i have enought time i will you send the code to correct this.
philg

hi all
like promiss, i put here the solution that i have find to resolve the save problem when user switch form a group to an other and when some field are saved with empty value.
In fact, in the file application/helpers/expressions/em_manager_helper.php
in the function function _UpdateValuesInDatabase($updatedValues, $finished=false) wich take and array updatedValues on argument, the first line in the function erase this parameter with this call : $updatedValues = $this->updatedValues;

In fact this function permiss to know wich field have to be update ( normaly the field of the current group ) and the value of this field.
When i have make test, i find that the method $this->updatedValues add other field for other group with null values ! Or the parameters initial parameters updatedValues where good ....

The simple solution to resolve my problem is to comment the first line of the function :
//$updatedValues = $this->updatedValues;

This bug report has become a mess.

Please create a new one after re-testing using the latest 2.05 version, and if your issue is still reproducible, attach a sample survey and a detailed description how to reproduce it. Thank you for your understanding.