]> source.dussan.org Git - vaadin-framework.git/commitdiff
Fix VaadinService.findUI to throw Error if UIConstants.UI_ID_PARAMETER
authormtzukanov <mtzukanov@vaadin.com>
Fri, 28 Jun 2013 12:52:48 +0000 (15:52 +0300)
committerVaadin Code Review <review@vaadin.com>
Fri, 28 Jun 2013 13:09:35 +0000 (13:09 +0000)
is not set (#11943)

Change-Id: I3cbb88e3fab4f39af8e4799d5a914ab9bd4733f5

server/src/com/vaadin/server/VaadinService.java

index 18dc468cb0282acd6cd375de29f0b28636672a3d..cfbf2606aec0c30b1477254e3645fd7e68a5a62d 100644 (file)
@@ -905,7 +905,7 @@ public abstract class VaadinService implements Serializable {
      * 
      * @param request
      *            the request for which a UI is desired
-     * @return the UI belonging to the request
+     * @return the UI belonging to the request or null if no UI is found
      * 
      */
     public UI findUI(VaadinRequest request) {
@@ -915,9 +915,11 @@ public abstract class VaadinService implements Serializable {
 
         // Get UI id from the request
         String uiIdString = request.getParameter(UIConstants.UI_ID_PARAMETER);
-        int uiId = Integer.parseInt(uiIdString);
-
-        UI ui = session.getUIById(uiId);
+        UI ui = null;
+        if (uiIdString != null) {
+            int uiId = Integer.parseInt(uiIdString);
+            ui = session.getUIById(uiId);
+        }
 
         UI.setCurrent(ui);
         return ui;