Custom Open Graph tags

alzander
Offline

Support Specialist

10 years 9 months ago #34443 by alzander

Replied by alzander on topic Custom Open Graph tags

Kaz,
I'm sorry that JFBConnect v5.0.1 didn't fix this issue. I've entered this issue into our tracker now and it should definitely be fixed in the 5.1 release scheduled for August.

Hopefully you've tested the Open Graph tag priority issue as well and that's working as expected. That was a bigger issue as it affected more users, but we also want to make sure the $ works in descriptions as well going forward.

Thanks,
Alex

The topic has been locked.

alzander
Offline

Support Specialist

10 years 9 months ago #34534 by alzander

Replied by alzander on topic Custom Open Graph tags

Kaz,
Wanted to send you an update about this. We implemented a fix for the upcoming JFBConnect v5.1 release for this issue. It's an very minor change that you're welcome to implement on your current release if you'd like to fix now.

To make the change, edit the /plugins/system/jfbcsystem/jfbcsystem.php file. At the very bottom, you'll see a block that looks like:

$contents = $openGraphLibrary->removeOverlappingTags($contents);
        $search = '/' . preg_quote($placeholder, '/') . '/';
        $contents = preg_replace($search, $graphTags, $contents, 1);

Update that block by adding the extra line as shown below:

$contents = $openGraphLibrary->removeOverlappingTags($contents);
        $search = '/' . preg_quote($placeholder, '/') . '/';
        $graphTags = str_replace('$', '\$', $graphTags);  // ADD THIS LINE
        $contents = preg_replace($search, $graphTags, $contents, 1);

Test and let us know how that goes. It should fix you right up.

Please let us know how it goes.

Thanks,
Alex

The topic has been locked.

SourceCoast Login

Custom Open Graph tags