View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
06708 | Bug reports | Survey participants (Tokens) | public | 2012-10-15 20:46 | 2012-10-17 10:39 |
Reporter | helper | Assigned To | c_schmitz | ||
Priority | low | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.00+ | ||||
Fixed in Version | 2.00+ | ||||
Summary | 06708: $register_errormsg not displaying at signup | ||||
Description | The global $register_errormsg is not showing up at register when errors occur (for example, missing email, bad answer for captcha). Note: the errormsg appears in NON-token mode. | ||||
Steps To Reproduce | Activate a survey with the following: basic template... Anonymized responses? Responses to this survey are NOT anonymized. Cannot be changed Generate tokens at activation Go to survey and attempt to sign up - do not answer the captcha or give a bad answer. No error appears to the end user. | ||||
Tags | No tags attached. | ||||
Bug heat | 4 | ||||
Complete LimeSurvey version number (& build) | 121014 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | IE8, FF, Chrome | ||||
Database type & version | mysqlnd 5.0.10 - 20111026 | ||||
Server OS (if known) | Windows 7 | ||||
Webserver software & version (if known) | Apache 2.4 | ||||
PHP Version | 5.4 | ||||
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=9886 |
|
New 2.00+ Build 121016 released. |
|
The fix induced another bug - did not know whether to open another bug or reopen this one. Steps to reproduce:
Undefined variable: surveyid C:\xampp\htdocs\limesurvey\application\helpers\replacements_helper.php(771) Suggestion for fix (replace line 771 with the following): if (isset($surveyid) && isset($SESSION['survey'.$surveyid]) && isset($SESSION['survey'.$surveyid]['register_errormsg'])) |
|
The fix to replacements_helper.php is missing a istantiation of the global $register_errormsg. Appears that there is a cross over here in there is a change towards an array ($thissurvey[register_errormsg]) - the new logic:
will not work with the $thissurvey[register_errormsg]. We need to either set a global for $register_errormsg or finish moving to the array. |
|
That part is obsolete and will be removed. |
|
LimeSurvey: master 13ae7772 2012-10-16 04:10 Details Diff |
Fixed issue 06708: Error messages not displaying at survey signup when user did not completely/correctly fill out the form |
Affected Issues 06708 |
|
mod - application/controllers/RegisterController.php | Diff File | ||
mod - application/helpers/replacements_helper.php | Diff File | ||
mod - scripts/admin/questions.js | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2012-10-15 20:46 | helper | New Issue | |
2012-10-16 10:55 | c_schmitz | Assigned To | => c_schmitz |
2012-10-16 10:55 | c_schmitz | Status | new => assigned |
2012-10-16 11:11 | c_schmitz | Status | assigned => resolved |
2012-10-16 11:11 | c_schmitz | Fixed in Version | => 2.00+ |
2012-10-16 11:11 | c_schmitz | Resolution | open => fixed |
2012-10-16 11:11 | c_schmitz | Changeset attached | => LimeSurvey master 13ae7772 |
2012-10-16 11:11 | c_schmitz | Note Added: 21351 | |
2012-10-16 12:23 | c_schmitz | Note Added: 21357 | |
2012-10-16 12:23 | c_schmitz | Status | resolved => closed |
2012-10-16 13:49 | helper | Note Added: 21363 | |
2012-10-16 13:49 | helper | Status | closed => feedback |
2012-10-16 13:49 | helper | Resolution | fixed => reopened |
2012-10-16 19:24 | helper | Note Added: 21383 | |
2012-10-16 19:24 | helper | Status | feedback => assigned |
2012-10-17 10:39 | c_schmitz | Note Added: 21399 | |
2012-10-17 10:39 | c_schmitz | Status | assigned => closed |
2012-10-17 10:39 | c_schmitz | Resolution | reopened => fixed |
2016-12-08 10:39 | c_schmitz | Category | Tokens => Survey participants (Tokens) |