Will the current tags in the Open Graph submodule do still work for the Video URL. There have been updates to the specification by Facebook. Instead of just og:video to identify the URL of the video it has now been changed to og:video:url.

Patch coming shortly.

CommentFileSizeAuthor
#9 metatag-n2567677-9.patch4.62 KBdamienmckenna
#8 metatag-n2567677-8.patch1.84 KBdamienmckenna
#6 metatag-update_og_video_tag_to_latest_spec-2567677-6-d7.patch3.98 KBperignon
#2 metatag-update_og_video_tag_to_latest_spec-2567677-2-d7.patch1.35 KBperignon

Comments

Perignon created an issue. See original summary.

perignon’s picture

perignon commented
Status: Active » Needs review
StatusFileSize
new metatag-update_og_video_tag_to_latest_spec-2567677-2-d7.patch1.35 KB

Here is a patch to change the og:video to og:video:url and a hook_update to warn users of the change and the need to manually update tags.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Needs work
Issue tags: -open graph, -facebook

Sigh. Thanks, Facebook.

This needs an update script to change the existing meta tags in both configs and entity records, see the update scripts in metatag_twitter_cards for an example.

perignon’s picture

perignon commented

Ah! I was looking for an example update hook to do this and the one in the module just says to do it manually. I'll go look at the twitter one.

perignon’s picture

perignon commented

Ok got the changes made. I'm going to run some tests on my site. I have over 2000 nodes for it to churn through.

perignon’s picture

perignon commented
Status: Needs work » Needs review
StatusFileSize
new metatag-update_og_video_tag_to_latest_spec-2567677-6-d7.patch3.98 KB

Tested on my local and it seemed to have worked. I used the example in the twitter submodule.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Category: Bug report » Task
Parent issue: » #2539438: Plan for Metatag 7.x-1.8 release
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new metatag-n2567677-8.patch1.84 KB

A minor adjustment - add the 'replaces' attribute of the meta tag definition.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new metatag-n2567677-9.patch4.62 KB

The last patch was missing the update script. I've also added a cache clear to the update script.

perignon’s picture

perignon commented

So this one has everything in it.

damienmckenna’s picture

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

Committed. Thanks.

Status: Fixed » Closed (fixed)

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