Remove unnecessary method.
[Sone.git] / src / main / java / net / pterodactylus / sone / core / WebOfTrustUpdater.java
index 5143053..67bf3ac 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Sone - WebOfTrustUpdater.java - Copyright © 2012 David Roden
+ * Sone - WebOfTrustUpdater.java - Copyright © 2013 David Roden
  *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -17,6 +17,8 @@
 
 package net.pterodactylus.sone.core;
 
+import static com.google.common.base.Preconditions.checkNotNull;
+
 import java.util.concurrent.BlockingQueue;
 import java.util.concurrent.LinkedBlockingQueue;
 import java.util.logging.Level;
@@ -32,6 +34,9 @@ import net.pterodactylus.sone.freenet.wot.WebOfTrustException;
 import net.pterodactylus.util.logging.Logging;
 import net.pterodactylus.util.service.AbstractService;
 
+import com.google.common.annotations.VisibleForTesting;
+import com.google.inject.Inject;
+
 /**
  * Updates WebOfTrust identity data in a background thread because communicating
  * with the WebOfTrust plugin can potentially last quite long.
@@ -45,7 +50,7 @@ public class WebOfTrustUpdater extends AbstractService {
 
        /** Stop job. */
        @SuppressWarnings("synthetic-access")
-       private static final WebOfTrustUpdateJob stopJob = new WebOfTrustUpdateJob();
+       private final WebOfTrustUpdateJob stopJob = new WebOfTrustUpdateJob();
 
        /** The web of trust connector. */
        private final WebOfTrustConnector webOfTrustConnector;
