View Issue Details

This bug affects 1 person(s).
 12
IDProjectCategoryView StatusLast Update
16867Bug reportsSurvey editingpublic2021-03-08 12:29
ReporterDenisChenu Assigned Togabrieljenik  
PrioritynoneSeverityminor 
Status closedResolutionfixed 
Product Version3.25.1 
Summary16867: Randomization happen in survey logic file and in export
Description

When a survey is tested : randomization happen in survey logic file and in export.

Steps To Reproduce

Import included survey
Activate survey
Launch survey
Move next to see 1st page with question
Close test
Go to Survey logic file : cehck order
Got to export : check order

Additional Information

The order are from the current survey.
We already have this issue before , fixed years ago …

TagsNo tags attached.
Attached Files
Bug heat12
Complete LimeSurvey version number (& build)3.25.1 github
I will donate to the project if issue is resolvedNo
Browsernot relevant ?
Database type & versionmariadb
Server OS (if known)fedora/linux
Webserver software & version (if known)nginx
PHP Versionphp7.4

Relationships

related to 17107 acknowledgedc_schmitz Survey seems without any question when testing (admin user) 
related to 17113 closedgabrieljenik Randomization happen in survey logic file and in export 
related to 17142 new Bad order of VV and database column with randomiza 

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2020-11-30 09:43

developer   ~60797

Limequery have the issue

Peek 30-11-2020 09-43.gif (231,593 bytes)
gabrieljenik

gabrieljenik

2021-01-12 22:20

manager   ~61508

Last edited: 2021-01-12 22:20

This only happens when running the survey and running the export at the same time (sharing the session)
The issue is the fieldmap is shared.

When running the survey, the order is randomized.

I believe this can be disregarded or delayed.

ping @cdorin

DenisChenu

DenisChenu

2021-01-13 07:52

developer   ~61511

Last edited: 2021-01-13 09:29

This only happens when running the survey and running the export at the same time (sharing the session)

Yes

I believe this can be disregarded or delayed.

it can be very annoying …
It was fixed in 2.6lts

c_schmitz

c_schmitz

2021-01-13 09:35

administrator   ~61517

Do you have a link to the original fix?

DenisChenu

DenisChenu

2021-01-13 10:45

developer   ~61523

I i remind : we «just» force reset …

I think we reset more Expression in 2.6lts , for example : we always force reset in Survey logic file here it seems starnge the survey logic can be outdated …

very old fix …

Mazi

Mazi

2021-02-03 18:59

updater   ~61906

This can indeed be annyoing and also leads to support requests by confised users.

gabrieljenik

gabrieljenik

2021-02-12 16:52

manager   ~62225

What do you think of this situation:
1 - Run survey on tab A
2 - Run Logic File on tab B
(I reset EM as suggested)
3 - When I return to Tab A thins will be weird

DenisChenu

DenisChenu

2021-02-12 17:20

developer   ~62228

3 - When I return to Tab A thins will be weird

It's always case …

  1. Launch survey in tab B
  2. Move next
  3. Update a question on tab A
  4. Try to move on tab B

In general : it's empty, with debug : something with gid not set.

DenisChenu

DenisChenu

2021-02-13 10:33

developer   ~62234

@gabrieljenik case proof : in 3LTS with debug and without debug
I didn't know but seems language changer do solething.

PS : i really think it's already reported year ago.

DenisChenu

DenisChenu

2021-02-18 09:44

developer   ~62337

https://bugs.limesurvey.org/view.php?id=17107

gabrieljenik

gabrieljenik

2021-02-18 22:19

manager   ~62374

PR: https://github.com/LimeSurvey/LimeSurvey/pull/1770
Created a similar ticket for LS4

ollehar

ollehar

2021-02-22 15:29

administrator   ~62439

Last edited: 2021-02-22 15:30

This is minor or major? I honestly surprised how many minor issues we have in this sprint...

gabrieljenik

gabrieljenik

2021-02-22 16:00

manager   ~62449

Don't know. I just grab stuff from the list from hig to low priority :)

ollehar

ollehar

2021-02-22 16:04

administrator   ~62452

Yeah, but this has "none" as priority?

gabrieljenik

