A lss in all in one with cookie enable

Faulty lines : https://github.com/LimeSurvey/LimeSurvey/blob/79683baa01edbec5ae2dfde4af82fc0b86a0da58/application/controllers/survey/index.php#L349

But if newtest=Y allow to do survey again : must remove cookie …

hmm, wasn't this one recently solved? Or Gabriel made a PR recently?

There are a related fix or PR, but clerly : must remove (!isset($param['newtest']) || $param['newtest'] != "Y")

Issue exist in the latest version(4.3.4) also

@cdorin There was an issue related to it. I think it was about cookies and admin previews.
Will review.

newTest param shouldn't be considered for this validation
PR: https://github.com/LimeSurvey/LimeSurvey/pull/1538

Tested the issue before pulling the PR, Issue exist. Tested the issue after pulling the PR, below are my findings:
When I test either with or without pull request I still see this error message.
Please refer the attachment for more details

16235_ErrorMessage.png (39,621 bytes)   

16235_ErrorMessage.png (39,621 bytes)   

Before PR: https://screencast-o-matic.com/watch/cYji6iGlVB
After PR: https://screencast-o-matic.com/watch/cYji62Gln2
Before the PR I was able to navigate to welcome page and take the survey for second time when "set cookie to prevent repetition" is enabled.
After pulling the PR when I click on survey for second time I get the error message.
Please refer the recordings for more details, working as expected.

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

Fixed in Release 4.3.10+200812