]> source.dussan.org Git - vaadin-framework.git/commitdiff
inner Event classes converted to static nested classes (#5855)
authorPekka Hyvönen <pekka@vaadin.com>
Tue, 4 Sep 2012 07:33:22 +0000 (10:33 +0300)
committerLeif Åstrand <leif@vaadin.com>
Tue, 4 Sep 2012 07:36:39 +0000 (10:36 +0300)
15 files changed:
server/src/com/vaadin/data/util/sqlcontainer/query/TableQuery.java
server/src/com/vaadin/event/FieldEvents.java
server/src/com/vaadin/event/MouseEvents.java
server/src/com/vaadin/server/Page.java
server/src/com/vaadin/ui/AbstractSelect.java
server/src/com/vaadin/ui/AbstractSplitPanel.java
server/src/com/vaadin/ui/AbstractTextField.java
server/src/com/vaadin/ui/Button.java
server/src/com/vaadin/ui/Component.java
server/src/com/vaadin/ui/ComponentContainer.java
server/src/com/vaadin/ui/LoginForm.java
server/src/com/vaadin/ui/PopupView.java
server/src/com/vaadin/ui/TabSheet.java
server/src/com/vaadin/ui/Window.java
server/tests/src/com/vaadin/tests/server/navigator/UriFragmentManagerTest.java

index a6a1a5a8fb8a3ce7033a8f28faec6bd75a9766b3..8a3dabbf0e6377f090a21542f239a5743fd9ffaf 100644 (file)
@@ -678,7 +678,7 @@ public class TableQuery implements QueryDelegate,
     /**
      * Simple RowIdChangeEvent implementation.
      */
-    public class RowIdChangeEvent extends EventObject implements
+    public static class RowIdChangeEvent extends EventObject implements
             QueryDelegate.RowIdChangeEvent {
         private final RowId oldId;
         private final RowId newId;
index 3da60ee46692d21e921e4c7b2c832119882af240..1a1bc95305211431461387b771a39b3929582728 100644 (file)
@@ -144,7 +144,7 @@ public interface FieldEvents {
      * @since 6.2
      */
     @SuppressWarnings("serial")
-    public class FocusEvent extends Component.Event {
+    public static class FocusEvent extends Component.Event {
 
         /**
          * Identifier for event that can be used in {@link EventRouter}
@@ -184,7 +184,7 @@ public interface FieldEvents {
      * @since 6.2
      */
     @SuppressWarnings("serial")
-    public class BlurEvent extends Component.Event {
+    public static class BlurEvent extends Component.Event {
 
         /**
          * Identifier for event that can be used in {@link EventRouter}
@@ -243,6 +243,7 @@ public interface FieldEvents {
      * @since 6.5
      */
     public static abstract class TextChangeEvent extends Component.Event {
+
         public TextChangeEvent(Component source) {
             super(source);
         }
index 6712ef79a59a8a142c18ed866cc60b7f2df1633d..e287055c2b383a8084c39d643d39bec078261133 100644 (file)
@@ -43,7 +43,7 @@ public interface MouseEvents {
      * @see ClickListener
      * @since 6.2
      */
-    public class ClickEvent extends Component.Event {
+    public static class ClickEvent extends Component.Event {
         public static final int BUTTON_LEFT = MouseEventDetails.BUTTON_LEFT;
         public static final int BUTTON_MIDDLE = MouseEventDetails.BUTTON_MIDDLE;
         public static final int BUTTON_RIGHT = MouseEventDetails.BUTTON_RIGHT;
@@ -202,7 +202,7 @@ public interface MouseEvents {
      * @author Vaadin Ltd.
      * @since 6.2
      */
-    public class DoubleClickEvent extends Component.Event {
+    public static class DoubleClickEvent extends Component.Event {
 
         public DoubleClickEvent(Component source) {
             super(source);
index b5718d10bb444c489eb601e9d55ae25ad2c1e628..015c6c907f91031452a2134291422b3e9ec548ac 100644 (file)
@@ -54,7 +54,7 @@ public class Page implements Serializable {
     /**
      * Event that is fired when a browser window containing a uI is resized.
      */
-    public class BrowserWindowResizeEvent extends EventObject {
+    public static class BrowserWindowResizeEvent extends EventObject {
 
         private final int width;
         private final int height;
@@ -233,7 +233,7 @@ public class Page implements Serializable {
     /**
      * Event fired when uri fragment changes.
      */
-    public class FragmentChangedEvent extends EventObject {
+    public static class FragmentChangedEvent extends EventObject {
 
         /**
          * The new uri fragment
index 2fc3bf40805b1af03a702880a23ff32ac45a2978..2f420b928666f95222dcc15ef1875c10916bfcb6 100644 (file)
@@ -19,6 +19,7 @@ package com.vaadin.ui;
 import java.io.Serializable;
 import java.util.Collection;
 import java.util.Collections;
+import java.util.EventObject;
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Iterator;
@@ -1612,7 +1613,8 @@ public abstract class AbstractSelect extends AbstractField<Object> implements
     protected void firePropertySetChange() {
         if (propertySetEventListeners != null
                 && !propertySetEventListeners.isEmpty()) {
-            final Container.PropertySetChangeEvent event = new PropertySetChangeEvent();
+            final Container.PropertySetChangeEvent event = new PropertySetChangeEvent(
+                    this);
             final Object[] listeners = propertySetEventListeners.toArray();
             for (int i = 0; i < listeners.length; i++) {
                 ((Container.PropertySetChangeListener) listeners[i])
@@ -1627,7 +1629,8 @@ public abstract class AbstractSelect extends AbstractField<Object> implements
      */
     protected void fireItemSetChange() {
         if (itemSetEventListeners != null && !itemSetEventListeners.isEmpty()) {
-            final Container.ItemSetChangeEvent event = new ItemSetChangeEvent();
+            final Container.ItemSetChangeEvent event = new ItemSetChangeEvent(
+                    this);
             final Object[] listeners = itemSetEventListeners.toArray();
             for (int i = 0; i < listeners.length; i++) {
                 ((Container.ItemSetChangeListener) listeners[i])
@@ -1640,8 +1643,12 @@ public abstract class AbstractSelect extends AbstractField<Object> implements
     /**
      * Implementation of item set change event.
      */
-    private class ItemSetChangeEvent implements Serializable,
-            Container.ItemSetChangeEvent {
+    private static class ItemSetChangeEvent extends EventObject implements
+            Serializable, Container.ItemSetChangeEvent {
+
+        private ItemSetChangeEvent(Container source) {
+            super(source);
+        }
 
         /**
          * Gets the Property where the event occurred.
@@ -1650,7 +1657,7 @@ public abstract class AbstractSelect extends AbstractField<Object> implements
          */
         @Override
         public Container getContainer() {
-            return AbstractSelect.this;
+            return (Container) getSource();
         }
 
     }
@@ -1658,9 +1665,13 @@ public abstract class AbstractSelect extends AbstractField<Object> implements
     /**
      * Implementation of property set change event.
      */
-    private class PropertySetChangeEvent implements
+    private static class PropertySetChangeEvent extends EventObject implements
             Container.PropertySetChangeEvent, Serializable {
 
+        private PropertySetChangeEvent(Container source) {
+            super(source);
+        }
+
         /**
          * Retrieves the Container whose contents have been modified.
          * 
@@ -1668,7 +1679,7 @@ public abstract class AbstractSelect extends AbstractField<Object> implements
          */
         @Override
         public Container getContainer() {
-            return AbstractSelect.this;
+            return (Container) getSource();
         }
 
     }
index 93c4ec4f59b72cbe7b8eda47b054c4f520f1b225..dd6ff50efbf8ac326bcfe3cb4120a47027ef8725 100644 (file)
@@ -392,8 +392,8 @@ public abstract class AbstractSplitPanel extends AbstractComponentContainer {
      *            Allowed units are UNITS_PERCENTAGE and UNITS_PIXELS
      */
     public void setMaxSplitPosition(float pos, Unit unit) {
-        setSplitPositionLimits(getSplitterState().minPosition, posMinUnit,
-                pos, unit);
+        setSplitPositionLimits(getSplitterState().minPosition, posMinUnit, pos,
+                unit);
     }
 
     /**
@@ -492,7 +492,7 @@ public abstract class AbstractSplitPanel extends AbstractComponentContainer {
         public void splitterClick(SplitterClickEvent event);
     }
 
-    public class SplitterClickEvent extends ClickEvent {
+    public static class SplitterClickEvent extends ClickEvent {
 
         public SplitterClickEvent(Component source,
                 MouseEventDetails mouseEventDetails) {
index 0cb74dae6d64c86d44c954bd5cac06b2bce4b14e..3dd2b4dae8e78432ebe0003c3acbb116a3d9082e 100644 (file)
@@ -570,7 +570,7 @@ public abstract class AbstractTextField extends AbstractField<String> implements
         return textChangeEventTimeout;
     }
 
-    public class TextChangeEventImpl extends TextChangeEvent {
+    public static class TextChangeEventImpl extends TextChangeEvent {
         private String curText;
         private int cursorPosition;
 
index ec75d2da9b3b7455b0c8f59ed305592fb21915a9..02b768925975e4361d3985b3e0675cbe3b8aedff 100644 (file)
@@ -108,7 +108,7 @@ public class Button extends AbstractComponent implements
      * @author Vaadin Ltd.
      * @since 3.0
      */
-    public class ClickEvent extends Component.Event {
+    public static class ClickEvent extends Component.Event {
 
         private final MouseEventDetails details;
 
index 01033ff560f7c6bfda85539cb612547ccf245e38..5bd1d53b86d41a4c88c5cf3e06aa5f5785668383 100644 (file)
@@ -670,7 +670,7 @@ public interface Component extends ClientConnector, Sizeable, Serializable {
      * @see Component.Listener
      */
     @SuppressWarnings("serial")
-    public class Event extends EventObject {
+    public static class Event extends EventObject {
 
         /**
          * Constructs a new event with the specified source component.
@@ -860,7 +860,7 @@ public interface Component extends ClientConnector, Sizeable, Serializable {
      * </p>
      */
     @SuppressWarnings("serial")
-    public class ErrorEvent extends Event {
+    public static class ErrorEvent extends Event {
 
         private final ErrorMessage message;
 
index cee6d7300b5b9f77f7d8ea11e0341efb216d99dd..50a6e7e148b14cdd9cfbe8b06870018cb20b255a 100644 (file)
@@ -174,7 +174,7 @@ public interface ComponentContainer extends HasComponents {
      * Component attach event sent when a component is attached to container.
      */
     @SuppressWarnings("serial")
-    public class ComponentAttachEvent extends Component.Event {
+    public static class ComponentAttachEvent extends Component.Event {
 
         private final Component component;
 
@@ -218,7 +218,7 @@ public interface ComponentContainer extends HasComponents {
      * Component detach event sent when a component is detached from container.
      */
     @SuppressWarnings("serial")
-    public class ComponentDetachEvent extends Component.Event {
+    public static class ComponentDetachEvent extends Component.Event {
 
         private final Component component;
 
index abcc7ff6074eae2a73fd1764626a4481db7daa42..61846eab4e8c4d5631a23553d74ed953e6fd6120 100644 (file)
@@ -108,7 +108,7 @@ public class LoginForm extends CustomComponent {
                     String value = (parameters.get(key))[0];
                     params.put(key, value);
                 }
-                LoginEvent event = new LoginEvent(params);
+                LoginEvent event = new LoginEvent(LoginForm.this, params);
                 fireEvent(event);
                 return true;
             }
@@ -200,12 +200,12 @@ public class LoginForm extends CustomComponent {
     /**
      * This event is sent when login form is submitted.
      */
-    public class LoginEvent extends Event {
+    public static class LoginEvent extends Event {
 
         private Map<String, String> params;
 
-        private LoginEvent(Map<String, String> params) {
-            super(LoginForm.this);
+        private LoginEvent(Component source, Map<String, String> params) {
+            super(source);
             this.params = params;
         }
 
index 8d6d28e121b1da87190b30b266396d951c5421db..182e71d6c9dd9c2bd303f9c1e054068a46dcedce 100644 (file)
@@ -20,10 +20,10 @@ import java.lang.reflect.Method;
 import java.util.Iterator;
 import java.util.Map;
 
+import com.vaadin.server.LegacyComponent;
 import com.vaadin.server.LegacyPaint;
 import com.vaadin.server.PaintException;
 import com.vaadin.server.PaintTarget;
-import com.vaadin.server.LegacyComponent;
 
 /**
  * 
@@ -421,7 +421,7 @@ public class PopupView extends AbstractComponentContainer implements
      * event with {@link #getPopupView()}.
      * 
      */
-    public class PopupVisibilityEvent extends Event {
+    public static class PopupVisibilityEvent extends Event {
 
         public PopupVisibilityEvent(PopupView source) {
             super(source);
index eb66de519f4826e9a9c12bbe72f116877b44cddd..291aea3bd64cccf749e18432e6ffeb47b407dfc4 100644 (file)
@@ -32,11 +32,11 @@ import com.vaadin.event.FieldEvents.FocusListener;
 import com.vaadin.event.FieldEvents.FocusNotifier;
 import com.vaadin.server.ErrorMessage;
 import com.vaadin.server.KeyMapper;
+import com.vaadin.server.LegacyComponent;
 import com.vaadin.server.LegacyPaint;
 import com.vaadin.server.PaintException;
 import com.vaadin.server.PaintTarget;
 import com.vaadin.server.Resource;
-import com.vaadin.server.LegacyComponent;
 import com.vaadin.shared.ui.tabsheet.TabsheetBaseConstants;
 import com.vaadin.shared.ui.tabsheet.TabsheetConstants;
 import com.vaadin.ui.Component.Focusable;
@@ -743,7 +743,7 @@ public class TabSheet extends AbstractComponentContainer implements Focusable,
      * @author Vaadin Ltd.
      * @since 3.0
      */
-    public class SelectedTabChangeEvent extends Component.Event {
+    public static class SelectedTabChangeEvent extends Component.Event {
 
         /**
          * New instance of selected tab change event
index 5c94a4c929f5adafde73fb190d76911b0ff1a87b..d7a33d742c8a348ca2d241bea78727dbd7bfc035 100644 (file)
@@ -299,7 +299,7 @@ public class Window extends Panel implements FocusNotifier, BlurNotifier,
         }
     }
 
-    public class CloseEvent extends Component.Event {
+    public static class CloseEvent extends Component.Event {
 
         /**
          * 
@@ -422,7 +422,7 @@ public class Window extends Panel implements FocusNotifier, BlurNotifier,
      * (e.g. the browser window is resized). The frequency may vary across
      * browsers.
      */
-    public class ResizeEvent extends Component.Event {
+    public static class ResizeEvent extends Component.Event {
 
         /**
          * 
index d58ff28b0054da98e112d0ae402b480111f510f8..905ddb529b09df03e1b0aaddfb12bfd116dd10fc 100644 (file)
@@ -56,8 +56,7 @@ public class UriFragmentManagerTest extends TestCase {
         navigator.navigateTo("test");
         control.replay();
 
-        FragmentChangedEvent event = page.new FragmentChangedEvent(page,
-                "oldtest");
+        FragmentChangedEvent event = new FragmentChangedEvent(page, "oldtest");
         manager.fragmentChanged(event);
     }
 }