DjangoGirls / tutorial

This is a tutorial we are using for Django Girls workshops
http://tutorial.djangogirls.org/
Other
1.53k stars 1.86k forks source link

Search results for polish are in spanish #1564

Open krzysztofzuraw opened 5 years ago

krzysztofzuraw commented 5 years ago

Issue description

When user searches for term in polish language search results are presented in spanish:

Screenshot 2019-09-07 at 17 39 24

Yet clicking on term results in polish entry 🤷‍♂

Language

Polish

Operating system

Tested on mac

das-g commented 5 years ago

I can reproduce this in Firefox on Linux and my locale is neither Spanish nor Polish:

LANG=de_CH.UTF-8
LC_CTYPE="de_CH.UTF-8"
LC_NUMERIC="de_CH.UTF-8"
LC_TIME="de_CH.UTF-8"
LC_COLLATE="de_CH.UTF-8"
LC_MONETARY="de_CH.UTF-8"
LC_MESSAGES="de_CH.UTF-8"
LC_PAPER="de_CH.UTF-8"
LC_NAME="de_CH.UTF-8"
LC_ADDRESS="de_CH.UTF-8"
LC_TELEPHONE="de_CH.UTF-8"
LC_MEASUREMENT="de_CH.UTF-8"
LC_IDENTIFICATION="de_CH.UTF-8"
LC_ALL=

To reproduce:

  1. go to https://tutorial.djangogirls.org
  2. choose "Polski"
  3. enter something in the top left search field (where it says "Wpisz szukaną frazę")

expected:

You get results from the Polish translation of the Django Girls tutorial

observed:

das-g commented 5 years ago

Is this a bug in GitBook legacy? I can't see any related settings on our side (i.e. in the content of this repo) that might be misconfigured.

krzysztofzuraw commented 5 years ago

It can be - I found that gitbook legacy is using https://github.com/GitbookIO/plugin-search which seems not maintained anymore

das-g commented 5 years ago

I've systematically tried in all languages listed on https://tutorial.djangogirls.org/

There seems to be a clear pattern linked to the order in which the translations are listed on https://tutorial.djangogirls.org/ (which probably also is the order in which they were initially published?): Everything after Spanish has Spanish search results, while those listed before Spanish have results in the respective proper language:

das-g commented 5 years ago

I wonder whether this is related to #177.

@SamyPesse, can you help us here?

magul commented 5 years ago

I have just bumped into probably root cause here. When I build tutorial locally I got

magul@bohr tutorial (crowdin-translation-de +) $ gitbook build
info: parsing multilingual book, with 18 languages 
info: 17 plugins are installed 
info: 15 explicitly listed 
info: loading plugin "heading-anchors"... OK 
info: loading plugin "ga"... OK 
info: loading plugin "richquotes"... OK 
info: loading plugin "github"... OK 
info: loading plugin "language-picker"... OK 
info: loading plugin "sidebar-ad"... OK 
info: loading plugin "codeblock-label"... OK 
info: loading plugin "sectionx-ex"... OK 
info: loading plugin "collapsible-menu"... OK 
info: loading plugin "highlight"... OK 
info: loading plugin "search"... OK 
info: loading plugin "lunr"... OK 
info: loading plugin "sharing"... OK 
info: loading plugin "fontsettings"... OK 
info: loading plugin "theme-default"... OK 
info: found 18 asset files 
warn: "options" property is deprecated, use config.get(key) instead 
info:  
info: generating language "en" 
info: found 26 pages 
info: found 57 asset files 
warn: "sections" property is deprecated, use page.content instead 
warn: "this.generator" property is deprecated, use "this.output.name" instead 
warn: "navigation" property is deprecated 
warn: "book" property is deprecated, use "this" directly instead 
info:  
info: generating language "cs" 
info: found 25 pages 
info: found 50 asset files 
info:  
info: generating language "de" 
info: found 26 pages 
info: found 56 asset files 
info:  
info: generating language "el" 
info: found 26 pages 
info: found 54 asset files 
info:  
info: generating language "es" 
info: found 26 pages 
info: found 55 asset files 
info:  
info: generating language "fa" 
info: found 26 pages 
info: found 56 asset files 
warn: search index is too big, indexing is now disabled 
info:  
info: generating language "fr" 
info: found 26 pages 
info: found 57 asset files 
info:  
info: generating language "hu" 
info: found 25 pages 
info: found 50 asset files 
info:  
info: generating language "it" 
info: found 25 pages 
info: found 50 asset files 
info:  
info: generating language "ja" 
info: found 26 pages 
info: found 55 asset files 
info:  
info: generating language "ko" 
info: found 26 pages 
info: found 75 asset files 
info:  
info: generating language "pl" 
info: found 26 pages 
info: found 54 asset files 
info:  
info: generating language "pt" 
info: found 26 pages 
info: found 61 asset files 
info:  
info: generating language "ru" 
info: found 26 pages 
info: found 74 asset files 
info:  
info: generating language "sk" 
info: found 26 pages 
info: found 53 asset files 
info:  
info: generating language "tr" 
info: found 26 pages 
info: found 53 asset files 
info:  
info: generating language "uk" 
info: found 25 pages 
info: found 49 asset files 
info:  
info: generating language "zh" 
info: found 25 pages 
info: found 49 asset files 
info: >> generation finished with success in 92.5s ! 

There's a warning during rendering Persian language:

warn: search index is too big, indexing is now disabled 

I would assume, that increasing index threshold should resolve that issue. (We shold alos check hot to do so on Gitbook service).

IllicLanthresh commented 1 year ago

I'm having the same issue but I see that the problem is not tied to Polish-Spanish, searching yields results in all languages, independently of the selected language. Probably better to modify the issue tittle