[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [libqapt] src/worker: Don't set the transaction status to finished until we have properly set the ex
From:       Jonathan Thomas <echidnaman () kubuntu ! org>
Date:       2013-01-31 23:28:53
Message-ID: 20130131232853.75659A6091 () git ! kde ! org
[Download RAW message or body]

Git commit e823e0ac3f2316948db30129e256755548c179b7 by Jonathan Thomas.
Committed on 01/02/2013 at 00:27.
Pushed by jmthomas into branch 'master'.

Don't set the transaction status to finished until we have properly set the exit code.

M  +1    -1    src/worker/transaction.cpp

http://commits.kde.org/libqapt/e823e0ac3f2316948db30129e256755548c179b7

diff --git a/src/worker/transaction.cpp b/src/worker/transaction.cpp
index b074977..84cdead 100644
--- a/src/worker/transaction.cpp
+++ b/src/worker/transaction.cpp
@@ -274,9 +274,9 @@ void Transaction::setExitStatus(QApt::ExitStatus exitStatus)
 {
     QMutexLocker lock(&m_dataMutex);
 
-    setStatus(QApt::FinishedStatus);
     m_exitStatus = exitStatus;
     emit propertyChanged(QApt::ExitStatusProperty, QDBusVariant(exitStatus));
+    setStatus(QApt::FinishedStatus);
     emit finished(exitStatus);
 }
 

[prev in list] [next in list] [prev in thread] [next in thread]