#5865. Removed some non-needed code from widgetset, prepared datefields and button...

svn changeset:15867/svn branch:6.5

diff --git a/src/com/vaadin/terminal/gwt/client/WidgetSet.java b/src/com/vaadin/terminal/gwt/client/WidgetSet.java
index 146e3b894901a73631e68f1e20ce1e3134ce504e..38984e75c30f7b0dfaeef1ea32a86ef9c5966d7e 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/WidgetSet.java
+++ b/src/com/vaadin/terminal/gwt/client/WidgetSet.java
@@ -8,18 +8,13 @@ import com.google.gwt.core.client.GWT;
 import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.terminal.gwt.client.ui.VButton;
 import com.vaadin.terminal.gwt.client.ui.VCheckBox;
-import com.vaadin.terminal.gwt.client.ui.VDateFieldCalendar;
 import com.vaadin.terminal.gwt.client.ui.VFilterSelect;
 import com.vaadin.terminal.gwt.client.ui.VListSelect;
-import com.vaadin.terminal.gwt.client.ui.VNativeSelect;
-import com.vaadin.terminal.gwt.client.ui.VOptionGroup;
 import com.vaadin.terminal.gwt.client.ui.VPasswordField;
-import com.vaadin.terminal.gwt.client.ui.VPopupCalendar;
 import com.vaadin.terminal.gwt.client.ui.VSplitPanelHorizontal;
 import com.vaadin.terminal.gwt.client.ui.VSplitPanelVertical;
 import com.vaadin.terminal.gwt.client.ui.VTextArea;
 import com.vaadin.terminal.gwt.client.ui.VTextField;
-import com.vaadin.terminal.gwt.client.ui.VTwinColSelect;
 import com.vaadin.terminal.gwt.client.ui.VUnknownComponent;
 import com.vaadin.terminal.gwt.client.ui.VView;
 import com.vaadin.terminal.gwt.client.ui.VWindow;
