View Issue Details

This bug affects 1 person(s).
 4
IDProjectCategoryView StatusLast Update
08234Bug reportsSurvey participants (Tokens)public2016-02-19 16:38
ReporterDenisChenu Assigned ToDenisChenu  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.00+ 
Target Version2.05 RCFixed in Version2.05+ 
Summary08234: Text email send with HTML email are not filtered
Description

When send a HTML email, there are a text email attched with it. The text of the text email get the style in the text, (and line feed is not in good place every time)

Steps To Reproduce

Create a new HTML, put something like this in the HTML source:
<html>
<head>
<style>
body{ color:green;background:red } // This color is very beautiful
</style>
</head>
<body>
Get may best survey <br />{SURVEYURL}

</body>
</html>

The text part of the email was:
\n
\n
\n
body{ color:green;background:red } // This color is very beautiful
\n
\n
\n
Get may best survey http://surveyurl.example.org
\n
\n
\n

Additional Information

PHPMailer don't have a clean system for this ?

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)130804
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database type & versionnot relevant
Server OS (if known)not relevant
Webserver software & version (if known)not relevant
PHP Versionnot relevant

Relationships

related to 10536 closedDenisChenu Higher risk that the emails are rated as Spam because of X_PRIORITY header 

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2013-10-04 18:14

developer   ~26518

altBody is for text email with html email.

in PHPmailer
$textMsg = trim(strip_tags(preg_replace('/<(head|title|style|script)[^>]>.?<\/\1>/s','',$message)));

Gloups

c_schmitz

c_schmitz

2013-10-07 22:02

administrator   ~26608

Can you take this for 2.05?

DenisChenu

DenisChenu

2013-10-08 08:27

developer   ~26612

Yes, sure

DenisChenu

DenisChenu

2014-01-02 17:26

developer   ~27712

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

DenisChenu

DenisChenu

2014-01-02 17:28

developer   ~27713

Leave PHPMailer do his job

c_schmitz

c_schmitz

2014-01-09 23:31

administrator   ~27808

2.05+ build 140109 released

Related Changesets

LimeSurvey: master 127ef2c0

2014-01-02 16:26:23

DenisChenu

Details Diff
Fixed issue 08234: Text email send with HTML email are not filtered
Dev: leave PHPmailer do his job
Affected Issues
08234
mod - application/helpers/common_helper.php Diff File

Issue History

Date Modified Username Field Change
2013-10-04 18:06 DenisChenu New Issue
2013-10-04 18:07 DenisChenu Steps to Reproduce Updated
2013-10-04 18:14 DenisChenu Note Added: 26518
2013-10-07 22:02 c_schmitz Assigned To => DenisChenu
2013-10-07 22:02 c_schmitz Status new => assigned
2013-10-07 22:02 c_schmitz Target Version => 2.05 RC
2013-10-07 22:02 c_schmitz Note Added: 26608
2013-10-08 08:27 DenisChenu Note Added: 26612
2014-01-02 17:26 DenisChenu Changeset attached => LimeSurvey master 127ef2c0
2014-01-02 17:26 DenisChenu Note Added: 27712
2014-01-02 17:26 DenisChenu Resolution open => fixed
2014-01-02 17:28 DenisChenu Note Added: 27713
2014-01-02 17:28 DenisChenu Status assigned => resolved
2014-01-02 17:28 DenisChenu Fixed in Version => 2.05+
2014-01-09 23:31 c_schmitz Note Added: 27808
2014-01-09 23:31 c_schmitz Status resolved => closed
2016-02-19 16:38 DenisChenu Relationship added related to 10536
2016-12-08 10:39 c_schmitz Category Tokens => Survey participants (Tokens)