Re: [Freeipa-devel] [PATCH] admiyo-0169-reset-target-section

2011-01-27 Thread Adam Young

On 01/26/2011 04:52 PM, Adam Young wrote:



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

Rebased
From c37d4a2499281980c9a73034a91b012c8fc97fc5 Mon Sep 17 00:00:00 2001
From: Adam Young ayo...@redhat.com
Date: Wed, 26 Jan 2011 16:50:43 -0500
Subject: [PATCH 3/3] reset target section
 target section needed to cache results for use in results
 moved load logic into reset and load now calls reset

---
 install/ui/aci.js |9 ++---
 1 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/install/ui/aci.js b/install/ui/aci.js
index 1cc1611caac78163bc7ae88bc2f03426f91f58be..3bad384a2b80e3842395ba54004b2808928f28cf 100644
--- a/install/ui/aci.js
+++ b/install/ui/aci.js
@@ -494,6 +494,12 @@ IPA.target_section = function () {
 };
 
 that.load = function(result) {
+that.result = result;
+that.reset();
+};
+
+that.reset = function() {
+var result = that.result;
 if(result.subtree){
 $('#aci_query_text').val(result.subtree);
 $('#aci_by_query').click();
@@ -524,9 +530,6 @@ IPA.target_section = function () {
 }
 };
 
-that.reset = function() {
-};
-
 that.save = function (record){
 
 var record_type = $(input[name='type']:checked).attr('id');
-- 
1.7.3.5

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

Re: [Freeipa-devel] [PATCH] admiyo-0169-reset-target-section

2011-01-27 Thread Adam Young

On 01/27/2011 01:57 PM, Adam Young wrote:

On 01/26/2011 04:52 PM, Adam Young wrote:



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

Rebased


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

ACKed in IRC by edewata and pushed to master
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel