[buddypress-trac] [BuddyPress Trac] #6933: Emails: Do not strip HTML content for email content

buddypress-trac noreply at wordpress.org
Wed Mar 2 21:15:52 UTC 2016


#6933: Emails: Do not strip HTML content for email content
--------------------------+-----------------------------
 Reporter:  r-a-y         |       Owner:
     Type:  enhancement   |      Status:  new
 Priority:  normal        |   Milestone:  Future Release
Component:  API - Emails  |     Version:  2.5.0
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+-----------------------------

Comment (by r-a-y):

 > If you want to propose changing this, it needs it own ticket, not being
 snuck onto this one.

 Agreed.  I bundled them initially because both are related to each other.
 See #6934.

 > Where do we strip HTML tags from the email content?

 I should have been more specific.

 I primarily meant the `usermessage` token located here:

 https://buddypress.trac.wordpress.org/browser/tags/2.5.0/src/bp-activity
 /bp-activity-notifications.php?marks=79#L76
 https://buddypress.trac.wordpress.org/browser/tags/2.5.0/src/bp-messages
 /bp-messages-notifications.php?marks=63#L61

 > Are you talking about ALL HTML tags?

 I think we should implement our own version of KSES for HTML email
 content, similar to what we currently do for the Activity component's
 version of KSES.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6933#comment:4>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list