Fixed some javadocs and added a listener to the FooComponent to show what intent…

… events are perceptible externally.

@@ -11,6 +11,9 @@ import org.onlab.onos.cluster.ClusterService;
 import org.onlab.onos.net.device.DeviceEvent;
 import org.onlab.onos.net.device.DeviceEvent;
 import org.onlab.onos.net.device.DeviceListener;
 import org.onlab.onos.net.device.DeviceListener;
 import org.onlab.onos.net.device.DeviceService;
 import org.onlab.onos.net.device.DeviceService;
+import org.onlab.onos.net.intent.IntentEvent;
+import org.onlab.onos.net.intent.IntentListener;
+import org.onlab.onos.net.intent.IntentService;
 import org.slf4j.Logger;
 import org.slf4j.Logger;
 
 
 import static org.slf4j.LoggerFactory.getLogger;
 import static org.slf4j.LoggerFactory.getLogger;
@@ -29,13 +32,18 @@ public class FooComponent {
     @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY)
     @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY)
     protected DeviceService deviceService;
     protected DeviceService deviceService;
 
 
+    @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY)
+    protected IntentService intentService;
+
     private final ClusterEventListener clusterListener = new InnerClusterListener();
     private final ClusterEventListener clusterListener = new InnerClusterListener();
     private final DeviceListener deviceListener = new InnerDeviceListener();
     private final DeviceListener deviceListener = new InnerDeviceListener();
+    private final IntentListener intentListener = new InnerIntentListener();
 
 
     @Activate
     @Activate
     public void activate() {
     public void activate() {
         clusterService.addListener(clusterListener);
         clusterService.addListener(clusterListener);
         deviceService.addListener(deviceListener);
         deviceService.addListener(deviceListener);
+        intentService.addListener(intentListener);
         log.info("Started");
         log.info("Started");
     }
     }
 
 
@@ -43,6 +51,7 @@ public class FooComponent {
     public void deactivate() {
     public void deactivate() {
         clusterService.removeListener(clusterListener);
         clusterService.removeListener(clusterListener);
         deviceService.removeListener(deviceListener);
         deviceService.removeListener(deviceListener);
+        intentService.removeListener(intentListener);
         log.info("Stopped");
         log.info("Stopped");
     }
     }
 
 
@@ -59,6 +68,23 @@ public class FooComponent {
             log.info("YEEEEHAAAAW! {}", event);
             log.info("YEEEEHAAAAW! {}", event);
         }
         }
     }
     }
+
+    private class InnerIntentListener implements IntentListener {
+        @Override
+        public void event(IntentEvent event) {
+            String message;
+            if (event.type() == IntentEvent.Type.SUBMITTED) {
+                message = "WOW! It looks like someone has some intentions: {}";
+            } else if (event.type() == IntentEvent.Type.INSTALLED) {
+                message = "AWESOME! So far things are going great: {}";
+            } else if (event.type() == IntentEvent.Type.WITHDRAWN) {
+                message = "HMMM! Ambitions are fading apparently: {}";
+            } else {
+                message = "CRAP!!! Things are not turning out as intended: {}";
+            }
+            log.info(message, event.subject());
+        }
+    }
 }
 }
 
 
 
 

@@ -10,12 +10,12 @@ public interface TopologyChangeDelegate {
     /**
     /**
      * Notifies that topology has changed in such a way that the specified
      * Notifies that topology has changed in such a way that the specified
      * intents should be recompiled. If the {@code compileAllFailed} parameter
      * intents should be recompiled. If the {@code compileAllFailed} parameter
-     * is true, the all intents in {@link org.onlab.onos.net.intent.IntentState#FAILED}
+     * is true, then all intents in {@link org.onlab.onos.net.intent.IntentState#FAILED}
      * state should be compiled as well.
      * state should be compiled as well.
      *
      *
      * @param intentIds intents that should be recompiled
      * @param intentIds intents that should be recompiled
-     * @param compileAllFailed true implies full compile is required; false for
-     *                         selective recompile only
+     * @param compileAllFailed true implies full compile of all failed intents
+     *                         is required; false for selective recompile only
      */
      */
     void triggerCompile(Iterable<IntentId> intentIds, boolean compileAllFailed);
     void triggerCompile(Iterable<IntentId> intentIds, boolean compileAllFailed);