View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
10226Bug reportsResponse browsingpublic2016-01-14 10:532017-10-23 11:02
Reporterervit Assigned ToLouisGac 
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.06+ 
Summary10226: {ANSWERTABLE} includes <script> placed in question text
Description

The email notifying that a survey has been completed and reporting the {ANSWERTABLE} also shows all scripts placed in the question text source.

Steps To Reproduce

Make a question with script in question source (not in the Help source), add {ASNSWERTABLE} in the notification email, complete a survey, read the email that was generated and sent.

Additional Information

I placed the script in question source and not in help source because if there is a script, the "i" icon appears even if there is no help text and it looks weird (people think the help text is missing or has bugged out, in some templates it also looks ugly)

TagsNo tags attached.
Bug heat6
Complete LimeSurvey version number (& build)151215
I will donate to the project if issue is resolvedNo
BrowserFirefox 43
Database type & versionMySQL
Server OS (if known)Linux
Webserver software & version (if known)GoDaddy hosting config 2.2
PHP Version5.3

Users monitoring this issue

There are no users monitoring this issue.

Activities

ervit

ervit

2016-01-14 11:03

reporter   ~34277

actually, in includes all HTML tags added to the question text, such as etc.
LouisGac

LouisGac

2016-06-29 15:07

developer   ~39807

Fix committed to 2.06lts branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=19528
LouisGac

LouisGac

2016-06-29 15:34

developer   ~39808

Fix committed to 2.06lts branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=19529
DenisChenu

DenisChenu

2016-07-04 10:24

developer   ~39816

@LouisGac : i have another idea for ANSWERTABLE : include Question->question (with subquestion) only one time. Acvtually : it's awfull : you can have 20 times the same text shown ....
LouisGac

LouisGac

2016-11-15 17:20

developer   ~41961

Denis, was this idea for 2.06 or 2.50 ?
DenisChenu

DenisChenu

2016-11-15 17:26

developer   ~41962

Last edited: 2016-11-15 17:27

2.50 (or 3.0 ;) ) :) not really a bug. More a misconception :).

Already done in my fork for 2.06 . With usage of * and for text email too.
LouisGac

LouisGac

2016-11-16 15:27

developer   ~41982

could you post a link to your fix, so I can port it to 2.50?
DenisChenu

DenisChenu

2016-11-16 15:40

developer   ~41983

If i don't put link, it's because my fix need fix ;).

https://github.com/SondagesPro/LimeSurvey-SondagesPro/commit/551e48fda876b74ea306c2af57e6ee5e53311755
LouisGac

LouisGac

2016-11-16 16:05

developer   ~41984

^^
LouisGac

LouisGac

2017-10-23 11:02

developer   ~44787

now we will not fix it (LS3 system is different )

Related Changesets

LimeSurvey: 2.06lts b4a63209

2016-06-29 15:07

LouisGac


Details Diff		 Fixed issue 10226: {ANSWERTABLE} includes <script> placed in question text Affected Issues
10226
mod - application/helpers/frontend_helper.php Diff File

LimeSurvey: 2.06lts 4d756170

2016-06-29 15:34

LouisGac


Details Diff		 Fixed issue 10226: {ANSWERTABLE} includes <script> placed in question text Affected Issues
10226
mod - application/helpers/frontend_helper.php Diff File

Issue History

Date Modified Username Field Change
2016-01-14 10:53 ervit New Issue
2016-01-14 11:03 ervit Note Added: 34277
2016-06-29 15:07 LouisGac Changeset attached => LimeSurvey 2.06lts b4a63209
2016-06-29 15:07 LouisGac Note Added: 39807
2016-06-29 15:07 LouisGac Assigned To => LouisGac
2016-06-29 15:07 LouisGac Resolution open => fixed
2016-06-29 15:34 LouisGac Changeset attached => LimeSurvey 2.06lts 4d756170
2016-06-29 15:34 LouisGac Note Added: 39808
2016-07-04 10:24 DenisChenu Note Added: 39816
2016-11-15 17:20 LouisGac Note Added: 41961
2016-11-15 17:26 DenisChenu Note Added: 41962
2016-11-15 17:27 DenisChenu Note Edited: 41962
2016-11-16 15:27 LouisGac Note Added: 41982
2016-11-16 15:40 DenisChenu Note Added: 41983
2016-11-16 16:05 LouisGac Note Added: 41984
2017-10-23 11:02 LouisGac Status new => closed
2017-10-23 11:02 LouisGac Note Added: 44787