]> source.dussan.org Git - vaadin-framework.git/commitdiff
More state getter removal
authorJohn Ahlroos <john@vaadin.com>
Fri, 31 Aug 2012 11:22:26 +0000 (14:22 +0300)
committerJohn Ahlroos <john@vaadin.com>
Fri, 31 Aug 2012 11:23:03 +0000 (14:23 +0300)
12 files changed:
client/src/com/vaadin/client/extensions/javascriptmanager/JavaScriptManagerConnector.java
client/src/com/vaadin/client/ui/MediaBaseConnector.java
client/src/com/vaadin/client/ui/audio/AudioConnector.java
server/src/com/vaadin/server/AbstractUIProvider.java
server/src/com/vaadin/server/DefaultUIProvider.java
server/src/com/vaadin/server/UIProvider.java
server/src/com/vaadin/ui/AbstractMedia.java
server/src/com/vaadin/ui/JavaScript.java
shared/src/com/vaadin/shared/JavaScriptExtensionState.java
shared/src/com/vaadin/shared/extension/javascriptmanager/JavaScriptManagerState.java
shared/src/com/vaadin/shared/ui/AbstractMediaState.java
uitest/src/com/vaadin/tests/components/loginform/LoginFormUIInLoginHandler.java

index 7cab90a90de9b9fd0ef386741fed305786f83251..f6d643d1ba90349c5d6e90c4d1703da73b3e3352 100644 (file)
@@ -1,4 +1,4 @@
-/* 
+/*
  * Copyright 2011 Vaadin Ltd.
  * 
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not
@@ -48,7 +48,7 @@ public class JavaScriptManagerConnector extends AbstractExtensionConnector {
     public void onStateChanged(StateChangeEvent stateChangeEvent) {
         super.onStateChanged(stateChangeEvent);
 
-        Set<String> newNames = getState().getNames();
+        Set<String> newNames = getState().names;
 
         // Current names now only contains orphan callbacks
         currentNames.removeAll(newNames);
index 8a17e052ba6deb800edc279c07a03a56fe4ccc6c..961ba4821e2e91e91f24f8e5b29fbb4ce36cb31c 100644 (file)
@@ -49,12 +49,12 @@ public abstract class MediaBaseConnector extends AbstractComponentConnector {
     public void onStateChanged(StateChangeEvent stateChangeEvent) {
         super.onStateChanged(stateChangeEvent);
 
-        for (int i = 0; i < getState().getSources().size(); i++) {
-            URLReference source = getState().getSources().get(i);
-            String sourceType = getState().getSourceTypes().get(i);
+        for (int i = 0; i < getState().sources.size(); i++) {
+            URLReference source = getState().sources.get(i);
+            String sourceType = getState().sourceTypes.get(i);
             getWidget().addSource(source.getURL(), sourceType);
         }
-        setAltText(getState().getAltText());
+        setAltText(getState().altText);
     }
 
     @Override
@@ -66,7 +66,7 @@ public abstract class MediaBaseConnector extends AbstractComponentConnector {
 
         if (altText == null || "".equals(altText)) {
             altText = getDefaultAltHtml();
-        } else if (!getState().isHtmlContentAllowed()) {
+        } else if (!getState().htmlContentAllowed) {
             altText = Util.escapeHTML(altText);
         }
         getWidget().setAltText(altText);
index 8c2fa179c89f58cce55d741553ce6e6608d0ffa0..bfa282ca09ba875db6ea7d5376f79196a7c28c48 100644 (file)
@@ -35,7 +35,7 @@ public class AudioConnector extends MediaBaseConnector {
         Style style = getWidget().getElement().getStyle();
 
         // Make sure that the controls are not clipped if visible.
-        if (getState().isShowControls()
+        if (getState().showControls
                 && (style.getHeight() == null || "".equals(style.getHeight()))) {
             if (BrowserInfo.get().isChrome()) {
                 style.setHeight(32, Unit.PX);
index 07b95fc7130c14e3f4c2783605f354873435c82e..f383368d9142e32b60a23d9ec7027eb235b8473d 100644 (file)
@@ -1,35 +1,35 @@
-/*\r
- * Copyright 2011 Vaadin Ltd.\r
- * \r
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not\r
- * use this file except in compliance with the License. You may obtain a copy of\r
- * the License at\r
- * \r
- * http://www.apache.org/licenses/LICENSE-2.0\r
- * \r
- * Unless required by applicable law or agreed to in writing, software\r
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT\r
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the\r
- * License for the specific language governing permissions and limitations under\r
- * the License.\r
- */\r
-\r
-package com.vaadin.server;\r
-\r
-import com.vaadin.Application;\r
-import com.vaadin.ui.UI;\r
-\r
-public abstract class AbstractUIProvider implements UIProvider {\r
-\r
-    @Override\r
-    public UI instantiateUI(Application application, Class<? extends UI> type,\r
-            WrappedRequest request) {\r
-        try {\r
-            return type.newInstance();\r
-        } catch (InstantiationException e) {\r
-            throw new RuntimeException("Could not instantiate root class", e);\r
-        } catch (IllegalAccessException e) {\r
-            throw new RuntimeException("Could not access root class", e);\r
-        }\r
-    }\r
-}\r
+/*
+ * Copyright 2011 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.server;
+
+import com.vaadin.Application;
+import com.vaadin.ui.UI;
+
+public abstract class AbstractUIProvider implements UIProvider {
+
+    @Override
+    public UI instantiateUI(Application application, Class<? extends UI> type,
+            WrappedRequest request) {
+        try {
+            return type.newInstance();
+        } catch (InstantiationException e) {
+            throw new RuntimeException("Could not instantiate root class", e);
+        } catch (IllegalAccessException e) {
+            throw new RuntimeException("Could not access root class", e);
+        }
+    }
+}
index 913402c89f46fcc82030383d4c1d587b1ab32830..d4490b164236e3822d04ae1c96e77d3be984d842 100644 (file)
@@ -1,51 +1,51 @@
-/*\r
- * Copyright 2011 Vaadin Ltd.\r
- * \r
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not\r
- * use this file except in compliance with the License. You may obtain a copy of\r
- * the License at\r
- * \r
- * http://www.apache.org/licenses/LICENSE-2.0\r
- * \r
- * Unless required by applicable law or agreed to in writing, software\r
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT\r
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the\r
- * License for the specific language governing permissions and limitations under\r
- * the License.\r
- */\r
-\r
-package com.vaadin.server;\r
-\r
-import com.vaadin.Application;\r
-import com.vaadin.UIRequiresMoreInformationException;\r
-import com.vaadin.ui.UI;\r
-\r
-public class DefaultUIProvider extends AbstractUIProvider {\r
-\r
-    @Override\r
-    public Class<? extends UI> getUIClass(Application application,\r
-            WrappedRequest request) throws UIRequiresMoreInformationException {\r
-        Object uiClassNameObj = application\r
-                .getProperty(Application.UI_PARAMETER);\r
-\r
-        if (uiClassNameObj instanceof String) {\r
-            String uiClassName = uiClassNameObj.toString();\r
-\r
-            ClassLoader classLoader = request.getDeploymentConfiguration()\r
-                    .getClassLoader();\r
-            if (classLoader == null) {\r
-                classLoader = getClass().getClassLoader();\r
-            }\r
-            try {\r
-                Class<? extends UI> uiClass = Class.forName(uiClassName, true,\r
-                        classLoader).asSubclass(UI.class);\r
-\r
-                return uiClass;\r
-            } catch (ClassNotFoundException e) {\r
-                throw new RuntimeException("Could not find UI class", e);\r
-            }\r
-        }\r
-\r
-        return null;\r
-    }\r
-}\r
+/*
+ * Copyright 2011 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.server;
+
+import com.vaadin.Application;
+import com.vaadin.UIRequiresMoreInformationException;
+import com.vaadin.ui.UI;
+
+public class DefaultUIProvider extends AbstractUIProvider {
+
+    @Override
+    public Class<? extends UI> getUIClass(Application application,
+            WrappedRequest request) throws UIRequiresMoreInformationException {
+        Object uiClassNameObj = application
+                .getProperty(Application.UI_PARAMETER);
+
+        if (uiClassNameObj instanceof String) {
+            String uiClassName = uiClassNameObj.toString();
+
+            ClassLoader classLoader = request.getDeploymentConfiguration()
+                    .getClassLoader();
+            if (classLoader == null) {
+                classLoader = getClass().getClassLoader();
+            }
+            try {
+                Class<? extends UI> uiClass = Class.forName(uiClassName, true,
+                        classLoader).asSubclass(UI.class);
+
+                return uiClass;
+            } catch (ClassNotFoundException e) {
+                throw new RuntimeException("Could not find UI class", e);
+            }
+        }
+
+        return null;
+    }
+}
index f262a3e225261f33c81043ec3a66277c7782f6f9..890809fdd5779cfa689edb68fd4179c3c934ae5c 100644 (file)
@@ -1,29 +1,29 @@
-/*\r
- * Copyright 2011 Vaadin Ltd.\r
- * \r
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not\r
- * use this file except in compliance with the License. You may obtain a copy of\r
- * the License at\r
- * \r
- * http://www.apache.org/licenses/LICENSE-2.0\r
- * \r
- * Unless required by applicable law or agreed to in writing, software\r
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT\r
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the\r
- * License for the specific language governing permissions and limitations under\r
- * the License.\r
- */\r
-\r
-package com.vaadin.server;\r
-\r
-import com.vaadin.Application;\r
-import com.vaadin.UIRequiresMoreInformationException;\r
-import com.vaadin.ui.UI;\r
-\r
-public interface UIProvider {\r
-    public Class<? extends UI> getUIClass(Application application,\r
-            WrappedRequest request) throws UIRequiresMoreInformationException;\r
-\r
-    public UI instantiateUI(Application application, Class<? extends UI> type,\r
-            WrappedRequest request);\r
-}\r
+/*
+ * Copyright 2011 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.server;
+
+import com.vaadin.Application;
+import com.vaadin.UIRequiresMoreInformationException;
+import com.vaadin.ui.UI;
+
+public interface UIProvider {
+    public Class<? extends UI> getUIClass(Application application,
+            WrappedRequest request) throws UIRequiresMoreInformationException;
+
+    public UI instantiateUI(Application application, Class<? extends UI> type,
+            WrappedRequest request);
+}
index 3f59b070b7b13a62722097f2c9b1940de8928fbd..219413974b4408d1872e0f037ed95c201130bc5f 100644 (file)
@@ -56,8 +56,8 @@ public abstract class AbstractMedia extends AbstractComponent {
     }
 
     private void clearSources() {
-        getState().getSources().clear();
-        getState().getSourceTypes().clear();
+        getState().sources.clear();
+        getState().sourceTypes.clear();
     }
 
     /**
@@ -71,10 +71,10 @@ public abstract class AbstractMedia extends AbstractComponent {
      */
     public void addSource(Resource source) {
         if (source != null) {
-            List<URLReference> sources = getState().getSources();
+            List<URLReference> sources = getState().sources;
             sources.add(new ResourceReference(source, this, Integer
                     .toString(sources.size())));
-            getState().getSourceTypes().add(source.getMIMEType());
+            getState().sourceTypes.add(source.getMIMEType());
         }
     }
 
