View Issue Details

This bug affects 1 person(s).
 10
IDProjectCategoryView StatusLast Update
16229Bug reports_ Unknownpublic2020-09-21 14:54
Reporterollehar Assigned Togabrieljenik  
PriorityhighSeveritypartial_block 
Status closedResolutionfixed 
Product Version4.2.2 
Target Version4.x.0-dev 
Summary16229: Random answer order currently broken
Description

See https://manual.limesurvey.org/Question_type_-_List_(Dropdown)#Random_answer_order_.28random_order.29

and

https://github.com/LimeSurvey/LimeSurvey/commit/c87c729e5c22decea62b5039a383c84cc34494c0

TagsNo tags attached.
Bug heat10
Complete LimeSurvey version number (& build)latest master
I will donate to the project if issue is resolvedNo
Browser-
Database type & version-
Server OS (if known)-
Webserver software & version (if known)-
PHP Version-

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2020-05-05 15:15

developer   ~57541

But : it's broken in 16032-randomize-also-randomizes-answers not in master ?
Right ?

ollehar

ollehar

2020-05-05 15:15

administrator   ~57542

Yes, but it will be merged, since the fix still improves the software overall (IMO).

ollehar

ollehar

2020-05-05 15:16

administrator   ~57543

By which I mean, fewer users have broken workflow with the "fix" than without.

DenisChenu

DenisChenu

2020-05-05 15:17

developer   ~57544

OK :)

Seems right : if you can show it in any order : you can show it with default order :)

cdorin

cdorin

2020-05-17 15:39

reporter   ~57856

Shouldn't random order show the default order as well?

DenisChenu

DenisChenu

2020-05-17 18:57

developer   ~57857

Shouldn't random order show the default order as well?

? random order must be always for public survey, never in stat or elsewhere

gabrieljenik

gabrieljenik

2020-09-10 19:23

manager   ~59774

Last edited: 2020-09-10 19:29

PR: https://github.com/LimeSurvey/LimeSurvey/pull/1587

gabrieljenik

gabrieljenik

2020-09-17 11:18

manager   ~59859

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

lime_release_bot

lime_release_bot

2020-09-21 14:54

administrator   ~59902

Fixed in Release 4.3.17+200921

Related Changesets

LimeSurvey: master ce507162

2020-09-10 21:21

gabrieljenik


Details Diff
Fixed issue 16229: Random answer order currently broken

Only ordering answers if the question have subquestions.
Affected Issues
16229
mod - application/models/Question.php Diff File

LimeSurvey: master 8b685f35

2020-09-17 13:18

ollehar1

Committer: GitHub


Details Diff
Fixed issue 16229: Random answer order currently broken

Dev: Merge pull request #1587 from gabrieljenik/bug/16229--Random-answer-order-currently-broken
Affected Issues
16229
mod - application/models/Question.php Diff File

Issue History

Date Modified Username Field Change
2020-05-05 15:13 ollehar New Issue
2020-05-05 15:15 DenisChenu Note Added: 57541
2020-05-05 15:15 ollehar Note Added: 57542
2020-05-05 15:16 ollehar Note Added: 57543
2020-05-05 15:17 DenisChenu Note Added: 57544
2020-05-15 15:06 ollehar Priority none => normal
2020-05-17 15:39 cdorin Note Added: 57856
2020-05-17 18:57 DenisChenu Note Added: 57857
2020-09-09 10:14 cdorin Priority normal => high
2020-09-09 10:14 cdorin Severity minor => partial_block
2020-09-09 15:39 cdorin Assigned To => gabrieljenik
2020-09-09 15:39 cdorin Status new => assigned
2020-09-09 15:56 cdorin Zoho Sprints => |Yes|
2020-09-10 19:23 gabrieljenik Note Added: 59774
2020-09-10 19:29 gabrieljenik Note Edited: 59774
2020-09-17 11:18 ollehar1 Changeset attached => LimeSurvey master 8b685f35
2020-09-17 11:18 gabrieljenik Changeset attached => LimeSurvey master ce507162
2020-09-17 11:18 gabrieljenik Note Added: 59859
2020-09-17 11:18 gabrieljenik Resolution open => fixed
2020-09-21 14:54 lime_release_bot Zoho Sprints Yes => |Yes|
2020-09-21 14:54 lime_release_bot Note Added: 59902
2020-09-21 14:54 lime_release_bot Status assigned => closed