GUAC-1429: Modify Directory API to set identifier on added object, and update javascript to use new models.

extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/glyptodon/guacamole/auth/jdbc/base/ModeledDirectoryObjectService.java

@@ -32,6 +32,7 @@ import org.glyptodon.guacamole.GuacamoleSecurityException;
 import org.glyptodon.guacamole.auth.jdbc.permission.ObjectPermissionMapper;
 import org.glyptodon.guacamole.auth.jdbc.permission.ObjectPermissionModel;
 import org.glyptodon.guacamole.auth.jdbc.user.UserModel;
+import org.glyptodon.guacamole.net.auth.Identifiable;
 import org.glyptodon.guacamole.net.auth.permission.ObjectPermission;
 import org.glyptodon.guacamole.net.auth.permission.ObjectPermissionSet;
 
@@ -54,7 +55,7 @@ import org.glyptodon.guacamole.net.auth.permission.ObjectPermissionSet;
  *     database.
  */
 public abstract class ModeledDirectoryObjectService<InternalType extends ModeledDirectoryObject<ModelType>,
-        ExternalType, ModelType extends ObjectModel>
+        ExternalType extends Identifiable, ModelType extends ObjectModel>
     implements DirectoryObjectService<InternalType, ExternalType> {
 
     /**
@@ -384,6 +385,9 @@ public abstract class ModeledDirectoryObjectService<InternalType extends Modeled
         // Create object
         getObjectMapper().insert(model);
 
+        // Set identifier on original object
+        object.setIdentifier(model.getIdentifier());
+
         // Add implicit permissions
         getPermissionMapper().insert(getImplicitPermissions(user, model));
 

extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/glyptodon/guacamole/auth/jdbc/base/ModeledGroupedDirectoryObjectService.java

@@ -28,6 +28,7 @@ import java.util.Collections;
 import org.glyptodon.guacamole.GuacamoleException;
 import org.glyptodon.guacamole.GuacamoleSecurityException;
 import org.glyptodon.guacamole.auth.jdbc.user.AuthenticatedUser;
+import org.glyptodon.guacamole.net.auth.Identifiable;
 import org.glyptodon.guacamole.net.auth.permission.ObjectPermission;
 import org.glyptodon.guacamole.net.auth.permission.ObjectPermissionSet;
 
@@ -50,7 +51,7 @@ import org.glyptodon.guacamole.net.auth.permission.ObjectPermissionSet;
  *     database.
  */
 public abstract class ModeledGroupedDirectoryObjectService<InternalType extends ModeledGroupedDirectoryObject<ModelType>,
-        ExternalType, ModelType extends GroupedObjectModel>
+        ExternalType extends Identifiable, ModelType extends GroupedObjectModel>
         extends ModeledDirectoryObjectService<InternalType, ExternalType, ModelType> {
 
     /**

guacamole-ext/src/main/java/org/glyptodon/guacamole/net/auth/Directory.java

@@ -28,16 +28,15 @@ import org.glyptodon.guacamole.GuacamoleException;
 
 /**
  * Provides access to a collection of all objects with associated identifiers,
- * and allows user manipulation and removal. Objects stored within a
- * Directory are not necessarily returned to the use as references to
- * the stored objects, thus updating an object requires calling an update
- * function.
+ * and allows user manipulation and removal. Objects returned by a Directory
+ * are not necessarily backed by the stored objects, thus updating an object
+ * always requires calling the update() function.
  *
  * @author Michael Jumper
  * @param <ObjectType>
  *     The type of objects stored within this Directory.
  */
-public interface Directory<ObjectType> {
+public interface Directory<ObjectType extends Identifiable> {
 
     /**
      * Returns the object having the given identifier. Note that changes to
@@ -91,12 +90,16 @@ public interface Directory<ObjectType> {
     Set<String> getIdentifiers() throws GuacamoleException;
 
     /**
-     * Adds the given object to the overall set.
+     * Adds the given object to the overall set. If a new identifier is
+     * created for the added object, that identifier will be automatically
+     * assigned via setIdentifier().
      *
-     * @param object The object to add.
+     * @param object
+     *     The object to add.
      *
-     * @throws GuacamoleException If an error occurs while adding the object , or
-     *                            if adding the object is not allowed.
+     * @throws GuacamoleException
+     *     If an error occurs while adding the object, or if adding the object
+     *     is not allowed.
      */
     void add(ObjectType object)
             throws GuacamoleException;

guacamole-ext/src/main/java/org/glyptodon/guacamole/net/auth/simple/SimpleDirectory.java

@@ -30,6 +30,7 @@ import java.util.Set;
 import org.glyptodon.guacamole.GuacamoleException;
 import org.glyptodon.guacamole.GuacamoleSecurityException;
 import org.glyptodon.guacamole.net.auth.Directory;
+import org.glyptodon.guacamole.net.auth.Identifiable;
 
 /**
  * An extremely simple read-only implementation of a Directory which provides
@@ -40,7 +41,8 @@ import org.glyptodon.guacamole.net.auth.Directory;
  * @param <ObjectType>
  *     The type of objects stored within this SimpleDirectory.
  */
-public class SimpleDirectory<ObjectType> implements Directory<ObjectType> {
+public class SimpleDirectory<ObjectType extends Identifiable>
+        implements Directory<ObjectType> {
 
     /**
      * The Map of objects to provide access to.

guacamole/src/main/webapp/app/rest/services/connectionGroupService.js

@@ -145,8 +145,8 @@ angular.module('rest').factory('connectionGroupService', ['$injector',
             })
 
             // Set the identifier on the new connection group and clear the cache
-            .success(function connectionGroupCreated(identifier){
-                connectionGroup.identifier = identifier;
+            .success(function connectionGroupCreated(newConnectionGroup){
+                connectionGroup.identifier = newConnectionGroup.identifier;
                 cacheService.connections.removeAll();
             });
         }

guacamole/src/main/webapp/app/rest/services/connectionService.js

@@ -152,8 +152,8 @@ angular.module('rest').factory('connectionService', ['$injector',
             })
 
             // Set the identifier on the new connection and clear the cache
-            .success(function connectionCreated(identifier){
-                connection.identifier = identifier;
+            .success(function connectionCreated(newConnection){
+                connection.identifier = newConnection.identifier;
                 cacheService.connections.removeAll();
             });
         }