From 945412268f1fcd3b6ac8a4bae87434db409f6b1c Mon Sep 17 00:00:00 2001 From: doitux Date: Sun, 5 Feb 2012 21:19:21 +0000 Subject: [PATCH] improvements to logfile remove action --- src/gui/qt/settingsdialog/settingsdialogimpl.cpp | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/gui/qt/settingsdialog/settingsdialogimpl.cpp b/src/gui/qt/settingsdialog/settingsdialogimpl.cpp index 86f55443..fe6e2a2b 100644 --- a/src/gui/qt/settingsdialog/settingsdialogimpl.cpp +++ b/src/gui/qt/settingsdialog/settingsdialogimpl.cpp @@ -1356,6 +1356,7 @@ void settingsDialogImpl::refreshLogFileList() item->setData(0, Qt::UserRole, dbFilesList.at(i).absoluteFilePath()); treeWidget_logFiles->addTopLevelItem(item); } + treeWidget_logFiles->sortItems(0, Qt::DescendingOrder); } void settingsDialogImpl::deleteLogFile() @@ -1368,12 +1369,10 @@ void settingsDialogImpl::deleteLogFile() QMessageBox::Yes | QMessageBox::No); if(ret == QMessageBox::Yes) { - for (int i = 0; i < selectedItemsList.size(); ++i) { - - QFile fileToDelete (selectedItemsList.at(i)->data(0, Qt::UserRole).toString()); - if(!fileToDelete.remove()) { - QMessageBox::warning(this, "Remove log file", "PokerTH cannot remove this log file, please check if you have write access to this file!", QMessageBox::Close ); + qDebug() << selectedItemsList.at(i)->data(0, Qt::UserRole).toString(); + if(!QFile::remove(selectedItemsList.at(i)->data(0, Qt::UserRole).toString())) { + QMessageBox::warning(this, "Remove log file", "PokerTH cannot remove this log file, please verify that you have write access to this file!", QMessageBox::Close ); } } refreshLogFileList();