Hi,

I was testing this module, came across this issue where it prints query string in canonical url.

For example if url is being crawl from social network or something similar service where they add query parameters for tracking purpose. And in this module same parameters gets printed in canonical url, which is bad for SEO i guess.

For example, request url:
http://www.xyz.com/some-article-path/page/0/1?utm_medium=social&utm_source=facebook.com

Resulted canonical url in this module:
http://www.xyz.com/some-article-path/page/0/1?utm_medium=social&utm_source=facebook.com

Should be:
http://www.xyz.com/some-article-path/page/0/1

So i think, if clean url/pagination is enabled then lets avoid printing query string from request url.
Suggested solution: Using stroke function to avoid parameters after '?'
Example: $link_current_page= strtok($link_current_page, '?');

Comments

Rameez created an issue. See original summary.

Rameez’s picture

Issue summary: View changes