[Nepomuk] Re: Review Request: storeResource should return the mappings

[Artem Serebriyskiy]

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


I may know how to pass QHash<QUrl,QUrl> via DBus. Could you please delay commiting, I want to try to implement it and send it to you so changes to be submitted in one commit, instead of two. I will try to send it in the next 12 hours. Ok?

- Artem


On July 29, 2011, 9:04 a.m., Vishesh Handa wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102128/
> -----------------------------------------------------------
> 
> (Updated July 29, 2011, 9:04 a.m.)
> 
> 
> Review request for Nepomuk and Sebastian Trueg.
> 
> 
> Summary
> -------
> 
> When SimpleResources are passed on to storeResources they are run through an identification process, so as to avoid duplicates. The resources which fail the identification process are then created.
> 
> It is nice to have a mapping of the SimpleResource uris and the uris that are finally pushed in the Nepomuk Repository.
> 
> I've tried for over 2 days to transfer the stupid QHash<QUrl,QUrl> over dbus. I've now given up and passed it as a QHash<QString,QString>.
> 
> 
> Diffs
> -----
> 
>   nepomuk/interfaces/org.kde.nepomuk.DataManagement.xml 0172373 
>   nepomuk/services/storage/datamanagementadaptor.h 05ef64c 
>   nepomuk/services/storage/datamanagementadaptor.cpp 4ca8114 
>   nepomuk/services/storage/datamanagementcommand.h c970404 
>   nepomuk/services/storage/datamanagementmodel.h 9722157 
>   nepomuk/services/storage/datamanagementmodel.cpp 062cdcc 
>   nepomuk/services/storage/lib/CMakeLists.txt 2189a7f 
>   nepomuk/services/storage/lib/datamanagement.h 5b31603 
>   nepomuk/services/storage/lib/datamanagement.cpp 9c570b0 
>   nepomuk/services/storage/lib/dbustypes.h 900e219 
>   nepomuk/services/storage/lib/dbustypes.cpp 27fd0f4 
>   nepomuk/services/storage/lib/simpleresourcegraph.h 386b8c1 
>   nepomuk/services/storage/lib/simpleresourcegraph.cpp 3bab264 
>   nepomuk/services/storage/lib/storeresourcesjob.h PRE-CREATION 
>   nepomuk/services/storage/lib/storeresourcesjob.cpp PRE-CREATION 
>   nepomuk/services/storage/test/asyncclientapitest.cpp c430f06 
>   nepomuk/services/strigi/indexer/nepomukindexwriter.cpp 8164b69 
> 
> Diff: http://git.reviewboard.kde.org/r/102128/diff
> 
> 
> Testing
> -------
> 
> Tested while using metadata sharing. The mappings are correctly returned.
> 
> 
> Thanks,
> 
> Vishesh
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/nepomuk/attachments/20110729/349d0871/attachment.htm