First, thanks for this module. It works as described, and is written really well.

The following patch runs the core URL filter on the message text before caching. This was necessary for a client project, and I saw another person in the queue that wanted this feature.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mdeltito’s picture

Status: Active » Needs review
FileSize
862 bytes
Jaapx’s picture

Issue summary: View changes

This patch works very well. Ik had to add it by hand because the module code has changed a little bit.
Thank you for publishing!

tostinni’s picture

Although this works fine, it is using a private function _filter_url().

So here is an improved patch to support this feature and improve it by passing Facebook message through Drupal's check_markup() function so you can add any filter you want.
I also added an admin settings to chose which filter should be applied.

tostinni’s picture

FileSize
1.26 KB

Small fix on my patch to move the sanitization to preprocess where it belongs.