From: Joonas Lehtinen <joonas.lehtinen@itmill.com>
Date: Mon, 29 Jan 2007 16:56:43 +0000 (+0000)
Subject: Removed dual adapter registration: in ajax mode the window sent the events both to... 
X-Git-Tag: 6.7.0.beta1~6724
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=d189afd876afc2dac2f8a9994fdf9505162f7b77;p=vaadin-framework.git

Removed dual adapter registration: in ajax mode the window sent the events both to application manager as well as
to application servlet. This causes various bugs in repainting.

Fixes #247

svn changeset:366/svn branch:toolkit
---

diff --git a/src/com/itmill/toolkit/terminal/web/ApplicationServlet.java b/src/com/itmill/toolkit/terminal/web/ApplicationServlet.java
index 1930547b8f..7129673e0d 100644
--- a/src/com/itmill/toolkit/terminal/web/ApplicationServlet.java
+++ b/src/com/itmill/toolkit/terminal/web/ApplicationServlet.java
@@ -1885,6 +1885,10 @@ public class ApplicationServlet extends HttpServlet implements
 			application.removeListener((Application.WindowAttachListener) this);
 			application.removeListener((Application.WindowDetachListener) this);
 
+			// Deregister all window listeners
+			for (Iterator wins = application.getWindows().iterator(); wins.hasNext();)
+				((Window)wins.next()).removeListener((Paintable.RepaintRequestListener)this);
+			
 			// Manager takes control over the application
 			mgr.takeControl();
 		}