Skip to content
This repository has been archived by the owner on Oct 29, 2019. It is now read-only.

Article fields in the wrong language are getting indexed #79

Closed
mikek opened this issue Feb 21, 2015 · 7 comments
Closed

Article fields in the wrong language are getting indexed #79

mikek opened this issue Feb 21, 2015 · 7 comments

Comments

@mikek
Copy link
Contributor

mikek commented Feb 21, 2015

At search_indexes.py there are safe_translation_getter() calls without current language argument (and it looks like parler does not try to detect the current language inside that method either). So, article's filelds translated in first available language are sometimes indexed in the Solr core for another language.
The aldryn-search was built with hvad in mind, so we probably need to take some extra steps in index methods to properly index translatable models.
@yakky, can you try to reproduce this and adjust the index?
CC: @czpython @intellectronica

@yakky yakky self-assigned this Feb 21, 2015
@yakky
Copy link
Contributor

yakky commented Feb 21, 2015

On it

@yakky
Copy link
Contributor

yakky commented Mar 4, 2015

@mikek could you check 0c314e7?

@mkoistinen
Copy link
Contributor

@mikek can you advise?

@mkoistinen mkoistinen added this to the 1.0.0 milestone Apr 10, 2015
@mkoistinen
Copy link
Contributor

@mikek Is this still an issue? Need your feedback here.

@mikek
Copy link
Contributor Author

mikek commented Apr 13, 2015

@mkoistinen, IIRC, I was not able to test (or id did not fix the issue for me) 0c314e7 and it can not be cleanly applied to master now. @chronossc is now working on search related code in our project, he might know for sure if newsblog search configuration is up to date.

@mkoistinen
Copy link
Contributor

@chronossc did you already fix this issue? If so, can we close this?

@mkoistinen
Copy link
Contributor

Closing due to age, please re-open if this issue still exists

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

No branches or pull requests

3 participants