search

adampmoss / CreareSEO

A free SEO extension for Magento 1
GNU General Public License v2.0
151 stars 72 forks source link

implemented more robust product canonical redirect, fixes #126 #135

Closed sprankhub closed 6 years ago

sprankhub commented 6 years ago

IMHO it is much cleaner to not use the $_SERVER global, but refer to the respective Magento methods. We have to compare with the escaped URL since getCurrentUrl also escapes the URL. But it makes more sense to redirect to the non-escaped version.

This should fix the redirect issue from #126. Can you confirm, @hejhog?

hejhog commented 6 years ago

Confirmation that the fix works on both my test and production server.