Sorry, I got a false positive on that last patch. This one has test OK. But, since I got tripped up on a one liner last time, I'll avoid pushing this one under the one line rule.


THe original problem was that ipa_cmd was missing the 'group' param. I fixed this, tested it, and then broke it again. This version redoes the fix, which is to pass in 'group' as the obj param for ipa_cmd.
>From 1e9015a0a05d795b117e9980faf66bb310ec4fd8 Mon Sep 17 00:00:00 2001
From: Adam Young <ayo...@redhat.com>
Date: Mon, 30 Aug 2010 10:13:10 -0400
Subject: [PATCH] quote obj param for group
 Correction for previous comit.  'group' not group.

---
 install/static/user.js |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/install/static/user.js b/install/static/user.js
index 72b1069..71bf3e5 100644
--- a/install/static/user.js
+++ b/install/static/user.js
@@ -265,7 +265,7 @@ function enrollUserInNextGroup(){
 
         ipa_cmd( 'add_member',args, options ,
                  enrollUserInGroupSuccess,
-                 enrollUserInGroupFailure,group );
+                 enrollUserInGroupFailure,'group' );
     }else{
         location.hash="tab=user&facet=group&pkey="+qs.pkey;
     }
-- 
1.7.1

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to