diff --git a/src/main/scala/app/UserManagementController.scala b/src/main/scala/app/UserManagementController.scala index f625126..369fe7a 100644 --- a/src/main/scala/app/UserManagementController.scala +++ b/src/main/scala/app/UserManagementController.scala @@ -54,7 +54,11 @@ )(EditGroupForm.apply) get("/admin/users")(adminOnly { - admin.users.html.list(getAllUsers()) + val users = getAllUsers() + val members = users.collect { case account if(account.isGroupAccount) => + account.userName -> getGroupMembers(account.userName) + }.toMap + admin.users.html.list(users, members) }) get("/admin/users/_new")(adminOnly { diff --git a/src/main/twirl/admin/users/group.scala.html b/src/main/twirl/admin/users/group.scala.html index 0fdfbd7..636b927 100644 --- a/src/main/twirl/admin/users/group.scala.html +++ b/src/main/twirl/admin/users/group.scala.html @@ -1,7 +1,7 @@ @(account: Option[model.Account], members: List[String])(implicit context: app.Context) @import context._ @import view.helpers._ -@html.main(if(account.isEmpty) "New User" else "Update User"){ +@html.main(if(account.isEmpty) "New Group" else "Update Group"){ @admin.html.menu("users"){
@@ -56,35 +56,47 @@ $('#error-memberName').text(''); var userName = $('#memberName').val(); + // check empty + if($.trim(userName) == ''){ + return false; + } + // TODO check existence + // check duplication var exists = $('#members li').filter(function(){ - return $(this).data('userName') == userName; + return $(this).data('name') == userName; }).length > 0; if(exists){ $('#error-memberName').text('User has been already added.'); return false; } + // add member $('#members').append($('