I just noticed that the canonical tag from this module is not using URL aliases. Even though I had URL aliases for every node, the canonical tag looked something like this:

<link rel="canonical" href="http://example.com/node/123"/>

That made Google display wrong URLs in the SERPs for the entire site.

I've turned off that feature in Nodewords, and I'm outputting that element with the Global Redirect Module now, but my site is probably not the only one affected by the problem.

CommentFileSizeAuthor
#3 nodewords-n1419172.patch1.72 KBdamienmckenna

Comments

damienmckenna’s picture

Status: Active » Postponed (maintainer needs more info)

Is it v1.3 or v1.13 you're using?

Did you turn on the "Canonical URL use path alias" option on the main settings page (admin/content/nodewords)?

Z2222’s picture

Version: 6.x-1.3 » 6.x-1.13
Priority: Critical » Normal

Thanks for your reply. Sorry, my mistake -- it's version 6.x-1.13.

I didn't see that setting before. I think it should be on by default otherwise people are going to mess up their sites with this module by accident -- like me. :)

Even better, have it use URL aliases by default, and a checkbox to disable the feature, but with a warning: "Only check this box if you know what you are doing".

I can't think of any scenario where one would want to have URL aliases but have the canonical URL be unaliased. The only thing it could possibly do is mess up the URLs in the SERPs.

damienmckenna’s picture

Title: The Canonical Tag Doesn't Use URL Aliases » Canonical tag should use aliases by default
Category: bug » feature
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new1.72 KB

You know, I agree, I'm not sure why I left it as FALSE before. This patch will be committed fairly soon.

Z2222’s picture

Thanks... :)

halloffame’s picture

+1.. Is this committed yet to the latest dev version?

damienmckenna’s picture

Status: Needs review » Fixed

I've committed this, thanks for bringing up the issue.

Status: Fixed » Closed (fixed)

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