From 76bb89940a3818456f535b7d4a8ba6ab9f382631 Mon Sep 17 00:00:00 2001 From: Arno Date: Sat, 6 Nov 2010 14:12:12 +0100 Subject: Bugfix in moveToBurn() Using the filename from the QModelIndex isn't enough to determine wether the file is a cover or not. A series can have more than one part. So check the real file type and only copy if it's not of type Movie. This fix should obsolete the new ConsistencyChecker :) --- filestreewidget.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'filestreewidget.cpp') diff --git a/filestreewidget.cpp b/filestreewidget.cpp index 7fbc0fc..b017d7d 100644 --- a/filestreewidget.cpp +++ b/filestreewidget.cpp @@ -90,7 +90,11 @@ void FilesTreeWidget::moveToBurn(){ foreach(QString name, files.keys()){ QString sourceFile = Helper::createArchivePath(name, files.value(name)); QString targetFile = QString("%1/%2/%3").arg(burnDir).arg(dirName).arg(name); - if(name == i.data(FilesTreeModel::FileNameRole).toString()){ + int fileType = mModel->fileType(files.value(name)); + if(fileType == -1){ + continue; + } + if(fileType == FilesTreeModel::Movie){ QFile::rename(sourceFile, targetFile); }else{ QFile::copy(sourceFile, targetFile); -- cgit v1.2.3-70-g09d2