View Issue Details

Related ChangesetsJump to NotesJump to History
This issue affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
20106Bug reportsSurvey editingpublic2025-05-15 14:022026-03-04 14:28
Reporterriku Assigned Toc_schmitz  
PrioritynoneSeverityminor 
Status resolvedResolutionfixed 
Summary20106: PR 04118 breaks the loading of Survey options with force_xmlsettings_for_survey_rendering enabled.
Description

Specifically the changes on lines 709 and 712 in application/core/LSETwigViewRenderer.php now give result in 'N/A' for the theme's config.xml options: https://github.com/LimeSurvey/LimeSurvey/pull/4118/files#diff-13389663d7f722bfdc7f856fb756237f993648facfd80697d9fe6f241f40027bR709-R712.

The issue occurs as is_string(SimpleXMLElement Object) is false.

Reverting these two lines fixes the issue.

Steps To Reproduce

The easiest way to see this in action would be to compare how the theme settings are applied in either case.

Steps to reproduce

  1. Check that the theme settings apply properly without the force_xmlsettings_for_survey_rendering setting enabled. For example, with the 'vanilla' theme, ensure that the Survey container setting is set to Yes and container-md is used when displaying the Survey.
  2. Set 'force_xmlsettings_for_survey_rendering' => true in config.
  3. Check the Survey again.

Expected result

The Survey content is aligned within container-md as the 'vanilla' theme by default has the container value set to 'on' in it's config.xml

Actual result

The Survey container setting cannot be retrieved (results in N/A) and therefore the content is aligned in container-fluid (edge to edge), even though the 'vanilla' theme by default has the container value set to 'on' in it's config.xml.

TagsNo tags attached.
Bug heat6
Complete LimeSurvey version number (& build)6.13.2 github
I will donate to the project if issue is resolvedNo
Browser
Database type & version10.11.11-MariaDB
Server OS (if known)
Webserver software & version (if known)Apache/2.4.62
PHP Version8.1.32

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2026-01-14 07:58

developer   ~84090

Previous dev are removed.
Set it as new.
tibor.pacalat

tibor.pacalat

2026-01-23 17:44

administrator   ~84144

@DenisChenu, sorry I don't understand your comment. Is this a non-issue? Can this be closed?
DenisChenu

DenisChenu

2026-01-24 12:00

developer   ~84145

See Issue History

tibor.pacalat Assigned To => marcolombardo

I have no idea on the issue here. Just assignation to an old dev :)
c_schmitz

c_schmitz

2026-03-04 14:28

administrator   ~84393

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=39392

Related Changesets

LimeSurvey: master c478e01b

2026-03-04 14:28

c_schmitz


Details Diff		 Fixed issue 20106: Broken loading of survey theme options with force_xmlsettings_for_survey_rendering enabled Affected Issues
20106
mod - application/core/LSETwigViewRenderer.php Diff File

Issue History

Date Modified Username Field Change
2025-05-15 14:02 riku New Issue
2025-05-15 16:27 tibor.pacalat Assigned To => marcolombardo
2025-05-15 16:27 tibor.pacalat Status new => assigned
2026-01-14 07:57 DenisChenu Assigned To marcolombardo => tibor.pacalat
2026-01-14 07:58 DenisChenu Status assigned => new
2026-01-14 07:58 DenisChenu Note Added: 84090
2026-01-14 07:58 DenisChenu Bug heat 0 => 2
2026-01-23 17:44 tibor.pacalat Note Added: 84144
2026-01-23 17:44 tibor.pacalat Bug heat 2 => 4
2026-01-24 12:00 DenisChenu Note Added: 84145
2026-03-04 14:15 c_schmitz Assigned To tibor.pacalat =>
2026-03-04 14:26 c_schmitz Assigned To => c_schmitz
2026-03-04 14:26 c_schmitz Status new => assigned
2026-03-04 14:28 c_schmitz Changeset attached => LimeSurvey master c478e01b
2026-03-04 14:28 c_schmitz Note Added: 84393
2026-03-04 14:28 c_schmitz Status assigned => resolved
2026-03-04 14:28 c_schmitz Resolution open => fixed
2026-03-04 14:28 c_schmitz Bug heat 4 => 6