View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
16849 | Bug reports | Other | public | 2020-11-20 13:15 | 2020-12-28 10:42 |
Reporter | cesoielampo | Assigned To | gabrieljenik | ||
Priority | normal | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Product Version | 3.23.6 | ||||
Summary | 16849: Question radio button with Question Theme "Bootstrap Buttons" does not keep value in "Other" field | ||||
Description | Question radio button with Question Theme "Bootstrap Buttons" does not keep value in "Other" field and does not show "Other field" going to the previous page of the survey | ||||
Steps To Reproduce |
See attached survey. | ||||
Additional Information | That doesn't happen without the theme "Bootstrap Button" | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Bug heat | 4 | ||||
Complete LimeSurvey version number (& build) | 3.23.6+200929 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | Any | ||||
Database type & version | MySQL | ||||
Server OS (if known) | |||||
Webserver software & version (if known) | |||||
PHP Version | 7.2.33 | ||||
related to | 16895 | closed | gabrieljenik | Question radio button with Question Theme "Bootstrap Buttons" does not keep value in "Other" field |
https://github.com/LimeSurvey/LimeSurvey/pull/1681/files To check if this happens on LS4 too. |
|
Fix committed to 3.x-LTS branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=30799 |
|
Fixed in Release 3.25.5+201222 |
|
LimeSurvey: 3.x-LTS 665c34d5 2020-12-14 12:49 Committer: GitHub Details Diff |
Fixed issue 16849: Question radio button with Question Theme "Bootstrap Buttons" does not keep value in "Other" field (#1681) There were two problems: 1) The 'other' answer value from the DB was never used by the theme to feed it into the screen ($answer_other was not used in answer_row_other.twig). 2) The show/hide code is part of the radio's 'change' event, which was only trigger on an actual change, not initially to setup the screen An additional problem is that, in this theme, the 'other' input control is outside the scope of answer_row_other.twig, so the value cannot be directly assigned when rendering. Instead, a hidden input had to be added to hold the value until it can be transfered by javascript to the actual control. At last, as the doBootstrapRadio is called multiple times (once per question in the screen), the event assignment happens multiple times as well. |
Affected Issues 16849 |
|
mod - themes/question/bootstrap_buttons/survey/questions/answer/listradio/assets/scripts/bootstrapbuttons.js | Diff File | ||
mod - themes/question/bootstrap_buttons/survey/questions/answer/listradio/rows/answer_row_other.twig | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2020-11-20 13:15 | cesoielampo | New Issue | |
2020-11-20 13:15 | cesoielampo | File Added: limesurvey_survey_789424.lss | |
2020-11-20 17:00 | cdorin | Priority | none => normal |
2020-11-20 17:00 | cdorin | Status | new => confirmed |
2020-11-20 17:00 | cdorin | Sync to Zoho Project | => |Yes| |
2020-12-03 20:45 | gabrieljenik | Note Added: 60834 | |
2020-12-10 15:25 | gabrieljenik | Issue cloned: 16895 | |
2020-12-10 15:25 | gabrieljenik | Relationship added | related to 16895 |
2020-12-14 11:49 | gabrieljenik | Changeset attached | => LimeSurvey 3.x-LTS 665c34d5 |
2020-12-14 11:49 | gabrieljenik | Note Added: 60943 | |
2020-12-14 11:49 | gabrieljenik | Assigned To | => gabrieljenik |
2020-12-14 11:49 | gabrieljenik | Resolution | open => fixed |
2020-12-28 10:42 | lime_release_bot | Sync to Zoho Project | Yes => |Yes| |
2020-12-28 10:42 | lime_release_bot | Note Added: 61173 | |
2020-12-28 10:42 | lime_release_bot | Status | confirmed => closed |