Revise admin groups endpoint

client/galaxy/scripts/apps/admin.js

@@ -33,6 +33,10 @@ window.app = function app( options, bootstrapped ){
             this.page.display( new GridView( { url_base: Galaxy.root + 'admin/roles_list', url_data: Galaxy.params, dict_format: true } ) );
         },
 
+        show_groups: function() {
+            this.page.display( new GridView( { url_base: Galaxy.root + 'admin/groups_list', url_data: Galaxy.params, dict_format: true } ) );
+        },
+
         show_forms : function( form_id ) {
             var form_defs = {
                 reset_user_password: {

client/galaxy/scripts/apps/panels/admin-panel.js

@@ -79,7 +79,8 @@ var AdminPanel = Backbone.View.extend({
                 target  : '__use_router__'
             },{
                 title   : 'Groups',
-                url     : 'admin/groups'
+                url     : 'admin/groups',
+                target  : '__use_router__'
             },{
                 title   : 'Roles',
                 url     : 'admin/roles',

lib/galaxy/webapps/galaxy/buildapp.py

@@ -105,6 +105,7 @@ def paste_app_factory( global_conf, **kwargs ):
 
     webapp.add_client_route( '/admin/users', 'admin' )
     webapp.add_client_route( '/admin/roles', 'admin' )
+    webapp.add_client_route( '/admin/groups', 'admin' )
     webapp.add_client_route( '/admin/forms/{form_id}', 'admin' )
     webapp.add_client_route( '/tours' )
     webapp.add_client_route( '/tours/{tour_id}' )

lib/galaxy/webapps/galaxy/controllers/admin.py

@@ -1223,9 +1223,9 @@ def purge_role( self, trans, ids ):
             message += " %s " % role.name
         return ( message, "done" )
 
-    @web.expose
+    @web.expose_api
     @web.require_admin
-    def groups( self, trans, **kwargs ):
+    def groups_list( self, trans, **kwargs ):
         if 'operation' in kwargs:
             operation = kwargs[ 'operation' ].lower().replace( '+', ' ' )
             if operation == "groups":
@@ -1243,6 +1243,7 @@ def groups( self, trans, **kwargs ):
             if operation == "rename":
                 return self.rename_group( trans, **kwargs )
         # Render the list view
+        kwargs[ 'dict_format' ] = True
         return self.group_list_grid( trans, **kwargs )
 
     @web.expose