Review Request: Helper Methods for FileType Enum in new Class FileTypeSupport, get Type in TagDialog from Database instead of Extension, some Code Changes as Example of the Advantage of the Helper Methods

[Stefan Derkits]

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

(Updated 2010-11-05 19:48:36.301655)


Review request for Amarok.


Changes
-------

some changes got lost in between the versions ... this patch now contains all changes


Summary
-------

-) Created new Class with some Helper Functions
-) Changed MetaQueryWidget to use the QStringList that contains all Filetypes
-) Meta::SqlTrack.type() now gets it's Information from the Database instead of the FileExtension


Diffs (updated)
-----

  shared/FileType.h 55c80b9 
  shared/FileType.cpp PRE-CREATION 
  src/CMakeLists.txt b114bcd 
  src/browsers/CollectionTreeItemModelBase.cpp 3f64a73 
  src/core-impl/collections/db/sql/CMakeLists.txt 858f5c3 
  src/core-impl/collections/db/sql/SqlMeta.h f1ee131 
  src/core-impl/collections/db/sql/SqlMeta.cpp 006c805 
  src/core-impl/collections/support/MemoryFilter.cpp e2509ac 
  src/widgets/MetaQueryWidget.cpp 0249edf 
  tests/synchronization/CMakeLists.txt e5df2df 
  utilities/collectionscanner/CMakeLists.txt 22f3f1c 
  utilities/collectionscanner/Track.cpp 9291662 

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


Testing
-------

Tested in the App, found no wrong Behaviour


Thanks,

Stefan

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20101105/9e8b7b8f/attachment-0001.htm