@tibor.pacalat: This is a blocker for one of our customers. Can you look into this and assign it? Should be easy to reproduce.

@Mazi, I am sorry, but yesterday was the last release of LS5.

@tibor.pacalat: We also have https://bugs.limesurvey.org/view.php?id=19527 pending to be integrated at a follow up 5.x release. Fixing this one doesn't seem too difficult...

Please bear in mind that many, many customers are still at 5.x e.g. it takes them a few more months to set up a new machine with latest PHP 8.x to switch to version 6. Furthermore, many of them need to adjust custom themes for the switch.
That still is a huge customer base and I'd really appreciate if we could help by fixing at least such blocker things.

@DenisChenu can you fix this real quick please?

Friday maybe ?

Merci beaucoup, @DenisChenu!

I say Friday ! (tomorrow ;))

@Mazi : i think we can allow invalid token here too ?

Doing both : set completed if exist, no update if not exist.
OK ?

I really don't understand the logic of this code … It sounds awfully complicated for a simple thing.

Currently it seems all edit/update action is disable with anonymous surey …

@DenisChenu RE "i think we can allow invalid token here too ?" -> I agree, since not token is written into the response table. We just need to make sure the token status (completed) gets updated correctly if the token exists.

5.X https://github.com/LimeSurvey/LimeSurvey/pull/3872

Yes about the token status

PS : you can do anything importing VV file ;)

master ; https://github.com/LimeSurvey/LimeSurvey/pull/3873

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

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

Tested and merged.

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

Fixed in Release 5.6.67+240612