From c5e435f9bd5f06ef44a453a3d3cb5ea22ff2c6fb Mon Sep 17 00:00:00 2001 From: Klaas Freitag Date: Fri, 6 Dec 2013 16:37:30 +0100 Subject: [PATCH] Let recursive removal also remove the top dir. --- src/mirall/syncjournaldb.cpp | 26 ++++++++++++++------------ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/src/mirall/syncjournaldb.cpp b/src/mirall/syncjournaldb.cpp index 502fa8558d..4209f143db 100644 --- a/src/mirall/syncjournaldb.cpp +++ b/src/mirall/syncjournaldb.cpp @@ -381,19 +381,21 @@ bool SyncJournalDb::deleteFileRecord(const QString& filename, bool recursively) QMutexLocker locker(&_mutex); if( checkConnect() ) { - if (!recursively) { - qlonglong phash = getPHash(filename); - _deleteFileRecordPhash->bindValue( 0, QString::number(phash) ); + // if (!recursively) { + // always delete the actual file. - if( !_deleteFileRecordPhash->exec() ) { - qWarning() << "Exec error of SQL statement: " - << _deleteFileRecordPhash->lastQuery() - << " : " << _deleteFileRecordPhash->lastError().text(); - return false; - } - qDebug() << _deleteFileRecordPhash->executedQuery() << phash << filename; - _deleteFileRecordPhash->finish(); - } else { + qlonglong phash = getPHash(filename); + _deleteFileRecordPhash->bindValue( 0, QString::number(phash) ); + + if( !_deleteFileRecordPhash->exec() ) { + qWarning() << "Exec error of SQL statement: " + << _deleteFileRecordPhash->lastQuery() + << " : " << _deleteFileRecordPhash->lastError().text(); + return false; + } + qDebug() << _deleteFileRecordPhash->executedQuery() << phash << filename; + _deleteFileRecordPhash->finish(); + if( recursively) { _deleteFileRecordRecursively->bindValue(0, filename); if( !_deleteFileRecordRecursively->exec() ) { qWarning() << "Exec error of SQL statement: "