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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mr.andrey’s picture

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

Reassigning to correct project...

mr.andrey’s picture

Title: Patch for individual permissions for OG » Patch for individual permissions for OG + Facebook
Version: 6.x-1.13 » 6.x-1.14
FileSize
2.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

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
FileSize
2.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

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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