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

Review request for Nepomuk.


Description
-------

This patch solves a bug that prevented mp3 files with "disc number" tag being 
indexed. Code is based in "track number" code, and in the solution implemented 
in "flacendanalyzer.cpp", with the exception of the "found_track" flag because 
I think that is not necessary.

The error message displayed by nepomukserver was:
Soprano: "Invalid argument (1)": 
"<http://www.semanticdesktop.org/ontologies/2009/02/19/nmm#setNumber> has a 
rdfs:domain of 
<http://www.semanticdesktop.org/ontologies/2009/02/19/nmm#MusicPiece>. <_:c> 
only has the following types 
<http://www.semanticdesktop.org/ontologies/2009/02/19/nmm#MusicAlbum>, 
<http://www.w3.org/2000/01/rdf-schema#Resource>"


Diffs
-----

  lib/endanalyzers/id3endanalyzer.h 4d33740 
  lib/endanalyzers/id3endanalyzer.cpp a3b33ae 

Diff: http://git.reviewboard.kde.org/r/102994/diff/diff


Testing
-------

Before applying the patch:
-Trying to index several mp3s with disc number tag: indexing fails
-Trying to index the same mp3s after disc number tag was removed: indexing works

After applying the patch:
-Trying to index several mp3s with disc number tag: indexing worked and proper 
value was set in nmm:setNumber


Thanks,

Ignacio Serantes

_______________________________________________
Nepomuk mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/nepomuk

Reply via email to