]> source.dussan.org Git - vaadin-framework.git/commitdiff
Rename DeploymentConfiguration -> VaadinService (#9402)
authorLeif Åstrand <leif@vaadin.com>
Thu, 6 Sep 2012 07:18:56 +0000 (10:18 +0300)
committerLeif Åstrand <leif@vaadin.com>
Thu, 6 Sep 2012 07:20:42 +0000 (10:20 +0300)
32 files changed:
server/src/com/vaadin/server/AbstractCommunicationManager.java
server/src/com/vaadin/server/AbstractDeploymentConfiguration.java [deleted file]
server/src/com/vaadin/server/AbstractVaadinService.java [new file with mode: 0644]
server/src/com/vaadin/server/AddonContext.java
server/src/com/vaadin/server/BootstrapHandler.java
server/src/com/vaadin/server/CombinedRequest.java
server/src/com/vaadin/server/DefaultUIProvider.java
server/src/com/vaadin/server/DeploymentConfiguration.java [deleted file]
server/src/com/vaadin/server/GAEVaadinServlet.java
server/src/com/vaadin/server/LegacyVaadinPortlet.java
server/src/com/vaadin/server/LegacyVaadinServlet.java
server/src/com/vaadin/server/PortletCommunicationManager.java
server/src/com/vaadin/server/ServletPortletHelper.java
server/src/com/vaadin/server/VaadinPortlet.java
server/src/com/vaadin/server/VaadinPortletSession.java
server/src/com/vaadin/server/VaadinService.java [new file with mode: 0644]
server/src/com/vaadin/server/VaadinServlet.java
server/src/com/vaadin/server/VaadinSession.java
server/src/com/vaadin/server/WrappedHttpServletRequest.java
server/src/com/vaadin/server/WrappedHttpServletResponse.java
server/src/com/vaadin/server/WrappedPortletRequest.java
server/src/com/vaadin/server/WrappedPortletResponse.java
server/src/com/vaadin/server/WrappedRequest.java
server/src/com/vaadin/server/WrappedResponse.java
server/tests/src/com/vaadin/data/util/AbstractBeanContainerTest.java
server/tests/src/com/vaadin/server/TestAbstractApplicationServletStaticFilesLocation.java
server/tests/src/com/vaadin/tests/server/component/root/CustomUIClassLoader.java
shared/src/com/vaadin/shared/ui/link/LinkConstants.java
uitest/src/com/vaadin/launcher/ApplicationRunnerServlet.java
uitest/src/com/vaadin/tests/resources/NonExistingFileResource.java
uitest/src/com/vaadin/tests/tickets/Ticket1975.java
uitest/src/com/vaadin/tests/util/SampleDirectory.java

index 4f408d6fe97f273c3fbac01d7fea0866fcc7f8b5..9feab9277d1814c53b4fd75ddc532364a1178f8b 100644 (file)
@@ -571,7 +571,7 @@ public abstract class AbstractCommunicationManager implements Serializable {
             if (!handleVariables(request, response, callback, application, uI)) {
 
                 // var inconsistency; the client is probably out-of-sync
-                SystemMessages ci = response.getDeploymentConfiguration()
+                SystemMessages ci = response.getVaadinService()
                         .getSystemMessages();
                 String msg = ci.getOutOfSyncMessage();
                 String cap = ci.getOutOfSyncCaption();
@@ -1023,8 +1023,7 @@ public abstract class AbstractCommunicationManager implements Serializable {
                 }
             }
 
-            SystemMessages ci = request.getDeploymentConfiguration()
-                    .getSystemMessages();
+            SystemMessages ci = request.getVaadinService().getSystemMessages();
 
             // meta instruction for client to enable auto-forward to
             // sessionExpiredURL after timer expires.
@@ -2483,8 +2482,8 @@ public abstract class AbstractCommunicationManager implements Serializable {
                 .substring(ApplicationConstants.CONNECTOR_RESOURCE_PREFIX
                         .length() + 2);
 
-        final String mimetype = response.getDeploymentConfiguration()
-                .getMimeType(resourceName);
+        final String mimetype = response.getVaadinService().getMimeType(
+                resourceName);
 
         // Security check: avoid accidentally serving from the UI of the
         // classpath instead of relative to the context class
diff --git a/server/src/com/vaadin/server/AbstractDeploymentConfiguration.java b/server/src/com/vaadin/server/AbstractDeploymentConfiguration.java
deleted file mode 100644 (file)
index 3d7efa6..0000000
+++ /dev/null
@@ -1,81 +0,0 @@
-/* 
- * 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 java.lang.reflect.Constructor;
-import java.util.Iterator;
-import java.util.ServiceLoader;
-
-public abstract class AbstractDeploymentConfiguration implements
-        DeploymentConfiguration {
-
-    private AddonContext addonContext;
-    private final ApplicationConfiguration applicationConfiguration;
-
-    public AbstractDeploymentConfiguration(
-            ApplicationConfiguration applicationConfiguration) {
-        this.applicationConfiguration = applicationConfiguration;
-    }
-
-    @Override
-    public ApplicationConfiguration getApplicationConfiguration() {
-        return applicationConfiguration;
-    }
-
-    @Override
-    public ClassLoader getClassLoader() {
-        final String classLoaderName = getApplicationConfiguration()
-                .getApplicationOrSystemProperty("ClassLoader", null);
-        ClassLoader classLoader;
-        if (classLoaderName == null) {
-            classLoader = getClass().getClassLoader();
-        } else {
-            try {
-                final Class<?> classLoaderClass = getClass().getClassLoader()
-                        .loadClass(classLoaderName);
-                final Constructor<?> c = classLoaderClass
-                        .getConstructor(new Class[] { ClassLoader.class });
-                classLoader = (ClassLoader) c
-                        .newInstance(new Object[] { getClass().getClassLoader() });
-            } catch (final Exception e) {
-                throw new RuntimeException(
-                        "Could not find specified class loader: "
-                                + classLoaderName, e);
-            }
-        }
-        return classLoader;
-    }
-
-    @Override
-    public Iterator<AddonContextListener> getAddonContextListeners() {
-        // Called once for init and then no more, so there's no point in caching
-        // the instance
-        ServiceLoader<AddonContextListener> contextListenerLoader = ServiceLoader
-                .load(AddonContextListener.class, getClassLoader());
-        return contextListenerLoader.iterator();
-    }
-
-    @Override
-    public void setAddonContext(AddonContext addonContext) {
-        this.addonContext = addonContext;
-    }
-
-    @Override
-    public AddonContext getAddonContext() {
-        return addonContext;
-    }
-}
diff --git a/server/src/com/vaadin/server/AbstractVaadinService.java b/server/src/com/vaadin/server/AbstractVaadinService.java
new file mode 100644 (file)
index 0000000..516ce23
--- /dev/null
@@ -0,0 +1,80 @@
+/* 
+ * 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 java.lang.reflect.Constructor;
+import java.util.Iterator;
+import java.util.ServiceLoader;
+
+public abstract class AbstractVaadinService implements VaadinService {
+
+    private AddonContext addonContext;
+    private final ApplicationConfiguration applicationConfiguration;
+
+    public AbstractVaadinService(
+            ApplicationConfiguration applicationConfiguration) {
+        this.applicationConfiguration = applicationConfiguration;
+    }
+
+    @Override
+    public ApplicationConfiguration getApplicationConfiguration() {
+        return applicationConfiguration;
+    }
+
+    @Override
+    public ClassLoader getClassLoader() {
+        final String classLoaderName = getApplicationConfiguration()
+                .getApplicationOrSystemProperty("ClassLoader", null);
+        ClassLoader classLoader;
+        if (classLoaderName == null) {
+            classLoader = getClass().getClassLoader();
+        } else {
+            try {
+                final Class<?> classLoaderClass = getClass().getClassLoader()
+                        .loadClass(classLoaderName);
+                final Constructor<?> c = classLoaderClass
+                        .getConstructor(new Class[] { ClassLoader.class });
+                classLoader = (ClassLoader) c
+                        .newInstance(new Object[] { getClass().getClassLoader() });
+            } catch (final Exception e) {
+                throw new RuntimeException(
+                        "Could not find specified class loader: "
+                                + classLoaderName, e);
+            }
+        }
+        return classLoader;
+    }
+
+    @Override
+    public Iterator<AddonContextListener> getAddonContextListeners() {
+        // Called once for init and then no more, so there's no point in caching
+        // the instance
+        ServiceLoader<AddonContextListener> contextListenerLoader = ServiceLoader
+                .load(AddonContextListener.class, getClassLoader());
+        return contextListenerLoader.iterator();
+    }
+
+    @Override
+    public void setAddonContext(AddonContext addonContext) {
+        this.addonContext = addonContext;
+    }
+
+    @Override
+    public AddonContext getAddonContext() {
+        return addonContext;
+    }
+}
index 2b1cedc16278ead2b2764110a206539595ebd1ed..2cf9b23c756a1fe7a69f7c8e335910d6b45e510e 100644 (file)
@@ -36,8 +36,7 @@ import com.vaadin.util.ReflectTools;
  * META-INF/services/com.vaadin.server.AddonContextListener will be checked for
  * lines containing fully qualified names of classes to use. This behavior can
  * however be overridden for custom deployment situations (e.g. to use CDI or
- * OSGi) by overriding
- * {@link DeploymentConfiguration#getAddonContextListeners()}.
+ * OSGi) by overriding {@link VaadinService#getAddonContextListeners()}.
  * 
  * @author Vaadin Ltd
  * @since 7.0.0
@@ -47,7 +46,7 @@ public class AddonContext {
             .findMethod(ApplicationStartedListener.class, "applicationStarted",
                     ApplicationStartedEvent.class);
 
-    private final DeploymentConfiguration deploymentConfiguration;
+    private final VaadinService vaadinService;
 
     private final EventRouter eventRouter = new EventRouter();
 
@@ -59,13 +58,12 @@ public class AddonContext {
      * Creates a new context using a given deployment configuration. Only the
      * framework itself should typically create AddonContext methods.
      * 
-     * @param deploymentConfiguration
-     *            the deployment configuration for the associated servlet or
-     *            portlet.
+     * @param vaadinService
+     *            the vaadin service for the associated servlet or portlet.
      */
-    public AddonContext(DeploymentConfiguration deploymentConfiguration) {
-        this.deploymentConfiguration = deploymentConfiguration;
-        deploymentConfiguration.setAddonContext(this);
+    public AddonContext(VaadinService vaadinService) {
+        this.vaadinService = vaadinService;
+        vaadinService.setAddonContext(this);
     }
 
     /**
@@ -73,22 +71,21 @@ public class AddonContext {
      * 
      * @return the deployment configuration
      */
-    public DeploymentConfiguration getDeploymentConfiguration() {
-        return deploymentConfiguration;
+    public VaadinService getVaadinService() {
+        return vaadinService;
     }
 
     /**
      * Initializes this context, causing all found listeners to be notified.
      * Listeners are by default found using {@link ServiceLoader}, but the
-     * {@link DeploymentConfiguration} can provide an alternative
-     * implementation.
+     * {@link VaadinService} can provide an alternative implementation.
      * <p>
      * This method is not intended to be used by add-ons, but instead by the
      * part of the framework that created this context object.
      */
     public void init() {
         AddonContextEvent event = new AddonContextEvent(this);
-        Iterator<AddonContextListener> listeners = deploymentConfiguration
+        Iterator<AddonContextListener> listeners = vaadinService
                 .getAddonContextListeners();
         while (listeners.hasNext()) {
             AddonContextListener listener = listeners.next();
index 6b8fb1952aacdbc83c6cfeb0e3924557e5e6b006..37b9649e555d4632e9d34b6749cb7214f25eb823 100644 (file)
@@ -131,13 +131,12 @@ public abstract class BootstrapHandler implements RequestHandler {
     private String getBootstrapHtml(BootstrapContext context) {
         WrappedRequest request = context.getRequest();
         WrappedResponse response = context.getResponse();
-        DeploymentConfiguration deploymentConfiguration = request
-                .getDeploymentConfiguration();
+        VaadinService vaadinService = request.getVaadinService();
 
         BootstrapFragmentResponse fragmentResponse = context
                 .getBootstrapResponse();
 
-        if (deploymentConfiguration.isStandalone(request)) {
+        if (vaadinService.isStandalone(request)) {
             Map<String, Object> headers = new LinkedHashMap<String, Object>();
             Document document = Document.createShell("");
             BootstrapPageResponse pageResponse = new BootstrapPageResponse(
@@ -274,8 +273,8 @@ public abstract class BootstrapHandler implements RequestHandler {
                 .getUiProvider(context.getRequest(), context.getUIClass())
                 .getWidgetsetForUI(context.getRequest(), context.getUIClass());
         if (widgetset == null) {
-            widgetset = request.getDeploymentConfiguration()
-                    .getConfiguredWidgetset(request);
+            widgetset = request.getVaadinService().getConfiguredWidgetset(
+                    request);
         }
 
         widgetset = VaadinServlet.stripSpecialChars(widgetset);
@@ -327,9 +326,8 @@ public abstract class BootstrapHandler implements RequestHandler {
 
         WrappedRequest request = context.getRequest();
 
-        DeploymentConfiguration deploymentConfiguration = request
-                .getDeploymentConfiguration();
-        String staticFileLocation = deploymentConfiguration
+        VaadinService vaadinService = request.getVaadinService();
+        String staticFileLocation = vaadinService
                 .getStaticFileLocation(request);
 
         fragmentNodes
@@ -421,12 +419,10 @@ public abstract class BootstrapHandler implements RequestHandler {
 
         WrappedRequest request = context.getRequest();
         VaadinSession application = context.getApplication();
-        DeploymentConfiguration deploymentConfiguration = request
-                .getDeploymentConfiguration();
+        VaadinService vaadinService = request.getVaadinService();
 
         // Get system messages
-        SystemMessages systemMessages = deploymentConfiguration
-                .getSystemMessages();
+        SystemMessages systemMessages = vaadinService.getSystemMessages();
         if (systemMessages != null) {
             // Write the CommunicationError -message to client
             JSONObject comErrMsg = new JSONObject();
@@ -448,7 +444,7 @@ public abstract class BootstrapHandler implements RequestHandler {
             defaults.put("authErrMsg", authErrMsg);
         }
 
-        String staticFileLocation = deploymentConfiguration
+        String staticFileLocation = vaadinService
                 .getStaticFileLocation(request);
         String widgetsetBase = staticFileLocation + "/"
                 + VaadinServlet.WIDGETSET_DIRECTORY_PATH;
@@ -458,11 +454,11 @@ public abstract class BootstrapHandler implements RequestHandler {
             defaults.put("debug", true);
         }
 
-        if (deploymentConfiguration.isStandalone(request)) {
+        if (vaadinService.isStandalone(request)) {
             defaults.put("standalone", true);
         }
 
-        defaults.put("heartbeatInterval", deploymentConfiguration
+        defaults.put("heartbeatInterval", vaadinService
                 .getApplicationConfiguration().getHeartbeatInterval());
 
         defaults.put("appUri", getAppUri(context));
@@ -485,7 +481,7 @@ public abstract class BootstrapHandler implements RequestHandler {
      */
     public String getThemeUri(BootstrapContext context, String themeName) {
         WrappedRequest request = context.getRequest();
-        final String staticFilePath = request.getDeploymentConfiguration()
+        final String staticFilePath = request.getVaadinService()
                 .getStaticFileLocation(request);
         return staticFilePath + "/" + VaadinServlet.THEME_DIRECTORY_PATH
                 + themeName;
@@ -513,8 +509,7 @@ public abstract class BootstrapHandler implements RequestHandler {
         String themeName = getThemeName(context);
         if (themeName == null) {
             WrappedRequest request = context.getRequest();
-            themeName = request.getDeploymentConfiguration()
-                    .getConfiguredTheme(request);
+            themeName = request.getVaadinService().getConfiguredTheme(request);
         }
 
         // XSS preventation, theme names shouldn't contain special chars anyway.
index cc336ffa73749201b563f4353a127390629d6fc1..3de5067a20424ce7357410d066b46d5295e79a98 100644 (file)
@@ -178,7 +178,7 @@ public class CombinedRequest implements WrappedRequest {
     }
 
     @Override
-    public DeploymentConfiguration getDeploymentConfiguration() {
-        return secondRequest.getDeploymentConfiguration();
+    public VaadinService getVaadinService() {
+        return secondRequest.getVaadinService();
     }
 }
index 9babf4704e9996ca001871a60fda0ee545ea5101..0a5613a58857f1c08c26579bbb45df9f3027895f 100644 (file)
@@ -29,7 +29,7 @@ public class DefaultUIProvider extends AbstractUIProvider {
         if (uiClassNameObj instanceof String) {
             String uiClassName = uiClassNameObj.toString();
 
-            ClassLoader classLoader = request.getDeploymentConfiguration()
+            ClassLoader classLoader = request.getVaadinService()
                     .getClassLoader();
             if (classLoader == null) {
                 classLoader = getClass().getClassLoader();
diff --git a/server/src/com/vaadin/server/DeploymentConfiguration.java b/server/src/com/vaadin/server/DeploymentConfiguration.java
deleted file mode 100644 (file)
index 0a70ef0..0000000
+++ /dev/null
@@ -1,141 +0,0 @@
-/*
- * 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 java.io.File;
-import java.io.Serializable;
-import java.util.Iterator;
-
-import javax.portlet.PortletContext;
-import javax.servlet.ServletContext;
-
-/**
- * Provide deployment specific settings that are required outside terminal
- * specific code.
- * 
- * @author Vaadin Ltd.
- * 
- * @since 7.0
- */
-public interface DeploymentConfiguration extends Serializable {
-    /**
-     * Return the URL from where static files, e.g. the widgetset and the theme,
-     * are served. In a standard configuration the VAADIN folder inside the
-     * returned folder is what is used for widgetsets and themes.
-     * 
-     * The returned folder is usually the same as the context path and
-     * independent of the application.
-     * 
-     * @param request
-     *            the request for which the location should be determined
-     * 
-     * @return The location of static resources (should contain the VAADIN
-     *         directory). Never ends with a slash (/).
-     */
-    public String getStaticFileLocation(WrappedRequest request);
-
-    /**
-     * Gets the widgetset that is configured for this deployment, e.g. from a
-     * parameter in web.xml.
-     * 
-     * @param request
-     *            the request for which a widgetset is required
-     * @return the name of the widgetset
-     */
-    public String getConfiguredWidgetset(WrappedRequest request);
-
-    /**
-     * Gets the theme that is configured for this deployment, e.g. from a portal
-     * parameter or just some sensible default value.
-     * 
-     * @param request
-     *            the request for which a theme is required
-     * @return the name of the theme
-     */
-    public String getConfiguredTheme(WrappedRequest request);
-
-    /**
-     * Checks whether the Vaadin application will be rendered on its own in the
-     * browser or whether it will be included into some other context. A
-     * standalone application may do things that might interfere with other
-     * parts of a page, e.g. changing the page title and requesting focus upon
-     * loading.
-     * 
-     * @param request
-     *            the request for which the application is loaded
-     * @return a boolean indicating whether the application should be standalone
-     */
-    public boolean isStandalone(WrappedRequest request);
-
-    /**
-     * Get the class loader to use for loading classes loaded by name, e.g.
-     * custom UI classes. <code>null</code> indicates that the default class
-     * loader should be used.
-     * 
-     * @return the class loader to use, or <code>null</code>
-     */
-    public ClassLoader getClassLoader();
-
-    /**
-     * Returns the MIME type of the specified file, or null if the MIME type is
-     * not known. The MIME type is determined by the configuration of the
-     * container, and may be specified in a deployment descriptor. Common MIME
-     * types are "text/html" and "image/gif".
-     * 
-     * @param resourceName
-     *            a String specifying the name of a file
-     * @return a String specifying the file's MIME type
-     * 
-     * @see ServletContext#getMimeType(String)
-     * @see PortletContext#getMimeType(String)
-     */
-    public String getMimeType(String resourceName);
-
-    /**
-     * Gets the application configuration.
-     * 
-     * @return the application configuration
-     */
-    public ApplicationConfiguration getApplicationConfiguration();
-
-    public Iterator<AddonContextListener> getAddonContextListeners();
-
-    public AddonContext getAddonContext();
-
-    public void setAddonContext(AddonContext vaadinContext);
-
-    /**
-     * Gets the system messages object
-     * 
-     * @return the system messages object
-     */
-    public SystemMessages getSystemMessages();
-
-    /**
-     * Returns application context base directory.
-     * 
-     * Typically an application is deployed in a such way that is has an
-     * application directory. For web applications this directory is the root
-     * directory of the web applications. In some cases applications might not
-     * have an application directory (for example web applications running
-     * inside a war).
-     * 
-     * @return The application base directory or null if the application has no
-     *         base directory.
-     */
-    public File getBaseDirectory();
-}
index 957c0a9fe1f900d0446f895b57bc78fc9e4558f6..98b5f94c6a6bbbd56f1f712fe0b4b1911c3fd4f6 100644 (file)
@@ -172,9 +172,9 @@ public class GAEVaadinServlet extends VaadinServlet {
             HttpServletResponse unwrappedResponse) throws ServletException,
             IOException {
         WrappedHttpServletRequest request = new WrappedHttpServletRequest(
-                unwrappedRequest, getDeploymentConfiguration());
+                unwrappedRequest, getVaadinService());
         WrappedHttpServletResponse response = new WrappedHttpServletResponse(
-                unwrappedResponse, getDeploymentConfiguration());
+                unwrappedResponse, getVaadinService());
 
         if (isCleanupRequest(request)) {
             cleanDatastore();
index e62bf5f4d43e32d91b753d596cb135a14bac36f9..0f71f102c7de6ecf9260b874ca3b53a121b6ab74 100644 (file)
@@ -28,7 +28,7 @@ public class LegacyVaadinPortlet extends VaadinPortlet {
             throws ClassNotFoundException {
         try {
             return ServletPortletHelper
-                    .getLegacyApplicationClass(getDeploymentConfiguration());
+                    .getLegacyApplicationClass(getVaadinService());
         } catch (ApplicationClassException e) {
             throw new RuntimeException(e);
         }
index 183d4389d618d4d57a28f4e31b2994161f8b547a..a631ae831981cb990785fa9139abb82db5e935de 100644 (file)
@@ -28,7 +28,7 @@ public class LegacyVaadinServlet extends VaadinServlet {
             throws ClassNotFoundException {
         try {
             return ServletPortletHelper
-                    .getLegacyApplicationClass(getDeploymentConfiguration());
+                    .getLegacyApplicationClass(getVaadinService());
         } catch (ApplicationClassException e) {
             throw new RuntimeException(e);
         }
index fcb0ae228ecb052710c68ada32654163ccbc50d1..5b6948e2e32923db23640cb07f8afcca51eb77ec 100644 (file)
@@ -127,9 +127,9 @@ public class PortletCommunicationManager extends AbstractCommunicationManager {
 
             @Override
             protected String getMainDivStyle(BootstrapContext context) {
-                DeploymentConfiguration deploymentConfiguration = context
-                        .getRequest().getDeploymentConfiguration();
-                return deploymentConfiguration.getApplicationConfiguration()
+                VaadinService vaadinService = context.getRequest()
+                        .getVaadinService();
+                return vaadinService.getApplicationConfiguration()
                         .getApplicationOrSystemProperty(
                                 VaadinPortlet.PORTLET_PARAMETER_STYLE, null);
             }
index 609168ee9618f118574217c5f2e269fac66e9f6d..f964415fdcdec967a5d059ce7331e8ffa4c71ff0 100644 (file)
@@ -42,12 +42,11 @@ class ServletPortletHelper implements Serializable {
     }
 
     static Class<? extends Application> getLegacyApplicationClass(
-            DeploymentConfiguration deploymentConfiguration)
-            throws ApplicationClassException {
-        Properties initParameters = deploymentConfiguration
-                .getApplicationConfiguration().getInitParameters();
+            VaadinService vaadinService) throws ApplicationClassException {
+        Properties initParameters = vaadinService.getApplicationConfiguration()
+                .getInitParameters();
         String applicationParameter = initParameters.getProperty("application");
-        ClassLoader classLoader = deploymentConfiguration.getClassLoader();
+        ClassLoader classLoader = vaadinService.getClassLoader();
 
         if (applicationParameter == null) {
             throw new ApplicationClassException(
@@ -133,13 +132,11 @@ class ServletPortletHelper implements Serializable {
     }
 
     public static void initDefaultUIProvider(VaadinSession application,
-            DeploymentConfiguration deploymentConfiguration)
-            throws ApplicationClassException {
-        String uiProperty = deploymentConfiguration
-                .getApplicationConfiguration().getInitParameters()
-                .getProperty(VaadinSession.UI_PARAMETER);
+            VaadinService vaadinService) throws ApplicationClassException {
+        String uiProperty = vaadinService.getApplicationConfiguration()
+                .getInitParameters().getProperty(VaadinSession.UI_PARAMETER);
         if (uiProperty != null) {
-            verifyUIClass(uiProperty, deploymentConfiguration.getClassLoader());
+            verifyUIClass(uiProperty, vaadinService.getClassLoader());
             application.addUIProvider(new DefaultUIProvider());
         }
     }
index 236b38b27ce6297e9091c370c2a7b26e725f675f..be571fbf6045a1bcbc5ba672ee60c63e42fcd07a 100644 (file)
@@ -74,11 +74,10 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
 
     public static final String RESOURCE_URL_ID = "APP";
 
-    public static class PortletDeploymentConfiguration extends
-            AbstractDeploymentConfiguration {
+    public static class PortletService extends AbstractVaadinService {
         private final VaadinPortlet portlet;
 
-        public PortletDeploymentConfiguration(VaadinPortlet portlet,
+        public PortletService(VaadinPortlet portlet,
                 ApplicationConfiguration applicationConfiguration) {
             super(applicationConfiguration);
             this.portlet = portlet;
@@ -185,9 +184,8 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
             WrappedPortletRequest {
 
         public WrappedHttpAndPortletRequest(PortletRequest request,
-                HttpServletRequest originalRequest,
-                DeploymentConfiguration deploymentConfiguration) {
-            super(request, deploymentConfiguration);
+                HttpServletRequest originalRequest, PortletService vaadinService) {
+            super(request, vaadinService);
             this.originalRequest = originalRequest;
         }
 
@@ -229,8 +227,8 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
     public static class WrappedGateinRequest extends
             WrappedHttpAndPortletRequest {
         public WrappedGateinRequest(PortletRequest request,
-                DeploymentConfiguration deploymentConfiguration) {
-            super(request, getOriginalRequest(request), deploymentConfiguration);
+                PortletService vaadinService) {
+            super(request, getOriginalRequest(request), vaadinService);
         }
 
         private static final HttpServletRequest getOriginalRequest(
@@ -252,8 +250,8 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
             WrappedHttpAndPortletRequest {
 
         public WrappedLiferayRequest(PortletRequest request,
-                DeploymentConfiguration deploymentConfiguration) {
-            super(request, getOriginalRequest(request), deploymentConfiguration);
+                PortletService vaadinService) {
+            super(request, getOriginalRequest(request), vaadinService);
         }
 
         @Override
@@ -320,7 +318,7 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
     // TODO Can we close the application when the portlet is removed? Do we know
     // when the portlet is removed?
 
-    private PortletDeploymentConfiguration deploymentConfiguration;
+    private PortletService vaadinService;
     private AddonContext addonContext;
 
     @Override
@@ -346,9 +344,9 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
         }
 
         ApplicationConfiguration applicationConfiguration = createApplicationConfiguration(applicationProperties);
-        deploymentConfiguration = createDeploymentConfiguration(applicationConfiguration);
+        vaadinService = createPortletService(applicationConfiguration);
 
-        addonContext = new AddonContext(deploymentConfiguration);
+        addonContext = new AddonContext(vaadinService);
         addonContext.init();
     }
 
@@ -358,10 +356,9 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
                 applicationProperties);
     }
 
-    protected PortletDeploymentConfiguration createDeploymentConfiguration(
+    protected PortletService createPortletService(
             ApplicationConfiguration applicationConfiguration) {
-        return new PortletDeploymentConfiguration(this,
-                applicationConfiguration);
+        return new PortletService(this, applicationConfiguration);
     }
 
     @Override
@@ -429,7 +426,7 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
         WrappedPortletRequest wrappedRequest = createWrappedRequest(request);
 
         WrappedPortletResponse wrappedResponse = new WrappedPortletResponse(
-                response, getDeploymentConfiguration());
+                response, getVaadinService());
 
         CurrentInstance.set(WrappedRequest.class, wrappedRequest);
         CurrentInstance.set(WrappedResponse.class, wrappedResponse);
@@ -611,20 +608,17 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
         String portalInfo = request.getPortalContext().getPortalInfo()
                 .toLowerCase();
         if (portalInfo.contains("liferay")) {
-            return new WrappedLiferayRequest(request,
-                    getDeploymentConfiguration());
+            return new WrappedLiferayRequest(request, getVaadinService());
         } else if (portalInfo.contains("gatein")) {
-            return new WrappedGateinRequest(request,
-                    getDeploymentConfiguration());
+            return new WrappedGateinRequest(request, getVaadinService());
         } else {
-            return new WrappedPortletRequest(request,
-                    getDeploymentConfiguration());
+            return new WrappedPortletRequest(request, getVaadinService());
         }
 
     }
 
-    protected PortletDeploymentConfiguration getDeploymentConfiguration() {
-        return deploymentConfiguration;
+    protected PortletService getVaadinService() {
+        return vaadinService;
     }
 
     private void handleUnknownRequest(PortletRequest request,
@@ -835,8 +829,8 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
         Locale locale = request.getLocale();
         newApplication.setLocale(locale);
         // No application URL when running inside a portlet
-        newApplication.start(new ApplicationStartEvent(null,
-                getDeploymentConfiguration().getApplicationConfiguration(),
+        newApplication.start(new ApplicationStartEvent(null, getVaadinService()
+                .getApplicationConfiguration(),
                 new PortletCommunicationManager(newApplication)));
         addonContext.fireApplicationStarted(newApplication);
 
@@ -849,7 +843,7 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
 
         try {
             ServletPortletHelper.initDefaultUIProvider(application,
-                    getDeploymentConfiguration());
+                    getVaadinService());
         } catch (ApplicationClassException e) {
             throw new PortletException(e);
         }
@@ -889,8 +883,7 @@ public class VaadinPortlet extends GenericPortlet implements Constants {
 
         // if this was an UIDL request, response UIDL back to client
         if (getRequestType(request) == RequestType.UIDL) {
-            SystemMessages ci = getDeploymentConfiguration()
-                    .getSystemMessages();
+            SystemMessages ci = getVaadinService().getSystemMessages();
             criticalNotification(request, response,
                     ci.getInternalErrorCaption(), ci.getInternalErrorMessage(),
                     null, ci.getInternalErrorURL());
index 8d1ae84fcac1a6441f0252c333e3a338331a879c..40c88ac8fe80adf5a137ed66bbac44f55dd98532 100644 (file)
@@ -84,8 +84,7 @@ public class VaadinPortletSession extends VaadinSession {
     public PortletConfig getPortletConfig() {
         WrappedPortletResponse response = (WrappedPortletResponse) CurrentInstance
                 .get(WrappedResponse.class);
-        return response.getDeploymentConfiguration().getPortlet()
-                .getPortletConfig();
+        return response.getVaadinService().getPortlet().getPortletConfig();
     }
 
     public void addPortletListener(PortletListener listener) {
diff --git a/server/src/com/vaadin/server/VaadinService.java b/server/src/com/vaadin/server/VaadinService.java
new file mode 100644 (file)
index 0000000..41c864f
--- /dev/null
@@ -0,0 +1,141 @@
+/*
+ * 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 java.io.File;
+import java.io.Serializable;
+import java.util.Iterator;
+
+import javax.portlet.PortletContext;
+import javax.servlet.ServletContext;
+
+/**
+ * Provide deployment specific settings that are required outside terminal
+ * specific code.
+ * 
+ * @author Vaadin Ltd.
+ * 
+ * @since 7.0
+ */
+public interface VaadinService extends Serializable {
+    /**
+     * Return the URL from where static files, e.g. the widgetset and the theme,
+     * are served. In a standard configuration the VAADIN folder inside the
+     * returned folder is what is used for widgetsets and themes.
+     * 
+     * The returned folder is usually the same as the context path and
+     * independent of the application.
+     * 
+     * @param request
+     *            the request for which the location should be determined
+     * 
+     * @return The location of static resources (should contain the VAADIN
+     *         directory). Never ends with a slash (/).
+     */
+    public String getStaticFileLocation(WrappedRequest request);
+
+    /**
+     * Gets the widgetset that is configured for this deployment, e.g. from a
+     * parameter in web.xml.
+     * 
+     * @param request
+     *            the request for which a widgetset is required
+     * @return the name of the widgetset
+     */
+    public String getConfiguredWidgetset(WrappedRequest request);
+
+    /**
+     * Gets the theme that is configured for this deployment, e.g. from a portal
+     * parameter or just some sensible default value.
+     * 
+     * @param request
+     *            the request for which a theme is required
+     * @return the name of the theme
+     */
+    public String getConfiguredTheme(WrappedRequest request);
+
+    /**
+     * Checks whether the Vaadin application will be rendered on its own in the
+     * browser or whether it will be included into some other context. A
+     * standalone application may do things that might interfere with other
+     * parts of a page, e.g. changing the page title and requesting focus upon
+     * loading.
+     * 
+     * @param request
+     *            the request for which the application is loaded
+     * @return a boolean indicating whether the application should be standalone
+     */
+    public boolean isStandalone(WrappedRequest request);
+
+    /**
+     * Get the class loader to use for loading classes loaded by name, e.g.
+     * custom UI classes. <code>null</code> indicates that the default class
+     * loader should be used.
+     * 
+     * @return the class loader to use, or <code>null</code>
+     */
+    public ClassLoader getClassLoader();
+
+    /**
+     * Returns the MIME type of the specified file, or null if the MIME type is
+     * not known. The MIME type is determined by the configuration of the
+     * container, and may be specified in a deployment descriptor. Common MIME
+     * types are "text/html" and "image/gif".
+     * 
+     * @param resourceName
+     *            a String specifying the name of a file
+     * @return a String specifying the file's MIME type
+     * 
+     * @see ServletContext#getMimeType(String)
+     * @see PortletContext#getMimeType(String)
+     */
+    public String getMimeType(String resourceName);
+
+    /**
+     * Gets the application configuration.
+     * 
+     * @return the application configuration
+     */
+    public ApplicationConfiguration getApplicationConfiguration();
+
+    public Iterator<AddonContextListener> getAddonContextListeners();
+
+    public AddonContext getAddonContext();
+
+    public void setAddonContext(AddonContext vaadinContext);
+
+    /**
+     * Gets the system messages object
+     * 
+     * @return the system messages object
+     */
+    public SystemMessages getSystemMessages();
+
+    /**
+     * Returns application context base directory.
+     * 
+     * Typically an application is deployed in a such way that is has an
+     * application directory. For web applications this directory is the root
+     * directory of the web applications. In some cases applications might not
+     * have an application directory (for example web applications running
+     * inside a war).
+     * 
+     * @return The application base directory or null if the application has no
+     *         base directory.
+     */
+    public File getBaseDirectory();
+}
index 3a60cf5009e28e5f52bdfd13cd8ef6fa836c99e3..cd62e292fbe7914fb48ec0b4a08d5fa9b114cfc8 100644 (file)
@@ -55,11 +55,10 @@ import com.vaadin.util.CurrentInstance;
 @SuppressWarnings("serial")
 public class VaadinServlet extends HttpServlet implements Constants {
 
-    public static class ServletDeploymentConfiguration extends
-            AbstractDeploymentConfiguration {
+    public static class ServletService extends AbstractVaadinService {
         private final VaadinServlet servlet;
 
-        public ServletDeploymentConfiguration(VaadinServlet servlet,
+        public ServletService(VaadinServlet servlet,
                 ApplicationConfiguration applicationProperties) {
             super(applicationProperties);
             this.servlet = servlet;
@@ -175,7 +174,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
 
     private final String resourcePath = null;
 
-    private DeploymentConfiguration deploymentConfiguration;
+    private ServletService servletService;
 
     private AddonContext addonContext;
 
@@ -214,9 +213,9 @@ public class VaadinServlet extends HttpServlet implements Constants {
         }
 
         ApplicationConfiguration applicationConfiguration = createApplicationConfiguration(applicationProperties);
-        deploymentConfiguration = createDeploymentConfiguration(applicationConfiguration);
+        servletService = createServletService(applicationConfiguration);
 
-        addonContext = new AddonContext(deploymentConfiguration);
+        addonContext = new AddonContext(servletService);
         addonContext.init();
     }
 
@@ -226,10 +225,9 @@ public class VaadinServlet extends HttpServlet implements Constants {
                 applicationProperties);
     }
 
-    protected ServletDeploymentConfiguration createDeploymentConfiguration(
+    protected ServletService createServletService(
             ApplicationConfiguration applicationConfiguration) {
-        return new ServletDeploymentConfiguration(this,
-                applicationConfiguration);
+        return new ServletService(this, applicationConfiguration);
     }
 
     @Override
@@ -393,7 +391,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
     private WrappedHttpServletResponse createWrappedResponse(
             HttpServletResponse response) {
         WrappedHttpServletResponse wrappedResponse = new WrappedHttpServletResponse(
-                response, getDeploymentConfiguration());
+                response, getVaadinService());
         return wrappedResponse;
     }
 
@@ -407,8 +405,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
      */
     protected WrappedHttpServletRequest createWrappedRequest(
             HttpServletRequest request) {
-        return new WrappedHttpServletRequest(request,
-                getDeploymentConfiguration());
+        return new WrappedHttpServletRequest(request, getVaadinService());
     }
 
     /**
@@ -416,8 +413,8 @@ public class VaadinServlet extends HttpServlet implements Constants {
      * 
      * @return the deployment configuration
      */
-    protected DeploymentConfiguration getDeploymentConfiguration() {
-        return deploymentConfiguration;
+    protected ServletService getVaadinService() {
+        return servletService;
     }
 
     /**
@@ -443,7 +440,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
             // This can be removed if cookieless mode (#3228) is supported
             if (request.getRequestedSessionId() == null) {
                 // User has cookies disabled
-                SystemMessages systemMessages = getDeploymentConfiguration()
+                SystemMessages systemMessages = getVaadinService()
                         .getSystemMessages();
                 criticalNotification(request, response,
                         systemMessages.getCookiesDisabledCaption(),
@@ -645,7 +642,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
         Locale locale = request.getLocale();
         newApplication.setLocale(locale);
         newApplication.start(new ApplicationStartEvent(applicationUrl,
-                getDeploymentConfiguration().getApplicationConfiguration(),
+                getVaadinService().getApplicationConfiguration(),
                 createCommunicationManager(newApplication)));
 
         addonContext.fireApplicationStarted(newApplication);
@@ -728,7 +725,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
 
         try {
             ServletPortletHelper.initDefaultUIProvider(newApplication,
-                    getDeploymentConfiguration());
+                    getVaadinService());
         } catch (ApplicationClassException e) {
             throw new ServletException(e);
         }
@@ -741,8 +738,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
             Throwable e) throws IOException, ServletException {
         // if this was an UIDL request, response UIDL back to client
         if (getRequestType(request) == RequestType.UIDL) {
-            SystemMessages ci = getDeploymentConfiguration()
-                    .getSystemMessages();
+            SystemMessages ci = getVaadinService().getSystemMessages();
             criticalNotification(request, response,
                     ci.getInternalErrorCaption(), ci.getInternalErrorMessage(),
                     null, ci.getInternalErrorURL());
@@ -805,8 +801,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
         }
 
         try {
-            SystemMessages ci = getDeploymentConfiguration()
-                    .getSystemMessages();
+            SystemMessages ci = getVaadinService().getSystemMessages();
             if (getRequestType(request) != RequestType.UIDL) {
                 // 'plain' http req - e.g. browser reload;
                 // just go ahead redirect the browser
@@ -848,8 +843,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
         }
 
         try {
-            SystemMessages ci = getDeploymentConfiguration()
-                    .getSystemMessages();
+            SystemMessages ci = getVaadinService().getSystemMessages();
             if (getRequestType(request) != RequestType.UIDL) {
                 // 'plain' http req - e.g. browser reload;
                 // just go ahead redirect the browser
@@ -931,8 +925,8 @@ public class VaadinServlet extends HttpServlet implements Constants {
 
             // strip leading "/" otherwise stream from JAR wont work
             filename = filename.substring(1);
-            resourceUrl = getDeploymentConfiguration().getClassLoader()
-                    .getResource(filename);
+            resourceUrl = getVaadinService().getClassLoader().getResource(
+                    filename);
 
             if (resourceUrl == null) {
                 // cannot serve requested file
@@ -1013,7 +1007,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
              * cache timeout can be configured by setting the resourceCacheTime
              * parameter in web.xml
              */
-            int resourceCacheTime = getDeploymentConfiguration()
+            int resourceCacheTime = getVaadinService()
                     .getApplicationConfiguration().getResourceCacheTime();
             response.setHeader("Cache-Control",
                     "max-age= " + String.valueOf(resourceCacheTime));
index 440fc02ee60b3af28f924ba85aa61427792612a0..c4b0680c47240dd0fcde1da186134b043d16f9bd 100644 (file)
@@ -1343,10 +1343,10 @@ public class VaadinSession implements Terminal.ErrorListener,
      * <p>
      * The default implementation in {@link VaadinSession} uses the
      * {@value #UI_PARAMETER} parameter from web.xml for finding the name of the
-     * UI class. If {@link DeploymentConfiguration#getClassLoader()} does not
-     * return <code>null</code>, the returned {@link ClassLoader} is used for
-     * loading the UI class. Otherwise the {@link ClassLoader} used to load this
-     * class is used.
+     * UI class. If {@link VaadinService#getClassLoader()} does not return
+     * <code>null</code>, the returned {@link ClassLoader} is used for loading
+     * the UI class. Otherwise the {@link ClassLoader} used to load this class
+     * is used.
      * 
      * </p>
      * 
@@ -1885,7 +1885,7 @@ public class VaadinSession implements Terminal.ErrorListener,
      * 
      * @see #getUidlRequestTimeout()
      * @see #closeInactiveUIs()
-     * @see DeploymentConfiguration#getHeartbeatInterval()
+     * @see ApplicationConfiguration#getHeartbeatInterval()
      * 
      * @since 7.0.0
      * 
@@ -1907,7 +1907,7 @@ public class VaadinSession implements Terminal.ErrorListener,
      * otherwise heartbeat requests are enough to extend UI lifetime
      * indefinitely.
      * 
-     * @see DeploymentConfiguration#isIdleUICleanupEnabled()
+     * @see ApplicationConfiguration#isIdleUICleanupEnabled()
      * @see #getHeartbeatTimeout()
      * @see #closeInactiveUIs()
      * 
index e86166545a70f9d5fae37fe03f483950a21c0a94..7b458dd3d447ac974b497c03e83714e0981e1f81 100644 (file)
@@ -19,6 +19,8 @@ package com.vaadin.server;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletRequestWrapper;
 
+import com.vaadin.server.VaadinServlet.ServletService;
+
 /**
  * Wrapper for {@link HttpServletRequest}.
  * 
@@ -31,7 +33,7 @@ import javax.servlet.http.HttpServletRequestWrapper;
 public class WrappedHttpServletRequest extends HttpServletRequestWrapper
         implements WrappedRequest {
 
-    private final DeploymentConfiguration deploymentConfiguration;
+    private final ServletService vaadinService;
 
     /**
      * Wraps a http servlet request and associates with a deployment
@@ -39,13 +41,13 @@ public class WrappedHttpServletRequest extends HttpServletRequestWrapper
      * 
      * @param request
      *            the http servlet request to wrap
-     * @param deploymentConfiguration
-     *            the associated deployment configuration
+     * @param vaadinService
+     *            the associated vaadin service
      */
     public WrappedHttpServletRequest(HttpServletRequest request,
-            DeploymentConfiguration deploymentConfiguration) {
+            ServletService vaadinService) {
         super(request);
-        this.deploymentConfiguration = deploymentConfiguration;
+        this.vaadinService = vaadinService;
     }
 
     @Override
@@ -68,8 +70,8 @@ public class WrappedHttpServletRequest extends HttpServletRequestWrapper
     }
 
     @Override
-    public DeploymentConfiguration getDeploymentConfiguration() {
-        return deploymentConfiguration;
+    public ServletService getVaadinService() {
+        return vaadinService;
     }
 
     @Override
index f2916ac0eb625161c958f03cee27864c7ac0b066..5fbfa451d8372a6c81b1d2310b36cf83cb6e42de 100644 (file)
@@ -19,6 +19,8 @@ package com.vaadin.server;
 import javax.servlet.http.HttpServletResponse;
 import javax.servlet.http.HttpServletResponseWrapper;
 
+import com.vaadin.server.VaadinServlet.ServletService;
+
 /**
  * Wrapper for {@link HttpServletResponse}.
  * 
@@ -31,20 +33,20 @@ import javax.servlet.http.HttpServletResponseWrapper;
 public class WrappedHttpServletResponse extends HttpServletResponseWrapper
         implements WrappedResponse {
 
-    private DeploymentConfiguration deploymentConfiguration;
+    private ServletService vaadinService;
 
     /**
      * Wraps a http servlet response and an associated deployment configuration
      * 
      * @param response
      *            the http servlet response to wrap
-     * @param deploymentConfiguration
-     *            the associated deployment configuration
+     * @param vaadinService
+     *            the associated vaadin service
      */
     public WrappedHttpServletResponse(HttpServletResponse response,
-            DeploymentConfiguration deploymentConfiguration) {
+            ServletService vaadinService) {
         super(response);
-        this.deploymentConfiguration = deploymentConfiguration;
+        this.vaadinService = vaadinService;
     }
 
     /**
@@ -78,7 +80,7 @@ public class WrappedHttpServletResponse extends HttpServletResponseWrapper
     }
 
     @Override
-    public DeploymentConfiguration getDeploymentConfiguration() {
-        return deploymentConfiguration;
+    public ServletService getVaadinService() {
+        return vaadinService;
     }
 }
\ No newline at end of file
index 65099add766da7357eb413a63eebcf344ca58729..bea002181b685abcb64e394945af3a785af2fd96 100644 (file)
@@ -25,6 +25,7 @@ import javax.portlet.ClientDataRequest;
 import javax.portlet.PortletRequest;
 import javax.portlet.ResourceRequest;
 
+import com.vaadin.server.VaadinPortlet.PortletService;
 import com.vaadin.shared.ApplicationConstants;
 
 /**
@@ -39,20 +40,20 @@ import com.vaadin.shared.ApplicationConstants;
 public class WrappedPortletRequest implements WrappedRequest {
 
     private final PortletRequest request;
-    private final DeploymentConfiguration deploymentConfiguration;
+    private final PortletService vaadinService;
 
     /**
      * Wraps a portlet request and an associated deployment configuration
      * 
      * @param request
      *            the portlet request to wrap
-     * @param deploymentConfiguration
-     *            the associated deployment configuration
+     * @param vaadinService
+     *            the associated vaadin service
      */
     public WrappedPortletRequest(PortletRequest request,
-            DeploymentConfiguration deploymentConfiguration) {
+            PortletService vaadinService) {
         this.request = request;
-        this.deploymentConfiguration = deploymentConfiguration;
+        this.vaadinService = vaadinService;
     }
 
     @Override
@@ -190,8 +191,8 @@ public class WrappedPortletRequest implements WrappedRequest {
     }
 
     @Override
-    public DeploymentConfiguration getDeploymentConfiguration() {
-        return deploymentConfiguration;
+    public PortletService getVaadinService() {
+        return vaadinService;
     }
 
     /**
index e3010501b62803b2caf53475019d7d3ab469a67f..af4288fbefe06d8ff7b90811be5347e6868ee3ed 100644 (file)
@@ -29,7 +29,7 @@ import javax.portlet.MimeResponse;
 import javax.portlet.PortletResponse;
 import javax.portlet.ResourceResponse;
 
-import com.vaadin.server.VaadinPortlet.PortletDeploymentConfiguration;
+import com.vaadin.server.VaadinPortlet.PortletService;
 
 /**
  * Wrapper for {@link PortletResponse} and its subclasses.
@@ -48,20 +48,20 @@ public class WrappedPortletResponse implements WrappedResponse {
     }
 
     private final PortletResponse response;
-    private PortletDeploymentConfiguration deploymentConfiguration;
+    private PortletService vaadinService;
 
     /**
      * Wraps a portlet response and an associated deployment configuration
      * 
      * @param response
      *            the portlet response to wrap
-     * @param deploymentConfiguration
-     *            the associated deployment configuration
+     * @param vaadinService
+     *            the associated vaadin service
      */
     public WrappedPortletResponse(PortletResponse response,
-            PortletDeploymentConfiguration deploymentConfiguration) {
+            PortletService vaadinService) {
         this.response = response;
-        this.deploymentConfiguration = deploymentConfiguration;
+        this.vaadinService = vaadinService;
     }
 
     @Override
@@ -116,7 +116,7 @@ public class WrappedPortletResponse implements WrappedResponse {
     }
 
     @Override
-    public PortletDeploymentConfiguration getDeploymentConfiguration() {
-        return deploymentConfiguration;
+    public PortletService getVaadinService() {
+        return vaadinService;
     }
 }
\ No newline at end of file
index 5d7ece9ef1f6d3b443640c1fd74607277a2a274b..9bd296130fc51f9c6eecb6cd59f7960408ddfef6 100644 (file)
@@ -246,12 +246,12 @@ public interface WrappedRequest extends Serializable {
     public String getHeader(String headerName);
 
     /**
-     * Gets the deployment configuration for the context of this request.
+     * Gets the vaadin service for the context of this request.
      * 
-     * @return the deployment configuration
+     * @return the vaadin service
      * 
-     * @see DeploymentConfiguration
+     * @see VaadinService
      */
-    public DeploymentConfiguration getDeploymentConfiguration();
+    public VaadinService getVaadinService();
 
 }
index c8d9342418578f732d33d920bdfab7e7a56262b7..90c9a356e10d5e16ad2b9fd049fd4e89889a7667 100644 (file)
@@ -149,11 +149,11 @@ public interface WrappedResponse extends Serializable {
     public void sendError(int errorCode, String message) throws IOException;
 
     /**
-     * Gets the deployment configuration for the context of this response.
+     * Gets the vaadin service for the context of this response.
      * 
-     * @return the deployment configuration
+     * @return the vaadin service
      * 
-     * @see DeploymentConfiguration
+     * @see VaadinService
      */
-    public DeploymentConfiguration getDeploymentConfiguration();
+    public VaadinService getVaadinService();
 }
index d393f1c9b227d071057183627f6fa6a36382a3e7..1e663afdd2348107f27357f1d5bc42483e28286c 100644 (file)
@@ -1,6 +1,5 @@
 package com.vaadin.data.util;
 
-
 /**
  * Automated test for {@link AbstractBeanContainer}.
  * 
index 2aa8d0436441b8f6d7680beabbeaaa074fa62a30..e6d84a2b83ca0dd59edb4b9e1cf751d56fd360fd 100644 (file)
@@ -14,7 +14,7 @@ import javax.servlet.http.HttpServletRequest;
 import junit.framework.TestCase;
 
 import com.vaadin.DefaultApplicationConfiguration;
-import com.vaadin.server.VaadinServlet.ServletDeploymentConfiguration;
+import com.vaadin.server.VaadinServlet.ServletService;
 
 public class TestAbstractApplicationServletStaticFilesLocation extends TestCase {
 
@@ -32,7 +32,7 @@ public class TestAbstractApplicationServletStaticFilesLocation extends TestCase
         Field f = VaadinServlet.class
                 .getDeclaredField("deploymentConfiguration");
         f.setAccessible(true);
-        f.set(servlet, new ServletDeploymentConfiguration(servlet,
+        f.set(servlet, new ServletService(servlet,
                 new DefaultApplicationConfiguration(servlet.getClass(),
                         new Properties())));
 
@@ -80,8 +80,8 @@ public class TestAbstractApplicationServletStaticFilesLocation extends TestCase
         // Set request into replay mode
         replay(request);
 
-        String location = servlet.getDeploymentConfiguration()
-                .getStaticFileLocation(servlet.createWrappedRequest(request));
+        String location = servlet.getVaadinService().getStaticFileLocation(
+                servlet.createWrappedRequest(request));
         return location;
     }
 
@@ -93,8 +93,8 @@ public class TestAbstractApplicationServletStaticFilesLocation extends TestCase
         // Set request into replay mode
         replay(request);
 
-        String location = servlet.getDeploymentConfiguration()
-                .getStaticFileLocation(servlet.createWrappedRequest(request));
+        String location = servlet.getVaadinService().getStaticFileLocation(
+                servlet.createWrappedRequest(request));
         return location;
     }
 
index c41183daf3a6ab81260de92c7a1a6d07b7903240..2b4676bc429760b9da4f8eeadb113f456653592e 100644 (file)
@@ -11,7 +11,7 @@ import org.easymock.EasyMock;
 import com.vaadin.DefaultApplicationConfiguration;
 import com.vaadin.server.ApplicationConfiguration;
 import com.vaadin.server.DefaultUIProvider;
-import com.vaadin.server.DeploymentConfiguration;
+import com.vaadin.server.VaadinService;
 import com.vaadin.server.VaadinSession;
 import com.vaadin.server.VaadinSession.ApplicationStartEvent;
 import com.vaadin.server.WrappedRequest;
@@ -73,14 +73,14 @@ public class CustomUIClassLoader extends TestCase {
 
     private static WrappedRequest createRequestMock(ClassLoader classloader) {
         // Mock a DeploymentConfiguration to give the passed classloader
-        DeploymentConfiguration configurationMock = EasyMock
-                .createMock(DeploymentConfiguration.class);
+        VaadinService configurationMock = EasyMock
+                .createMock(VaadinService.class);
         EasyMock.expect(configurationMock.getClassLoader()).andReturn(
                 classloader);
 
         // Mock a WrappedRequest to give the mocked deployment configuration
         WrappedRequest requestMock = EasyMock.createMock(WrappedRequest.class);
-        EasyMock.expect(requestMock.getDeploymentConfiguration()).andReturn(
+        EasyMock.expect(requestMock.getVaadinService()).andReturn(
                 configurationMock);
 
         EasyMock.replay(configurationMock, requestMock);
index 80768a073b4c584eb6220d9427d47ce6fc2eb030..4752cb0d5116030529b7a693b0f233c5e35f20c9 100644 (file)
@@ -16,7 +16,6 @@
 
 package com.vaadin.shared.ui.link;
 
-
 public class LinkConstants {
     public static String HREF_RESOURCE = "href";
 }
index 675fa4e319129fc9d120ea13856884a5c044ee1e..230a7b92300886e93f243ee4680b7901048b8c73 100644 (file)
@@ -61,8 +61,7 @@ public class ApplicationRunnerServlet extends LegacyVaadinServlet {
             Collections.addAll(defaultPackages, initParameter.split(","));
         }
         String str = TestBase.class.getName().replace('.', '/') + ".class";
-        URL url = getDeploymentConfiguration().getClassLoader()
-                .getResource(str);
+        URL url = getVaadinService().getClassLoader().getResource(str);
         if ("file".equals(url.getProtocol())) {
             File comVaadinTests = new File(url.getPath()).getParentFile()
                     .getParentFile();
@@ -270,10 +269,9 @@ public class ApplicationRunnerServlet extends LegacyVaadinServlet {
     }
 
     @Override
-    protected ServletDeploymentConfiguration createDeploymentConfiguration(
+    protected ServletService createServletService(
             ApplicationConfiguration applicationConfiguration) {
-        return new ServletDeploymentConfiguration(this,
-                applicationConfiguration) {
+        return new ServletService(this, applicationConfiguration) {
             @Override
             public String getStaticFileLocation(WrappedRequest request) {
                 URIS uris = getApplicationRunnerURIs(WrappedHttpServletRequest
@@ -291,8 +289,7 @@ public class ApplicationRunnerServlet extends LegacyVaadinServlet {
     @Override
     protected WrappedHttpServletRequest createWrappedRequest(
             HttpServletRequest request) {
-        return new WrappedHttpServletRequest(request,
-                getDeploymentConfiguration()) {
+        return new WrappedHttpServletRequest(request, getVaadinService()) {
             @Override
             public String getRequestPathInfo() {
                 return ApplicationRunnerServlet.this.getRequestPathInfo(this);
index 45552df39cbdcb7b8035d4b5311d070fc765e53c..c102291b5021f2ff5e142911a25652d447a65773 100644 (file)
@@ -27,7 +27,7 @@ public class NonExistingFileResource extends TestBase {
             @Override
             public void buttonClick(ClickEvent event) {
                 FileResource res = new FileResource(new File(CurrentInstance
-                        .get(WrappedRequest.class).getDeploymentConfiguration()
+                        .get(WrappedRequest.class).getVaadinService()
                         .getBaseDirectory()
                         + "/" + filename));
                 getMainWindow().open(res);
index 93ecf4b6eb34d2b5e0a363b21575d93a76520414..e2804e9c3cd4c4bbfc18d3b43a095246d5799f99 100644 (file)
@@ -48,8 +48,7 @@ public class Ticket1975 extends Application {
 
                     }));
             File baseDir = CurrentInstance.get(WrappedRequest.class)
-                    .getDeploymentConfiguration().getBaseDirectory()
-                    .getAbsoluteFile();
+                    .getVaadinService().getBaseDirectory().getAbsoluteFile();
             File f = new File(baseDir + "/VAADIN/themes/" + getTheme()
                     + "/layouts/Ticket1975.html");
 
index be98de67cc800c4360d6d094b09fe156084ea940..609606b50194dc4efbc0e19e1518478d48120e9a 100644 (file)
@@ -49,7 +49,7 @@ public class SampleDirectory {
                 + "possible security constraint with Application "
                 + "Server or Servlet Container.<br />";
         File file = CurrentInstance.get(WrappedRequest.class)
-                .getDeploymentConfiguration().getBaseDirectory();
+                .getVaadinService().getBaseDirectory();
         if ((file == null) || (!file.canRead())
                 || (file.getAbsolutePath() == null)) {
             // cannot access example directory, possible security issue with