Skip to:
Content

BuddyPress.org

Opened 17 years ago

Closed 17 years ago

#778 closed defect (bug) (fixed)

erroneous call to strip_tags in bp-wire.php bp_wire_new_post

Reported by: DJPaul Owned by:
Priority: minor Milestone: 1.1
Component: Version:
Severity: Keywords:
Cc:

Description

line 101, bp-wire.php. The call to strip_tags() is not needed and is stripping out tags prior to being saved into the DB.

Change History (7)

#1 @DJPaul
17 years ago

Whilst above stores the tags in DB, doesn't display them on the wire pages.
I added the following to the top of my bp-wire-filters.php which seems to work (they displayed + are escaped in the source)

add_filter( 'bp_get_wire_post_content', 'attribute_escape', 1 );

#2 @DJPaul
17 years ago

  • Keywords has-patch removed

I'm withdrwaing that idea for a fix ;) as i'm not sure it works properly

#3 @DJPaul
17 years ago

sample string for testing in the wire is like an IRC quote:

"<Person> Message"

#4 @(none)
17 years ago

  • Milestone Wire 1.1

Milestone Wire 1.1 deleted

#5 @apeatling
17 years ago

  • Milestone → 1.0.4

#6 @DJPaul
17 years ago

  • Milestone 1.0.4 → 1.1

#7 @apeatling
17 years ago

  • Resolution → fixed
  • Status new → closed

(In [1777]) Strip tags not needed as this is being run through a whole host of filters, stripping out the nasties already. Props DJPaul. Fixes #778

Note: See TracTickets for help on using tickets.