improvements to logfile remove action

This commit is contained in:
doitux
2012-02-05 21:19:21 +00:00
parent 8857a25780
commit 945412268f
@@ -1356,6 +1356,7 @@ void settingsDialogImpl::refreshLogFileList()
item->setData(0, Qt::UserRole, dbFilesList.at(i).absoluteFilePath()); item->setData(0, Qt::UserRole, dbFilesList.at(i).absoluteFilePath());
treeWidget_logFiles->addTopLevelItem(item); treeWidget_logFiles->addTopLevelItem(item);
} }
treeWidget_logFiles->sortItems(0, Qt::DescendingOrder);
} }
void settingsDialogImpl::deleteLogFile() void settingsDialogImpl::deleteLogFile()
@@ -1368,12 +1369,10 @@ void settingsDialogImpl::deleteLogFile()
QMessageBox::Yes | QMessageBox::No); QMessageBox::Yes | QMessageBox::No);
if(ret == QMessageBox::Yes) { if(ret == QMessageBox::Yes) {
for (int i = 0; i < selectedItemsList.size(); ++i) { for (int i = 0; i < selectedItemsList.size(); ++i) {
qDebug() << selectedItemsList.at(i)->data(0, Qt::UserRole).toString();
QFile fileToDelete (selectedItemsList.at(i)->data(0, Qt::UserRole).toString()); if(!QFile::remove(selectedItemsList.at(i)->data(0, Qt::UserRole).toString())) {
if(!fileToDelete.remove()) { QMessageBox::warning(this, "Remove log file", "PokerTH cannot remove this log file, please verify that you have write access to this file!", QMessageBox::Close );
QMessageBox::warning(this, "Remove log file", "PokerTH cannot remove this log file, please check if you have write access to this file!", QMessageBox::Close );
} }
} }
refreshLogFileList(); refreshLogFileList();