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 <>
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 ,
-                 enrollUserInGroupFailure,group );
+                 enrollUserInGroupFailure,'group' );

Freeipa-devel mailing list

Reply via email to