From 97f40bd20bf3f44d662f89149a19c3007347f402 Mon Sep 17 00:00:00 2001
From: akwizgran <michael@briarproject.org>
Date: Tue, 24 Apr 2018 16:45:28 +0100
Subject: [PATCH] Check whether hooks need to be called before loading contact.
---
.../bramble/versioning/ClientVersioningManagerImpl.java | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/bramble-core/src/main/java/org/briarproject/bramble/versioning/ClientVersioningManagerImpl.java b/bramble-core/src/main/java/org/briarproject/bramble/versioning/ClientVersioningManagerImpl.java
index 3d028196ed..002001d7e1 100644
--- a/bramble-core/src/main/java/org/briarproject/bramble/versioning/ClientVersioningManagerImpl.java
+++ b/bramble-core/src/main/java/org/briarproject/bramble/versioning/ClientVersioningManagerImpl.java
@@ -217,8 +217,10 @@ class ClientVersioningManagerImpl implements ClientVersioningManager, Client,
Map<ClientMajorVersion, Visibility> after =
getVisibilities(newLocalStates, newRemoteStates);
// Call hooks for any visibilities that have changed
- Contact c = getContact(txn, m.getGroupId());
- callVisibilityHooks(txn, c, before, after);
+ if (!before.equals(after)) {
+ Contact c = getContact(txn, m.getGroupId());
+ callVisibilityHooks(txn, c, before, after);
+ }
} catch (FormatException e) {
throw new InvalidMessageException(e);
}
--
GitLab