Search redirects defined in navigator account are not working properly

EmicoEcommerce / Magento2Tweakwise-archived

Magento 2 module for Tweakwise integration

Other

9 stars 25 forks source link

Search redirects defined in navigator account are not working properly #26

Closed VesnaS closed 6 years ago

VesnaS commented 6 years ago

Environment

PHP Version: 7.0
Magento Version: 2.1.6
Tweakwise Version: latest release
Tweakwise Export Version: latest release
Magento Deploy Mode: production
Third party modules: yes

Steps to reproduce

Create redirect in Tweakwise navigator account

redirect

Publish data and search for that term in Magento

Actual result

We always get redirected to the home page

Expected result

Magento should redirect to defined URL

EduardSelles commented 6 years ago

Hello,

I think it will work if you select a store view as category instead of the 'root'.

VesnaS commented 6 years ago

Hello,

I tried to change the configuration as you suggested, but to no avail. The result is the same.

EduardSelles commented 6 years ago

Unfortunately we can not reproduce the problem in our two (CE & EE) demo environments.

Maybe you can try to adjust the url again to a full path including 'http://'?

I am afraid that Emico can not solve the problem if it is not reproducible in a clean Magento installation.

VesnaS commented 6 years ago

Hello Eduard,

Thank you for your reply.

Yes, setting redirects as full path with protocol and domain solved the issue. I also tried clean Magento installation, and paths beginning with "/" also don't work - because of the issue that Magento UrlInterface has in the getUrl method that you used in your extension.

In conclusion, only full paths or relative paths (without beginning "/") will work properly.

Maybe you should consider looking into this issue in some future release.

Fgruntjes commented 6 years ago

If request by TW we will add support for full paths without domain in a feature release. Thank you for your test work on this.