Bug 115935 - khelpcenter sends user into a loop
Summary: khelpcenter sends user into a loop
Status: RESOLVED FIXED
Alias: None
Product: khelpcenter
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR wishlist
Target Milestone: ---
Assignee: Documentation Editorial Team
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-11-08 17:09 UTC by Thorsten Staerk
Modified: 2016-03-12 23:24 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
program correction (4.53 KB, patch)
2005-11-08 19:51 UTC, Thorsten Staerk
Details
one example how it looked @ me after applying the patch (82.37 KB, image/jpeg)
2005-11-13 17:52 UTC, Thorsten Staerk
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Thorsten Staerk 2005-11-08 17:09:47 UTC
Version:            (using KDE KDE 3.4.2)
Installed from:    SuSE RPMs
OS:                Linux

When in konqueror, I click onto Help -> Konqueror Handbook and search for "Bookmarks". I expect to get some hints about konqueror's bookmarks. Instead, I get links to "Suse Linux Documentation". Ok, I ignore that. Then, I get a link to "Konqueror handbook" which sends me into a loop because that is where I started the search. 
Please implement that konqueror handbook chapters are linked instead of the konqueror handbook, that's what the user expects from a search functionality.
Comment 1 Thorsten Staerk 2005-11-08 19:51:23 UTC
Created attachment 13348 [details]
program correction

Here is the patch I sent to Cornelius Schumacher about one year ago.
Comment 2 Cornelius Schumacher 2005-11-09 00:30:16 UTC
Sorry for not taking care of the patch. Unfortunately I didn't have any time left for the help center.
Comment 3 Thorsten Staerk 2005-11-13 17:52:30 UTC
Created attachment 13418 [details]
one example how it looked @ me after applying the patch
Comment 4 Pino Toscano 2016-03-12 23:06:10 UTC
Git commit 583ed246c75ce2c686957b6e1dc34aaed31206b4 by Pino Toscano.
Committed on 12/03/2016 at 23:01.
Pushed by pino into branch 'master'.

Add Xapian support for indexing and searching

Start using Xapian for indexing and searching the documentation: it
works much better than htp://dig, it provides a C++ API, and it is well
maintained.

Introduce two helper tools to create/update an index, and search on it,
starting from the cache.bz2 files generated by meinproc: this way it is
possible to index the whole text, with the entities already expanded
(something which would not be there when parsing the docbook files
directly). Make use of libxml2 to parse the HTML documents, and get all
the text in them.

Xapian and libxml2 are currently considered as mandatory dependencies:
they are portable, generally available everywhere, and not requiring
themselves too many extra dependencies.
Related: bug 126710, bug 93664, bug 103266, bug 122437, bug 125276, bug 152671, bug 158633, bug 209415, bug 256397, bug 266290

M  +13   -0    CMakeLists.txt
M  +35   -0    searchhandlers/CMakeLists.txt
A  +134  -0    searchhandlers/cachereader.cpp     [License: GPL (v2+)]
A  +49   -0    searchhandlers/cachereader.h     [License: GPL (v2+)]
A  +107  -0    searchhandlers/htmltextdump.cpp     [License: GPL (v2+)]
A  +29   -0    searchhandlers/htmltextdump.h     [License: GPL (v2+)]
A  +5    -0    searchhandlers/xapian.desktop.cmake
A  +126  -0    searchhandlers/xapiancommon.cpp     [License: GPL (v2+)]
A  +50   -0    searchhandlers/xapiancommon.h     [License: GPL (v2+)]
A  +269  -0    searchhandlers/xapianindexer.cpp     [License: GPL (v2+)]
A  +151  -0    searchhandlers/xapiansearch.cpp     [License: GPL (v2+)]

http://commits.kde.org/khelpcenter/583ed246c75ce2c686957b6e1dc34aaed31206b4
Comment 5 Pino Toscano 2016-03-12 23:24:42 UTC
Git commit cafde04221d0f0d20b042f9afcf63669f23b7a1d by Pino Toscano.
Committed on 12/03/2016 at 23:02.
Pushed by pino into branch 'master'.

Re-enable the full-index search

Now that
- Xapian is in place for indexing and searching
- ht://dig is removed
- indexes are updated automatically each startup
we can enable the full-index search again.
Related: bug 57897, bug 93664, bug 103266, bug 112958, bug 161514, bug 173767, bug 207519, bug 222494, bug 237329, bug 266290, bug 269260, bug 269619

M  +1    -3    navigator.cpp

http://commits.kde.org/khelpcenter/cafde04221d0f0d20b042f9afcf63669f23b7a1d