From 0941a57c913f984e11b223223356d98239147740 Mon Sep 17 00:00:00 2001
From: Michael Hackstein <michael@arangodb.com>
Date: Wed, 1 Mar 2017 22:07:52 +0100
Subject: [PATCH] Fixed a classical bug. true/false is always way harder to get
 right than extremly complicated datatypes...

---
 arangod/MMFiles/MMFilesWalRecoverState.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arangod/MMFiles/MMFilesWalRecoverState.cpp b/arangod/MMFiles/MMFilesWalRecoverState.cpp
index 60f96a0536..cc23d1c8aa 100644
--- a/arangod/MMFiles/MMFilesWalRecoverState.cpp
+++ b/arangod/MMFiles/MMFilesWalRecoverState.cpp
@@ -733,7 +733,7 @@ bool MMFilesWalRecoverState::ReplayMarker(TRI_df_marker_t const* marker,
         // dropped later
         bool const forceSync = state->willBeDropped(databaseId, collectionId);
         CollectionResult res = collection->updateProperties(payloadSlice, forceSync);
-        if (res.successful()) {
+        if (!res.successful()) {
           LOG_TOPIC(WARN, arangodb::Logger::FIXME) << "cannot change collection properties for collection "
                     << collectionId << " in database " << databaseId << ": "
                     << res.errorMessage;