Dependency Graph

Dependency Graph
related to related to child of child of duplicate of duplicate of

View Issue Details

This bug affects 1 person(s).
 10
IDProjectCategoryView StatusLast Update
16235Bug reportsSurvey takingpublic2020-08-12 10:10
ReporterDenisChenu Assigned Togabrieljenik  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version4.2.2 
Summary16235: Welcome page show again if cookies are enable and survey is already done
Description

If « Set cookie to prevent repeated participation: » is on, after survey was completed, laucnh again the survey show the welcome page.

Steps To Reproduce

Import included survey
Activate
Launch (with newtest=Y)
Submit
Laucch again : see welcome page
Move next : Error about cookie (it's OK)

Additional Information

3.X have same issue
If survey are All in one : all survey can be shown, but disable submit

TagsNo tags attached.
Attached Files
Peek 06-05-2020 15-22.gif (148,656 bytes)
Bug heat10
Complete LimeSurvey version number (& build)3.22.15 AND 4.2.2
I will donate to the project if issue is resolvedNo
Browserff
Database type & versionnot relevant?
Server OS (if known)not relevant ?
Webserver software & version (if known)not relevant ?
PHP Versionnot relevant ?

Relationships

related to 16219 closed Testing a question will not be possible if admin participated in the survey. 
related to 16569 closedgabrieljenik Welcome page show again if cookies are enable and survey is already done 

Activities

DenisChenu

DenisChenu

2020-05-06 15:28

developer   ~57588

A lss in all in one with cookie enable

DenisChenu

DenisChenu

2020-05-06 15:28

developer   ~57589

Last edited: 2020-08-10 16:31

DenisChenu

DenisChenu

2020-05-06 15:37

developer   ~57592

Last edited: 2020-08-10 16:31

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 …

cdorin

cdorin

2020-07-09 17:09

reporter   ~58876

Last edited: 2020-08-10 16:31

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

DenisChenu

DenisChenu

2020-07-09 17:36

developer   ~58877

Last edited: 2020-08-10 16:31

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

user225042

2020-07-14 14:42

  ~58944

Last edited: 2020-08-10 16:31

Issue exist in the latest version(4.3.4) also

gabrieljenik

gabrieljenik

2020-08-05 00:49

manager   ~59315

Last edited: 2020-08-10 16:31

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

gabrieljenik

gabrieljenik

2020-08-06 22:22

manager   ~59357

Last edited: 2020-08-10 16:31

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

user225042

2020-08-07 10:41

  ~59365

Last edited: 2020-08-10 16:31

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)   

user225042

2020-08-07 12:20

  ~59367

Last edited: 2020-08-10 16:31

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.

gabrieljenik

gabrieljenik

2020-08-10 15:52

manager   ~59389

Last edited: 2020-08-10 16:31

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

lime_release_bot

lime_release_bot

2020-08-12 10:10

administrator   ~59430

Fixed in Release 4.3.10+200812

Related Changesets

LimeSurvey: master a380df5e

2020-08-10 15:41:52

gabrieljenik


Committer: GitHub Details Diff
Fixed issue 16235: Welcome page show again if cookies are enable and survey is already done (#1538)

newTest param shouldn't be considered for this validation
Affected Issues
16235
mod - application/controllers/survey/index.php Diff File

Issue History

Date Modified Username Field Change
2020-05-06 15:27 DenisChenu New Issue
2020-05-06 15:27 DenisChenu File Added: Peek 06-05-2020 15-22.gif
2020-05-06 15:28 DenisChenu Note Added: 57588
2020-05-06 15:28 DenisChenu Note Added: 57589
2020-05-06 15:28 DenisChenu File Added: limesurvey_survey_cookiesAllinone3X.lss
2020-05-06 15:28 DenisChenu Relationship added related to 16219
2020-05-06 15:37 DenisChenu Note Added: 57592
2020-05-11 09:51 ollehar Priority none => normal
2020-07-09 17:09 cdorin Note Added: 58876
2020-07-09 17:36 DenisChenu Note Added: 58877
2020-07-14 14:42 user225042 Note Added: 58944
2020-07-14 14:43 user225042 Status new => confirmed
2020-07-29 23:08 cdorin Zoho Sprints => |Yes|
2020-07-29 23:08 swendrich Zoho Sprints ID => 14469000000197133
2020-08-05 00:49 gabrieljenik Note Added: 59315
2020-08-05 00:50 gabrieljenik Issue cloned: 16569
2020-08-05 00:50 gabrieljenik Relationship added related to 16569
2020-08-06 22:22 gabrieljenik Note Added: 59357
2020-08-07 10:41 user225042 Note Added: 59365
2020-08-07 10:41 user225042 File Added: 16235_ErrorMessage.png
2020-08-07 12:20 user225042 Note Added: 59367
2020-08-10 15:52 gabrieljenik Changeset attached => LimeSurvey master a380df5e
2020-08-10 15:52 gabrieljenik Note Added: 59389
2020-08-10 15:52 gabrieljenik Assigned To => gabrieljenik
2020-08-10 15:52 gabrieljenik Resolution open => fixed
2020-08-10 16:31 swendrich Zoho Sprints Yes => |Yes|
2020-08-10 16:31 swendrich Status confirmed => resolved
2020-08-10 16:31 swendrich Zoho Sprints Yes => |Yes|
2020-08-12 10:10 lime_release_bot Zoho Sprints Yes => |Yes|
2020-08-12 10:10 lime_release_bot Note Added: 59430
2020-08-12 10:10 lime_release_bot Status resolved => closed