Skip to:
Content

BuddyPress.org

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#2632 closed defect (bug) (fixed)

Activity Stream doesn't close HTML tags

Reported by: paulhastings0 Owned by:
Priority: major Milestone: 1.2.6
Component: Activity Version:
Severity: 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 16 years ago.
v1

Download all attachments as: .zip

Change History (4)

@DJPaul
16 years ago

v1

#1 @DJPaul
16 years ago

  • Keywords has-patch tested added; HTML automatically close HTML tags removed
  • Milestone 1.3 → 1.2.6
  • Priority normal → 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
16 years ago

  • Resolution → fixed
  • Status new → closed

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

#3 @johnjamesjacoby
16 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.