Review Request 112589: Use the Nepomuk Query Builder widget to add syntax-highlighting in Dolphin search

[Denis Steckelmacher]

> On Sept. 8, 2013, 10:50 p.m., Frank Reininghaus wrote:
> > dolphin/src/search/dolphinsearchbox.cpp, line 247
> > <http://git.reviewboard.kde.org/r/112589/diff/1/?file=188172#file188172line247>
> >
> >     I'm wondering if this function could get a name that makes it more obvious what it is about. Maybe "updateSearchInputParsing" or something like that?
> >     
> >     In the places where this slot is called directly, "slotWhatGroupClicked" looks a bit weird ;-)

I replaced "slotWhatGroupClicked" with "updateSearchInputParsing" as you suggest. An other possible name is "enableParsingForContentSearch".


- Denis


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112589/#review39614
-----------------------------------------------------------


On Sept. 9, 2013, 11:45 a.m., Denis Steckelmacher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112589/
> -----------------------------------------------------------
> 
> (Updated Sept. 9, 2013, 11:45 a.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Description
> -------
> 
> 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.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/search/dolphinsearchbox.h f3cc535 
>   dolphin/src/search/dolphinsearchbox.cpp ef9c2bf 
> 
> Diff: http://git.reviewboard.kde.org/r/112589/diff/
> 
> 
> Testing
> -------
> 
> 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 ?
> 
> 
> Thanks,
> 
> Denis Steckelmacher
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20130909/3db054a2/attachment.htm>