View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
20180Bug reportsQuestion editorpublic2025-07-17 19:442025-07-23 14:03
Reporterholch Assigned Totibor.pacalat  
PrioritynoneSeverityminor 
Status resolvedResolutionfixed 
Product Version6.6.x 
Summary20180: Double escaping bug
Description

In the question editor for single choice questions, there is a double escaping bug in "Position for other option", see screenshot.

Steps To Reproduce

Steps to reproduce

Add a new question of the type "single choice - list radio"
Go to "position for other option"

Expected result

Clean item, text only.

Actual result

There is some code appearing.

TagsNo tags attached.
Attached Files
Captura de tela 2025-07-15 164217.png (12,604 bytes)   
Captura de tela 2025-07-15 164217.png (12,604 bytes)   
Bug heat6
Complete LimeSurvey version number (& build)6.15.4
I will donate to the project if issue is resolvedNo
BrowserChrome
Database type & versionNA
Server OS (if known)NA
Webserver software & version (if known)NA
PHP VersionNA

Users monitoring this issue

There are no users monitoring this issue.

Activities

gabrieljenik

gabrieljenik

2025-07-22 16:11

manager   ~83130

https://github.com/LimeSurvey/LimeSurvey/pull/4374

We have identified some double gt()
Widget and View

Ex views:
application\extensions\AdvancedSettingWidget\views\singleselect.php
application\extensions\AdvancedSettingWidget\views\buttongroup.php

For thhese already done on widget.
Not sure if we should do it on the widget or on the view.

02456
Here Denis said on the Widget, but I am having second thoughts now.
If implemented on the view, there will be more flexibility.
DenisChenu

DenisChenu

2025-07-22 16:50

developer   ~83134

Are you sure about 02546 ? 02456: Answering the multiple numerical question doesn't update conditions when typing answers
DenisChenu

DenisChenu

2025-07-22 16:52

developer   ~83135

PS : my opinion is still valid with your fix :) escape must come from widget

I have to report issue about HTML allowed in attribute help in 3 and not in 6 ;)
gabrieljenik

gabrieljenik

2025-07-23 02:30

manager   ~83137

Are you sure about 02546 ? 02456: Answering the multiple numerical question doesn't update conditions when typing answers

Sorry, I meant to say PR 2546
https://github.com/LimeSurvey/LimeSurvey/pull/2456#issuecomment-1167334374
guest

guest

2025-07-23 14:03

viewer   ~83138

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

Related Changesets

LimeSurvey: master 4ab4f875

2025-07-23 14:03

Gabriel Jenik

Committer: GitHub


Details Diff		 Fixed issue 20180: Double escaping bug (#4374)

Co-authored-by: lapiudevgit <devgit@lapiu.biz>		 Affected Issues
20180
mod - application/extensions/AdvancedSettingWidget/AdvancedSettingWidget.php Diff File

Issue History

Date Modified Username Field Change
2025-07-17 19:44 holch New Issue
2025-07-17 19:44 holch File Added: Captura de tela 2025-07-15 164217.png
2025-07-18 13:04 tibor.pacalat Assigned To => gabrieljenik
2025-07-18 13:04 tibor.pacalat Status new => assigned
2025-07-22 16:11 gabrieljenik Assigned To gabrieljenik => DenisChenu
2025-07-22 16:11 gabrieljenik Status assigned => ready for code review
2025-07-22 16:11 gabrieljenik Note Added: 83130
2025-07-22 16:11 gabrieljenik Bug heat 0 => 2
2025-07-22 16:50 DenisChenu Note Added: 83134
2025-07-22 16:50 DenisChenu Bug heat 2 => 4
2025-07-22 16:52 DenisChenu Note Added: 83135
2025-07-22 16:53 DenisChenu Assigned To DenisChenu => tibor.pacalat
2025-07-22 16:53 DenisChenu Status ready for code review => ready for testing
2025-07-23 02:30 gabrieljenik Note Added: 83137
2025-07-23 14:03 Changeset attached => LimeSurvey master 4ab4f875
2025-07-23 14:03 guest Note Added: 83138
2025-07-23 14:03 guest Bug heat 4 => 6
2025-07-23 14:03 tibor.pacalat Status ready for testing => resolved
2025-07-23 14:03 tibor.pacalat Resolution open => fixed