According to the documentation of the hreflang tag (https://support.google.com/webmasters/answer/189077?hl=en), the rel value should be rel="alternate", but for some reason the module is currently outputting rel="alternative".

CommentFileSizeAuthor
#2 hreflang_rel_value-2704805-2.patch510 bytesjcnventura
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jcnventura created an issue. See original summary.

jcnventura’s picture

jcnventura CreditAttribution: jcnventura at Wunder commented
Status: Active » Reviewed & tested by the community
FileSize
hreflang_rel_value-2704805-2.patch510 bytes

It's a one liner fix.. No review needed other than looking at the patch.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Version: 7.x-1.14 » 7.x-1.x-dev
Status: Reviewed & tested by the community » Needs review
Parent issue: » #2699045: Plan for Metatag 7.x-1.15 release

Thanks for spotting the oversight, and the patch.

One minor note - please do not mark your own patches are "RTBC", instead please mark them "needs review", that'll kick off the testbot and let me know there's a patch to look at.

Thanks again.

charlesj’s picture

charlesj CreditAttribution: charlesj commented

Patch works for me. Thank you!
(I am using Drupal 7.43, PHP 5.3.3)

giupenni’s picture

giupenni CreditAttribution: giupenni commented

Patch works for me.
Thank you!

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Fixed

Committed. Thanks.

Status: Fixed » Closed (fixed)

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