View Issue Details

Related ChangesetsJump to NotesJump to History
This issue affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
11713Bug reportsSurvey takingpublic2016-09-27 12:272016-10-04 17:11
ReporterMazi Assigned Toc_schmitz  
PrioritynoneSeverityminor 
Status closedResolutionfixed 
Product Version2.52.x 
Fixed in Version2.52.x 
Summary11713: Misleading error message when setting an expiration date
Description

If an expiration date is set for a survey and someone still tries to access the survey, they see an error message like "This survey is no longer available". This sounds like the survey was deleted.
Please extend the message by adding something like "It expired YYYY-MM-DD" to make it more clear that the survey just expired.

Steps To Reproduce

Set an expiry date in the past and start the survey.

TagsNo tags attached.
Bug heat6
Complete LimeSurvey version number (& build)160821
I will donate to the project if issue is resolvedNo
BrowserChrome
Database type & versionMySQL 5
Server OS (if known)Linux
Webserver software & version (if known)Apache 2
PHP Version5.5

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2016-09-27 12:30

developer   ~40972

Solution : https://framagit.org/SondagePro-LimeSurvey-plugin/adaptEnterErrorSurveyText :)

See https://framagit.org/SondagePro-LimeSurvey-plugin/adaptEnterErrorSurveyText/blob/master/adaptEnterErrorSurveyText.php#L47 for available sentence :)
Mazi

Mazi

2016-09-27 12:34

updater   ~40974

Nice, but I think this has to be improved in core, not by another plugin.
DenisChenu

DenisChenu

2016-09-27 12:42

developer   ~40975

In fact : i'm unsure all user want to see the date in this screen. I really think default is OK except in some situation, and in this situation : all sentence must be updatable.
c_schmitz

c_schmitz

2016-09-27 15:12

administrator   ~40976

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

c_schmitz

2016-09-27 15:13

administrator   ~40977

I agree with Denis. I made the message a bit more clear but left out the date.
Mazi

Mazi

2016-09-27 15:34

updater   ~40978

That new message is far better, thanks!
DenisChenu

DenisChenu

2016-09-27 16:02

developer   ~40980

Yep, great :) really best
c_schmitz

c_schmitz

2016-10-04 17:11

administrator   ~41115

Version 2.53+161004 released

Related Changesets

LimeSurvey: master 4348c76d

2016-09-27 17:12

c_schmitz


Details Diff		 Fixed issue 11713: Unclear message when setting an expiration date Affected Issues
11713
mod - application/controllers/survey/index.php Diff File

Issue History

Date Modified Username Field Change
2016-09-27 12:27 Mazi New Issue
2016-09-27 12:30 DenisChenu Note Added: 40972
2016-09-27 12:34 Mazi Note Added: 40974
2016-09-27 12:42 DenisChenu Note Added: 40975
2016-09-27 15:12 c_schmitz Changeset attached => LimeSurvey master 4348c76d
2016-09-27 15:12 c_schmitz Note Added: 40976
2016-09-27 15:12 c_schmitz Assigned To => c_schmitz
2016-09-27 15:12 c_schmitz Resolution open => fixed
2016-09-27 15:13 c_schmitz Status new => resolved
2016-09-27 15:13 c_schmitz Fixed in Version => 2.52.x
2016-09-27 15:13 c_schmitz Note Added: 40977
2016-09-27 15:34 Mazi Note Added: 40978
2016-09-27 16:02 DenisChenu Note Added: 40980
2016-10-04 17:11 c_schmitz Note Added: 41115
2016-10-04 17:11 c_schmitz Status resolved => closed