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 kiwix-serve fulltext result page template wording (english) #27

Closed
tim-moody opened this issue May 1, 2017 · 12 comments
Closed

Fix kiwix-serve fulltext result page template wording (english) #27

tim-moody opened this issue May 1, 2017 · 12 comments
Assignees
Labels

Comments

@tim-moody
Copy link

No result were found for XXX should read

No results were found for XXX

@kelson42
Copy link
Contributor

kelson42 commented May 1, 2017

@tim-moody Can you please make a detailed bug report with all the details needed to reproduce the problem?

@tim-moody
Copy link
Author

easy. enter any search argument that returns no results and look at the message.

@kelson42
Copy link
Contributor

kelson42 commented May 1, 2017

Which ZIM file?
What is the kiwix-serve command you have started?
Which version of kiwix-serve?
What is the string you exactly enter?
What do you get exactly?
What do you expect and why?

@tim-moody
Copy link
Author

  1. download and untar http://download.kiwix.org/nightly/2017-04-30/kiwix-tools_armhf_2017-04-30.tar.gz
  2. run /opt/schoolserver/kiwix/bin/kiwix-serve --port 3000 /library/zims/content/wikipedia_en_medicine_2017-04.zim
  3. navigate browser to :3000
  4. open medical wiki by clicking appropriately
  5. enter XXXXXXX in search field
  6. click on containing 'XXXXXXX'
    7A. screen displays No result were found for XXXXXXX
    7B. screen should display No results were found for XXXXXXX
    The reason I expect this is because English has agreement between subject and verb with respect to number and the plural is idiomatically correct.

@tim-moody
Copy link
Author

I see that this bug existed in prior versions

@kelson42 kelson42 added the bug label May 1, 2017
@kelson42
Copy link
Contributor

kelson42 commented May 1, 2017

This is a bit less trivial as it looks like because of 2d29753271123d9dc670d96306e192e11c860853. The point is that is looks like we can not change it easily now, @rgaudin you confirm? I'm not sure what would be the best approach. now. @mgautierfr Any ideas?

@rgaudin
Copy link
Member

rgaudin commented May 1, 2017

@kelson42, can you add a link to that commit?

Last time I looked at that code (2years?), it was an English-only template results.tmpl which could be easily fixed but maybe search (and its results) have moved to kiwix-lib now?

@kelson42
Copy link
Contributor

kelson42 commented May 2, 2017

@rgaudin It's still english-only, but how to generate the results.ct2 file?

@mgautierfr
Copy link
Member

It seems that the ct2 file is generated by ctpp2c binary coming from ctpp2 dependency.

The results.ct2 is stored in the kiwix-lib repository (it's the searcher from kiwix-lib who use it).
We should store the results.tmpl in the repository and make the build system automatically generate the results.ct2.

@tim-moody
Copy link
Author

this remains in nightly 5-29

@kelson42 kelson42 changed the title kiwix-serve English Fix kiwix-serve fulltext result page template wording (english) Jun 6, 2017
@kelson42
Copy link
Contributor

kelson42 commented Jun 6, 2017

We need to fix this first kiwix/libkiwix#50

@kelson42
Copy link
Contributor

Fixed with kiwix/libkiwix@b6a58d1

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

No branches or pull requests

4 participants