@@ -83,7 +83,7 @@ public abstract class AbstractMedia extends AbstractComponent {
             WrappedResponse response, String path) throws IOException {
         Matcher matcher = Pattern.compile("(\\d+)(/.*)?").matcher(path);
         if (matcher.matches()) {
-            List<URLReference> sources = getState().getSources();
+            List<URLReference> sources = getState().sources;
 
             int sourceIndex = Integer.parseInt(matcher.group(1));
 
@@ -128,7 +128,7 @@ public abstract class AbstractMedia extends AbstractComponent {
      */
     public List<Resource> getSources() {
         ArrayList<Resource> sources = new ArrayList<Resource>();
-        for (URLReference ref : getState().getSources()) {
+        for (URLReference ref : getState().sources) {
             sources.add(((ResourceReference) ref).getResource());
         }
         return sources;
@@ -140,14 +140,14 @@ public abstract class AbstractMedia extends AbstractComponent {
      * @param showControls
      */
     public void setShowControls(boolean showControls) {
-        getState().setShowControls(showControls);
+        getState().showControls = showControls;
     }
 
     /**
      * @return true if the browser is to show native media controls.
      */
     public boolean isShowControls() {
-        return getState().isShowControls();
+        return getState().showControls;
     }
 
     /**
@@ -162,7 +162,7 @@ public abstract class AbstractMedia extends AbstractComponent {
      * @param altText
      */
     public void setAltText(String altText) {
-        getState().setAltText(altText);
+        getState().altText = altText;
     }
 
     /**
@@ -170,7 +170,7 @@ public abstract class AbstractMedia extends AbstractComponent {
      *         HTML5.
      */
     public String getAltText() {
-        return getState().getAltText();
+        return getState().altText;
     }
 
     /**
@@ -180,7 +180,7 @@ public abstract class AbstractMedia extends AbstractComponent {
      * @param htmlContentAllowed
      */
     public void setHtmlContentAllowed(boolean htmlContentAllowed) {
-        getState().setHtmlContentAllowed(htmlContentAllowed);
+        getState().htmlContentAllowed = htmlContentAllowed;
     }
 
     /**
@@ -188,7 +188,7 @@ public abstract class AbstractMedia extends AbstractComponent {
      *         be rendered as HTML.
      */
     public boolean isHtmlContentAllowed() {
-        return getState().isHtmlContentAllowed();
+        return getState().htmlContentAllowed;
     }
 
     /**
@@ -198,14 +198,14 @@ public abstract class AbstractMedia extends AbstractComponent {
      * @param autoplay
      */
     public void setAutoplay(boolean autoplay) {
-        getState().setAutoplay(autoplay);
+        getState().autoplay = autoplay;
     }
 
     /**
      * @return true if the media is set to automatically start playback.
      */
     public boolean isAutoplay() {
-        return getState().isAutoplay();
+        return getState().autoplay;
     }
 
     /**
@@ -214,14 +214,14 @@ public abstract class AbstractMedia extends AbstractComponent {
      * @param muted
      */
     public void setMuted(boolean muted) {
-        getState().setMuted(muted);
+        getState().muted = muted;
     }
 
     /**
      * @return true if the audio is muted.
      */
     public boolean isMuted() {
-        return getState().isMuted();
+        return getState().muted;
     }
 
     /**
index f3e8564fabfb14708032a14ee0fcf5d63226ca4f..f9324ba32120c7d95f2b03f61a8d9713541c5c95 100644 (file)
@@ -1,4 +1,4 @@
-/* 
+/*
  * Copyright 2011 Vaadin Ltd.
  * 
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not
@@ -93,7 +93,7 @@ public class JavaScript extends AbstractExtension {
      */
     public void addFunction(String name, JavaScriptFunction function) {
         functions.put(name, function);
-        getState().getNames().add(name);
+        getState().names.add(name);
     }
 
     /**
@@ -109,7 +109,7 @@ public class JavaScript extends AbstractExtension {
      */
     public void removeFunction(String name) {
         functions.remove(name);
-        getState().getNames().remove(name);
+        getState().names.remove(name);
     }
 
     /**
index 3e97294f7d1b2bebd5566035ee72e48ac359c805..fb864c6ee114b28feb57040d6f61cb307f40ae4a 100644 (file)
@@ -1,4 +1,4 @@
-/* 
+/*
  * Copyright 2011 Vaadin Ltd.
  * 
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not
index 7d1938d735eedbad2146c0fe1a6bcca2da8bdada..71af5e9598e1e3b4ac0f43748b47da727e744084 100644 (file)
@@ -1,4 +1,4 @@
-/* 
+/*
  * Copyright 2011 Vaadin Ltd.
  * 
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not
@@ -22,13 +22,5 @@ import java.util.Set;
 import com.vaadin.shared.communication.SharedState;
 
 public class JavaScriptManagerState extends SharedState {
-    private Set<String> names = new HashSet<String>();
-
-    public Set<String> getNames() {
-        return names;
-    }
-
-    public void setNames(Set<String> names) {
-        this.names = names;
-    }
+    public Set<String> names = new HashSet<String>();
 }
index 80d41dd7976f15a43af9bea0a1264697d87c6f0f..76d4e1b00008904caa46f8e2fd232e7332bf13ea 100644 (file)
@@ -19,80 +19,20 @@ import java.util.ArrayList;
 import java.util.List;
 
 import com.vaadin.shared.ComponentState;
-import com.vaadin.shared.annotations.DelegateToWidget;
 import com.vaadin.shared.communication.URLReference;
 
 public class AbstractMediaState extends ComponentState {
-    private boolean showControls;
+    public boolean showControls;
 
-    private String altText;
+    public String altText;
 
-    private boolean htmlContentAllowed;
+    public boolean htmlContentAllowed;
 
-    private boolean autoplay;
+    public boolean autoplay;
 
-    private boolean muted;
+    public boolean muted;
 
-    private List<URLReference> sources = new ArrayList<URLReference>();
-    private List<String> sourceTypes = new ArrayList<String>();
-
-    public boolean isShowControls() {
-        return showControls;
-    }
-
-    @DelegateToWidget("setControls")
-    public void setShowControls(boolean showControls) {
-        this.showControls = showControls;
-    }
-
-    public String getAltText() {
-        return altText;
-    }
-
-    public void setAltText(String altText) {
-        this.altText = altText;
-    }
-
-    public boolean isHtmlContentAllowed() {
-        return htmlContentAllowed;
-    }
-
-    public void setHtmlContentAllowed(boolean htmlContentAllowed) {
-        this.htmlContentAllowed = htmlContentAllowed;
-    }
-
-    public boolean isAutoplay() {
-        return autoplay;
-    }
-
-    @DelegateToWidget
-    public void setAutoplay(boolean autoplay) {
-        this.autoplay = autoplay;
-    }
-
-    public boolean isMuted() {
-        return muted;
-    }
-
-    @DelegateToWidget
-    public void setMuted(boolean muted) {
-        this.muted = muted;
-    }
-
-    public List<URLReference> getSources() {
-        return sources;
-    }
-
-    public void setSources(List<URLReference> sources) {
-        this.sources = sources;
-    }
-
-    public List<String> getSourceTypes() {
-        return sourceTypes;
-    }
-
-    public void setSourceTypes(List<String> sourceTypes) {
-        this.sourceTypes = sourceTypes;
-    }
+    public List<URLReference> sources = new ArrayList<URLReference>();
 
+    public List<String> sourceTypes = new ArrayList<String>();
 }
index b3ebb02751596b1a19c03032afc11dfbead6b76a..58656cb7bf4187b191804f65c26a36166f2f09bf 100755 (executable)
@@ -1,52 +1,52 @@
-package com.vaadin.tests.components.loginform;\r
-\r
-import com.vaadin.tests.components.TestBase;\r
-import com.vaadin.ui.Label;\r
-import com.vaadin.ui.LoginForm;\r
-import com.vaadin.ui.LoginForm.LoginEvent;\r
-import com.vaadin.ui.LoginForm.LoginListener;\r
-import com.vaadin.ui.UI;\r
-\r
-public class LoginFormUIInLoginHandler extends TestBase {\r
-\r
-    @Override\r
-    protected void setup() {\r
-        LoginForm lf = new LoginForm();\r
-        lf.addListener(new LoginListener() {\r
-\r
-            @Override\r
-            public void onLogin(LoginEvent event) {\r
-                UI r1 = UI.getCurrent();\r
-                if (r1 != null) {\r
-                    addComponent(new Label("UI.getCurrent().data: "\r
-                            + r1.getData()));\r
-                } else {\r
-                    addComponent(new Label("UI.getCurrent() is null"));\r
-                }\r
-                UI r2 = ((LoginForm) event.getSource()).getUI();\r
-                if (r2 != null) {\r
-                    addComponent(new Label("event.getSource().data: "\r
-                            + r2.getData()));\r
-                } else {\r
-                    addComponent(new Label(\r
-                            "event.getSource().getRoot() is null"));\r
-                }\r
-            }\r
-        });\r
-        addComponent(lf);\r
-        getLayout().getUI().setData("This UI");\r
-    }\r
-\r
-    @Override\r
-    protected String getDescription() {\r
-        // TODO Auto-generated method stub\r
-        return null;\r
-    }\r
-\r
-    @Override\r
-    protected Integer getTicketNumber() {\r
-        // TODO Auto-generated method stub\r
-        return null;\r
-    }\r
-\r
-}\r
+package com.vaadin.tests.components.loginform;
+
+import com.vaadin.tests.components.TestBase;
+import com.vaadin.ui.Label;
+import com.vaadin.ui.LoginForm;
+import com.vaadin.ui.LoginForm.LoginEvent;
+import com.vaadin.ui.LoginForm.LoginListener;
+import com.vaadin.ui.UI;
+
+public class LoginFormUIInLoginHandler extends TestBase {
+
+    @Override
+    protected void setup() {
+        LoginForm lf = new LoginForm();
+        lf.addListener(new LoginListener() {
+
+            @Override
+            public void onLogin(LoginEvent event) {
+                UI r1 = UI.getCurrent();
+                if (r1 != null) {
+                    addComponent(new Label("UI.getCurrent().data: "
+                            + r1.getData()));
+                } else {
+                    addComponent(new Label("UI.getCurrent() is null"));
+                }
+                UI r2 = ((LoginForm) event.getSource()).getUI();
+                if (r2 != null) {
+                    addComponent(new Label("event.getSource().data: "
+                            + r2.getData()));
+                } else {
+                    addComponent(new Label(
+                            "event.getSource().getRoot() is null"));
+                }
+            }
+        });
+        addComponent(lf);
+        getLayout().getUI().setData("This UI");
+    }
+
+    @Override
+    protected String getDescription() {
+        // TODO Auto-generated method stub
+        return null;
+    }
+
+    @Override
+    protected Integer getTicketNumber() {
+        // TODO Auto-generated method stub
+        return null;
+    }
+
+}