Skip to:
Content

BuddyPress.org

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#2632 closed defect (bug) (fixed)

Activity Stream doesn't close HTML tags

Reported by: paulhastings0's profile paulhastings0 Owned by:
Milestone: 1.2.6 Priority: major
Severity: Version:
Component: Activity Keywords: has-patch, tested
Cc:

Description

I noticed today that the activity stream doesn't close HTML tags. One of our users left a comment that began an <i> tag but he forgot to close it. The rest of the site content underneath it then changed into italics.

This video shows what it looked like: http://screencast.com/t/MjAzZmQ0ZTkt

I believe that WordPress posts try to automatically close all commonly used HTML tags. Could BuddyPress do that as well?

Attachments (1)

patch-2632.patch (729 bytes) - added by DJPaul 14 years ago.
v1

Download all attachments as: .zip

Change History (4)

@DJPaul
14 years ago

v1

#1 @DJPaul
14 years ago

  • Keywords has-patch tested added; HTML automatically close HTML tags removed
  • Milestone changed from 1.3 to 1.2.6
  • Priority changed from normal to major

I think this should have been caught by the force_balance_tags filter on bp_get_activity_content_body, but it doesn't seem to fix this, so the patch adds the filter on the way into the database which it probably always should have had anyway.

#2 @johnjamesjacoby
14 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [3268]) Fixes #2632(branch) props Paul Gibbs

#3 @johnjamesjacoby
14 years ago

(In [3269]) Fixes #2632(trunk) props Paul Gibbs. (Also patches missing branch updates from merge.)

Note: See TracTickets for help on using tickets.