View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
16236 | Bug reports | Theme editor | public | 2020-05-06 18:15 | 2020-07-07 16:20 |
Reporter | DenisChenu | Assigned To | |||
Priority | high | Severity | crash | ||
Status | closed | Resolution | fixed | ||
Product Version | 4.2.2 | ||||
Summary | 16236: Option of Vanilla theme are not saved | ||||
Description | When try to set an option on Vanilla theme : nothing are saved | ||||
Steps To Reproduce | See screencast | ||||
Additional Information | Seems to ba saved in Global part | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Bug heat | 18 | ||||
Complete LimeSurvey version number (& build) | 4.2.2 github | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | ff | ||||
Database type & version | not relevant? | ||||
Server OS (if known) | not relevant ? | ||||
Webserver software & version (if known) | not relevant ? | ||||
PHP Version | not relevant ? | ||||
@cdorin, this also seems to break all our templates which are based on vanilla. Please fix soon! |
|
it is in our to-do for this sprint |
|
Thanks for the feedback! |
|
Tested in latest version. When i click on Save it is redirected to the attached screen |
|
@cdorin, this should have been included at a sprint last month. Sushmanadendla has tested but it looks like this is still not working as expected, did I get that right? |
|
@mazi Yeah well it was included in the last sprint but there is lot of workaround required to fix the issue it is taking time,hopefully it will be available in next sprint. |
|
Vanilla is not using the core options page, and the custom one is not properly triggering the form submit. So there are two options: I think option 2 is the best one as to have What do you think? |
|
With option 2 : you broke all theme (again and again and again …)
NO ! because there are system that can NOT BE DONE WITH CURENT OPTION. |
|
Option #1 implemented: BTW, I have tried option 2 twice and haven't had any problems. I understand themes which inherited from Vanilla maybe imepacted (not sure, I am doubting about that) . |
|
OK : if it comes by I think you replace core part, not template part |
|
Tested issue by pulling the PR. Able to save the changes successfully. Please refer the screenshot for more details |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=30239 |
|
Fixed in Release 4.3.3+200707 |
|
I also did some testing. If I didn't miss anything, everyone using a custom theme with adjusted theme options also has to update the theme's options.js file with the updated one of the vanilla theme?! |
|
|
|
@denis: what do you mean about this? |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2020-05-06 18:15 | DenisChenu | New Issue | |
2020-05-06 18:15 | DenisChenu | File Added: Peek 06-05-2020 18-13.gif | |
2020-05-07 13:35 | ollehar | Priority | none => high |
2020-05-07 13:35 | ollehar | Severity | minor => crash |
2020-05-17 23:48 | cdorin | Status | new => confirmed |
2020-05-17 23:48 | cdorin | Zoho Sprints | => |Yes| |
2020-05-17 23:48 |
|
Zoho Sprints ID | => 14469000000091037 |
2020-05-19 15:47 | Mazi | Note Added: 57936 | |
2020-05-19 16:46 | cdorin | Note Added: 57940 | |
2020-05-19 17:50 | Mazi | Note Added: 57944 | |
2020-06-03 16:25 |
|
Zoho Sprints | Yes => |Yes| |
2020-06-03 16:26 |
|
Zoho Sprints | Yes => |Yes| |
2020-06-03 16:27 |
|
Note Added: 58160 | |
2020-06-03 16:27 |
|
File Added: VanillaTheme_16236.png | |
2020-06-03 16:29 |
|
Zoho Sprints | Yes => |Yes| |
2020-06-03 16:29 |
|
Note Edited: 58160 | |
2020-06-15 11:35 | sdsAdm1n | Issue Monitored: sdsAdm1n | |
2020-06-21 10:30 | medhat | Issue Monitored: medhat | |
2020-06-24 17:25 | Mazi | Note Added: 58425 | |
2020-06-24 17:29 |
|
Note Added: 58426 | |
2020-06-29 21:18 | gabrieljenik | Note Added: 58564 | |
2020-06-30 08:18 | DenisChenu | Note Added: 58568 | |
2020-06-30 08:18 | DenisChenu | Note Edited: 58568 | |
2020-06-30 17:50 | gabrieljenik | Note Added: 58594 | |
2020-06-30 18:14 | DenisChenu | Note Added: 58598 | |
2020-07-01 14:45 |
|
Note Added: 58614 | |
2020-07-01 14:45 |
|
File Added: 16236_Vanillatheme_edit.png | |
2020-07-01 15:21 | gabrieljenik | Changeset attached | => LimeSurvey master 14a8d26f |
2020-07-01 15:21 | guest | Note Added: 58616 | |
2020-07-02 12:37 |
|
Zoho Sprints | Yes => |Yes| |
2020-07-02 12:37 |
|
Status | confirmed => resolved |
2020-07-06 13:21 | lime_release_bot | Zoho Sprints | Yes => |Yes| |
2020-07-06 13:21 | lime_release_bot | Note Added: 58713 | |
2020-07-06 13:21 | lime_release_bot | Status | resolved => closed |
2020-07-06 13:21 | lime_release_bot | Resolution | open => fixed |
2020-07-07 13:16 | Mazi | Note Added: 58734 | |
2020-07-07 14:05 | DenisChenu | Note Added: 58739 | |
2020-07-07 16:19 | gabrieljenik | Note Added: 58754 | |
2020-07-07 16:20 | gabrieljenik | Note Edited: 58754 | |
2021-08-07 01:11 | guest | Bug heat | 14 => 18 |