What are you using to generate the canonical link? I just tried installing the canonical header tag module on my test site, and it correctly generates the link without a session ID.
I was using the header tag module that came with the Ultimate SEO Urls addon, removed that and installed the stock one and it worked
Edit - I spoke to soon.
The SEO analysis says the link points to
and then this below
The specified canonical link points to a different page.
I think it might have something to do with .htaccess but not sure
Code: Select all
## BEGIN USU URL rewrite rules ##
## USU Version: 3.0.4 ##
<IfModule mod_rewrite.c>
RewriteEngine On
RewriteBase /
RewriteRule ^([a-z0-9/-]+)-c-([0-9_]+).html$ index.php [NC,L,QSA]
RewriteRule ^([a-z0-9/-]+)-m-([0-9]+).html$ index.php [NC,L,QSA]
RewriteRule ^([a-z0-9/-]+)-p-([0-9]+).html$ product_info.php [NC,L,QSA]
RewriteRule ^([a-z0-9/-]+)-pi-([0-9]+).html$ info.php [NC,L,QSA]
## For rewrite, path_rewrite
RewriteRule ^([a-z0-9/-]+)-pr-([0-9]+).html$ ext/modules/content/reviews/write.php [NC,L,QSA]
## For standard, path_standard
RewriteRule ^write.php/([a-z0-9/-]+)-pr-([0-9]+) ext/modules/content/reviews/write.php [NC,L,QSA]
</IfModule>
Perhaps it wants it set in the HTML_PARAMS, e.g. for German:
https://github.com/cupidare/PhoenixCart ... an.php#L30 Code: Select all
const HTML_PARAMS = ' dir="ltr" lang="de"';
Don't forget to include a space at the beginning of the string. Also, you may want the language to be en-GB (rather than just en). For German, the language is de (i.e. de is the thing that you want to replace in that line).
Strange one this as my line was this
changed it to
Code: Select all
const HTML_PARAMS = ' dir="ltr" lang="en"';
and it works, happy days.
on to the next task.............