From b3b511082aaf71b5edc3e6f3d9c490801e80b780 Mon Sep 17 00:00:00 2001
From: Jani Laakso <jani.laakso@itmill.com>
Date: Fri, 7 Mar 2008 13:25:29 +0000
Subject: [PATCH] Continued fix for #1475: setReadOnly(true) fails for:
 Checkbox, Button, Upload

Checkbox and Button setReadonly(false) now ignores variable changes.

svn changeset:3999/svn branch:trunk
---
 src/com/itmill/toolkit/ui/Button.java | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/com/itmill/toolkit/ui/Button.java b/src/com/itmill/toolkit/ui/Button.java
index 538c8953d6..0ece73eb7a 100644
--- a/src/com/itmill/toolkit/ui/Button.java
+++ b/src/com/itmill/toolkit/ui/Button.java
@@ -152,7 +152,11 @@ public class Button extends AbstractField {
      * @param variables
      */
     public void changeVariables(Object source, Map variables) {
-        if (variables.containsKey("state")) {
+        if (isReadOnly()) {
+            System.err
+                    .println("Button: ignoring change variable for read-only component, caption="
+                            + getCaption());
+        } else if (variables.containsKey("state")) {
             // Gets the new and old button states
             final Boolean newValue = (Boolean) variables.get("state");
             final Boolean oldValue = (Boolean) getValue();
-- 
2.39.5