Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Elasticsuite Tracker does not properly support ampersand in categories title (and probably in product names, cms pages title, etc) #771

Closed
rbayet opened this issue Feb 20, 2018 · 1 comment

Comments

@rbayet
Copy link
Collaborator

rbayet commented Feb 20, 2018

Preconditions

Magento Version : EE 2.2.2

ElasticSuite Version : 2.5.0

Environment : Developer mode

Third party modules : lots, none related to search

Steps to reproduce

  1. Name a category with an ampersand (&) in the title, for instance "Baby & Child"
  2. Place a product in that category
  3. Navigate in frontend on the home page, the "Baby & Child" category, the product page, perform a search to find that product

Expected result

  1. After the cron:run reindexing of tracker events, events document should be present in the Elasticsearch index with the correct page title (category page event), category label (product page event)

Actual result

  1. The ampersand generates additional request parameters/variables
  2. The original parameters/variables contain partial value and those extra parameters are pushed "as is" as empty ES document fields
  3. Those bogus document fields become part of the mapping

category_hit_event

product_hit_event

mapping

afoucret pushed a commit to afoucret/elasticsuite that referenced this issue Feb 20, 2018
@afoucret
Copy link
Contributor

Fixed by PR #773

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants