Review Request: Get FileType of SQLTracks from Database instead of File Extension

[Stefan Derkits]

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

Review request for amarok.


Summary
-------

-) Added a new Class Variable to SqlTrack to hold the FileType of a Track
-) Retrieve FileType from Database and save it in the Class Variable
-) Return a QString Representation of the Amarok::FileType when calling the Method SqlTrack::type()


Diffs
-----

  src/core-impl/collections/sqlcollection/SqlMeta.h ee3ec21 
  src/core-impl/collections/sqlcollection/SqlMeta.cpp 2da0333 

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


Testing
-------

Hard to test, but the TagDialog displays the correct Filetypes
Wanted to test it with a Testcase where FileType in Database is different then the FileExtension but realized that only certain FileExtensions are read by the CollectionScanner/TagLib :)
So actually atm, the Code doesn't make much difference, but in my opinion this is a cleaner (and more future proof) Solution to display the Filetype then just displaying the Extension of the File


Thanks,

Stefan

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20100927/01f31769/attachment.htm