Review Request: Proof of concept for the 2012 GSoC project idea 'Semantic Collection for Amarok'.

[Phalgun Guduthur]

Like Edward said it was not meant to be shipped.

"How about a real Nepumuk backed collection not using sql at all?"
It is done and have submitted the request for the same.

Sorry for not closing it earlier. Closed it now.

Thanks Ralf.

On Thu, Aug 16, 2012 at 5:57 PM, Ralf Engels <ralf-engels at gmx.de> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104369/
>
> I wouldn't ship it.
> Ratings will not longer be written back to the files. Search would not work (since the search is done in the sql database and the rating is not longer written there) and so on.
> How about a real Nepumuk backed collection not using sql at all?
>
> I would like to reject this request if you agree.
>
>
> - Ralf
>
> On March 22nd, 2012, 4:58 p.m., Phalgun Guduthur wrote:
>   Review request for Amarok and Teo Mrnjavac.
> By Phalgun Guduthur.
>
> *Updated March 22, 2012, 4:58 p.m.*
> Description
>
> I have tried to demonstrate a basic read and write of Nepomuk index through Amarok by altering how song ratings are stored.
>
> When applied, this patch stores any changed rating of a song into the Nepomuk index and not the Sql backend. To test this, change the rating of any song through Amarok and check the attributes of that song using Dolphin (song->properties). The song rating would have changed accordingly.
>
> To demonstrate the read part, Each song's rating is fetched from the Nepomuk index instead of the Sql backend. To test this, change the rating of any song through Dolphin and the same would be reciprocated in Amarok.
>
> Please note, this is only for proof of concept. This is not intended to be shipped.
> The code changes I have made are only temporary. The actual project will have Nepomuk classes and handlers to do the same task.
>
>   Testing
>
> The existing test cases work.
> It has been tried on numerous songs successfully on my computer.
>
>   Diffs
>
>    - src/core-impl/collections/db/sql/CMakeLists.txt (bdb3966)
>    - src/core-impl/collections/db/sql/SqlMeta.cpp (e663adf)
>
> View Diff <http://git.reviewboard.kde.org/r/104369/diff/>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20120816/ceb5808b/attachment-0001.html>