This addresses Endi's comment regarding the 'this' pointer from path admiyo-0019. This patch is required to get the associations working.
>From 221351809b94f71572219177b36443a60c157cf7 Mon Sep 17 00:00:00 2001
From: Adam Young <ayo...@redhat.com>
Date: Tue, 7 Sep 2010 13:13:59 -0400
Subject: [PATCH] local param for this in closure

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

diff --git a/install/static/associate.js b/install/static/associate.js
index 748c40e..c11642b 100644
--- a/install/static/associate.js
+++ b/install/static/associate.js
@@ -85,7 +85,7 @@ function BulkAssociator(form, pkey, manyObjPkeys){
         options[form.manyObj] = option;
 
         var args = [this.pkey];
-
+        var associator = this;
         ipa_cmd( form.method,args, options ,
                  function(response){
                      var qs = ipa_parse_qs();
@@ -93,7 +93,7 @@ function BulkAssociator(form, pkey, manyObjPkeys){
                          alert("error adding memeber");
                      }else{
                          location.hash="tab=" +form.oneObj
-                             +"&facet=details&pkey="+this.pkey;
+                             +"&facet=details&pkey="+associator.pkey;
                      }
                  },
                  function(response){
-- 
1.7.1

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

Reply via email to