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

Fix locale_negotiator for fulltextsearch #2091

Merged
merged 1 commit into from
Mar 23, 2016
Merged

Fix locale_negotiator for fulltextsearch #2091

merged 1 commit into from
Mar 23, 2016

Conversation

pvalsecc
Copy link
Contributor

This function was relying on Piramid to use default_locale_name if no locale
is found. But fulltextsearch is using this function directly and was returing
status=500 if the browser's languages were not matching the list of available
languages.

@sbrunner
Copy link
Member

I agree but Travis don't ...

@sbrunner
Copy link
Member

maven.ibiblio.org looks down ...

@sbrunner sbrunner added this to the 1.6.8 milestone Mar 22, 2016
@sbrunner
Copy link
Member

Should be rebased :-)

@pvalsecc pvalsecc force-pushed the fulltext_locale branch 2 times, most recently from e980de5 to a5d5990 Compare March 23, 2016 07:52
This function was relying on Piramid to use default_locale_name if no locale
is found. But fulltextsearch is using this function directly and was returing
status=500 if the browser's languages were not matching the list of available
languages.
@sbrunner
Copy link
Member

Thanks, merge :-)

@pvalsecc pvalsecc merged commit 60ed566 into 1.6 Mar 23, 2016
@pvalsecc pvalsecc deleted the fulltext_locale branch March 23, 2016 14:18
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

Successfully merging this pull request may close these issues.

2 participants