Use the Nepomuk Query Builder widget to add syntax-highlighting in Dolphin search

Review Request #112589 - Created Sept. 8, 2013 and submitted

Information
Denis Steckelmacher
kde-baseapps
frameworks
Reviewers
dolphin

When Dolphin is compiled with Nepomuk support, this patch replaces the plain line edit in which the user enters a search query with the new auto-completed QueryBuilder widget.

The user interface of the widget is shown in http://steckdenis.be/static/images/dolphin-query-builder.png .

The QueryBuilder widget can be found in the "steckdenis-gsoc2013" branch of nepomuk-widgets. You will also need the "steckdenis-gsoc2013" branch of nepomuk-core. This patch is not yet ready for merging as we have to wait that my other branches get merged in nepomuk-core and nepomuk-widgets. Some small changes in the API of the QueryBuilder are still possible.

Compiling with the CMake variable WITH_NepomukWidgets set to ON is possible and the resulting Dolphin works as expected (the query builder widget is displayed and queries can be entered in it). When this variable is set to OFF, Dolphin still builds and runs correctly, except that there is no auto-completion available.

The only problem is that even without the patch (and Nepomuk enabled system-wide and at compile time), Dolphin does not seem to use Nepomuk queries. Every search operation is slow and there is a huge amount of disk activity. Nepomuk queries like "modified last week" work perfectly well in KRunner. This problem (likely due to a misconfiguration on my desktop as it is present in Dolphin master and Dolphin 4.11, with Nepomuk-Core master or 4.11) prevents me from testing that queries entered in the query builder actually work. Can somebody with a working dolphin test that ?

Issues

  • 0
  • 10
  • 0
  • 10
Description From Last Updated
Frank Reininghaus
Denis Steckelmacher
Emmanuel Pescosta
Denis Steckelmacher
Emmanuel Pescosta
Denis Steckelmacher
Emmanuel Pescosta
Denis Steckelmacher
Review request changed

Status: Closed (submitted)

Hrvoje Senjan

   
dolphin/src/search/dolphinsearchbox.cpp (Diff revision 4)
 
 

this looks wrong? other defintions use HAVE_BALOO

  1. Good catch, I've pushed a fix for this. I had looked for "Nepomuk" but had forgotten that Vim performs case-sensitive searching.

Loading...