Fixed #191.

4 files changed, 11 insertions, 4 deletions

diff --git a/HISTORY b/HISTORY
index 3d90f3b0..b4cf59a3 100644
--- a/HISTORY
+++ b/HISTORY
@@ -1,8 +1,10 @@
 Version 3.0.2 July 2, 2006

 ==========================

 BUG: Ticket#182 - List and validator controls cause problem in child classes (Qiang)

+BUG: Ticket#191 - Duplicated postbacks occur when using TButton with validators (Qiang)

 BUG: Ticket#213 - PRADO Requirements Checker charset error (Qiang)

 CHG: ensureChildControls() is now invoked in TControl::initRecursive (Qiang)

+CHG: Postback enabled control will always disable default client-side browser action. (Qiang)

 

 Version 3.0.1 June 4, 2006

 ==========================

diff --git a/UPGRADE b/UPGRADE
index afd64286..4def476e 100644
--- a/UPGRADE
+++ b/UPGRADE
@@ -16,7 +16,9 @@ for both A and B.
 

 Upgrading from v3.0.1

 ---------------------

-

+- Postback enabled control will always disable default client-side browser action.

+  This is due to google toolbar's interference of event stopping scheme.

+  This modification should only affect user-derived postback javascripts.

 

 Upgrading from v3.0.0

 ---------------------

diff --git a/framework/Web/Javascripts/js/prado.js b/framework/Web/Javascripts/js/prado.js
index 0946e192..1ec69786 100644
--- a/framework/Web/Javascripts/js/prado.js
+++ b/framework/Web/Javascripts/js/prado.js
@@ -243,8 +243,7 @@ form.action=options['PostBackUrl'];if(options['TrackFocus'])
 {var active=document.activeElement;if(active)
 lastFocus.value=active.id;else
 lastFocus.value=options['EventTarget'];}}
-$('PRADO_POSTBACK_TARGET').value=options['EventTarget'];$('PRADO_POSTBACK_PARAMETER').value=options['EventParameter'];if(options['StopEvent'])
-Event.stop(event);Event.fireEvent(form,"submit");}
+$('PRADO_POSTBACK_TARGET').value=options['EventTarget'];$('PRADO_POSTBACK_PARAMETER').value=options['EventParameter'];Event.stop(event);Event.fireEvent(form,"submit");}
 Prado.Element={setValue:function(element,value)
 {var el=$(element);if(el&&typeof(el.value)!="undefined")
 el.value=value;},select:function(element,method,value)
diff --git a/framework/Web/Javascripts/prado/form.js b/framework/Web/Javascripts/prado/form.js
index aec9395d..beea9248 100644
--- a/framework/Web/Javascripts/prado/form.js
+++ b/framework/Web/Javascripts/prado/form.js
@@ -129,7 +129,11 @@ Prado.PostBack = function(event,options)
 

 	$('PRADO_POSTBACK_TARGET').value = options['EventTarget'];

 	$('PRADO_POSTBACK_PARAMETER').value = options['EventParameter'];

-	if(options['StopEvent']) 

+	/**

+	 * Since google toolbar prevents browser default action,

+	 * we will always disable default client-side browser action

+	 */

+	/*if(options['StopEvent']) */

 		Event.stop(event);

 	Event.fireEvent(form,"submit");

 }