Fixing a defect where logical port numbers are listed as part of CLI arg completion.

Fixing a defect where devices imported with device id in uppercase are considered as different from those discovered in lowercase.
Fixed javadocs error from a rebase.

Change-Id: I76741022fb95d10a9a16fc9ce6d6443b166822ab

@@ -15,9 +15,6 @@
  */
  */
 package org.onlab.onos.cli.net;
 package org.onlab.onos.cli.net;
 
 
-import java.util.List;
-import java.util.SortedSet;
-
 import org.apache.karaf.shell.console.Completer;
 import org.apache.karaf.shell.console.Completer;
 import org.apache.karaf.shell.console.completer.StringsCompleter;
 import org.apache.karaf.shell.console.completer.StringsCompleter;
 import org.onlab.onos.cli.AbstractShellCommand;
 import org.onlab.onos.cli.AbstractShellCommand;
@@ -25,6 +22,9 @@ import org.onlab.onos.net.Device;
 import org.onlab.onos.net.Port;
 import org.onlab.onos.net.Port;
 import org.onlab.onos.net.device.DeviceService;
 import org.onlab.onos.net.device.DeviceService;
 
 
+import java.util.List;
+import java.util.SortedSet;
+
 /**
 /**
  * ConnectPoint completer.
  * ConnectPoint completer.
  */
  */
@@ -40,11 +40,12 @@ public class ConnectPointCompleter implements Completer {
         // Generate the device ID/port number identifiers
         // Generate the device ID/port number identifiers
         for (Device device : service.getDevices()) {
         for (Device device : service.getDevices()) {
             SortedSet<String> strings = delegate.getStrings();
             SortedSet<String> strings = delegate.getStrings();
-
             for (Port port : service.getPorts(device.id())) {
             for (Port port : service.getPorts(device.id())) {
+                if (!port.number().isLogical()) {
                     strings.add(device.id().toString() + "/" + port.number());
                     strings.add(device.id().toString() + "/" + port.number());
                 }
                 }
             }
             }
+        }
 
 
         // Now let the completer do the work for figuring out what to offer.
         // Now let the completer do the work for figuring out what to offer.
         return delegate.complete(buffer, cursor, candidates);
         return delegate.complete(buffer, cursor, candidates);

@@ -43,6 +43,7 @@ public interface CodecService {
      *
      *
      * @param entityClass entity class
      * @param entityClass entity class
      * @param codec       JSON codec
      * @param codec       JSON codec
+     * @param <T>         entity type
      */
      */
     <T> void registerCodec(Class<T> entityClass, JsonCodec<T> codec);
     <T> void registerCodec(Class<T> entityClass, JsonCodec<T> codec);
 
 

@@ -34,7 +34,7 @@ public final class DeviceId extends ElementId {
     // Public construction is prohibited
     // Public construction is prohibited
     private DeviceId(URI uri) {
     private DeviceId(URI uri) {
         this.uri = uri;
         this.uri = uri;
-        this.str = uri.toString();
+        this.str = uri.toString().toLowerCase();
     }
     }
 
 
 
 

@@ -9,8 +9,8 @@ public interface DatabaseService {
     /**
     /**
      * Reads the specified key.
      * Reads the specified key.
      * @param tableName name of the table associated with this operation.
      * @param tableName name of the table associated with this operation.
-     * @return key key to read.
-     * @returns value (and version) associated with this key. This calls returns null if the key does not exist.
+     * @param key key to read.
+     * @return value (and version) associated with this key. This calls returns null if the key does not exist.
      */
      */
     VersionedValue get(String tableName, String key);
     VersionedValue get(String tableName, String key);
 
 

@@ -249,6 +249,7 @@ public class DefaultTopology extends AbstractModel implements Topology {
      *
      *
      * @param src    source device
      * @param src    source device
      * @param dst    destination device
      * @param dst    destination device
+     * @param weight edge weight function
      * @return set of shortest paths
      * @return set of shortest paths
      */
      */
     Set<Path> getPaths(DeviceId src, DeviceId dst, LinkWeight weight) {
     Set<Path> getPaths(DeviceId src, DeviceId dst, LinkWeight weight) {