- First commit to issue fork.
The canonical URL that is being generated by smart_paging has a few problems.
1. The url is inaccurately putting the current page and nopaging arguments together. It is also url encoding the ampersand.
Example: {url}?page=0%2C2&nopaging=1
2. The canonical added by smart_paging does not take into consideration canonicals added by the system or other modules. In most cases this ends up generating two canonical links in the head.
3. We cannot assume that we should always override the canonical with a paginated URL. There are many modules that can modify/override the canonical and all we know is that there is paging applied to 'this' URL. So, if the current paged URL matches an existing canonical URL, we can modify/override it. If there is NO canonical URL, then we can add one. Otherwise we shouldn't do anything.
I have been working on a patch for this issue and should be able to post one within the next day.
Fixed
1.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.