From e9d29d5cf3cb2f5ab3606692afc07be27f1810f4 Mon Sep 17 00:00:00 2001
From: Matti Tahvonen <matti.tahvonen@itmill.com>
Date: Fri, 29 Aug 2008 09:40:59 +0000
Subject: [PATCH] fixed bug sending duplicate variables in some cases

svn changeset:5297/svn branch:trunk
---
 .../gwt/client/ApplicationConnection.java     | 23 ++++++++++---------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ApplicationConnection.java b/src/com/itmill/toolkit/terminal/gwt/client/ApplicationConnection.java
index 30c43e7046..ef73ff90e5 100755
--- a/src/com/itmill/toolkit/terminal/gwt/client/ApplicationConnection.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ApplicationConnection.java
@@ -32,10 +32,10 @@ import com.google.gwt.user.client.ui.FocusWidget;
 import com.google.gwt.user.client.ui.HasFocus;
 import com.google.gwt.user.client.ui.HasWidgets;
 import com.google.gwt.user.client.ui.Widget;
-import com.itmill.toolkit.terminal.gwt.client.ui.IContextMenu;
 import com.itmill.toolkit.terminal.gwt.client.ui.Field;
-import com.itmill.toolkit.terminal.gwt.client.ui.IView;
+import com.itmill.toolkit.terminal.gwt.client.ui.IContextMenu;
 import com.itmill.toolkit.terminal.gwt.client.ui.INotification;
+import com.itmill.toolkit.terminal.gwt.client.ui.IView;
 import com.itmill.toolkit.terminal.gwt.client.ui.INotification.HideEvent;
 
 /**
@@ -572,7 +572,7 @@ public class ApplicationConnection {
      * Returns Paintable element by its id
      * 
      * @param id
-     *            Paintable ID
+     *                Paintable ID
      */
     public Paintable getPaintable(String id) {
         return (Paintable) idToPaintable.get(id);
@@ -614,6 +614,7 @@ public class ApplicationConnection {
                         || pendingVariableBursts.size() == 0) {
                     Vector burst = (Vector) pendingVariables.clone();
                     pendingVariableBursts.add(burst);
+                    pendingVariables.clear();
                 }
             } else {
                 buildAndSendVariableBurst(pendingVariables);
@@ -715,13 +716,13 @@ public class ApplicationConnection {
      * when needed.
      * 
      * @param component
-     *            Widget to be updated, expected to implement an instance of
-     *            Paintable
+     *                Widget to be updated, expected to implement an instance of
+     *                Paintable
      * @param uidl
-     *            UIDL to be painted
+     *                UIDL to be painted
      * @param manageCaption
-     *            True if you want to delegate caption, icon, description and
-     *            error message management to parent.
+     *                True if you want to delegate caption, icon, description
+     *                and error message management to parent.
      * 
      * @return Returns true iff no further painting is needed by caller
      */
@@ -857,7 +858,7 @@ public class ApplicationConnection {
      * parent.
      * 
      * @param uidl
-     *            UIDL to create Paintable from.
+     *                UIDL to create Paintable from.
      * @return Either existing or new Paintable corresponding to UIDL.
      */
     public Paintable getPaintable(UIDL uidl) {
@@ -897,7 +898,7 @@ public class ApplicationConnection {
      * to browser due URI's in UIDL may contain custom protocols like theme://.
      * 
      * @param toolkitUri
-     *            toolkit URI from uidl
+     *                toolkit URI from uidl
      * @return translated URI ready for browser
      */
     public String translateToolkitUri(String toolkitUri) {
@@ -977,7 +978,7 @@ public class ApplicationConnection {
      * Adds PNG-fix conditionally (only for IE6) to the specified IMG -element.
      * 
      * @param el
-     *            the IMG element to fix
+     *                the IMG element to fix
      */
     public void addPngFix(Element el) {
         BrowserInfo b = BrowserInfo.get();
-- 
2.39.5