Comments on: Canonical URL links – osCommerce Contribution

By: Matt

Matt — Fri, 03 Sep 2010 14:34:31 +0000

Hi Pete, You are right. Search engines SHOULDN'T be indexing those session id's, but even Google wants to be naughty once in a while and index them anyway. You CAN'T prevent SID from being indexed when a customer decides to link to a specific URL with (unfortunately) the SID. There are still a lot URL's with SID indexed, this addon prevents it by just removing the SID. For filters, variables and sessions is even why Google introduced the canonical tag in the first way.

By: Pete

Pete — Mon, 30 Aug 2010 21:57:13 +0000

I am clearly missing something with this osCommerce contribution. What else does it do, other than remove the SID from the URL?
I ask because if a store is set up correct, a bot shouldn’t be starting a session. Thanks.

By: Thomas

Thomas — Tue, 04 May 2010 08:08:55 +0000

To Admin – I posted this in the previous post but it might be more relevant in this one…
–

I’m having the following issue after some changes in oscommerce: our shop exists in 3 languages. Product info pages are accessible through the following URLs:

/product-name-dutch-p-1.html
/product-name-english-p-1.html?language=dutch
/product-name-french-p-1.html?language=french

Will this contribution take care of this problem? Obviously I want the canonical URL be the URL in the correct language, and that the URLs in the 2 other languages with the language variable behind redirect to the first one.

Thanks for your answer

By: » Canonical URL Affiliate edition | Webshoptimizer

» Canonical URL Affiliate edition | Webshoptimizer — Tue, 03 Nov 2009 22:38:54 +0000

[...] Get the original Canonical URL contribution [...]

By: GamersTronics.com, LLC

GamersTronics.com, LLC — Tue, 27 Oct 2009 17:33:12 +0000

Here is the Solution

okay i figured out a rough way to fix the problem.

it looks like the author was looking to remove trailing slashes in the url. however my server doesn’t put this trailing slash so it was making my .com or .net into .co and .ne

$domain = substr((($request_type == ‘SSL’) ? HTTPS_SERVER : HTTP_SERVER), 0, -1); // gets the base URL minus the trailing slash

Remove the [, -1] from the code

so it looks like this

$domain = substr((($request_type == ‘SSL’) ? HTTPS_SERVER : HTTP_SERVER), 0); // gets the base URL minus the trailing slash

works fine for me now.

By: GL Customs

GL Customs — Mon, 21 Sep 2009 18:40:06 +0000

I changed the $domain to this with good results:

$domain = HTTP_SERVER;

There shouldn’t be a need to call the SSL as I don’t believe SSL pages will get indexed.

By: Cycil Borggus

Cycil Borggus — Wed, 26 Aug 2009 08:19:40 +0000

So how do we get the link to show a correct url.
It truncates all TLD. ie:

Drops or cuts .net=ne, .com=co, org=or, no matter hos DNS is setup..

ADMIN:
In your case, replace the $domain line with this one:

$domain = “http://www.yourdomain.nl”

This will replace the variable server check with your hard coded domain name with/without trailing slash

By: Yannick

Yannick — Wed, 01 Apr 2009 03:42:29 +0000

Same results as Lukas. URLs are not truncated.

By: Lukas

Lukas — Mon, 02 Mar 2009 21:13:16 +0000

Sorry, I see no difference with this contribution.

By: Sabine

Sabine — Sat, 28 Feb 2009 14:08:49 +0000

Correction, it’s probably an issue with how your WordPress parses the quotes.
Still, the trailing slash code is messing up at the tld.

Admin: Yes, WordPress parses the quotes all wrong. Sorry about that. And it looks like in your case your server don’t put a trailing slash behind your domain name. In your case, replace the $domain line with this one:

$domain = “http://www.yourdomain.nl”

This will replace the variable server check with your hard coded domain name with/without trailing slash