View Issue Details

This bug affects 1 person(s).
 8
IDProjectCategoryView StatusLast Update
10643Bug reportsStatisticspublic2016-04-01 11:36
ReporterDrewskie Assigned ToLouisGac 
PriorityhighSeverityminor 
Status closedResolutionfixed 
Product Version2.50.x 
Fixed in Version2.50.x 
Summary10643: Active Surveys included expired
Description

The active surveys listed on the admin panel in Survey List include expired.

Once a survey is expired it is no longer active.

Steps To Reproduce

Set survey as expired and will still be shown in the list as active

TagsNo tags attached.
Attached Files
active survey.JPG (48,344 bytes)   
active survey.JPG (48,344 bytes)   
survey is expired.JPG (103,006 bytes)
Bug heat8
Complete LimeSurvey version number (& build)160228
I will donate to the project if issue is resolvedNo
BrowserIE11
Database type & versionSQL 2008
Server OS (if known)Windows 2012
Webserver software & version (if known)IIS8
PHP Version5.4.24

Users monitoring this issue

Drewskie

Activities

c_schmitz

c_schmitz

2016-03-11 15:20

administrator   ~36430

Strictly speaking the expired surveys are activated, so the survey list itself is correct.
However we see your point and so will fix the 'Active' view to only list really active surveys.

We will hoever also add back in a visual indicator (icon) to indicate if a survey is not activated, activated, activated but yet started, activated but expired.

LouisGac

LouisGac

2016-03-15 13:05

developer   ~36507

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

Drewskie

Drewskie

2016-03-15 14:04

reporter   ~36509

Last edited: 2016-03-15 14:04

Need to update drop down list (Any state) / Yes / No / Expired
Also on the top of the screen I think it should only show the count for currently running (not include expired)

and thanks for the fix :)

LouisGac

LouisGac

2016-03-15 14:26

developer   ~36510

ok why not

LouisGac

LouisGac

2016-03-16 11:57

developer   ~36566

https://github.com/LimeSurvey/LimeSurvey/commit/9d623e1a61cb42120bd0d64f673bbcd2f6ea2db6

Drewskie

Drewskie

2016-03-30 10:21

reporter   ~36827

Thanks for the update, the only option missing is for ones that are not expired and active.

Would think it would fall under yes but this includes both.

LouisGac

LouisGac

2016-03-30 10:46

developer   ~36828

"active: yes" shows all survey with statistics.

Drewskie

Drewskie

2016-03-30 11:53

reporter   ~36831

If a person wants to send out reminders quickly, the eisiest way would be to show a list of active not expired surveys.

Just my 2c.

LouisGac

LouisGac

2016-03-30 12:05

developer   ~36832

ok can be easy added. question is : what filter name ? I'd say "running" ?

LouisGac

LouisGac

2016-03-30 12:12

developer   ~36833

Last edited: 2016-03-30 12:20

by the way, carsten asked me to remove the delete button from the listing (it's a "only once action"). Maybe I could replace it by a button to "send out reminders" ?

Drewskie

Drewskie

2016-03-30 12:15

reporter   ~36834

running sounds good.

Reminder may be a nice addition, less clicks always make admins happy.

c_schmitz

c_schmitz

2016-04-01 11:36

administrator   ~36883

Version 2.50+ Build 160401 released

Related Changesets

LimeSurvey: master 0b29644b

2016-03-15 12:05:10

LouisGac

Details Diff
Fixed issue 10643: Active Surveys included expired Affected Issues
10643
mod - application/models/Survey.php Diff File
mod - application/views/admin/survey/listSurveys_view.php Diff File

Issue History

Date Modified Username Field Change
2016-03-02 07:25 Drewskie New Issue
2016-03-02 07:25 Drewskie File Added: active survey.JPG
2016-03-02 07:25 Drewskie Issue Monitored: Drewskie
2016-03-02 07:25 Drewskie File Added: survey is expired.JPG
2016-03-02 15:30 c_schmitz Assigned To => c_schmitz
2016-03-02 15:30 c_schmitz Status new => assigned
2016-03-11 15:20 c_schmitz Note Added: 36430
2016-03-11 17:14 c_schmitz Assigned To c_schmitz => ollehar
2016-03-15 10:56 LouisGac Assigned To ollehar => LouisGac
2016-03-15 10:57 LouisGac Priority normal => high
2016-03-15 13:05 LouisGac Changeset attached => LimeSurvey master 0b29644b
2016-03-15 13:05 LouisGac Note Added: 36507
2016-03-15 13:05 LouisGac Resolution open => fixed
2016-03-15 14:04 Drewskie Note Added: 36509
2016-03-15 14:04 Drewskie Note Edited: 36509
2016-03-15 14:26 LouisGac Note Added: 36510
2016-03-16 11:57 LouisGac Note Added: 36566
2016-03-16 11:57 LouisGac Status assigned => resolved
2016-03-16 11:58 LouisGac Fixed in Version => 2.5
2016-03-30 10:21 Drewskie Note Added: 36827
2016-03-30 10:46 LouisGac Note Added: 36828
2016-03-30 11:53 Drewskie Note Added: 36831
2016-03-30 12:05 LouisGac Note Added: 36832
2016-03-30 12:12 LouisGac Note Added: 36833
2016-03-30 12:15 Drewskie Note Added: 36834
2016-03-30 12:20 LouisGac Note Edited: 36833
2016-04-01 11:36 c_schmitz Note Added: 36883
2016-04-01 11:36 c_schmitz Status resolved => closed
2021-08-08 06:01 guest Bug heat 6 => 8