Move read-only to AbstractFieldState (#8514)

Change-Id: I0baea991bd80075d63c5d57e3b07f4e9fcb12676

client/src/main/java/com/vaadin/client/ComponentConnector.java

@@ -85,15 +85,6 @@ public interface ComponentConnector extends ServerConnector {
      */
     public boolean isRelativeHeight();
 
-    /**
-     * Checks if the connector is read only.
-     *
-     * @deprecated This belongs in AbstractFieldConnector, see #8514
-     * @return true
-     */
-    @Deprecated
-    public boolean isReadOnly();
-
     /**
      * Return true if parent handles caption, false if the paintable handles the
      * caption itself.

client/src/main/java/com/vaadin/client/connectors/AbstractListingConnector.java

@@ -17,7 +17,7 @@
 
 import com.vaadin.client.connectors.data.HasDataSource;
 import com.vaadin.client.data.DataSource;
-import com.vaadin.client.ui.AbstractComponentConnector;
+import com.vaadin.client.ui.AbstractFieldConnector;
 import com.vaadin.shared.data.DataCommunicatorConstants;
 import com.vaadin.shared.data.selection.SelectionModel;
 import com.vaadin.ui.AbstractListing;
@@ -36,7 +36,7 @@
  * @since 8.0
  */
 public abstract class AbstractListingConnector<SELECTIONMODEL extends SelectionModel<?>>
-        extends AbstractComponentConnector implements HasDataSource {
+        extends AbstractFieldConnector implements HasDataSource {
 
     private DataSource<JsonObject> dataSource = null;
 
@@ -64,7 +64,7 @@ public void setSelectionModel(SELECTIONMODEL selectionModel) {
 
     /**
      * Returns the selection model instance used.
-     * 
+     *
      * @return the selection model
      */
     public SELECTIONMODEL getSelectionModel() {
@@ -73,7 +73,7 @@ public SELECTIONMODEL getSelectionModel() {
 
     /**
      * Returns the key of the given data row.
-     * 
+     *
      * @param row
      *            the row
      * @return the row key
@@ -84,7 +84,7 @@ protected static String getRowKey(JsonObject row) {
 
     /**
      * Returns the data of the given data row.
-     * 
+     *
      * @param row
      *            the row
      * @return the row data
@@ -95,7 +95,7 @@ protected static JsonValue getRowData(JsonObject row) {
 
     /**
      * Returns whether the given row is selected.
-     * 
+     *
      * @param row
      *            the row
      * @return {@code true} if the row is selected, {@code false} otherwise

client/src/main/java/com/vaadin/client/ui/AbstractComponentConnector.java

@@ -628,10 +628,6 @@ protected void updateWidgetStyleNames() {
         // Set the core 'v' style name for the widget
         setWidgetStyleName(StyleConstants.UI_WIDGET, true);
 
-        // should be in AbstractFieldConnector ?
-        // add / remove read-only style name
-        setWidgetStyleName("v-readonly", isReadOnly());
-
         // add / remove error style name
         setWidgetStyleNameWithPrefix(primaryStyleName, StyleConstants.ERROR_EXT,
                 null != state.errorMessage);
@@ -730,17 +726,6 @@ protected void setWidgetStyleNameWithPrefix(String prefix, String styleName,
         getWidget().setStyleName(prefix + styleName, add);
     }
 
-    /*
-     * (non-Javadoc)
-     *
-     * @see com.vaadin.client.ComponentConnector#isReadOnly()
-     */
-    @Override
-    @Deprecated
-    public boolean isReadOnly() {
-        return getState().readOnly;
-    }
-
     @Override
     public LayoutManager getLayoutManager() {
         return LayoutManager.get(getConnection());

client/src/main/java/com/vaadin/client/ui/AbstractFieldConnector.java

@@ -35,8 +35,20 @@ public boolean isRequiredIndicatorVisible() {
     protected void updateWidgetStyleNames() {
         super.updateWidgetStyleNames();
 
+        // add / remove read-only style name
+        setWidgetStyleName("v-readonly", isReadOnly());
+
         // add / remove error style name to Fields
         setWidgetStyleNameWithPrefix(getWidget().getStylePrimaryName(),
                 StyleConstants.REQUIRED_EXT, isRequiredIndicatorVisible());
     }
+
+    /**
+     * Checks if the connector is read only.
+     *
+     * @return true
+     */
+    public boolean isReadOnly() {
+        return getState().readOnly;
+    }
 }

client/src/main/java/com/vaadin/client/ui/upload/UploadConnector.java

@@ -84,7 +84,7 @@ public void updateFromUIDL(UIDL uidl, ApplicationConnection client) {
         }
         getWidget().fu.setName(getWidget().paintableId + "_file");
 
-        if (!isEnabled() || isReadOnly()) {
+        if (!isEnabled()) {
             getWidget().disableUpload();
         } else if (!uidl.getBooleanAttribute("state")) {
             // Enable the button only if an upload is not in progress

compatibility-client/src/main/java/com/vaadin/v7/client/ComponentConnector.java

@@ -0,0 +1,36 @@
+/*
+ * Copyright 2000-2016 Vaadin Ltd.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License. You may obtain a copy of
+ * the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+ * License for the specific language governing permissions and limitations under
+ * the License.
+ */
+package com.vaadin.v7.client;
+
+/**
+ * Legacy ComponentConnector with {@code isReadOnly}.
+ *
+ * @author teemusa
+ *
+ */
+public interface ComponentConnector
+        extends com.vaadin.client.ComponentConnector {
+
+    /**
+     * Checks if the connector is read only.
+     *
+     * @deprecated This belongs in AbstractFieldConnector, see #8514
+     * @return true
+     */
+    @Deprecated
+    public boolean isReadOnly();
+
+}

compatibility-client/src/main/java/com/vaadin/v7/client/ui/AbstractLegacyComponentConnector.java

@@ -18,6 +18,7 @@
 import com.vaadin.client.ui.AbstractComponentConnector;
 import com.vaadin.client.ui.AbstractConnector;
 import com.vaadin.shared.communication.ServerRpc;
+import com.vaadin.v7.client.ComponentConnector;
 import com.vaadin.v7.shared.AbstractLegacyComponentState;
 
 /**
@@ -30,8 +31,8 @@
  * @deprecated only used for Vaadin 7 compatiblity components
  */
 @Deprecated
-public class AbstractLegacyComponentConnector
-        extends AbstractComponentConnector {
+public class AbstractLegacyComponentConnector extends AbstractComponentConnector
+        implements ComponentConnector {
 
     // overridden to be visible to VUpload in the same package. Without making
     // it public in VUploadConnector
@@ -44,4 +45,17 @@ protected <T extends ServerRpc> T getRpcProxy(Class<T> rpcInterface) {
     public AbstractLegacyComponentState getState() {
         return (AbstractLegacyComponentState) super.getState();
     }
+
+    @Override
+    public boolean isReadOnly() {
+        return getState().readOnly;
+    }
+
+    @Override
+    protected void updateWidgetStyleNames() {
+        super.updateWidgetStyleNames();
+
+        // add / remove read-only style name
+        setWidgetStyleName("v-readonly", isReadOnly());
+    }
 }

compatibility-client/src/main/java/com/vaadin/v7/client/ui/form/FormConnector.java

@@ -1,12 +1,12 @@
 /*
  * Copyright 2000-2016 Vaadin Ltd.
- * 
+ *
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not
  * use this file except in compliance with the License. You may obtain a copy of
  * the License at
- * 
+ *
  * http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
  * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
@@ -40,7 +40,8 @@
 
 @Connect(Form.class)
 public class FormConnector extends AbstractComponentContainerConnector
-        implements Paintable, MayScrollChildren {
+        implements Paintable, MayScrollChildren,
+        com.vaadin.v7.client.ComponentConnector {
 
     private final ElementResizeListener footerResizeListener = new ElementResizeListener() {
         @Override
@@ -179,7 +180,7 @@ public VForm getWidget() {
 
     @Override
     public boolean isReadOnly() {
-        return super.isReadOnly() || getState().propertyReadOnly;
+        return getState().readOnly || getState().propertyReadOnly;
     }
 
     @Override

compatibility-client/src/main/java/com/vaadin/v7/client/ui/tree/TreeConnector.java

@@ -31,10 +31,10 @@
 import com.vaadin.client.VConsole;
 import com.vaadin.client.WidgetUtil;
 import com.vaadin.client.communication.StateChangeEvent;
-import com.vaadin.client.ui.AbstractComponentConnector;
 import com.vaadin.shared.MouseEventDetails;
 import com.vaadin.shared.ui.Connect;
 import com.vaadin.shared.ui.MultiSelectMode;
+import com.vaadin.v7.client.ui.AbstractLegacyComponentConnector;
 import com.vaadin.v7.client.ui.VTree;
 import com.vaadin.v7.client.ui.VTree.TreeNode;
 import com.vaadin.v7.shared.ui.tree.TreeConstants;
@@ -43,7 +43,7 @@
 import com.vaadin.v7.ui.Tree;
 
 @Connect(Tree.class)
-public class TreeConnector extends AbstractComponentConnector
+public class TreeConnector extends AbstractLegacyComponentConnector
         implements Paintable {
 
     protected final Map<TreeNode, TooltipInfo> tooltipMap = new HashMap<TreeNode, TooltipInfo>();

compatibility-shared/src/main/java/com/vaadin/v7/shared/AbstractLegacyComponentState.java

@@ -29,4 +29,5 @@
 public class AbstractLegacyComponentState extends AbstractComponentState {
     @NoLayout
     public boolean immediate = false;
+    public boolean readOnly = false;
 }

server/src/main/java/com/vaadin/ui/AbstractComponent.java

@@ -1053,8 +1053,16 @@ public boolean isResponsive() {
      *            read-only mode or not
      */
     protected void setReadOnly(boolean readOnly) {
-        if (readOnly != isReadOnly()) {
-            getState().readOnly = readOnly;
+        if (getState(false) instanceof AbstractFieldState) {
+            if (readOnly != isReadOnly()) {
+                ((AbstractFieldState) getState()).readOnly = readOnly;
+            }
+        } else {
+            throw new IllegalStateException(
+                    "This component does not support the read-only mode, since state is of type "
+                            + getStateType().getSimpleName()
+                            + " and does not inherit "
+                            + AbstractFieldState.class.getSimpleName());
         }
     }
 
@@ -1067,7 +1075,14 @@ protected void setReadOnly(boolean readOnly) {
      * @see #setReadOnly(boolean)
      */
     protected boolean isReadOnly() {
-        return getState(false).readOnly;
+        if (getState(false) instanceof AbstractFieldState) {
+            return ((AbstractFieldState) getState(false)).readOnly;
+        }
+        throw new IllegalStateException(
+                "This component does not support the read-only mode, since state is of type "
+                        + getStateType().getSimpleName()
+                        + " and does not inherit "
+                        + AbstractFieldState.class.getSimpleName());
     }
 
     /**
@@ -1424,13 +1439,12 @@ protected void setRequiredIndicatorVisible(boolean visible) {
     protected boolean isRequiredIndicatorVisible() {
         if (getState(false) instanceof AbstractFieldState) {
             return ((AbstractFieldState) getState(false)).required;
-        } else {
-            throw new IllegalStateException(
-                    "This component does not support the required indicator, since state is of type "
-                            + getStateType().getSimpleName()
-                            + " and does not inherit "
-                            + AbstractFieldState.class.getSimpleName());
         }
+        throw new IllegalStateException(
+                "This component does not support the required indicator, since state is of type "
+                        + getStateType().getSimpleName()
+                        + " and does not inherit "
+                        + AbstractFieldState.class.getSimpleName());
     }
 
     private static final Logger getLogger() {

shared/src/main/java/com/vaadin/shared/AbstractComponentState.java

@@ -31,7 +31,6 @@
 public class AbstractComponentState extends SharedState {
     public String height = "";
     public String width = "";
-    public boolean readOnly = false;
     @NoLayout
     public String description = "";
     // Note: for the caption, there is a difference between null and an empty

shared/src/main/java/com/vaadin/shared/AbstractFieldState.java

@@ -31,4 +31,9 @@ public class AbstractFieldState extends TabIndexState {
      */
     public boolean required = false;
 
+    /**
+     * Is the field read-only or not.
+     */
+    public boolean readOnly = false;
+
 }