@@ -57,8 +62,9 @@ public class WebOfTrustUpdater extends AbstractService {
         * Creates a new trust updater.
         *
         * @param webOfTrustConnector
-        *            The web of trust connector
+        *              The web of trust connector
         */
+       @Inject
        public WebOfTrustUpdater(WebOfTrustConnector webOfTrustConnector) {
                super("Trust Updater");
                this.webOfTrustConnector = webOfTrustConnector;
@@ -69,68 +75,129 @@ public class WebOfTrustUpdater extends AbstractService {
        //
 
        /**
-        * Retrieves the trust relation between the truster and the trustee. This
-        * method will return immediately and perform a trust update in the
-        * background.
+        * Updates the trust relation between the truster and the trustee. This method
+        * will return immediately and perform a trust update in the background.
         *
         * @param truster
-        *            The identity giving the trust
+        *              The identity giving the trust
         * @param trustee
-        *            The identity receiving the trust
+        *              The identity receiving the trust
+        * @param score
+        *              The new level of trust (from -100 to 100, may be {@code null} to remove
+        *              the trust completely)
+        * @param comment
+        *              The comment of the trust relation
+        */
+       public void setTrust(OwnIdentity truster, Identity trustee, Integer score, String comment) {
+               SetTrustJob setTrustJob = new SetTrustJob(truster, trustee, score, comment);
+               if (updateJobs.contains(setTrustJob)) {
+                       updateJobs.remove(setTrustJob);
+               }
+               logger.log(Level.FINER, "Adding Trust Update Job: " + setTrustJob);
+               try {
+                       updateJobs.put(setTrustJob);
+               } catch (InterruptedException e) {
+                       /* the queue is unbounded so it should never block. */
+               }
+       }
+
+       /**
+        * Adds the given context to the given own identity, waiting for completion of
+        * the operation.
+        *
+        * @param ownIdentity
+        *              The own identity to add the context to
+        * @param context
+        *              The context to add
+        * @return {@code true} if the context was added successfully, {@code false}
+        *         otherwise
         */
-       public void getTrust(OwnIdentity truster, Identity trustee) {
-               GetTrustJob getTrustJob = new GetTrustJob(truster, trustee);
-               if (!updateJobs.contains(getTrustJob)) {
-                       logger.log(Level.FINER, "Adding Trust Update Job: " + getTrustJob);
+       public boolean addContextWait(OwnIdentity ownIdentity, String context) {
+               AddContextJob addContextJob = new AddContextJob(ownIdentity, context);
+               if (!updateJobs.contains(addContextJob)) {
+                       logger.log(Level.FINER, "Adding Context Job: " + addContextJob);
                        try {
-                               updateJobs.put(getTrustJob);
+                               updateJobs.put(addContextJob);
                        } catch (InterruptedException ie1) {
                                /* the queue is unbounded so it should never block. */
                        }
+                       return addContextJob.waitForCompletion();
+               } else {
+                       for (WebOfTrustUpdateJob updateJob : updateJobs) {
+                               if (updateJob.equals(addContextJob)) {
+                                       return updateJob.waitForCompletion();
+                               }
+                       }
                }
+               return false;
        }
 
        /**
-        * Updates the trust relation between the truster and the trustee. This
-        * method will return immediately and perform a trust update in the
-        * background.
+        * Removes the given context from the given own identity.
         *
-        * @param truster
-        *            The identity giving the trust
-        * @param trustee
-        *            The identity receiving the trust
-        * @param score
-        *            The new level of trust (from -100 to 100, may be {@code null}
-        *            to remove the trust completely)
-        * @param comment
-        *            The comment of the trust relation
+        * @param ownIdentity
+        *              The own identity to remove the context from
+        * @param context
+        *              The context to remove
         */
-       public void setTrust(OwnIdentity truster, Identity trustee, Integer score, String comment) {
-               SetTrustJob setTrustJob = new SetTrustJob(truster, trustee, score, comment);
-               if (updateJobs.contains(setTrustJob)) {
-                       updateJobs.remove(setTrustJob);
+       public void removeContext(OwnIdentity ownIdentity, String context) {
+               RemoveContextJob removeContextJob = new RemoveContextJob(ownIdentity, context);
+               if (!updateJobs.contains(removeContextJob)) {
+                       logger.log(Level.FINER, "Adding Context Job: " + removeContextJob);
+                       try {
+                               updateJobs.put(removeContextJob);
+                       } catch (InterruptedException ie1) {
+                               /* the queue is unbounded so it should never block. */
+                       }
                }
-               logger.log(Level.FINER, "Adding Trust Update Job: " + setTrustJob);
+       }
+
+       /**
+        * Sets a property on the given own identity.
+        *
+        * @param ownIdentity
+        *              The own identity to set the property on
+        * @param propertyName
+        *              The name of the property to set
+        * @param propertyValue
+        *              The value of the property to set
+        */
+       public void setProperty(OwnIdentity ownIdentity, String propertyName, String propertyValue) {
+               SetPropertyJob setPropertyJob = new SetPropertyJob(ownIdentity, propertyName, propertyValue);
+               if (updateJobs.contains(setPropertyJob)) {
+                       updateJobs.remove(setPropertyJob);
+               }
+               logger.log(Level.FINER, "Adding Property Job: " + setPropertyJob);
                try {
-                       updateJobs.put(setTrustJob);
+                       updateJobs.put(setPropertyJob);
                } catch (InterruptedException e) {
                        /* the queue is unbounded so it should never block. */
                }
        }
 
+       /**
+        * Removes a property from the given own identity.
+        *
+        * @param ownIdentity
+        *              The own identity to remove the property from
+        * @param propertyName
+        *              The name of the property to remove
+        */
+       public void removeProperty(OwnIdentity ownIdentity, String propertyName) {
+               setProperty(ownIdentity, propertyName, null);
+       }
+
        //
        // SERVICE METHODS
        //
 
-       /**
-        * {@inheritDoc}
-        */
+       /** {@inheritDoc} */
        @Override
        protected void serviceRun() {
                while (!shouldStop()) {
                        try {
                                WebOfTrustUpdateJob updateJob = updateJobs.take();
-                               if (shouldStop() || (updateJob == stopJob)) {
+                               if (shouldStop()) {
                                        break;
                                }
                                logger.log(Level.FINE, "Running Trust Update Job: " + updateJob);
@@ -144,9 +211,7 @@ public class WebOfTrustUpdater extends AbstractService {
                }
        }
 
-       /**
-        * {@inheritDoc}
-        */
+       /** {@inheritDoc} */
        @Override
        protected void serviceStop() {
                try {
@@ -161,7 +226,18 @@ public class WebOfTrustUpdater extends AbstractService {
         *
         * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
         */
-       private static class WebOfTrustUpdateJob {
+       @VisibleForTesting
+       class WebOfTrustUpdateJob implements Runnable {
+
+               /** Object for synchronization. */
+               @SuppressWarnings("hiding")
+               private final Object syncObject = new Object();
+
+               /** Whether the job has finished. */
+               private boolean finished;
+
+               /** Whether the job was successful. */
+               private boolean success;
 
                //
                // ACTIONS
@@ -169,168 +245,348 @@ public class WebOfTrustUpdater extends AbstractService {
 
                /**
                 * Performs the actual update operation.
-                * <p>
+                * <p/>
                 * The implementation of this class does nothing.
                 */
+               @Override
                public void run() {
                        /* does nothing. */
                }
 
+               /**
+                * Waits for completion of this job or stopping of the WebOfTrust updater.
+                *
+                * @return {@code true} if this job finished successfully, {@code false}
+                *         otherwise
+                * @see WebOfTrustUpdater#stop()
+                */
+               @SuppressWarnings("synthetic-access")
+               public boolean waitForCompletion() {
+                       synchronized (syncObject) {
+                               while (!finished && !shouldStop()) {
+                                       try {
+                                               syncObject.wait();
+                                       } catch (InterruptedException ie1) {
+                                               /* we’re looping, ignore. */
+                                       }
+                               }
+                               return success;
+                       }
+               }
+
+               //
+               // PROTECTED METHODS
+               //
+
+               /**
+                * Signals that this job has finished.
+                *
+                * @param success
+                *              {@code true} if this job finished successfully, {@code false} otherwise
+                */
+               protected void finish(boolean success) {
+                       synchronized (syncObject) {
+                               finished = true;
+                               this.success = success;
+                               syncObject.notifyAll();
+                       }
+               }
+
        }
 
        /**
-        * Base class for WebOfTrust trust update jobs.
+        * Update job that sets the trust relation between two identities.
         *
         * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
         */
-       private static class WebOfTrustTrustUpdateJob extends WebOfTrustUpdateJob {
+       @VisibleForTesting
+       class SetTrustJob extends WebOfTrustUpdateJob {
 
                /** The identity giving the trust. */
-               protected final OwnIdentity truster;
+               private final OwnIdentity truster;
 
                /** The identity receiving the trust. */
-               protected final Identity trustee;
+               private final Identity trustee;
+
+               /** The score of the relation. */
+               private final Integer score;
+
+               /** The comment of the relation. */
+               private final String comment;
 
                /**
-                * Creates a new trust update job.
+                * Creates a new set trust job.
                 *
                 * @param truster
-                *            The identity giving the trust
+                *              The identity giving the trust
                 * @param trustee
-                *            The identity receiving the trust
+                *              The identity receiving the trust
+                * @param score
+                *              The score of the trust (from -100 to 100, may be {@code null} to remote
+                *              the trust relation completely)
+                * @param comment
+                *              The comment of the trust relation
                 */
+               public SetTrustJob(OwnIdentity truster, Identity trustee, Integer score, String comment) {
+                       this.truster = checkNotNull(truster, "truster must not be null");
+                       this.trustee = checkNotNull(trustee, "trustee must not be null");
+                       this.score = score;
+                       this.comment = comment;
+               }
+
+               /** {@inheritDoc} */
+               @Override
                @SuppressWarnings("synthetic-access")
-               public WebOfTrustTrustUpdateJob(OwnIdentity truster, Identity trustee) {
-                       super();
-                       this.truster = truster;
-                       this.trustee = trustee;
+               public void run() {
+                       try {
+                               if (score != null) {
+                                       webOfTrustConnector.setTrust(truster, trustee, score, comment);
+                                       trustee.setTrust(truster, new Trust(score, null, 0));
+                               } else {
+                                       webOfTrustConnector.removeTrust(truster, trustee);
+                                       trustee.removeTrust(truster);
+                               }
+                               finish(true);
+                       } catch (WebOfTrustException wote1) {
+                               logger.log(Level.WARNING, "Could not set Trust value for " + truster + " -> " + trustee + " to " + score + " (" + comment + ")!", wote1);
+                               finish(false);
+                       }
                }
 
                //
                // OBJECT METHODS
                //
 
-               /**
-                * {@inheritDoc}
-                */
+               /** {@inheritDoc} */
                @Override
                public boolean equals(Object object) {
                        if ((object == null) || !object.getClass().equals(getClass())) {
                                return false;
                        }
-                       WebOfTrustTrustUpdateJob updateJob = (WebOfTrustTrustUpdateJob) object;
-                       return ((truster == null) ? (updateJob.truster == null) : updateJob.truster.equals(truster)) && ((trustee == null) ? (updateJob.trustee == null) : updateJob.trustee.equals(trustee));
+                       SetTrustJob updateJob = (SetTrustJob) object;
+                       return updateJob.truster.equals(truster) && updateJob.trustee.equals(trustee);
                }
 
-               /**
-                * {@inheritDoc}
-                */
+               /** {@inheritDoc} */
                @Override
                public int hashCode() {
-                       return getClass().hashCode() ^ ((truster == null) ? 0 : truster.hashCode()) ^ ((trustee == null) ? 0 : trustee.hashCode());
+                       return getClass().hashCode() ^ truster.hashCode() ^ trustee.hashCode();
                }
 
-               /**
-                * {@inheritDoc}
-                */
+               /** {@inheritDoc} */
                @Override
                public String toString() {
-                       return String.format("%s[truster=%s,trustee=%s]", getClass().getSimpleName(), (truster == null) ? null : truster.getId(), (trustee == null) ? null : trustee.getId());
+                       return String.format("%s[truster=%s,trustee=%s]", getClass().getSimpleName(), truster.getId(), trustee.getId());
                }
 
        }
 
        /**
-        * Update job that sets the trust relation between two identities.
+        * Base class for context updates of an {@link OwnIdentity}.
         *
         * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
         */
-       private class SetTrustJob extends WebOfTrustTrustUpdateJob {
+       @VisibleForTesting
+       class WebOfTrustContextUpdateJob extends WebOfTrustUpdateJob {
 
-               /** The score of the relation. */
-               private final Integer score;
+               /** The own identity whose contexts to manage. */
+               protected final OwnIdentity ownIdentity;
 
-               /** The comment of the relation. */
-               private final String comment;
+               /** The context to update. */
+               protected final String context;
 
                /**
-                * Creates a new set trust job.
+                * Creates a new context update job.
                 *
-                * @param truster
-                *            The identity giving the trust
-                * @param trustee
-                *            The identity receiving the trust
-                * @param score
-                *            The score of the trust (from -100 to 100, may be
-                *            {@code null} to remote the trust relation completely)
-                * @param comment
-                *            The comment of the trust relation
+                * @param ownIdentity
+                *              The own identity to update
+                * @param context
+                *              The context to update
                 */
-               public SetTrustJob(OwnIdentity truster, Identity trustee, Integer score, String comment) {
-                       super(truster, trustee);
-                       this.score = score;
-                       this.comment = comment;
+               @SuppressWarnings("synthetic-access")
+               public WebOfTrustContextUpdateJob(OwnIdentity ownIdentity, String context) {
+                       this.ownIdentity = checkNotNull(ownIdentity, "ownIdentity must not be null");
+                       this.context = checkNotNull(context, "context must not be null");
                }
 
+               //
+               // OBJECT METHODS
+               //
+
+               /** {@inheritDoc} */
+               @Override
+               public boolean equals(Object object) {
+                       if ((object == null) || !object.getClass().equals(getClass())) {
+                               return false;
+                       }
+                       WebOfTrustContextUpdateJob updateJob = (WebOfTrustContextUpdateJob) object;
+                       return updateJob.ownIdentity.equals(ownIdentity) && updateJob.context.equals(context);
+               }
+
+               /** {@inheritDoc} */
+               @Override
+               public int hashCode() {
+                       return getClass().hashCode() ^ ownIdentity.hashCode() ^ context.hashCode();
+               }
+
+               /** {@inheritDoc} */
+               @Override
+               public String toString() {
+                       return String.format("%s[ownIdentity=%s,context=%s]", getClass().getSimpleName(), ownIdentity, context);
+               }
+
+       }
+
+       /**
+        * Job that adds a context to an {@link OwnIdentity}.
+        *
+        * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
+        */
+       @VisibleForTesting
+       class AddContextJob extends WebOfTrustContextUpdateJob {
+
                /**
-                * {@inheritDoc}
+                * Creates a new add-context job.
+                *
+                * @param ownIdentity
+                *              The own identity whose contexts to manage
+                * @param context
+                *              The context to add
                 */
+               public AddContextJob(OwnIdentity ownIdentity, String context) {
+                       super(ownIdentity, context);
+               }
+
+               /** {@inheritDoc} */
                @Override
                @SuppressWarnings("synthetic-access")
                public void run() {
                        try {
-                               if (score != null) {
-                                       if (trustee instanceof DefaultIdentity) {
-                                               ((DefaultIdentity) trustee).setTrust(truster, new Trust(score, null, 0));
-                                       }
-                                       webOfTrustConnector.setTrust(truster, trustee, score, comment);
-                               } else {
-                                       if (trustee instanceof DefaultIdentity) {
-                                               ((DefaultIdentity) trustee).setTrust(truster, null);
-                                       }
-                                       webOfTrustConnector.removeTrust(truster, trustee);
-                               }
-                       } catch (WebOfTrustException wote1) {
-                               logger.log(Level.WARNING, "Could not set Trust value for " + truster + " -> " + trustee + " to " + score + " (" + comment + ")!", wote1);
+                               webOfTrustConnector.addContext(ownIdentity, context);
+                               ownIdentity.addContext(context);
+                               finish(true);
+                       } catch (PluginException pe1) {
+                               logger.log(Level.WARNING, String.format("Could not add Context “%2$s” to Own Identity %1$s!", ownIdentity, context), pe1);
+                               finish(false);
                        }
                }
 
        }
 
        /**
-        * Update job that retrieves the trust relation between two identities.
+        * Job that removes a context from an {@link OwnIdentity}.
         *
         * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
         */
-       private class GetTrustJob extends WebOfTrustTrustUpdateJob {
+       @VisibleForTesting
+       class RemoveContextJob extends WebOfTrustContextUpdateJob {
 
                /**
-                * Creates a new trust update job.
+                * Creates a new remove-context job.
                 *
-                * @param truster
-                *            The identity giving the trust
-                * @param trustee
-                *            The identity receiving the trust
+                * @param ownIdentity
+                *              The own identity whose contexts to manage
+                * @param context
+                *              The context to remove
                 */
-               public GetTrustJob(OwnIdentity truster, Identity trustee) {
-                       super(truster, trustee);
+               public RemoveContextJob(OwnIdentity ownIdentity, String context) {
+                       super(ownIdentity, context);
                }
 
+               /** {@inheritDoc} */
+               @Override
+               @SuppressWarnings("synthetic-access")
+               public void run() {
+                       try {
+                               webOfTrustConnector.removeContext(ownIdentity, context);
+                               ownIdentity.removeContext(context);
+                               finish(true);
+                       } catch (PluginException pe1) {
+                               logger.log(Level.WARNING, String.format("Could not remove Context “%2$s” to Own Identity %1$s!", ownIdentity, context), pe1);
+                               finish(false);
+                       }
+               }
+
+       }
+
+       /**
+        * WebOfTrust update job that sets a property on an {@link OwnIdentity}.
+        *
+        * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
+        */
+       @VisibleForTesting
+       class SetPropertyJob extends WebOfTrustUpdateJob {
+
+               /** The own identity to update properties on. */
+               private final OwnIdentity ownIdentity;
+
+               /** The name of the property to update. */
+               private final String propertyName;
+
+               /** The value of the property to set. */
+               private final String propertyValue;
+
                /**
-                * {@inheritDoc}
+                * Creates a new set-property job.
+                *
+                * @param ownIdentity
+                *              The own identity to set the property on
+                * @param propertyName
+                *              The name of the property to set
+                * @param propertyValue
+                *              The value of the property to set
                 */
+               public SetPropertyJob(OwnIdentity ownIdentity, String propertyName, String propertyValue) {
+                       this.ownIdentity = ownIdentity;
+                       this.propertyName = propertyName;
+                       this.propertyValue = propertyValue;
+               }
+
+               /** {@inheritDoc} */
                @Override
                @SuppressWarnings("synthetic-access")
                public void run() {
                        try {
-                               Trust trust = webOfTrustConnector.getTrust(truster, trustee);
-                               if (trustee instanceof DefaultIdentity) {
-                                       ((DefaultIdentity) trustee).setTrust(truster, trust);
+                               if (propertyValue == null) {
+                                       webOfTrustConnector.removeProperty(ownIdentity, propertyName);
+                                       ownIdentity.removeProperty(propertyName);
+                               } else {
+                                       webOfTrustConnector.setProperty(ownIdentity, propertyName, propertyValue);
+                                       ownIdentity.setProperty(propertyName, propertyValue);
                                }
+                               finish(true);
                        } catch (PluginException pe1) {
-                               logger.log(Level.WARNING, "Could not get Trust value for " + truster + " -> " + trustee + "!", pe1);
+                               logger.log(Level.WARNING, String.format("Could not set Property “%2$s” to “%3$s” on Own Identity %1$s!", ownIdentity, propertyName, propertyValue), pe1);
+                               finish(false);
                        }
                }
+
+               //
+               // OBJECT METHODS
+               //
+
+               /** {@inheritDoc} */
+               @Override
+               public boolean equals(Object object) {
+                       if ((object == null) || !object.getClass().equals(getClass())) {
+                               return false;
+                       }
+                       SetPropertyJob updateJob = (SetPropertyJob) object;
+                       return updateJob.ownIdentity.equals(ownIdentity) && updateJob.propertyName.equals(propertyName);
+               }
+
+               /** {@inheritDoc} */
+               @Override
+               public int hashCode() {
+                       return getClass().hashCode() ^ ownIdentity.hashCode() ^ propertyName.hashCode();
+               }
+
+               /** {@inheritDoc} */
+               @Override
+               public String toString() {
+                       return String.format("%s[ownIdentity=%s,propertyName=%s]", getClass().getSimpleName(), ownIdentity, propertyName);
+               }
+
        }
 
 }