Hi there,

It seems that nodewords_og doesn't have the same permission control as nodewords_basic. Here's a patch that enables that functionality.

Cheers,
Andrey.

CommentFileSizeAuthor
#3 nodewords-n2049693-3.patch2.68 KBdamienmckenna
#2 nodewords_og.patch.txt2.95 KBmr.andrey
nodewords_og.patch.txt733 bytesmr.andrey

Comments

mr.andrey’s picture

mr.andrey commented
Project: Open graph » Nodewords: D6 Meta Tags
Version: » 6.x-1.13

Reassigning to correct project...

mr.andrey’s picture

mr.andrey commented
Title: Patch for individual permissions for OG » Patch for individual permissions for OG + Facebook
Version: 6.x-1.13 » 6.x-1.14
StatusFileSize
new nodewords_og.patch.txt2.95 KB

Also, the video secure_url OG tag is missing. It's a requirement for Facebook to show your video content with a play button.

Attached is a combined patch to give configurable permissions and video secure URL field.

I should probably post these as two separate patches, but... well, that's just not going to happen at the moment :-)

Cheers,
Andrey.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: Patch for individual permissions for OG + Facebook » Missing OG permissions, og:video URL validation incorrect
Version: 6.x-1.14 » 6.x-1.x-dev
Issue summary: View changes
Status: Active » Needs review
Parent issue: » #2533956: Plan for Nodewords v6.x-1.15 release
StatusFileSize
new nodewords-n2049693-3.patch2.68 KB

The nodewords_og permissions are added via the Nodewords API via the 'permission' attribute in hook_nodewords_tags_info(), so there's no need to add nodewords_og_perm().

This has been rerolled, I've removed nodewords_og_perm() and fixed some spelling mistakes.

Also, don't forget to change the issue status to "needs review" after uploading a patch :) Thanks!

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.