]> source.dussan.org Git - vaadin-framework.git/commitdiff
Rename OSGi classes for Java naming conventions (#9320)
authorHenri Sara <henri.sara@gmail.com>
Mon, 15 May 2017 19:34:43 +0000 (22:34 +0300)
committerPekka Hyvönen <pekka@vaadin.com>
Mon, 15 May 2017 19:34:43 +0000 (22:34 +0300)
- Rename OSGi to Osgi in class names.
- Rename VaadinOSGiPortlet to OSGiVaadinPortlet for consistency with
Spring, CDI etc. add-ons
- Add missing javadoc

24 files changed:
client-compiled/src/main/java/com/vaadin/osgi/widgetset/DefaultWidgetsetContribution.java
compatibility-client-compiled/src/main/java/com/vaadin/osgi/compatibility/widgetset/CompatibilityWidgetsetContribution.java
compatibility-themes/src/main/java/com/vaadin/osgi/compatibility/themes/LegacyThemeContributions.java
documentation/advanced/advanced-osgi.asciidoc
liferay-integration/src/main/java/com/vaadin/osgi/liferay/OSGiUIProvider.java [deleted file]
liferay-integration/src/main/java/com/vaadin/osgi/liferay/OSGiVaadinPortletService.java [deleted file]
liferay-integration/src/main/java/com/vaadin/osgi/liferay/OsgiUIProvider.java [new file with mode: 0644]
liferay-integration/src/main/java/com/vaadin/osgi/liferay/OsgiVaadinPortlet.java [new file with mode: 0644]
liferay-integration/src/main/java/com/vaadin/osgi/liferay/OsgiVaadinPortletService.java [new file with mode: 0644]
liferay-integration/src/main/java/com/vaadin/osgi/liferay/PortletUIServiceTrackerCustomizer.java
liferay-integration/src/main/java/com/vaadin/osgi/liferay/VaadinOSGiPortlet.java [deleted file]
liferay-integration/src/main/java/com/vaadin/osgi/liferay/VaadinPortletProvider.java
osgi-integration/src/main/java/com/vaadin/osgi/servlet/VaadinServletRegistration.java
push/src/main/java/com/vaadin/osgi/push/PushResourcesContribution.java
server/src/main/java/com/vaadin/server/osgi/BootstrapContribution.java
shared/bnd.bnd
shared/src/main/java/com/vaadin/osgi/resources/OSGiVaadinResources.java [deleted file]
shared/src/main/java/com/vaadin/osgi/resources/OSGiVaadinTheme.java [deleted file]
shared/src/main/java/com/vaadin/osgi/resources/OSGiVaadinWidgetset.java [deleted file]
shared/src/main/java/com/vaadin/osgi/resources/OsgiVaadinResources.java [new file with mode: 0644]
shared/src/main/java/com/vaadin/osgi/resources/OsgiVaadinTheme.java [new file with mode: 0644]
shared/src/main/java/com/vaadin/osgi/resources/OsgiVaadinWidgetset.java [new file with mode: 0644]
shared/src/main/java/com/vaadin/osgi/resources/impl/VaadinResourceTrackerComponent.java
themes/src/main/java/com/vaadin/osgi/themes/ValoThemeContribution.java

index 8e793779300e89c1d66e41a7d87b53cfe27502ac..632bc2c7553045c158f6df69bd2f963c944a4d9c 100644 (file)
@@ -21,7 +21,7 @@ import org.osgi.service.component.annotations.Component;
 import org.osgi.service.component.annotations.Reference;
 import org.osgi.service.http.HttpService;
 
-import com.vaadin.osgi.resources.OSGiVaadinResources;
+import com.vaadin.osgi.resources.OsgiVaadinResources;
 import com.vaadin.osgi.resources.VaadinResourceService;
 
 @Component(immediate = true)
@@ -32,7 +32,7 @@ public class DefaultWidgetsetContribution {
 
     @Activate
     void startup(ComponentContext context) throws Exception {
-        VaadinResourceService service = OSGiVaadinResources.getService();
+        VaadinResourceService service = OsgiVaadinResources.getService();
         service.publishWidgetset(WIDGETSET_NAME, httpService);
     }
 
index f9c4c4d121e66d663f0441afb7d14a9cd30f132e..173827d1f8c0ac557bed38a8950a9b69a25f54b3 100644 (file)
@@ -21,7 +21,7 @@ import org.osgi.service.component.annotations.Component;
 import org.osgi.service.component.annotations.Reference;
 import org.osgi.service.http.HttpService;
 
-import com.vaadin.osgi.resources.OSGiVaadinResources;
+import com.vaadin.osgi.resources.OsgiVaadinResources;
 import com.vaadin.osgi.resources.VaadinResourceService;
 
 @Component(immediate = true)
@@ -32,7 +32,7 @@ public class CompatibilityWidgetsetContribution {
 
     @Activate
     void startup(ComponentContext context) throws Exception {
-        VaadinResourceService service = OSGiVaadinResources.getService();
+        VaadinResourceService service = OsgiVaadinResources.getService();
         service.publishWidgetset(WIDGETSET_NAME, httpService);
     }
 
index dd1b933bb980b8e1a64f99e75b9f3ab5e460cd03..2214e40b5e4cc6fbfe738b0e65e195c8124880b4 100644 (file)
@@ -20,7 +20,7 @@ import org.osgi.service.component.annotations.Component;
 import org.osgi.service.component.annotations.Reference;
 import org.osgi.service.http.HttpService;
 
-import com.vaadin.osgi.resources.OSGiVaadinResources;
+import com.vaadin.osgi.resources.OsgiVaadinResources;
 import com.vaadin.osgi.resources.VaadinResourceService;
 
 @Component(immediate = true)
@@ -32,7 +32,7 @@ public class LegacyThemeContributions {
 
     @Activate
     void startup() throws Exception {
-        VaadinResourceService service = OSGiVaadinResources.getService();
+        VaadinResourceService service = OsgiVaadinResources.getService();
         for (String themeName : LEGACY_THEMES) {
             service.publishTheme(themeName, httpService);
         }
index b5c241f23fb6d2c5a0b7678e5f970d480381e99e..0f81106f51885c8408085cbe6a4558ae9edc0432 100644 (file)
@@ -41,12 +41,12 @@ public static class MyUIServlet extends VaadinServlet {
 
 Vaadin Framework 8.1 and later versions provide two supported ways of publishing static resources for OSGi: by making OSGi services implementing an interface or by explicit calls to a service.
 
-The easiest way to publish a theme or a widgetset is to create a class implementing the interface [interfacename]#OSGiVaadinTheme# or [interfacename]#OSGiVaadinWidgetset# and annotating it with [interfacename]#@Component# to make it an OSGi service. This automatically publishes the theme or the widget set from the bundle at a path that contains the Vaadin Framework version used by the application.
+The easiest way to publish a theme or a widgetset is to create a class implementing the interface [interfacename]#OsgiVaadinTheme# or [interfacename]#OsgiVaadinWidgetset# and annotating it with [interfacename]#@Component# to make it an OSGi service. This automatically publishes the theme or the widget set from the bundle at a path that contains the Vaadin Framework version used by the application.
 
 [source, java]
 ----
 @Component
-public class MyTheme extends ValoTheme implements OSGiVaadinTheme {
+public class MyTheme extends ValoTheme implements OsgiVaadinTheme {
     public static final String THEME_NAME = "mytheme";
 
     @Override
@@ -57,11 +57,11 @@ public class MyTheme extends ValoTheme implements OSGiVaadinTheme {
 }
 ----
 
-Alternatively, an OSGi bundle activator or an SCR Component [interfacename]#@Activate# method can obtain an instance of [classname]#VaadinResourceService# from [classname]#OSGiVaadinResources# and explicitly call its methods to publish a theme, a widget set or an individual file in the bundle as a static resource at the correct path.
+Alternatively, an OSGi bundle activator or an SCR Component [interfacename]#@Activate# method can obtain an instance of [classname]#VaadinResourceService# from [classname]#OsgiVaadinResources# and explicitly call its methods to publish a theme, a widget set or an individual file in the bundle as a static resource at the correct path.
 
 [source, java]
 ----
-  VaadinResourceService service = OSGiVaadinResources.getService();
+  VaadinResourceService service = OsgiVaadinResources.getService();
   service.publishTheme("mytheme", httpService);
 ----
 
diff --git a/liferay-integration/src/main/java/com/vaadin/osgi/liferay/OSGiUIProvider.java b/liferay-integration/src/main/java/com/vaadin/osgi/liferay/OSGiUIProvider.java
deleted file mode 100644 (file)
index ccbd2fd..0000000
+++ /dev/null
@@ -1,59 +0,0 @@
-/*
- * Copyright 2000-2016 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.osgi.liferay;
-
-import org.osgi.framework.ServiceObjects;
-
-import com.vaadin.server.UIClassSelectionEvent;
-import com.vaadin.server.UIProvider;
-import com.vaadin.ui.UI;
-
-/**
- * Vaadin {@link com.vaadin.server.UIProvider} that provides a single {@link UI}
- * class provided through the registration of a {@link UI} as an OSGi service.
- * <p>
- * This only applies to Liferay Portal 7+ with OSGi support.
- *
- * @author Sampsa Sohlman
- *
- * @since 8.1
- */
-@SuppressWarnings("serial")
-public class OSGiUIProvider extends UIProvider {
-    private Class<UI> uiClass;
-
-    @SuppressWarnings("unchecked")
-    public OSGiUIProvider(ServiceObjects<UI> serviceObjects) {
-        super();
-        UI ui = serviceObjects.getService();
-        uiClass = (Class<UI>) ui.getClass();
-        serviceObjects.ungetService(ui);
-    }
-
-    @Override
-    public Class<? extends UI> getUIClass(UIClassSelectionEvent event) {
-        return uiClass;
-    }
-
-    public String getDefaultPortletName() {
-        return uiClass.getName();
-    }
-
-    public String getDefaultDisplayName() {
-        return uiClass.getSimpleName();
-    }
-
-}
diff --git a/liferay-integration/src/main/java/com/vaadin/osgi/liferay/OSGiVaadinPortletService.java b/liferay-integration/src/main/java/com/vaadin/osgi/liferay/OSGiVaadinPortletService.java
deleted file mode 100644 (file)
index c203fc4..0000000
+++ /dev/null
@@ -1,59 +0,0 @@
-/*
- * Copyright 2000-2016 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.osgi.liferay;
-
-import com.vaadin.server.DeploymentConfiguration;
-import com.vaadin.server.ServiceException;
-import com.vaadin.server.VaadinPortlet;
-import com.vaadin.server.VaadinPortletService;
-import com.vaadin.server.VaadinPortletSession;
-import com.vaadin.server.VaadinRequest;
-import com.vaadin.server.VaadinSession;
-import com.vaadin.ui.UI;
-
-/**
- * {@link VaadinPortletService} class that uses the {@link OSGiUIProvider} to
- * configure the {@link UI} class for a {@link VaadinPortlet}.
- * <p>
- * This only applies to Liferay Portal 7+ with OSGi support.
- *
- * @author Sampsa Sohlman
- *
- * @since 8.1
- */
-@SuppressWarnings("serial")
-public class OSGiVaadinPortletService extends VaadinPortletService {
-    private OSGiUIProvider osgiUIProvider;
-
-    public OSGiVaadinPortletService(VaadinPortlet portlet,
-            DeploymentConfiguration deploymentConfiguration,
-            OSGiUIProvider osgiUIProvider) throws ServiceException {
-
-        super(portlet, deploymentConfiguration);
-        this.osgiUIProvider = osgiUIProvider;
-    }
-
-    @Override
-    protected VaadinSession createVaadinSession(VaadinRequest request)
-            throws ServiceException {
-
-        VaadinSession vaadinSession = new VaadinPortletSession(this);
-        vaadinSession.addUIProvider(osgiUIProvider);
-
-        return vaadinSession;
-    }
-
-}
diff --git a/liferay-integration/src/main/java/com/vaadin/osgi/liferay/OsgiUIProvider.java b/liferay-integration/src/main/java/com/vaadin/osgi/liferay/OsgiUIProvider.java
new file mode 100644 (file)
index 0000000..072dcbd
--- /dev/null
@@ -0,0 +1,59 @@
+/*
+ * Copyright 2000-2016 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.osgi.liferay;
+
+import org.osgi.framework.ServiceObjects;
+
+import com.vaadin.server.UIClassSelectionEvent;
+import com.vaadin.server.UIProvider;
+import com.vaadin.ui.UI;
+
+/**
+ * Vaadin {@link com.vaadin.server.UIProvider} that provides a single {@link UI}
+ * class provided through the registration of a {@link UI} as an OSGi service.
+ * <p>
+ * This only applies to Liferay Portal 7+ with OSGi support.
+ *
+ * @author Sampsa Sohlman
+ *
+ * @since 8.1
+ */
+@SuppressWarnings("serial")
+public class OsgiUIProvider extends UIProvider {
+    private Class<UI> uiClass;
+
+    @SuppressWarnings("unchecked")
+    public OsgiUIProvider(ServiceObjects<UI> serviceObjects) {
+        super();
+        UI ui = serviceObjects.getService();
+        uiClass = (Class<UI>) ui.getClass();
+        serviceObjects.ungetService(ui);
+    }
+
+    @Override
+    public Class<? extends UI> getUIClass(UIClassSelectionEvent event) {
+        return uiClass;
+    }
+
+    public String getDefaultPortletName() {
+        return uiClass.getName();
+    }
+
+    public String getDefaultDisplayName() {
+        return uiClass.getSimpleName();
+    }
+
+}
diff --git a/liferay-integration/src/main/java/com/vaadin/osgi/liferay/OsgiVaadinPortlet.java b/liferay-integration/src/main/java/com/vaadin/osgi/liferay/OsgiVaadinPortlet.java
new file mode 100644 (file)
index 0000000..0d6c648
--- /dev/null
@@ -0,0 +1,50 @@
+/*
+ * Copyright 2000-2016 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.osgi.liferay;
+
+import com.vaadin.server.DeploymentConfiguration;
+import com.vaadin.server.ServiceException;
+import com.vaadin.server.VaadinPortlet;
+import com.vaadin.server.VaadinPortletService;
+import com.vaadin.ui.UI;
+
+/**
+ * {@link VaadinPortlet} that uses an {@link OsgiUIProvider} to configure its
+ * {@link UI}.
+ * <p>
+ * This only applies to Liferay Portal 7+ with OSGi support.
+ *
+ * @author Sampsa Sohlman
+ *
+ * @since 8.1
+ */
+@SuppressWarnings("serial")
+public class OsgiVaadinPortlet extends VaadinPortlet {
+    private OsgiUIProvider uiProvider;
+
+    public OsgiVaadinPortlet(OsgiUIProvider uiProvider) {
+        this.uiProvider = uiProvider;
+    }
+
+    @Override
+    protected VaadinPortletService createPortletService(
+            DeploymentConfiguration configuration) throws ServiceException {
+        OsgiVaadinPortletService osgiVaadinPortletService = new OsgiVaadinPortletService(
+                this, configuration, uiProvider);
+        osgiVaadinPortletService.init();
+        return osgiVaadinPortletService;
+    }
+}
diff --git a/liferay-integration/src/main/java/com/vaadin/osgi/liferay/OsgiVaadinPortletService.java b/liferay-integration/src/main/java/com/vaadin/osgi/liferay/OsgiVaadinPortletService.java
new file mode 100644 (file)
index 0000000..38858ec
--- /dev/null
@@ -0,0 +1,59 @@
+/*
+ * Copyright 2000-2016 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.osgi.liferay;
+
+import com.vaadin.server.DeploymentConfiguration;
+import com.vaadin.server.ServiceException;
+import com.vaadin.server.VaadinPortlet;
+import com.vaadin.server.VaadinPortletService;
+import com.vaadin.server.VaadinPortletSession;
+import com.vaadin.server.VaadinRequest;
+import com.vaadin.server.VaadinSession;
+import com.vaadin.ui.UI;
+
+/**
+ * {@link VaadinPortletService} class that uses the {@link OsgiUIProvider} to
+ * configure the {@link UI} class for a {@link VaadinPortlet}.
+ * <p>
+ * This only applies to Liferay Portal 7+ with OSGi support.
+ *
+ * @author Sampsa Sohlman
+ *
+ * @since 8.1
+ */
+@SuppressWarnings("serial")
+public class OsgiVaadinPortletService extends VaadinPortletService {
+    private OsgiUIProvider osgiUIProvider;
+
+    public OsgiVaadinPortletService(VaadinPortlet portlet,
+            DeploymentConfiguration deploymentConfiguration,
+            OsgiUIProvider osgiUIProvider) throws ServiceException {
+
+        super(portlet, deploymentConfiguration);
+        this.osgiUIProvider = osgiUIProvider;
+    }
+
+    @Override
+    protected VaadinSession createVaadinSession(VaadinRequest request)
+            throws ServiceException {
+
+        VaadinSession vaadinSession = new VaadinPortletSession(this);
+        vaadinSession.addUIProvider(osgiUIProvider);
+
+        return vaadinSession;
+    }
+
+}
index 0fc7c1eba8b77234789aba45e209be3bb3a31d4f..077730d2a85f4ba829face5ee21bf14838e0d6c0 100644 (file)
@@ -102,7 +102,7 @@ class PortletUIServiceTrackerCustomizer
         ServiceObjects<UI> serviceObjects = bundleContext
                 .getServiceObjects(reference);
 
-        OSGiUIProvider uiProvider = new OSGiUIProvider(serviceObjects);
+        OsgiUIProvider uiProvider = new OsgiUIProvider(serviceObjects);
 
         Dictionary<String, Object> properties = null;
         if (configuration != null) {
@@ -112,7 +112,7 @@ class PortletUIServiceTrackerCustomizer
             properties = createPortletProperties(reference);
         }
 
-        VaadinOSGiPortlet portlet = new VaadinOSGiPortlet(uiProvider);
+        OsgiVaadinPortlet portlet = new OsgiVaadinPortlet(uiProvider);
 
         ServiceRegistration<Portlet> serviceRegistration = bundleContext
                 .registerService(Portlet.class, portlet, properties);
@@ -123,7 +123,7 @@ class PortletUIServiceTrackerCustomizer
     }
 
     private Dictionary<String, Object> createPortletProperties(
-            OSGiUIProvider uiProvider, ServiceReference<UI> reference,
+            OsgiUIProvider uiProvider, ServiceReference<UI> reference,
             VaadinLiferayPortletConfiguration configuration) {
 
         Hashtable<String, Object> properties = new Hashtable<String, Object>();
diff --git a/liferay-integration/src/main/java/com/vaadin/osgi/liferay/VaadinOSGiPortlet.java b/liferay-integration/src/main/java/com/vaadin/osgi/liferay/VaadinOSGiPortlet.java
deleted file mode 100644 (file)
index e604ec3..0000000
+++ /dev/null
@@ -1,50 +0,0 @@
-/*
- * Copyright 2000-2016 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.osgi.liferay;
-
-import com.vaadin.server.DeploymentConfiguration;
-import com.vaadin.server.ServiceException;
-import com.vaadin.server.VaadinPortlet;
-import com.vaadin.server.VaadinPortletService;
-import com.vaadin.ui.UI;
-
-/**
- * {@link VaadinPortlet} that uses an {@link OSGiUIProvider} to configure its
- * {@link UI}.
- * <p>
- * This only applies to Liferay Portal 7+ with OSGi support.
- *
- * @author Sampsa Sohlman
- *
- * @since 8.1
- */
-@SuppressWarnings("serial")
-public class VaadinOSGiPortlet extends VaadinPortlet {
-    private OSGiUIProvider uiProvider;
-
-    public VaadinOSGiPortlet(OSGiUIProvider uiProvider) {
-        this.uiProvider = uiProvider;
-    }
-
-    @Override
-    protected VaadinPortletService createPortletService(
-            DeploymentConfiguration configuration) throws ServiceException {
-        OSGiVaadinPortletService osgiVaadinPortletService = new OSGiVaadinPortletService(
-                this, configuration, uiProvider);
-        osgiVaadinPortletService.init();
-        return osgiVaadinPortletService;
-    }
-}
index 046fc489ae23b40aa9bd412383b745e8755f1d25..78d0bff3833edb67896f0559f7017ca5df965ff6 100644 (file)
@@ -23,7 +23,7 @@ import org.osgi.service.component.annotations.Component;
 import org.osgi.service.component.annotations.Deactivate;
 import org.osgi.util.tracker.ServiceTracker;
 
-import com.vaadin.osgi.resources.OSGiVaadinResources;
+import com.vaadin.osgi.resources.OsgiVaadinResources;
 import com.vaadin.osgi.resources.VaadinResourceService;
 import com.vaadin.ui.UI;
 
@@ -46,7 +46,7 @@ public class VaadinPortletProvider {
     @Activate
     void activate(ComponentContext componentContext) throws Exception {
         BundleContext bundleContext = componentContext.getBundleContext();
-        VaadinResourceService service = OSGiVaadinResources.getService();
+        VaadinResourceService service = OsgiVaadinResources.getService();
 
         portletUIServiceTrackerCustomizer = new PortletUIServiceTrackerCustomizer(
                 service);
index c74e830b458ef2d41d773101f68846fd0467acbe..d7ec6289ce7bc2b4245a08c02fff709ce4aa30df 100644 (file)
@@ -33,8 +33,8 @@ import org.osgi.service.component.annotations.ReferencePolicy;
 import org.osgi.service.http.whiteboard.HttpWhiteboardConstants;
 import org.osgi.service.log.LogService;
 
-import com.vaadin.osgi.resources.OSGiVaadinResources;
-import com.vaadin.osgi.resources.OSGiVaadinResources.ResourceBundleInactiveException;
+import com.vaadin.osgi.resources.OsgiVaadinResources;
+import com.vaadin.osgi.resources.OsgiVaadinResources.ResourceBundleInactiveException;
 import com.vaadin.osgi.resources.VaadinResourceService;
 import com.vaadin.server.Constants;
 import com.vaadin.server.VaadinServlet;
@@ -118,7 +118,7 @@ public class VaadinServletRegistration {
     }
 
     private String getResourcePath() throws ResourceBundleInactiveException {
-        VaadinResourceService service = OSGiVaadinResources.getService();
+        VaadinResourceService service = OsgiVaadinResources.getService();
         return String.format("/%s", service.getResourcePathPrefix());
     }
 
index 8d4564c58295f805fbae30e063188652e92827a1..9b31af12a95e86a1f0c0f49e629965cbd9c1f343 100644 (file)
@@ -21,7 +21,7 @@ import org.osgi.service.component.annotations.Component;
 import org.osgi.service.component.annotations.Reference;
 import org.osgi.service.http.HttpService;
 
-import com.vaadin.osgi.resources.OSGiVaadinResources;
+import com.vaadin.osgi.resources.OsgiVaadinResources;
 import com.vaadin.osgi.resources.VaadinResourceService;
 
 @Component(immediate = true)
@@ -34,7 +34,7 @@ public class PushResourcesContribution {
 
     @Activate
     void startup(ComponentContext context) throws Exception {
-        VaadinResourceService service = OSGiVaadinResources.getService();
+        VaadinResourceService service = OsgiVaadinResources.getService();
         for (String resourceName : RESOURCES) {
             service.publishResource(resourceName, httpService);
         }
index c68ed518ba971a8744bea6974f101c8ee8c1f1f1..e04594273f0f4288d8dc28d81b4a75e7dd76b2d2 100644 (file)
@@ -21,8 +21,8 @@ import org.osgi.service.component.annotations.Reference;
 import org.osgi.service.http.HttpService;
 import org.osgi.service.http.NamespaceException;
 
-import com.vaadin.osgi.resources.OSGiVaadinResources;
-import com.vaadin.osgi.resources.OSGiVaadinResources.ResourceBundleInactiveException;
+import com.vaadin.osgi.resources.OsgiVaadinResources;
+import com.vaadin.osgi.resources.OsgiVaadinResources.ResourceBundleInactiveException;
 import com.vaadin.osgi.resources.VaadinResourceService;
 
 /**
@@ -40,7 +40,7 @@ public class BootstrapContribution {
 
     @Activate
     void startup() throws NamespaceException, ResourceBundleInactiveException {
-        VaadinResourceService service = OSGiVaadinResources.getService();
+        VaadinResourceService service = OsgiVaadinResources.getService();
         for (String resourceName : RESOURCES) {
             service.publishResource(resourceName, httpService);
         }
index ff4e85732f4949f66c675965430ef37e3fc97b86..9821bc67a7c4ce813d7b3cbb47a23540739a92cb 100644 (file)
@@ -1,5 +1,5 @@
 Bundle-SymbolicName: ${project.groupId}.shared
-Bundle-Activator: com.vaadin.osgi.resources.OSGiVaadinResources
+Bundle-Activator: com.vaadin.osgi.resources.OsgiVaadinResources
 Bundle-Name: Vaadin Shared
 Bundle-RequiredExecutionEnvironment: JavaSE-1.8
 Bundle-License: http://www.apache.org/licenses/LICENSE-2.0
diff --git a/shared/src/main/java/com/vaadin/osgi/resources/OSGiVaadinResources.java b/shared/src/main/java/com/vaadin/osgi/resources/OSGiVaadinResources.java
deleted file mode 100644 (file)
index 96f6f99..0000000
+++ /dev/null
@@ -1,86 +0,0 @@
-/*
- * Copyright 2000-2016 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.osgi.resources;
-
-import org.osgi.framework.BundleActivator;
-import org.osgi.framework.BundleContext;
-import org.osgi.framework.Version;
-
-import com.vaadin.osgi.resources.impl.VaadinResourceServiceImpl;
-
-/**
- * {@link BundleActivator} used to provide access to the
- * {@link VaadinResourceService} singleton for publishing themes, widgetsets and
- * other necessary resources.
- * 
- * @author Vaadin Ltd.
- * 
- * @since 8.1
- */
-public class OSGiVaadinResources implements BundleActivator {
-
-    /**
-     * Thrown if a method is called when the Resource bundle is not active.
-     * 
-     * @author Vaadin Ltd.
-     *
-     * @since 8.1
-     */
-    @SuppressWarnings("serial")
-    public static class ResourceBundleInactiveException extends Exception {
-        public ResourceBundleInactiveException(String message) {
-            super(message);
-        }
-    }
-
-    private static OSGiVaadinResources instance;
-
-    private VaadinResourceServiceImpl service;
-    private Version version;
-
-    /**
-     * Returns the {@link VaadinResourceService} instance. Always returns
-     * non-null.
-     * 
-     * @return the {@link VaadinResourceService resource service} to use for
-     *         publishing themes, widgetsets and other necessary resources
-     * @throws ResourceBundleInactiveException
-     *             if the bundle is not active
-     */
-    public static VaadinResourceService getService()
-            throws ResourceBundleInactiveException {
-        if (instance == null) {
-            throw new ResourceBundleInactiveException(
-                    "Vaadin Shared is not active!");
-        }
-        return instance.service;
-    }
-
-    @Override
-    public void start(BundleContext context) throws Exception {
-        version = context.getBundle().getVersion();
-        service = new VaadinResourceServiceImpl();
-        service.setBundleVersion(version.toString());
-        instance = this;
-    }
-
-    @Override
-    public void stop(BundleContext context) throws Exception {
-        instance = null;
-        service = null;
-        version = null;
-    }
-}
diff --git a/shared/src/main/java/com/vaadin/osgi/resources/OSGiVaadinTheme.java b/shared/src/main/java/com/vaadin/osgi/resources/OSGiVaadinTheme.java
deleted file mode 100644 (file)
index 852dd93..0000000
+++ /dev/null
@@ -1,35 +0,0 @@
-/*
- * Copyright 2000-2016 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.osgi.resources;
-
-/**
- * Used to declare a Vaadin Theme for use in OSGi. The theme is expected to be
- * in the same OSGi bundle as the class implementing this interface, under the
- * path "/VAADIN/themes/{themeName}" where {themeName} is what is returned by
- * {@link OSGiVaadinTheme#getName()}.
- * <p>
- * To publish a theme, an implementation of this interface needs to be
- * registered as an OSGi service, which makes
- * <code>VaadinResourceTrackerComponent</code> automatically publish the theme
- * with the given name.
- *
- * @author Vaadin Ltd.
- *
- * @since 8.1
- */
-public interface OSGiVaadinTheme {
-    public String getName();
-}
diff --git a/shared/src/main/java/com/vaadin/osgi/resources/OSGiVaadinWidgetset.java b/shared/src/main/java/com/vaadin/osgi/resources/OSGiVaadinWidgetset.java
deleted file mode 100644 (file)
index c594443..0000000
+++ /dev/null
@@ -1,35 +0,0 @@
-/*
- * Copyright 2000-2016 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.osgi.resources;
-
-/**
- * Used to declare a Vaadin Widgetset for use in OSGi. The widgetset is expected
- * to be in the same OSGi bundle as the class implementing this interface, under
- * the path "/VAADIN/widgetsets/{widgetsetName}" where {widgetsetName} is what
- * is returned by {@link OSGiVaadinWidgetset#getName()}.
- * <p>
- * To publish a widgetset, an implementation of this interface needs to be
- * registered as an OSGi service, which makes
- * <code>VaadinResourceTrackerComponent</code> automatically publish the
- * widgetset with the given name.
- *
- * @author Vaadin Ltd.
- *
- * @since 8.1
- */
-public interface OSGiVaadinWidgetset {
-    public String getName();
-}
diff --git a/shared/src/main/java/com/vaadin/osgi/resources/OsgiVaadinResources.java b/shared/src/main/java/com/vaadin/osgi/resources/OsgiVaadinResources.java
new file mode 100644 (file)
index 0000000..a423791
--- /dev/null
@@ -0,0 +1,86 @@
+/*
+ * Copyright 2000-2016 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.osgi.resources;
+
+import org.osgi.framework.BundleActivator;
+import org.osgi.framework.BundleContext;
+import org.osgi.framework.Version;
+
+import com.vaadin.osgi.resources.impl.VaadinResourceServiceImpl;
+
+/**
+ * {@link BundleActivator} used to provide access to the
+ * {@link VaadinResourceService} singleton for publishing themes, widgetsets and
+ * other necessary resources.
+ * 
+ * @author Vaadin Ltd.
+ * 
+ * @since 8.1
+ */
+public class OsgiVaadinResources implements BundleActivator {
+
+    /**
+     * Thrown if a method is called when the Resource bundle is not active.
+     * 
+     * @author Vaadin Ltd.
+     *
+     * @since 8.1
+     */
+    @SuppressWarnings("serial")
+    public static class ResourceBundleInactiveException extends Exception {
+        public ResourceBundleInactiveException(String message) {
+            super(message);
+        }
+    }
+
+    private static OsgiVaadinResources instance;
+
+    private VaadinResourceServiceImpl service;
+    private Version version;
+
+    /**
+     * Returns the {@link VaadinResourceService} instance. Always returns
+     * non-null.
+     * 
+     * @return the {@link VaadinResourceService resource service} to use for
+     *         publishing themes, widgetsets and other necessary resources
+     * @throws ResourceBundleInactiveException
+     *             if the bundle is not active
+     */
+    public static VaadinResourceService getService()
+            throws ResourceBundleInactiveException {
+        if (instance == null) {
+            throw new ResourceBundleInactiveException(
+                    "Vaadin Shared is not active!");
+        }
+        return instance.service;
+    }
+
+    @Override
+    public void start(BundleContext context) throws Exception {
+        version = context.getBundle().getVersion();
+        service = new VaadinResourceServiceImpl();
+        service.setBundleVersion(version.toString());
+        instance = this;
+    }
+
+    @Override
+    public void stop(BundleContext context) throws Exception {
+        instance = null;
+        service = null;
+        version = null;
+    }
+}
diff --git a/shared/src/main/java/com/vaadin/osgi/resources/OsgiVaadinTheme.java b/shared/src/main/java/com/vaadin/osgi/resources/OsgiVaadinTheme.java
new file mode 100644 (file)
index 0000000..0b4866b
--- /dev/null
@@ -0,0 +1,40 @@
+/*
+ * Copyright 2000-2016 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.osgi.resources;
+
+/**
+ * Used to declare a Vaadin Theme for use in OSGi. The theme is expected to be
+ * in the same OSGi bundle as the class implementing this interface, under the
+ * path "/VAADIN/themes/{themeName}" where {themeName} is what is returned by
+ * {@link OsgiVaadinTheme#getName()}.
+ * <p>
+ * To publish a theme, an implementation of this interface needs to be
+ * registered as an OSGi service, which makes
+ * <code>VaadinResourceTrackerComponent</code> automatically publish the theme
+ * with the given name.
+ *
+ * @author Vaadin Ltd.
+ *
+ * @since 8.1
+ */
+public interface OsgiVaadinTheme {
+    /**
+     * Return the theme name to publish for OSGi.
+     *
+     * @return theme name, not null
+     */
+    public String getName();
+}
diff --git a/shared/src/main/java/com/vaadin/osgi/resources/OsgiVaadinWidgetset.java b/shared/src/main/java/com/vaadin/osgi/resources/OsgiVaadinWidgetset.java
new file mode 100644 (file)
index 0000000..81e957e
--- /dev/null
@@ -0,0 +1,40 @@
+/*
+ * Copyright 2000-2016 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.osgi.resources;
+
+/**
+ * Used to declare a Vaadin Widgetset for use in OSGi. The widgetset is expected
+ * to be in the same OSGi bundle as the class implementing this interface, under
+ * the path "/VAADIN/widgetsets/{widgetsetName}" where {widgetsetName} is what
+ * is returned by {@link OsgiVaadinWidgetset#getName()}.
+ * <p>
+ * To publish a widgetset, an implementation of this interface needs to be
+ * registered as an OSGi service, which makes
+ * <code>VaadinResourceTrackerComponent</code> automatically publish the
+ * widgetset with the given name.
+ *
+ * @author Vaadin Ltd.
+ *
+ * @since 8.1
+ */
+public interface OsgiVaadinWidgetset {
+    /**
+     * Return the widgetset name to publish for OSGi.
+     *
+     * @return widgetset name, not null
+     */
+    public String getName();
+}
index 0cf6aaeb18a7f593892d5d1c3fe329d8b744ac60..adbde7d79842d75dd25d3a90ba8c6c5866180486 100644 (file)
@@ -36,14 +36,14 @@ import org.osgi.service.http.HttpContext;
 import org.osgi.service.http.HttpService;
 import org.osgi.service.http.NamespaceException;
 
-import com.vaadin.osgi.resources.OSGiVaadinResources;
-import com.vaadin.osgi.resources.OSGiVaadinResources.ResourceBundleInactiveException;
-import com.vaadin.osgi.resources.OSGiVaadinTheme;
-import com.vaadin.osgi.resources.OSGiVaadinWidgetset;
+import com.vaadin.osgi.resources.OsgiVaadinResources;
+import com.vaadin.osgi.resources.OsgiVaadinResources.ResourceBundleInactiveException;
+import com.vaadin.osgi.resources.OsgiVaadinTheme;
+import com.vaadin.osgi.resources.OsgiVaadinWidgetset;
 import com.vaadin.osgi.resources.VaadinResourceService;
 
 /**
- * Tracks {@link OSGiVaadinWidgetset} and {@link OSGiVaadinTheme} registration
+ * Tracks {@link OsgiVaadinWidgetset} and {@link OsgiVaadinTheme} registration
  * and uses {@link HttpService} to register them.
  * 
  * @author Vaadin Ltd.
@@ -59,18 +59,18 @@ public class VaadinResourceTrackerComponent {
     private Map<Long, String> widgetsetToAlias = Collections
             .synchronizedMap(new LinkedHashMap<>());
 
-    @Reference(cardinality = ReferenceCardinality.MULTIPLE, service = OSGiVaadinTheme.class, policy = ReferencePolicy.DYNAMIC)
-    void bindTheme(ServiceReference<OSGiVaadinTheme> themeRef)
+    @Reference(cardinality = ReferenceCardinality.MULTIPLE, service = OsgiVaadinTheme.class, policy = ReferencePolicy.DYNAMIC)
+    void bindTheme(ServiceReference<OsgiVaadinTheme> themeRef)
             throws ResourceBundleInactiveException, NamespaceException {
 
         Bundle bundle = themeRef.getBundle();
         BundleContext context = bundle.getBundleContext();
 
-        OSGiVaadinTheme theme = context.getService(themeRef);
+        OsgiVaadinTheme theme = context.getService(themeRef);
         if (theme == null)
             return;
 
-        VaadinResourceService resourceService = OSGiVaadinResources
+        VaadinResourceService resourceService = OsgiVaadinResources
                 .getService();
 
         try {
@@ -90,7 +90,7 @@ public class VaadinResourceTrackerComponent {
         }
     }
 
-    void unbindTheme(ServiceReference<OSGiVaadinTheme> themeRef) {
+    void unbindTheme(ServiceReference<OsgiVaadinTheme> themeRef) {
         Long serviceId = (Long) themeRef.getProperty(Constants.SERVICE_ID);
         String themeAlias = themeToAlias.remove(serviceId);
         if (themeAlias != null && httpService != null) {
@@ -98,17 +98,17 @@ public class VaadinResourceTrackerComponent {
         }
     }
 
-    @Reference(cardinality = ReferenceCardinality.MULTIPLE, service = OSGiVaadinWidgetset.class, policy = ReferencePolicy.DYNAMIC)
-    void bindWidgetset(ServiceReference<OSGiVaadinWidgetset> widgetsetRef)
+    @Reference(cardinality = ReferenceCardinality.MULTIPLE, service = OsgiVaadinWidgetset.class, policy = ReferencePolicy.DYNAMIC)
+    void bindWidgetset(ServiceReference<OsgiVaadinWidgetset> widgetsetRef)
             throws ResourceBundleInactiveException, NamespaceException {
         Bundle bundle = widgetsetRef.getBundle();
         BundleContext context = bundle.getBundleContext();
 
-        OSGiVaadinWidgetset widgetset = context.getService(widgetsetRef);
+        OsgiVaadinWidgetset widgetset = context.getService(widgetsetRef);
         if (widgetset == null)
             return;
 
-        VaadinResourceService service = OSGiVaadinResources.getService();
+        VaadinResourceService service = OsgiVaadinResources.getService();
         try {
             String pathPrefix = service.getResourcePathPrefix();
 
@@ -129,7 +129,7 @@ public class VaadinResourceTrackerComponent {
 
     }
 
-    void unbindWidgetset(ServiceReference<OSGiVaadinWidgetset> widgetsetRef) {
+    void unbindWidgetset(ServiceReference<OsgiVaadinWidgetset> widgetsetRef) {
         Long serviceId = (Long) widgetsetRef.getProperty(Constants.SERVICE_ID);
         String widgetsetAlias = widgetsetToAlias.remove(serviceId);
         if (widgetsetAlias != null && httpService != null) {
index cd1b7a4a8e9658dd1fc8c88d03777cf9f8aaf1aa..a31b6cf40403f741b1c80f0a0b8c63f4594f8eb2 100644 (file)
@@ -20,7 +20,7 @@ import org.osgi.service.component.annotations.Component;
 import org.osgi.service.component.annotations.Reference;
 import org.osgi.service.http.HttpService;
 
-import com.vaadin.osgi.resources.OSGiVaadinResources;
+import com.vaadin.osgi.resources.OsgiVaadinResources;
 import com.vaadin.osgi.resources.VaadinResourceService;
 
 @Component(immediate = true)
@@ -30,7 +30,7 @@ public class ValoThemeContribution {
 
     @Activate
     void startup() throws Exception {
-        VaadinResourceService service = OSGiVaadinResources.getService();
+        VaadinResourceService service = OsgiVaadinResources.getService();
         service.publishTheme("valo", httpService);
     }