From 8b8752c520c9d2d1f0673b7edf95e56eaeea160f Mon Sep 17 00:00:00 2001
From: Henri Sara <hesara@vaadin.com>
Date: Thu, 14 Jul 2016 15:22:39 +0300
Subject: Use shared state for OptionGroup htmlContentAllowed

Change-Id: I626165e19a1b634bd27b2e25efab88066552b190
---
 server/src/main/java/com/vaadin/ui/OptionGroup.java | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

(limited to 'server/src/main')

diff --git a/server/src/main/java/com/vaadin/ui/OptionGroup.java b/server/src/main/java/com/vaadin/ui/OptionGroup.java
index a8d0e61a97..af2afeb4ad 100644
--- a/server/src/main/java/com/vaadin/ui/OptionGroup.java
+++ b/server/src/main/java/com/vaadin/ui/OptionGroup.java
@@ -44,7 +44,6 @@ public class OptionGroup extends AbstractSelect implements
         FieldEvents.BlurNotifier, FieldEvents.FocusNotifier {
 
     private Set<Object> disabledItemIds = new HashSet<Object>();
-    private boolean htmlContentAllowed = false;
 
     public OptionGroup() {
         super();
@@ -62,14 +61,6 @@ public class OptionGroup extends AbstractSelect implements
         super(caption);
     }
 
-    @Override
-    public void paintContent(PaintTarget target) throws PaintException {
-        if (isHtmlContentAllowed()) {
-            target.addAttribute(OptionGroupConstants.HTML_CONTENT_ALLOWED, true);
-        }
-        super.paintContent(target);
-    }
-
     @Override
     protected void paintItem(PaintTarget target, Object itemId)
             throws PaintException {
@@ -242,8 +233,7 @@ public class OptionGroup extends AbstractSelect implements
      *            text
      */
     public void setHtmlContentAllowed(boolean htmlContentAllowed) {
-        this.htmlContentAllowed = htmlContentAllowed;
-        markAsDirty();
+        getState().htmlContentAllowed = htmlContentAllowed;
     }
 
     /**
@@ -254,7 +244,7 @@ public class OptionGroup extends AbstractSelect implements
      * @see #setHtmlContentAllowed(boolean)
      */
     public boolean isHtmlContentAllowed() {
-        return htmlContentAllowed;
+        return getState(false).htmlContentAllowed;
     }
 
     @Override
@@ -293,4 +283,9 @@ public class OptionGroup extends AbstractSelect implements
     protected OptionGroupState getState() {
         return (OptionGroupState) super.getState();
     }
+
+    @Override
+    protected OptionGroupState getState(boolean markAsDirty) {
+        return (OptionGroupState) super.getState(markAsDirty);
+    }
 }
-- 
cgit v1.2.3