 |
@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. |
|
|
@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. |
|
|
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. |
|
|
|
|
|
Yes about the token status PS : you can do anything importing VV file ;) |
|
|
|
|
|
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 |
- Anonymous
