using shortcodes in activity stream always result in [Read more] links if shortcode filter priority is wrong

[magnus78]

If I use shortcodes in the activity stream I always get [Read more] links inserted.

This is due to the fix for #3452, specifically the if ( $excerpt != $text ) check in bp_activity_truncate_entry

[magnus78]

proposed fix

[DJPaul]

Moving to 1.6 for review. Do you mean any type of activity item, or only those from blog posts?

[magnus78]

Turns out this happened to me because of an action priority problem. I had it for all activity items.

I think the fix is still correct though. If shortcodes are stripped you will always get the more link since texts obviously aren't the same anymore - and typically not because a length is exceeded.

If shortcodes are expanded before the code in question runs it doesn't matter of course (this is the normal scenario).

[DJPaul]

magnus78, was the action priority problem in your code/plugin, or was it in BuddyPress? Can you name names?

[magnus78]

The action priority problem was in my own plugin.

[boonebgorges]

Problem confirmed.

Since we are currently stripping shortcodes in the stream, we should continue to do so. Instead, let's also strip the shortcodes from the $text we're comparing against.

[boonebgorges]

(In [6095]) Strip shortcodes from comparison text before adding [Read More] link in activity stream

We test to see whether a 'Read More' link should be added to activity items
in the stream by comparing the excerpted version against the unprocessed text.
Since the excerpt is created using bp_create_excerpt() with the
'filter_shortcodes' flag set to true, we must also strip shortcodes on the
unprocessed text to get an accurate comparison for length.

This prevents the [Read More] link from being appended erroneously when
shortcodes are present in activity updates.

Fixes #4051. Props magnus78