View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
08410 | Bug reports | Survey taking | public | 2013-11-26 13:57 | 2013-12-02 21:26 |
Reporter | fairsay | Assigned To | c_schmitz | ||
Priority | normal | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.00+ | ||||
Fixed in Version | 2.00+ | ||||
Summary | 08410: Save fails on final submit for token with participant_id | ||||
Description | I have a survey (using build 131122) which is 'closed' but for which people can register and has a token table populated from the participant database. Whenever someone new registers, they complete the survey with no problem (it seems). But when someone with a token and a participant_id goes to complete the survey, it fails on the last step. The error message when the config.php debug = 2 is: | ||||
Steps To Reproduce | 1) Populate participants | ||||
Tags | No tags attached. | ||||
Bug heat | 4 | ||||
Complete LimeSurvey version number (& build) | 131122 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | Chrome, Safari | ||||
Database type & version | MySQL 5.0 | ||||
Server OS (if known) | Amazon Linux | ||||
Webserver software & version (if known) | Apache 2.2.25 | ||||
PHP Version | 5.3.27 | ||||
You should not enforce strict standards on production machines. |
|
I only did it temporarily (<30 seconds) to see if I could find the error that was causing the problems I was encountering. |
|
I've resorted to a workaround that seems to work: setting participant_id to NULL in to tokens table. Not sure if anyone can replicate this or if it is a problem only I have encountered (logging it here in case others encounter it). Description looks similar to 07278 except that this issue (08410) also includes a participant_id value. |
|
You wrote 'it fails on the last step.' How does it exactly fail? |
|
Sorry - should have been clearer. In production mode, it fails in two ways: When I make the participant_id null in the tokens table, both these issues go away and things work normally. |
|
I see - Although it is also a kink in the source code the basic problem is still that your PHP is configured to strict standards (it does not matter if debug is on or off). PHP recommends to not show use STRICT error level on production machines. |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13517 |
|
Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13518 |
|
Great - I just implemented the code on my current version and it seems to work with both the scenario where participant_id exists in the token table and where it is null. As for the the php strict error level, I'll investigate how to reduce it. I hadn't set it explicitly to anything but will now look to take it off strict. |
|
Version 2.00+ Build 131202 released |
|
LimeSurvey: master 55cb1972 2013-11-27 13:56 Details Diff |
Fixed issue 08410: Save fails on final submit for token with participant_id |
Affected Issues 08410 |
|
mod - application/helpers/frontend_helper.php | Diff File | ||
LimeSurvey: 2.05 a9bc05a8 2013-11-27 14:05 Details Diff |
Fixed issue 08410: Save fails on final submit for token with participant_id |
Affected Issues 08410 |
|
mod - application/helpers/frontend_helper.php | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2013-11-26 13:57 | fairsay | New Issue | |
2013-11-26 14:11 | c_schmitz | Note Added: 27406 | |
2013-11-26 14:48 | fairsay | Note Added: 27407 | |
2013-11-26 17:22 | fairsay | Note Added: 27411 | |
2013-11-26 22:17 | fairsay | Note Edited: 27411 | |
2013-11-27 12:07 | c_schmitz | Note Added: 27418 | |
2013-11-27 12:07 | c_schmitz | Assigned To | => c_schmitz |
2013-11-27 12:07 | c_schmitz | Status | new => feedback |
2013-11-27 12:14 | fairsay | Note Added: 27419 | |
2013-11-27 12:14 | fairsay | Status | feedback => assigned |
2013-11-27 13:52 | c_schmitz | Note Added: 27424 | |
2013-11-27 13:56 | c_schmitz | Changeset attached | => LimeSurvey master 55cb1972 |
2013-11-27 13:56 | c_schmitz | Note Added: 27425 | |
2013-11-27 13:56 | c_schmitz | Resolution | open => fixed |
2013-11-27 13:56 | c_schmitz | Status | assigned => resolved |
2013-11-27 13:56 | c_schmitz | Fixed in Version | => 2.00+ |
2013-11-27 14:04 | c_schmitz | Changeset attached | => LimeSurvey 2.05 a9bc05a8 |
2013-11-27 14:04 | c_schmitz | Note Added: 27426 | |
2013-11-27 14:23 | fairsay | Note Added: 27427 | |
2013-12-02 21:26 | c_schmitz | Note Added: 27470 | |
2013-12-02 21:26 | c_schmitz | Status | resolved => closed |