Review Request: Nepomuk desktop query API

[Sebastian Trueg]

> On 2009-11-05 17:58:15, Volker Krause wrote:
> > Looks good, just a few minor details.

Why add the explicit keyword? I don't see any possibility for misuse of the constructors. Plus I really like to be able to write

ComparisionTerm( Vocabulary::NAO::hasTag(), QString("nepomuk") );

instead of

ComparisionTerm( Nepomuk::Types::Property(Vocabulary::NAO::hasTag()), LiteralTerm(Soprano::LiteralValue(QString("nepomuk"))));


- Sebastian


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


On 2009-11-04 14:54:51, Sebastian Trueg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2061/
> -----------------------------------------------------------
> 
> (Updated 2009-11-04 14:54:51)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> One thing has been missing ever since KDE 4.0: A good Nepomuk query API. We cannot expect app developers to all learn SPARQL.
> This API is the final version (Virtuoso-extensions-based) of the query API that has already been in kdebase/workspace/libs/nepomukquery and kdebase/runtime/nepomuk/libnepomukquery. The API is already used by the Nepomuk query service, the Nepomuk search kio slave, the Nepomuk Plasma runner, and the simple search client in playground.
> 
> It makes perfect sense to include this API in KDE 4.4.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdelibs/includes/CMakeLists.txt 1044143 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/AndTerm PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/ComparisonTerm PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/GroupTerm PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/LiteralTerm PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/NegationTerm PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/OrTerm PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/Query PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/QueryParser PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/QueryServiceClient PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/ResourceTerm PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/Result PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/SimpleTerm PRE-CREATION 
>   trunk/KDE/kdelibs/includes/Nepomuk/Query/Term PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/CMakeLists.txt 1044143 
>   trunk/KDE/kdelibs/nepomuk/query/CMakeLists.txt PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/andterm.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/andterm.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/andterm_p.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/comparisonterm.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/comparisonterm.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/comparisonterm_p.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/dateparser.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/dateparser.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/dbusoperators.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/dbusoperators.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/groupterm.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/groupterm.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/groupterm_p.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/literalterm.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/literalterm.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/literalterm_p.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/negationterm.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/negationterm.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/negationterm_p.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/nepomukquery_export.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/nie.trig PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/org.kde.nepomuk.Query.xml PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/org.kde.nepomuk.QueryService.xml PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/orterm.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/orterm.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/orterm_p.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/query.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/query.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/query_p.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/querybuilderdata.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/queryparser.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/queryparser.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/queryserviceclient.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/queryserviceclient.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/resourceterm.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/resourceterm.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/resourceterm_p.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/result.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/result.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/simpleterm.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/simpleterm.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/simpleterm_p.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/term.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/term.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/term_p.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/test/CMakeLists.txt PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/test/pimo.trig PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/test/queryparsertest.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/test/queryparsertest.cpp PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/test/querytest.h PRE-CREATION 
>   trunk/KDE/kdelibs/nepomuk/query/test/querytest.cpp PRE-CREATION 
> 
> Diff: http://reviewboard.kde.org/r/2061/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sebastian
> 
>