View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
08477 | Bug reports | Survey taking | public | 2013-12-19 18:38 | 2014-01-25 17:25 |
Reporter | abezverkhyy | Assigned To | c_schmitz | ||
Priority | normal | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.50.x | ||||
Fixed in Version | 2.05+ | ||||
Summary | 08477: regression when session expires: in 1.92 LS showed a dedicated message, now it silently returns to the first page | ||||
Description | Regression when session expires: in 1.92 LS showed a dedicated message, now it silently returns to the first page. Therefore the user isn't informed that he potentially lost data he typed in. | ||||
Steps To Reproduce | Make a survey with several steps. | ||||
Additional Information | The code handling the session expiration message has been commented out in application/controllers/survey/index.php line 147 when LS migrated to Yii. I don't know if Yii supports detection of lost session. An alternative solution can be to add a meta refresh tag with a timeout equal to the session max lifetime and destination equal to an url which displays the lost session message. | ||||
Tags | No tags attached. | ||||
Bug heat | 6 | ||||
Complete LimeSurvey version number (& build) | c51d934 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | not relevant | ||||
Server OS (if known) | not relevant | ||||
Webserver software & version (if known) | not relevant | ||||
PHP Version | not relevant | ||||
Hi, Maybe we can do something like this: I don't like alert, surely it's better if we have already a alert replacement. Maybe time to have a 'alert' replacement (jquery dialog for example). Denis |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13699 |
|
I fixed it with a JS timer and redirection, when the user is redirected to ?sessionExpired the old error message is displayed. What do you think about this solution ? |
|
Sorry don't found this one. I think redirectiong must be a template issue only. If i do it , i do same system than token mismatch. And here you are not sure for ?/& need. Create the url with Yii seem better. I think show a clean message in HTML with a link to restart the same survey (but WITHHOUT token if exist) is clean enough. |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13700 |
|
2.05+ Build 140125 released |
|
LimeSurvey: master 22b0ced5 2014-01-16 17:56 Details Diff |
Fixed issue 08477: a clear error message is now displayed when a session expires |
Affected Issues 08477 |
|
mod - application/controllers/survey/index.php | Diff File | ||
LimeSurvey: master f5b5a018 2014-01-17 08:05 Details Diff |
Fixed issue 08477: a clear error message is now displayed when a session expires Dev Also reverted commit 22b0ced540286ea6c0d796ea64ef8009007c400e |
Affected Issues 08477 |
|
mod - application/controllers/survey/index.php | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2013-12-19 18:38 | abezverkhyy | New Issue | |
2013-12-21 19:23 | DenisChenu | Note Added: 27689 | |
2014-01-14 12:07 | c_schmitz | Assigned To | => c_schmitz |
2014-01-14 12:07 | c_schmitz | Status | new => assigned |
2014-01-16 17:58 | abezverkhyy | Changeset attached | => LimeSurvey master 22b0ced5 |
2014-01-16 17:58 | abezverkhyy | Note Added: 27919 | |
2014-01-16 17:58 | abezverkhyy | Assigned To | c_schmitz => abezverkhyy |
2014-01-16 17:58 | abezverkhyy | Resolution | open => fixed |
2014-01-16 18:00 | abezverkhyy | Note Added: 27920 | |
2014-01-16 18:01 | abezverkhyy | Status | assigned => resolved |
2014-01-16 18:42 | DenisChenu | Note Added: 27923 | |
2014-01-17 08:04 | c_schmitz | Changeset attached | => LimeSurvey master f5b5a018 |
2014-01-17 08:04 | c_schmitz | Note Added: 27929 | |
2014-01-17 08:04 | c_schmitz | Assigned To | abezverkhyy => c_schmitz |
2014-01-25 17:25 | c_schmitz | Note Added: 28169 | |
2014-01-25 17:25 | c_schmitz | Status | resolved => closed |
2014-01-25 17:25 | c_schmitz | Fixed in Version | => 2.05+ |