View Issue Details

IDProjectCategoryView StatusLast Update
14974Bug reports[All Projects] Otherpublic2019-06-25 13:01
ReporterMaziAssigned To 
PrioritynoneSeverityminor 
Status newResolutionopen 
Product Version3.17.x 
Target VersionFixed in Version 
Summary14974: Survey list can't be sorted by completes (and similar columns)
Description

At the survey list overview the list can't be sorted by the last 4 columns, all other columns are working fine, see screenshot.

Steps To Reproduce

Call the survey list and sort e.g. by completes.

TagsNo tags attached.
Complete LimeSurvey version number (& build)Version 3.17.5+190604
I will donate to the project if issue is resolvedNo
BrowserChrome
Database & DB-VersionMySQL 5
Server OS (if known)Ubuntu 14 TLS
Webserver software & version (if known)Apache 2
PHP Version7.2

Activities

Mazi

Mazi

2019-06-14 12:12

partner  

survey_list.png (62,492 bytes)
survey_list.png (62,492 bytes)
cdorin

cdorin

2019-06-24 17:05

manager   ~52515

I am not sure if you were able to sort them in the past (just tested it in 2.73 and 2.50). Looks more like a feature request.

@DenisChenu, do you know more?

DenisChenu

DenisChenu

2019-06-24 17:10

developer   ~52517

It's a feature request :).

Previously (2.6lts) : we load WHOLE survey and use jqgrid , then we can sort by anything. BUT : if you more than 150 survey it take a lot of time for loading.

Here : we need to create false attributes in https://github.com/LimeSurvey/LimeSurvey/blob/7f14f95efcdac03b59d6fdf705905afbcfcf7dfa/application/models/Survey.php#L1525 it can be done, but it's more a new feature iin my opinon.

PS : another feature is have a filter to each column , and can have dropdown for some (Group/Owner/Anonymized).

Mazi

Mazi

2019-06-24 17:34

partner   ~52520

@cdorin, we should actually name it a regression because at previous LS versions like 2.6.7 LTS that was doable.

DenisChenu

DenisChenu

2019-06-24 23:17

developer   ~52521

2.6.7lts with more than 300 survey‘s survey list i unusuable.
3.X with more than 2000 survey's still usauble without issue.

Not really a regression

cdorin

cdorin

2019-06-25 12:24

manager   ~52532

As Denis underlined, this will not work with big instances where there are lots of surveys in the survey list. I also talked to Carsten and it seems not feasible (at least not at the moment)

DenisChenu

DenisChenu

2019-06-25 12:40

developer   ~52535

It's feasible, see https://github.com/LimeSurvey/LimeSurvey/commit/3337b5ae3efee04a5153485c610aeea997985698 for a sample

But : this need lot of work since is adding «false» attribute and create new setter/getter
And Survey::search need to be completely reworked before … to use cleaner Yii way.

cdorin

cdorin

2019-06-25 13:01

manager   ~52539

Will bring the topic again after the new devs join :)

Issue History

Date Modified Username Field Change
2019-06-14 12:12 Mazi New Issue
2019-06-14 12:12 Mazi File Added: survey_list.png
2019-06-24 17:05 cdorin Note Added: 52515
2019-06-24 17:10 DenisChenu Note Added: 52517
2019-06-24 17:34 Mazi Note Added: 52520
2019-06-24 23:17 DenisChenu Note Added: 52521
2019-06-25 12:24 cdorin Note Added: 52532
2019-06-25 12:40 DenisChenu Note Added: 52535
2019-06-25 13:01 cdorin Note Added: 52539