Canonical links should use the link formatter provided as default to ensure the link is formed correctly and the default Drupal canonical link is replaced.

CommentFileSizeAuthor
#10 canonical-formatter-2153577-7.patch561 bytesmexicoder
#6 canonical-formatter-2153577-6.patch4.64 KBmexicoder
#4 canonical-formatter-2153577-4.patch4.64 KBmexicoder
#3 canonical-formatter-2153577-2.patch4.63 KBmexicoder
#1 canonical-formatter-2153577-1.patch4.62 KBmexicoder
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mexicoder’s picture

mexicoder CreditAttribution: mexicoder commented
FileSize
canonical-formatter-2153577-1.patch4.62 KB

This patch should apply the formatter defined for canonical link in known_tags.inc. All new attached fields will include this. Run update to update all previous tags to use the defined formatter.

mexicoder’s picture

mexicoder CreditAttribution: mexicoder commented

Same patch as #1 but against 7.x-2.7

mexicoder’s picture

mexicoder CreditAttribution: mexicoder commented
FileSize
canonical-formatter-2153577-2.patch4.63 KB

Patched against 2.7

mexicoder’s picture

mexicoder CreditAttribution: mexicoder commented
FileSize
canonical-formatter-2153577-4.patch4.64 KB

Corrected check in code

mexicoder’s picture

mexicoder CreditAttribution: mexicoder commented
Status: Active » Needs review
mexicoder’s picture

mexicoder CreditAttribution: mexicoder commented
FileSize
canonical-formatter-2153577-6.patch4.64 KB

Removed blank space from file

sandykadam’s picture

sandykadam CreditAttribution: sandykadam commented
Status: Needs review » Reviewed & tested by the community

@mexicoder Tested the canonical field and now it's returning exact formatter of link. Looks fine now, we can move the patch to module.

valthebald’s picture

valthebald
Location Sofia
CreditAttribution: valthebald commented
Version: 7.x-2.7 » 7.x-2.x-dev
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

mexicoder’s picture

mexicoder CreditAttribution: mexicoder commented
FileSize
canonical-formatter-2153577-7.patch561 bytes

Updated hook_update number as previous patch was against 2.7

mexicoder’s picture

mexicoder CreditAttribution: mexicoder commented
Status: Closed (fixed) » Needs review
valthebald’s picture

valthebald
Location Sofia
CreditAttribution: valthebald commented
Status: Needs review » Fixed

Committed to 7.x-2.x and pushed. Thanks @mexicoder!

Status: Fixed » Closed (fixed)

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