@@ -64,14 +59,6 @@ public class WidgetSet {
             VUnknownComponent c = GWT.create(VUnknownComponent.class);
             c.setServerSideClassName(serverSideName);
             return c;
-        } else if (VSplitPanelVertical.class == classType) {
-            return GWT.create(VSplitPanelVertical.class);
-        } else if (VTextArea.class == classType) {
-            return GWT.create(VTextArea.class);
-        } else if (VDateFieldCalendar.class == classType) {
-            return GWT.create(VDateFieldCalendar.class);
-        } else if (VPasswordField.class == classType) {
-            return GWT.create(VPasswordField.class);
         } else if (VWindow.class == classType) {
             return GWT.create(VWindow.class);
         } else {
@@ -98,21 +85,8 @@ public class WidgetSet {
             return VWindow.class;
         } else if (widgetClass == VFilterSelect.class) {
             if (uidl.hasAttribute("type")) {
-                // TODO check if all type checks are really needed
                 final String type = uidl.getStringAttribute("type").intern();
-                if ("twincol".equals(type)) {
-                    return VTwinColSelect.class;
-                } else if ("optiongroup".equals(type)) {
-                    return VOptionGroup.class;
-                } else if ("native".equals(type)) {
-                    return VNativeSelect.class;
-                } else if ("list".equals(type)) {
-                    return VListSelect.class;
-                } else if ("legacy-multi".equals(type)) {
-                    return VListSelect.class;
-                } else if (uidl.hasAttribute("selectmode")
-                        && uidl.getStringAttribute("selectmode")
-                                .equals("multi")) {
+                if ("legacy-multi" == type) {
                     return VListSelect.class;
                 }
             }
@@ -122,11 +96,6 @@ public class WidgetSet {
             } else if (uidl.hasAttribute("secret")) {
                 return VPasswordField.class;
             }
-        } else if (widgetClass == VPopupCalendar.class) {
-            if (uidl.hasAttribute("type")
-                    && uidl.getStringAttribute("type").equals("inline")) {
-                return VDateFieldCalendar.class;
-            }
         } else if (widgetClass == VSplitPanelHorizontal.class
                 && uidl.hasAttribute("vertical")) {
             return VSplitPanelVertical.class;
@@ -177,15 +146,10 @@ public class WidgetSet {
         if (fullyqualifiedName.equals("com.vaadin.ui.Button")) {
             loadImplementation(VCheckBox.class);
         } else if (fullyqualifiedName.equals("com.vaadin.ui.Select")) {
-            loadImplementation(VTwinColSelect.class);
-            loadImplementation(VOptionGroup.class);
-            loadImplementation(VNativeSelect.class);
             loadImplementation(VListSelect.class);
         } else if (fullyqualifiedName.equals("com.vaadin.ui.TextField")) {
             loadImplementation(VTextArea.class);
             loadImplementation(VPasswordField.class);
-        } else if (fullyqualifiedName.equals("com.vaadin.ui.DateField")) {
-            loadImplementation(VDateFieldCalendar.class);
         } else if (fullyqualifiedName.equals("com.vaadin.ui.SplitPanel")) {
             loadImplementation(VSplitPanelVertical.class);
         }

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VDateFieldCalendar.java b/src/com/vaadin/terminal/gwt/client/ui/VDateFieldCalendar.java
index 26f2949870456bbf8a91ae80611d4cd16a449d6d..454ea5e3ceef5ff8870f418cede6b437b2a9afab 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VDateFieldCalendar.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VDateFieldCalendar.java
@@ -15,6 +15,9 @@ import com.vaadin.terminal.gwt.client.ui.VCalendarPanel.FocusOutListener;
 import com.vaadin.terminal.gwt.client.ui.VCalendarPanel.SubmitListener;\r
 import com.vaadin.terminal.gwt.client.ui.VCalendarPanel.TimeChangeListener;\r
 \r
+/**\r
+ * A client side implementation for InlineDateField\r
+ */\r
 public class VDateFieldCalendar extends VDateField {\r
 \r
     private final VCalendarPanel calendarPanel;\r

diff --git a/src/com/vaadin/ui/Button.java b/src/com/vaadin/ui/Button.java
index 35843365f2d2ea037bdf3b818cef84d728b8e37b..0d193168e6754e50a999d158bea27ce6a039236f 100644 (file)
--- a/src/com/vaadin/ui/Button.java
+++ b/src/com/vaadin/ui/Button.java
@@ -211,6 +211,8 @@ public class Button extends AbstractField implements FieldEvents.BlurNotifier,
      * 
      * @param switchMode
      *            The switchMode to set.
+     * @deprecated the {@link CheckBox} component should be used instead of
+     *             Button in switch mode
      */
     public void setSwitchMode(boolean switchMode) {
         this.switchMode = switchMode;

diff --git a/src/com/vaadin/ui/CheckBox.java b/src/com/vaadin/ui/CheckBox.java
index 0ffafe3d02d47a4328cbc3becf5a3599cc937a49..e8b46c1d83c30fac5d2479dee2bbf936ef9f3392 100644 (file)
--- a/src/com/vaadin/ui/CheckBox.java
+++ b/src/com/vaadin/ui/CheckBox.java
@@ -91,6 +91,7 @@ public class CheckBox extends Button {
         super(caption, false);
     }
 
+    @Deprecated
     @Override
     public void setSwitchMode(boolean switchMode)
             throws UnsupportedOperationException {

diff --git a/src/com/vaadin/ui/DateField.java b/src/com/vaadin/ui/DateField.java
index 5e8e004a0e19e67a30a58fc2f0af36121e002f38..0300315dd70ed3781cccd7aae76d3315a1969bc6 100644 (file)
--- a/src/com/vaadin/ui/DateField.java
+++ b/src/com/vaadin/ui/DateField.java
@@ -84,21 +84,6 @@ public class DateField extends AbstractField implements
      */
     public static final int RESOLUTION_YEAR = 6;
 
-    /**
-     * Popup date selector (calendar).
-     */
-    protected static final String TYPE_POPUP = "popup";
-
-    /**
-     * Inline date selector (calendar).
-     */
-    protected static final String TYPE_INLINE = "inline";
-
-    /**
-     * Specified widget type.
-     */
-    protected String type = TYPE_POPUP;
-
     /**
      * Specified smallest modifiable unit.
      */
@@ -227,7 +212,6 @@ public class DateField extends AbstractField implements
             target.addAttribute("strict", true);
         }
 
-        target.addAttribute("type", type);
         target.addAttribute(VDateField.WEEK_NUMBERS, isShowISOWeekNumbers());
         target.addAttribute("parsable", parsingSucceeded);
 

diff --git a/src/com/vaadin/ui/InlineDateField.java b/src/com/vaadin/ui/InlineDateField.java
index 88f0c3e0b40bbbf54f40553adc68589e76d17d45..d1066dc28a7bb5145effdba2937fd3db7486f712 100644 (file)
--- a/src/com/vaadin/ui/InlineDateField.java
+++ b/src/com/vaadin/ui/InlineDateField.java
@@ -7,6 +7,7 @@ package com.vaadin.ui;
 import java.util.Date;\r
 \r
 import com.vaadin.data.Property;\r
+import com.vaadin.terminal.gwt.client.ui.VDateFieldCalendar;\r
 \r
 /**\r
  * <p>\r
@@ -21,32 +22,27 @@ import com.vaadin.data.Property;
  * @VERSION@\r
  * @since 5.0\r
  */\r
-@SuppressWarnings("serial")\r
+@ClientWidget(VDateFieldCalendar.class)\r
 public class InlineDateField extends DateField {\r
 \r
     public InlineDateField() {\r
         super();\r
-        type = TYPE_INLINE;\r
     }\r
 \r
     public InlineDateField(Property dataSource) throws IllegalArgumentException {\r
         super(dataSource);\r
-        type = TYPE_INLINE;\r
     }\r
 \r
     public InlineDateField(String caption, Date value) {\r
         super(caption, value);\r
-        type = TYPE_INLINE;\r
     }\r
 \r
     public InlineDateField(String caption, Property dataSource) {\r
         super(caption, dataSource);\r
-        type = TYPE_INLINE;\r
     }\r
 \r
     public InlineDateField(String caption) {\r
         super(caption);\r
-        type = TYPE_INLINE;\r
     }\r
 \r
 }\r

diff --git a/src/com/vaadin/ui/PopupDateField.java b/src/com/vaadin/ui/PopupDateField.java
index e975cb23d2c9ecf81c75e4a1ac8954ac692081ab..7501c9fe155bd95a0b09ae87a675ccd72fb3abf0 100644 (file)
--- a/src/com/vaadin/ui/PopupDateField.java
+++ b/src/com/vaadin/ui/PopupDateField.java
@@ -23,34 +23,28 @@ import com.vaadin.terminal.PaintTarget;
  * @VERSION@\r
  * @since 5.0\r
  */\r
-@SuppressWarnings("serial")\r
 public class PopupDateField extends DateField {\r
 \r
     private String inputPrompt = null;\r
 \r
     public PopupDateField() {\r
         super();\r
-        type = TYPE_POPUP;\r
     }\r
 \r
     public PopupDateField(Property dataSource) throws IllegalArgumentException {\r
         super(dataSource);\r
-        type = TYPE_POPUP;\r
     }\r
 \r
     public PopupDateField(String caption, Date value) {\r
         super(caption, value);\r
-        type = TYPE_POPUP;\r
     }\r
 \r
     public PopupDateField(String caption, Property dataSource) {\r
         super(caption, dataSource);\r
-        type = TYPE_POPUP;\r
     }\r
 \r
     public PopupDateField(String caption) {\r
         super(caption);\r
-        type = TYPE_POPUP;\r
     }\r
 \r
     @Override\r

diff --git a/src/com/vaadin/ui/Select.java b/src/com/vaadin/ui/Select.java
index e5c19ccdd38225c50e9c552352322bc8a82d236d..0eefdeacb49d841831a61007f97320eff578cb66 100644 (file)
--- a/src/com/vaadin/ui/Select.java
+++ b/src/com/vaadin/ui/Select.java
@@ -518,4 +518,25 @@ public class Select extends AbstractSelect implements AbstractSelect.Filtering,
 
     }
 
+    /**
+     * @deprecated use {@link ListSelect}, {@link OptionGroup} or
+     *             {@link TwinColSelect} instead
+     * @see com.vaadin.ui.AbstractSelect#setMultiSelect(boolean)
+     */
+    @Override
+    public void setMultiSelect(boolean multiSelect) {
+        super.setMultiSelect(multiSelect);
+    }
+
+    /**
+     * @deprecated use {@link ListSelect}, {@link OptionGroup} or
+     *             {@link TwinColSelect} instead
+     * 
+     * @see com.vaadin.ui.AbstractSelect#isMultiSelect()
+     */
+    @Override
+    public boolean isMultiSelect() {
+        return super.isMultiSelect();
+    }
+
 }