gabrieljenik

2021-03-01 12:16

manager   ~62571

Fix committed to 3.x-LTS branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=31229

lime_release_bot

lime_release_bot

2021-03-08 12:29

administrator   ~62879

Fixed in Release 3.25.16+210302

Related Changesets

LimeSurvey: 3.x-LTS 3d099ee9

2021-03-01 12:16:31

gabrieljenik


Committer: GitHub Details Diff
Fixed issue 16867: Randomization happen in survey logic file and in export (#1770)

* Fixed issue 16867: Randomization happen in survey logic file and in export

* Fixed issue 16867: Randomization happen in survey logic file and in export

Reviewing approach
Affected Issues
16867
mod - application/controllers/admin/export.php Diff File
mod - application/controllers/admin/expressions.php Diff File

Issue History

Date Modified Username Field Change
2020-11-30 09:36 DenisChenu New Issue
2020-11-30 09:36 DenisChenu File Added: limesurvey_survey_randomQuestion.lss
2020-11-30 09:36 DenisChenu File Added: Capture d’écran du 2020-11-30 09-31-05.png
2020-11-30 09:36 DenisChenu File Added: Capture d’écran du 2020-11-30 09-31-51.png
2020-11-30 09:36 DenisChenu File Added: Capture d’écran du 2020-11-30 09-32-09.png
2020-11-30 09:43 DenisChenu Note Added: 60797
2020-11-30 09:43 DenisChenu File Added: Peek 30-11-2020 09-43.gif
2020-12-28 14:56 cdorin Status new => confirmed
2020-12-28 14:56 cdorin Sync to Zoho Project => |Yes|
2021-01-12 22:20 gabrieljenik Note Added: 61508
2021-01-12 22:20 gabrieljenik Note Edited: 61508
2021-01-13 07:52 DenisChenu Note Added: 61511
2021-01-13 09:29 DenisChenu Note Edited: 61511
2021-01-13 09:35 c_schmitz Note Added: 61517
2021-01-13 10:45 DenisChenu Note Added: 61523
2021-02-03 18:59 Mazi Note Added: 61906
2021-02-12 16:52 gabrieljenik Note Added: 62225
2021-02-12 17:20 DenisChenu Note Added: 62228
2021-02-13 10:33 DenisChenu Note Added: 62234
2021-02-13 10:33 DenisChenu File Added: Peek 13-02-2021 10-30.gif
2021-02-13 10:33 DenisChenu File Added: Peek 13-02-2021 10-31.gif
2021-02-18 09:44 DenisChenu File Deleted: Peek 13-02-2021 10-30.gif
2021-02-18 09:44 DenisChenu File Deleted: Peek 13-02-2021 10-31.gif
2021-02-18 09:44 DenisChenu Note Added: 62337
2021-02-18 09:44 DenisChenu Relationship added related to 17017
2021-02-18 09:44 DenisChenu Relationship deleted related to 17017
2021-02-18 09:44 DenisChenu Relationship added related to 17107
2021-02-18 22:11 gabrieljenik Issue cloned: 17113
2021-02-18 22:11 gabrieljenik Relationship added related to 17113
2021-02-18 22:19 gabrieljenik Note Added: 62374
2021-02-22 15:29 ollehar Note Added: 62439
2021-02-22 15:30 ollehar Note Edited: 62439
2021-02-22 16:00 gabrieljenik Note Added: 62449
2021-02-22 16:04 ollehar Note Added: 62452
2021-02-28 10:08 DenisChenu Relationship added related to 17142
2021-03-01 12:16 gabrieljenik Changeset attached => LimeSurvey 3.x-LTS 3d099ee9
2021-03-01 12:16 gabrieljenik Note Added: 62571
2021-03-01 12:16 gabrieljenik Assigned To => gabrieljenik
2021-03-01 12:16 gabrieljenik Resolution open => fixed
2021-03-05 17:31 ollehar Status confirmed => resolved
2021-03-08 12:29 lime_release_bot Sync to Zoho Project Yes => |Yes|
2021-03-08 12:29 lime_release_bot Note Added: 62879
2021-03-08 12:29 lime_release_bot Status resolved => closed