Skip to:

Opened 15 years ago

Closed 15 years ago

#778 closed defect (bug) (fixed)

erroneous call to strip_tags in bp-wire.php bp_wire_new_post

Reported by: djpaul's profile DJPaul Owned by:
Milestone: 1.1 Priority: minor
Severity: Version:
Component: Keywords:


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
15 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
15 years ago

  • Keywords has-patch removed

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

#3 @DJPaul
15 years ago

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

"<Person> Message"

#4 @(none)
15 years ago

  • Milestone Wire 1.1 deleted

Milestone Wire 1.1 deleted

#5 @apeatling
15 years ago

  • Milestone set to 1.0.4

#6 @DJPaul
15 years ago

  • Milestone changed from 1.0.4 to 1.1

#7 @apeatling
15 years ago

  • Resolution set to fixed
  • Status changed from new to 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.