From: Jani Laakso <jani.laakso@itmill.com>
Date: Fri, 7 Mar 2008 13:59:31 +0000 (+0000)
Subject: Continued fix for #1475: setReadOnly(true) fails for: Checkbox, Button, Upload
X-Git-Tag: 6.7.0.beta1~4963
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=712a704db3129cd3bc5c8d2a8e78df011c371ca6;p=vaadin-framework.git

Continued fix for #1475: setReadOnly(true) fails for: Checkbox, Button, Upload

Upload setReadonly(false) now ignores file upload when set as readonly

svn changeset:4002/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/terminal/gwt/server/CommunicationManager.java b/src/com/itmill/toolkit/terminal/gwt/server/CommunicationManager.java
index 5fe7a7cec6..6c90cd24fc 100644
--- a/src/com/itmill/toolkit/terminal/gwt/server/CommunicationManager.java
+++ b/src/com/itmill/toolkit/terminal/gwt/server/CommunicationManager.java
@@ -129,6 +129,10 @@ public class CommunicationManager implements Paintable.RepaintRequestListener {
                     final String pid = name.split("_")[0];
                     final Upload uploadComponent = (Upload) idPaintableMap
                             .get(pid);
+                    if (uploadComponent.isReadOnly()) {
+                        throw new FileUploadException(
+                                "Warning: ignored file upload because upload component is set as read-only");
+                    }
                     if (uploadComponent == null) {
                         throw new FileUploadException(
                                 "Upload component not found");