[Freeipa-devel] [PATCH] small group fixes

Kevin McCarthy kmccarth at redhat.com
Mon Oct 8 16:23:48 UTC 2007


Missed submitting this patch, so sending it before I send the IPAClient
fixes patch.

-Kevin

-------------- next part --------------
# HG changeset patch
# User Kevin McCarthy <kmccarth at redhat.com>
# Date 1191860523 25200
# Node ID 7ee0841d54cf9bd415c1429891c2676a4177e716
# Parent  8a6b70b1ba1adb37bfdced6b2f6607efc9acaaff
Small group fixes: remove index, change to use hidden_fields (like UserFields)

diff -r 8a6b70b1ba1a -r 7ee0841d54cf ipa-server/ipa-gui/ipagui/forms/group.py
--- a/ipa-server/ipa-gui/ipagui/forms/group.py	Mon Oct 08 09:18:44 2007 -0700
+++ b/ipa-server/ipa-gui/ipagui/forms/group.py	Mon Oct 08 09:22:03 2007 -0700
@@ -21,8 +21,9 @@ class GroupNewForm(widgets.Form):
 class GroupNewForm(widgets.Form):
     params = ['group']
 
-    fields = [GroupFields.cn, GroupFields.description,
-              GroupFields.dn_to_info_json]
+    hidden_fields = [
+      GroupFields.dn_to_info_json
+    ]
 
     validator = GroupNewValidator()
 
@@ -46,10 +47,11 @@ class GroupEditForm(widgets.Form):
 class GroupEditForm(widgets.Form):
     params = ['members', 'group']
 
-    fields = [GroupFields.gidnumber, GroupFields.description,
-              GroupFields.cn_hidden, GroupFields.editprotected_hidden,
-              GroupFields.group_orig, GroupFields.member_data,
-              GroupFields.dn_to_info_json]
+    hidden_fields = [
+      GroupFields.cn_hidden, GroupFields.editprotected_hidden,
+      GroupFields.group_orig, GroupFields.member_data,
+      GroupFields.dn_to_info_json
+    ]
 
     validator = GroupEditValidator()
 
diff -r 8a6b70b1ba1a -r 7ee0841d54cf ipa-server/ipa-gui/ipagui/subcontrollers/group.py
--- a/ipa-server/ipa-gui/ipagui/subcontrollers/group.py	Mon Oct 08 09:18:44 2007 -0700
+++ b/ipa-server/ipa-gui/ipagui/subcontrollers/group.py	Mon Oct 08 09:22:03 2007 -0700
@@ -33,11 +33,10 @@ class GroupController(IPAController):
     # Group #
     #########
 
-    @expose("ipagui.templates.groupindex")
+    @expose()
     @identity.require(identity.not_anonymous())
     def index(self, tg_errors=None):
-        client.set_krbccache(os.environ["KRB5CCNAME"])
-        return dict()
+        raise turbogears.redirect("/group/list")
 
     @expose("ipagui.templates.groupnew")
     @identity.require(identity.not_anonymous())
diff -r 8a6b70b1ba1a -r 7ee0841d54cf ipa-server/ipa-gui/ipagui/templates/groupindex.kid
--- a/ipa-server/ipa-gui/ipagui/templates/groupindex.kid	Mon Oct 08 09:18:44 2007 -0700
+++ /dev/null	Thu Jan 01 00:00:00 1970 +0000
@@ -1,11 +0,0 @@
-<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
-<html xmlns="http://www.w3.org/1999/xhtml" xmlns:py="http://purl.org/kid/ns#"
-    py:extends="'grouplayout.kid'">
-<head>
-<meta content="text/html; charset=utf-8" http-equiv="Content-Type" py:replace="''"/>
-<title>Group Listing</title>
-</head>
-<body>
-       Groups go here.
-</body>
-</html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 4054 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20071008/84039c30/attachment.bin>


More information about the Freeipa-devel mailing list