Changeset 8156

Timestamp:

03/25/2014 06:15:09 PM (11 years ago)

Author:

boonebgorges

Message:

Run stripslashes filter on activity strings earlier than priority 10

Running stripslashes() on priority 10 caused race conditions with other filters
on the same content. This, in turn, causes conflicts with wptexturize(), which
cannot properly parse certain character combinations due to the incorrect
presence of escaping slashes. This problem exhibited itself most obviously
with the use of guillemet-style quotation marks (the slashes fooled
wptexturize() into thinking that the opening quote was actually the closing
one), but could also occur with other formatting rules.

Because the requirement to stripslashes() is due to BuddyPress's incorrect
escaping of much input content, we work around the race condition by running
stripslashes_deep() earlier than priority 10. This ensures that plugins hooking
to these filters with the default priority can expect properly formatted and
sanitized content. See #2776.

Props chouf1, imath, needle

File:

• trunk/bp-activity/bp-activity-filters.php (modified) (1 diff)

trunk/bp-activity/bp-activity-filters.php

@@ -68 +68 @@
 add_filter( 'bp_get_activity_feed_item_description', 'make_clickable', 9 );
 
-add_filter( 'bp_acomment_name',                      'stripslashes_deep' );
-add_filter( 'bp_get_activity_action',                'stripslashes_deep' );
-add_filter( 'bp_get_activity_content',               'stripslashes_deep' );
-add_filter( 'bp_get_activity_content_body',          'stripslashes_deep' );
-add_filter( 'bp_get_activity_parent_content',        'stripslashes_deep' );
-add_filter( 'bp_get_activity_latest_update',         'stripslashes_deep' );
-add_filter( 'bp_get_activity_latest_update_excerpt', 'stripslashes_deep' );
-add_filter( 'bp_get_activity_feed_item_description', 'stripslashes_deep' );
+add_filter( 'bp_acomment_name',                      'stripslashes_deep', 5 );
+add_filter( 'bp_get_activity_action',                'stripslashes_deep', 5 );
+add_filter( 'bp_get_activity_content',               'stripslashes_deep', 5 );
+add_filter( 'bp_get_activity_content_body',          'stripslashes_deep', 5 );
+add_filter( 'bp_get_activity_parent_content',        'stripslashes_deep', 5 );
+add_filter( 'bp_get_activity_latest_update',         'stripslashes_deep', 5 );
+add_filter( 'bp_get_activity_latest_update_excerpt', 'stripslashes_deep', 5 );
+add_filter( 'bp_get_activity_feed_item_description', 'stripslashes_deep', 5 );
 
 add_filter( 'bp_activity_primary_link_before_save',  'esc_url_raw' );