GUAC-932: Begin migration of admin to new guacGroupList. Remove unused util module.

guacamole/src/main/webapp/app/manage/controllers/connectionEditModalController.js

@@ -28,7 +28,6 @@ angular.module('manage').controller('connectionEditModalController', ['$scope',
             
     var connectionEditModal             = $injector.get('connectionEditModal');
     var connectionService               = $injector.get('connectionService');
-    var displayObjectPreparationService = $injector.get('displayObjectPreparationService');
     var Connection                      = $injector.get('Connection');
     var HistoryEntryWrapper             = $injector.get('HistoryEntryWrapper');
     
@@ -39,9 +38,6 @@ angular.module('manage').controller('connectionEditModalController', ['$scope',
     $scope.connection = new Connection($scope.connection);
     
     var newConnection = !$scope.connection.identifier;
-    if(newConnection)
-        // Prepare this connection for display
-        displayObjectPreparationService.prepareConnection($scope.connection);
     
     // Set it to VNC by default
     if(!$scope.connection.protocol)
@@ -50,9 +46,13 @@ angular.module('manage').controller('connectionEditModalController', ['$scope',
     $scope.historyEntryWrappers = [];
     
     // Wrap all the history entries
-    $scope.connection.history.forEach(function wrapHistoryEntry(historyEntry) {
-       $scope.historyEntryWrappers.push(new HistoryEntryWrapper(historyEntry)); 
-    });
+    if (!newConnection) {
+        connectionService.getConnectionHistory($scope.connection.identifier).success(function wrapHistoryEntries(historyEntries) {
+            historyEntries.forEach(function wrapHistoryEntry(historyEntry) {
+               $scope.historyEntryWrappers.push(new HistoryEntryWrapper(historyEntry)); 
+            });
+        });
+    }
     
     /**
      * Close the modal.

guacamole/src/main/webapp/app/manage/controllers/connectionGroupEditModalController.js

@@ -28,7 +28,6 @@ angular.module('manage').controller('connectionGroupEditModalController', ['$sco
             
     var connectionGroupEditModal        = $injector.get('connectionGroupEditModal');
     var connectionGroupService          = $injector.get('connectionGroupService');
-    var displayObjectPreparationService = $injector.get('displayObjectPreparationService');
     
     // Make a copy of the old connection group so that we can copy over the changes when done
     var oldConnectionGroup = $scope.connectionGroup;
@@ -66,9 +65,6 @@ angular.module('manage').controller('connectionGroupEditModalController', ['$sco
     $scope.save = function save() {
         connectionGroupService.saveConnectionGroup($scope.connectionGroup).success(function successfullyUpdatedConnectionGroup() {
             
-            // Prepare this connection group for display
-            displayObjectPreparationService.prepareConnectionGroup($scope.connectionGroup);
-            
             var oldParentID = oldConnectionGroup.parentIdentifier;
             var newParentID = $scope.connectionGroup.parentIdentifier;
             

guacamole/src/main/webapp/app/manage/controllers/manageController.js

@@ -60,83 +60,41 @@ angular.module('manage').controller('manageController', ['$scope', '$injector',
 
     });
     
-    /**
-     * Move the connection or connection group within the group heirarchy, 
-     * initially place a new item, or remove an item from the heirarchy.
-     * @param {object} item The connection or connection group to move.
-     * @param {string} fromID The ID of the group to move the item from, if relevant.
-     * @param {string} toID The ID of the group to move the item to, if relevant.
-     */
-    $scope.moveItem = function moveItem(item, fromID, toID) {
-        
-        // Remove the item from the old group, if there was one
-        if(fromID) {
-            var oldParent   = findGroup($scope.rootGroup, fromID),
-                oldChildren = oldParent.children;
-            
-            // Find and remove the item from the old group
-            for(var i = 0; i < oldChildren.length; i++) {
-                var child = oldChildren[i];
-                if(child.isConnection === item.isConnection &&
-                        child.identifier === item.identifier) {
-                    oldChildren.splice(i, 1);
-                    break;
-                }
-            }
-        }
-        
-        // Add the item to the new group, if there is one
-        if(toID) {
-            var newParent = findGroup($scope.rootGroup, toID);
-            newParent.children.push(item);
-        }
-    };
-    
-    function findGroup(group, parentID) {
-        // Only searching in groups
-        if(group.isConnection)
-            return;
-        
-        if(group.identifier === parentID)
-            return group;
-        
-        for(var i = 0; i < group.children.length; i++) {
-            var child = group.children[i];
-            var foundGroup = findGroup(child, parentID);
-            if(foundGroup) return foundGroup;
-        }
-    }
-        
-    
     $scope.protocols = {};
     
     // Get the protocol information from the server and copy it into the scope
     protocolService.getProtocols().success(function fetchProtocols(protocols) {
-        angular.extend($scope.protocols, protocols);
+        $scope.protocols = protocols;
     });
+
+    // Expose object edit functions to group list template
+    $scope.groupListContext = {
     
-    /**
-     * Toggle the open/closed status of the connectionGroup.
-     * 
-     * @param {object} connectionGroup The connection group to toggle.
-     */
-    $scope.toggleExpanded = function toggleExpanded(connectionGroup) {
-        connectionGroup.expanded = !connectionGroup.expanded;
-    };
-    
-    /**
-     * Open a modal to edit the connection.
-     *  
-     * @param {object} connection The connection to edit.
-     */
-    $scope.editConnection = function editConnection(connection) {
-        connectionEditModal.activate(
-        {
-            connection : connection, 
-            protocols  : $scope.protocols,
-            moveItem   : $scope.moveItem,
-            rootGroup  : $scope.rootGroup
-        });
+        /**
+         * Open a modal to edit the connection.
+         *  
+         * @param {object} connection The connection to edit.
+         */
+        editConnection : function editConnection(connection) {
+            connectionEditModal.activate({
+                connection : connection, 
+                protocols  : $scope.protocols,
+                rootGroup  : $scope.rootGroup
+            });
+        },
+
+        /**
+         * Open a modal to edit the connection group.
+         *  
+         * @param {object} connection The connection group to edit.
+         */
+        editConnectionGroup : function editConnectionGroup(connectionGroup) {
+            connectionGroupEditModal.activate({
+                connectionGroup : connectionGroup, 
+                rootGroup       : $scope.rootGroup
+            });
+        }
+
     };
     
     /**
@@ -164,20 +122,6 @@ angular.module('manage').controller('manageController', ['$scope', '$injector',
         });
     };
     
-    /**
-     * Open a modal to edit the connection group.
-     *  
-     * @param {object} connection The connection group to edit.
-     */
-    $scope.editConnectionGroup = function editConnectionGroup(connectionGroup) {
-        connectionGroupEditModal.activate(
-        {
-            connectionGroup : connectionGroup, 
-            moveItem        : $scope.moveItem,
-            rootGroup       : $scope.rootGroup
-        });
-    };
-    
     // Remove the user from the current list of users
     function removeUser(user) {
         for(var i = 0; i < $scope.users.length; i++) {