You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2017/02/27 10:38:31 UTC

svn commit: r1784522 - in /sling/trunk/bundles/servlets/post: ./ src/main/java/org/apache/sling/servlets/post/ src/main/java/org/apache/sling/servlets/post/impl/ src/main/java/org/apache/sling/servlets/post/impl/helper/

Author: cziegeler
Date: Mon Feb 27 10:38:31 2017
New Revision: 1784522

URL: http://svn.apache.org/viewvc?rev=1784522&view=rev
Log:
Move to official OSGi annotations for package exports

Added:
    sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/package-info.java   (with props)
Modified:
    sling/trunk/bundles/servlets/post/pom.xml
    sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/SlingPostServlet.java
    sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/helper/ChunkCleanUpTask.java

Modified: sling/trunk/bundles/servlets/post/pom.xml
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/servlets/post/pom.xml?rev=1784522&r1=1784521&r2=1784522&view=diff
==============================================================================
--- sling/trunk/bundles/servlets/post/pom.xml (original)
+++ sling/trunk/bundles/servlets/post/pom.xml Mon Feb 27 10:38:31 2017
@@ -58,12 +58,6 @@
                 <extensions>true</extensions>
                 <configuration>
                     <instructions>
-                        <Export-Package>
-                            org.apache.sling.servlets.post;version=2.3.0
-                        </Export-Package>
-                        <Private-Package>
-                            org.apache.sling.servlets.post.impl.*
-                        </Private-Package>
                         <Sling-Bundle-Resources>
                             /system/sling.js
                         </Sling-Bundle-Resources>
@@ -74,7 +68,7 @@
                             sling=http://sling.apache.org/jcr/sling/1.0
                         </Sling-Namespaces>
                         <Sling-Nodetypes>
-                            SLING-INF/nodetypes/chunk.cnd,
+                            SLING-INF/nodetypes/chunk.cnd
                         </Sling-Nodetypes>
                     </instructions>
                 </configuration>
@@ -168,11 +162,6 @@
             <scope>provided</scope>
         </dependency>
         <dependency>
-            <groupId>org.apache.felix</groupId>
-            <artifactId>org.apache.felix.scr.annotations</artifactId>
-            <scope>provided</scope>
-        </dependency>
-        <dependency>
             <groupId>org.apache.sling</groupId>
             <artifactId>org.apache.sling.commons.testing</artifactId>
             <version>2.0.10</version>

Modified: sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/SlingPostServlet.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/SlingPostServlet.java?rev=1784522&r1=1784521&r2=1784522&view=diff
==============================================================================
--- sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/SlingPostServlet.java (original)
+++ sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/SlingPostServlet.java Mon Feb 27 10:38:31 2017
@@ -75,7 +75,6 @@ import org.apache.sling.servlets.post.im
 import org.osgi.framework.BundleContext;
 import org.osgi.framework.Constants;
 import org.osgi.framework.ServiceRegistration;
-import org.osgi.service.component.ComponentContext;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -324,7 +323,7 @@ public class SlingPostServlet extends Sl
                 log.warn("given redirect target ({}) is not a valid uri: {}", result, e);
                 return null;
             }
-            
+
             log.debug("redirect requested as [{}] for path [{}]", result, ctx.getPath());
 
             // redirect to created/modified Resource
@@ -398,14 +397,13 @@ public class SlingPostServlet extends Sl
     // ---------- SCR Integration ----------------------------------------------
 
     @Activate
-    protected void activate(final ComponentContext context,
+    protected void activate(final BundleContext bundleContext,
             final Map<String, Object> configuration) {
         // configure now
         this.configure(configuration);
 
         // other predefined operations
         final ArrayList<ServiceRegistration> providedServices = new ArrayList<ServiceRegistration>();
-        final BundleContext bundleContext = context.getBundleContext();
         providedServices.add(registerOperation(bundleContext,
             SlingPostConstants.OPERATION_MODIFY, modifyOperation));
         providedServices.add(registerOperation(bundleContext,

Modified: sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/helper/ChunkCleanUpTask.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/helper/ChunkCleanUpTask.java?rev=1784522&r1=1784521&r2=1784522&view=diff
==============================================================================
--- sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/helper/ChunkCleanUpTask.java (original)
+++ sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/helper/ChunkCleanUpTask.java Mon Feb 27 10:38:31 2017
@@ -36,7 +36,6 @@ import org.apache.felix.scr.annotations.
 import org.apache.sling.commons.osgi.OsgiUtil;
 import org.apache.sling.jcr.api.SlingRepository;
 import org.apache.sling.servlets.post.SlingPostConstants;
-import org.osgi.service.component.ComponentContext;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -68,7 +67,7 @@ import org.slf4j.LoggerFactory;
         + "of the format for this value."),
     @Property(name = "service.description", value = "Periodic Chunk Cleanup Job", propertyPrivate = true),
     @Property(name = "service.vendor", value = "The Apache Software Foundation", propertyPrivate = true),
-    @Property(name = "scheduler.concurrent", label = "scheduler.concurrent", boolValue = false, 
+    @Property(name = "scheduler.concurrent", label = "scheduler.concurrent", boolValue = false,
         description = "Allow Chunk Cleanup Task to run concurrently (default: false).")})
 public class ChunkCleanUpTask implements Runnable {
 
@@ -91,6 +90,7 @@ public class ChunkCleanUpTask implements
     /**
      * Executes the job. Is called for each triggered schedule point.
      */
+    @Override
     public void run() {
         log.debug("ChunkCleanUpTask: Starting cleanup");
         cleanup();
@@ -175,8 +175,7 @@ public class ChunkCleanUpTask implements
     }
 
     @Activate
-    protected void activate(final ComponentContext context,
-            final Map<String, Object> configuration) {
+    protected void activate(final Map<String, Object> configuration) {
         chunkCleanUpAge = OsgiUtil.toInteger(
             configuration.get(CHUNK_CLEANUP_AGE), 1) * 60 * 1000;
         log.info("scheduler config [{}], chunkGarbageTime  [{}] ms",

Added: sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/package-info.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/package-info.java?rev=1784522&view=auto
==============================================================================
--- sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/package-info.java (added)
+++ sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/package-info.java Mon Feb 27 10:38:31 2017
@@ -0,0 +1,24 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you 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.
+ */
+
+@Version("2.3.0")
+package org.apache.sling.servlets.post;
+
+import org.osgi.annotation.versioning.Version;
+

Propchange: sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/package-info.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/package-info.java
------------------------------------------------------------------------------
    svn:keywords = author date id revision rev url