UsageStats: track more information

Specifically, let's track name, label, description and version.

From a performance perspective, for each identifiable object (e.g., a
ModuleInfo) we only need to interrogate and set this information once
(until statistics are flushed), so hopefully it does not add too much
overhead.

The alternative -- reconstructing the information after the fact --
is tough because the framework lacks a unified way of looking up objects
based on their identifiers. It is not required for Identifiable objects
to register themselves in any way, so it may not be possible to learn
anything else once the usage event is reported to the service.

Author: ctrueden

src/main/java/org/scijava/usage/DefaultUsageService.java

@@ -89,8 +89,7 @@ public UsageStats getUsage(final Object o) {
 			return null;
 		}
 		final String id = ((Identifiable) o).getIdentifier();
-		final String url = ((Locatable) o).getLocation();
-		if (!stats.containsKey(id)) stats.put(id, new UsageStats(id, url));
+		if (!stats.containsKey(id)) stats.put(id, new UsageStats(o));
 		return stats.get(id);
 	}
 

src/main/java/org/scijava/usage/UsageStats.java

@@ -31,28 +31,48 @@
 
 package org.scijava.usage;
 
+import org.scijava.AbstractBasicDetails;
+import org.scijava.BasicDetails;
 import org.scijava.Identifiable;
 import org.scijava.Locatable;
+import org.scijava.Versioned;
 
 /**
  * Data structure storing usage statistics for a particular identifier.
  * 
  * @author Curtis Rueden
  */
-public class UsageStats implements Identifiable, Locatable {
+public class UsageStats extends AbstractBasicDetails implements Identifiable,
+	Locatable, Versioned
+{
 
 	/** The object's unique identifier. */
 	private String id;
 
 	/** This object's location URL. */
 	private String url;
 
+	/** The object's version. */
+	private String version;
+
 	/** Number of times the object was used. */
 	private long count;
 
-	public UsageStats(final String id, final String url) {
-		this.id = id;
-		this.url = url;
+	/**
+	 * Creates usage statistics for the given object. Note that while several
+	 * pieces of information are initially extracted from the object, no reference
+	 * is retained to the object itself.
+	 */
+	public UsageStats(final Object o) {
+		if (o instanceof BasicDetails) {
+			final BasicDetails basicDetails = (BasicDetails) o;
+			setName(basicDetails.getName());
+			setLabel(basicDetails.getLabel());
+			setDescription(basicDetails.getDescription());
+		}
+		id = o instanceof Identifiable ? ((Identifiable) o).getIdentifier() : null;
+		url = o instanceof Locatable ? ((Locatable) o).getLocation() : null;
+		version = o instanceof Versioned ? ((Versioned) o).getVersion() : null;
 	}
 
 	/** Gets the number of times the object has been used. */
@@ -79,4 +99,11 @@ public String getLocation() {
 		return url;
 	}
 
+	// -- Versioned methods --
+
+	@Override
+	public String getVersion() {
+		return version;
+	}
+
 }