X-Git-Url: https://git.pterodactylus.net/?p=WoTNS.git;a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fwotns%2Ffreenet%2Fwot%2FIdentityManager.java;h=ea90dacf9d2a27a029214bab98ed23e3061dcb87;hp=3086f276579d378dd0b1094c9b117edaae11f311;hb=83c0fe680b6212c66b7eecc9efaf12de96bffe16;hpb=ab09aee8c42db059abbb568f5e2a3047f6fc3002

diff --git a/src/main/java/net/pterodactylus/wotns/freenet/wot/IdentityManager.java b/src/main/java/net/pterodactylus/wotns/freenet/wot/IdentityManager.java
index 3086f27..ea90dac 100644
--- a/src/main/java/net/pterodactylus/wotns/freenet/wot/IdentityManager.java
+++ b/src/main/java/net/pterodactylus/wotns/freenet/wot/IdentityManager.java
@@ -27,9 +27,7 @@ import java.util.Map.Entry;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
-import net.pterodactylus.util.collection.SetBuilder;
 import net.pterodactylus.util.logging.Logging;
-import net.pterodactylus.util.object.Default;
 import net.pterodactylus.util.service.AbstractService;
 import net.pterodactylus.wotns.freenet.plugin.PluginException;
 
@@ -173,15 +171,31 @@ public class IdentityManager extends AbstractService {
 		}
 	}
 
+	/**
+	 * Returns all identities that are trusted by the given own identity. In
+	 * addition to all non-own identities the given own identity is also
+	 * returned.
+	 *
+	 * @param ownIdentity
+	 *            The own identity to get the trusted identities for
+	 * @return The identities trusted by the given own identity
+	 */
 	public Set<Identity> getTrustedIdentities(OwnIdentity ownIdentity) {
-		SetBuilder<Identity> identities = new SetBuilder<Identity>();
+		Set<Identity> identities = new HashSet<Identity>();
 		if ((context == null) || ownIdentity.getContexts().contains(context)) {
 			identities.add(ownIdentity);
 		}
-		synchronized (syncObject) {
-			identities.addAll(Default.forNull(currentTrustedIdentities.get(ownIdentity), Collections.<Identity> emptySet()));
+		try {
+			Set<Identity> trustedIdentities = webOfTrustConnector.loadTrustedIdentities(ownIdentity, context);
+			Map<String, Identity> newTrustedIdentities = new HashMap<String, Identity>();
+			for (Identity trustedIdentity : trustedIdentities) {
+				newTrustedIdentities.put(trustedIdentity.getId(), trustedIdentity);
+			}
+			checkTrustedIdentities(ownIdentity, newTrustedIdentities);
+		} catch (WebOfTrustException wote1) {
+			logger.log(Level.WARNING, String.format("Could not load all trusted identities for %s.", ownIdentity), wote1);
 		}
-		return identities.get();
+		return identities;
 	}
 
 	//
@@ -218,7 +232,10 @@ public class IdentityManager extends AbstractService {
 						identities.put(identity.getId(), identity);
 					}
 
-					/* add own identities, too, as long as the WoT doesn’t do that. */
+					/*
+					 * add own identities, too, as long as the WoT doesn’t do
+					 * that.
+					 */
 					for (OwnIdentity additionalOwnIdentity : ownIdentities) {
 						if (additionalOwnIdentity == ownIdentity) {
 							continue;
@@ -268,6 +285,7 @@ public class IdentityManager extends AbstractService {
 			for (OwnIdentity oldOwnIdentity : currentOwnIdentities.values()) {
 				if (!newOwnIdentities.containsKey(oldOwnIdentity.getId())) {
 					identityListenerManager.fireOwnIdentityRemoved(oldOwnIdentity);
+					currentTrustedIdentities.remove(oldOwnIdentity);
 				}
 			}