View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
14039 | Bug reports | Survey taking | public | 2018-09-08 00:58 | 2021-07-12 11:53 |
Reporter | jelo | Assigned To | gabrieljenik | ||
Priority | high | Severity | block | ||
Status | closed | Resolution | fixed | ||
Product Version | 3.19.2 | ||||
Summary | 14039: Captcha prevent panel integration. GET URL parameter not captured. | ||||
Description | When Captchas are activated GET URL parameter are not captured by LimeSurvey. | ||||
Tags | No tags attached. | ||||
Bug heat | 14 | ||||
Complete LimeSurvey version number (& build) | 3.14.8+180829 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | MySQL | ||||
Server OS (if known) | CentOS 7 | ||||
Webserver software & version (if known) | Apache 2.4.X | ||||
PHP Version | 7.2.9 | ||||
is it in relation with this forum post? https://www.limesurvey.org/forum/installation-a-update-issues/116365-passthru-version-3-0#173625 |
|
I also hit this problem, anyone knows that the issue is being investigated? |
|
I have the same problem. The get parameter lost if I enable "Use CAPTCHA for survey access" |
|
Can we please get an lss file and a detailed step-by-step description on how to reproduce? Developers read and test multiple bug reports per day, and guessing what should be done takes a lot of time. Thank you. |
|
Hi Ollehar, Bug shown on Step 2 and Step 2a. Step 1. Under Publication & Access: Use CAPTCHA for survey access OFF example if the survey number is 175717 Step 2. Under Publication & Access: Use CAPTCHA for survey access ON Expect text "panelIDebug14039" capture at question Q1 |
|
What do you think? Problem DiagnosisThe function that catches URL parameters is not called when showing token form or captcha. Technically SpeakingPrefill values are read from the URL parameters on prefillFromCommandLine(), which is called from buildsurveysession(). But buildsurveysession() is not called when the captcha page needs to be displayed. Solution Alternative AAdd URL parameters to the captcha form, so they are included in the next request:
Solution Alternative BIf the captcha form needs to be displayed, store the additional URL parameter in the session:
into this:
Option B caveats: If the captcha form is displayed and, instead of submitting the form, the user goes to a different page, the "saved" query parameters will remain in the session. Solution Alternative CSame as A but, parameters wouldn't forwarded in the URL but in hidden. We would need to create a specific twig extension that creates hidden inputs for query string parameters. Downside: Custom themes would need amendment. |
|
Solution A sounds to be the most reasonable to me. |
|
@jelo Time to test this PR? |
|
Fix committed to 3.x-LTS branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=31951 |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=31952 |
|
Release done. |
|
LimeSurvey: 3.x-LTS 395f9f83 2021-06-04 15:56 Committer: GitHub Details Diff |
Fixed issue 14039: Captcha prevent panel integration. GET URL parameter not captured (#1896) Co-authored-by: encuestabizdevgit <devgit@encuesta.biz> |
Affected Issues 14039 |
|
mod - application/core/LSHttpRequest.php | Diff File | ||
mod - application/helpers/frontend_helper.php | Diff File | ||
LimeSurvey: master fca9950f 2021-06-04 15:56 Committer: ollehar Details Diff |
Fixed issue 14039: Captcha prevent panel integration. GET URL parameter not captured (#1896) Co-authored-by: encuestabizdevgit <devgit@encuesta.biz> |
Affected Issues 14039 |
|
mod - application/core/LSHttpRequest.php | Diff File | ||
mod - application/helpers/frontend_helper.php | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2018-09-08 00:58 | jelo | New Issue | |
2018-09-10 11:05 |
|
Assigned To | => LouisGac |
2018-09-10 11:05 |
|
Status | new => feedback |
2018-09-10 11:51 |
|
Note Added: 48997 | |
2019-03-07 09:46 | hkjoscsc | Issue Monitored: hkjoscsc | |
2019-03-07 09:53 | hkjoscsc | Note Added: 50821 | |
2021-02-08 16:51 | ollehar | Product Version | => 3.19.2 |
2021-02-24 03:15 | thomasleeca | Note Added: 62475 | |
2021-03-01 15:53 | c_schmitz | Assigned To | LouisGac => |
2021-03-01 15:53 | c_schmitz | Status | feedback => acknowledged |
2021-03-01 21:36 | ollehar | Priority | none => high |
2021-03-01 21:36 | ollehar | Severity | crash => block |
2021-03-05 14:52 | ollehar | Assigned To | => ollehar |
2021-03-05 14:52 | ollehar | Status | acknowledged => feedback |
2021-03-05 14:52 | ollehar | Note Added: 62764 | |
2021-03-05 16:29 | thomasleeca | Note Added: 62785 | |
2021-03-05 16:29 | thomasleeca | File Added: limesurvey_survey_175717.lss | |
2021-03-05 16:46 | ollehar | Status | feedback => assigned |
2021-05-21 15:09 | gabrieljenik | Note Added: 64510 | |
2021-05-21 16:15 | c_schmitz | Note Added: 64511 | |
2021-05-26 18:30 | gabrieljenik | Note Added: 64574 | |
2021-05-27 16:06 | ollehar | Status | assigned => ready for testing |
2021-05-27 16:06 | ollehar | Note Added: 64597 | |
2021-06-02 14:41 | ollehar | Sync to Zoho Project | => |Yes| |
2021-06-04 13:56 | gabrieljenik | Changeset attached | => LimeSurvey 3.x-LTS 395f9f83 |
2021-06-04 13:56 | gabrieljenik | Note Added: 64740 | |
2021-06-04 13:56 | gabrieljenik | Assigned To | ollehar => gabrieljenik |
2021-06-04 13:56 | gabrieljenik | Resolution | open => fixed |
2021-06-04 13:58 | ollehar | Changeset attached | => LimeSurvey master fca9950f |
2021-06-04 13:58 | gabrieljenik | Note Added: 64741 | |
2021-06-05 11:19 | ollehar | Status | ready for testing => resolved |
2021-07-12 11:53 | c_schmitz | Note Added: 65285 | |
2021-07-12 11:53 | c_schmitz | Status | resolved => closed |
2021-08-02 20:30 | guest | Bug heat | 12 => 14 |