View Issue Details

This bug affects 1 person(s).
 4
IDProjectCategoryView StatusLast Update
19747Bug reportsSurvey takingpublic2024-09-12 16:30
Reportergabrieljenik Assigned To 
PrioritynoneSeverityminor 
Status newResolutionopen 
Product Version6.6.x 
Summary19747: Sometimes the theme configuration hirearchy loading is not correct
Description

Sometimes when loading the the theme configuration hirearchy, if a theme configuration is missing for the surveyGroup, the woraround creates chaos.

I understand that getInstanceFromSurveyGroup should return an instance of the configuration associated with a survey group, while getInstanceFromTemplateName should return a “generic” instance.

When there is no specific configuration for a surveygroup, the code takes what getInstanceFromTemplateName returns (a general instance for the template) and modifies the instance, which is cached.

By not making a clone, what it ends up doing is modifying the cached instance, modifying not only for the surveygroup (as intended) but for the instance for the template.

So, when you call getInstanceFromTemplateName again, it returns something that is not what is supposed to be returned.

Steps To Reproduce

Steps to reproduce

Hard to reproduce. Need to keep on working on it.

Expected result

Get a specifc instance for the survey group.

Actual result

Update the instance used for the base template configuracion

TagsNo tags attached.
Attached Files
image.png (122,113 bytes)
Bug heat4
Complete LimeSurvey version number (& build)6.x
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMysql
Server OS (if known)
Webserver software & version (if known)
PHP Version8

Relationships

related to 19685 resolvedgabrieljenik php8.3 compatibility - LS Core 

Users monitoring this issue

There are no users monitoring this issue.

Activities

gabrieljenik

gabrieljenik

2024-09-12 15:44

manager   ~80997

Found when reviewing 19685

DenisChenu

DenisChenu

2024-09-12 16:30

developer   ~81002

Yep, all this code are really complex…
Same for all inheritance …

Issue History

Date Modified Username Field Change
2024-09-12 15:43 gabrieljenik New Issue
2024-09-12 15:43 gabrieljenik File Added: image.png
2024-09-12 15:44 gabrieljenik Relationship added related to 19685
2024-09-12 15:44 gabrieljenik Note Added: 80997
2024-09-12 15:44 gabrieljenik Bug heat 0 => 2
2024-09-12 16:29 DenisChenu Relationship added child of 19632
2024-09-12 16:29 DenisChenu Relationship deleted child of 19632
2024-09-12 16:30 DenisChenu Note Added: 81002
2024-09-12 16:30 DenisChenu Bug heat 2 => 4