You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by pa...@apache.org on 2017/06/01 20:14:40 UTC
svn commit: r1797261 - in /sling/trunk/samples/mail-archive/stats: pom.xml
src/main/java/org/apache/sling/mailarchive/stats/impl/StatsDataServlet.java
Author: pauls
Date: Thu Jun 1 20:14:40 2017
New Revision: 1797261
URL: http://svn.apache.org/viewvc?rev=1797261&view=rev
Log:
SLING-6909: Remove commons.json from samples/mail-archive/stats
Modified:
sling/trunk/samples/mail-archive/stats/pom.xml
sling/trunk/samples/mail-archive/stats/src/main/java/org/apache/sling/mailarchive/stats/impl/StatsDataServlet.java
Modified: sling/trunk/samples/mail-archive/stats/pom.xml
URL: http://svn.apache.org/viewvc/sling/trunk/samples/mail-archive/stats/pom.xml?rev=1797261&r1=1797260&r2=1797261&view=diff
==============================================================================
--- sling/trunk/samples/mail-archive/stats/pom.xml (original)
+++ sling/trunk/samples/mail-archive/stats/pom.xml Thu Jun 1 20:14:40 2017
@@ -55,6 +55,9 @@
<Sling-Initial-Content>
initial-content; uninstall:=true
</Sling-Initial-Content>
+ <Embed-Dependency>
+ org.apache.felix.utils;inline=org/apache/felix/utils/json/JSONWriter.class
+ </Embed-Dependency>
</instructions>
</configuration>
</plugin>
@@ -63,6 +66,12 @@
<dependencies>
<dependency>
+ <groupId>org.apache.felix</groupId>
+ <artifactId>org.apache.felix.utils</artifactId>
+ <version>1.9.0</version>
+ <scope>provided</scope>
+ </dependency>
+ <dependency>
<groupId>org.osgi</groupId>
<artifactId>org.osgi.core</artifactId>
</dependency>
@@ -115,11 +124,6 @@
</dependency>
<dependency>
<groupId>org.apache.sling</groupId>
- <artifactId>org.apache.sling.commons.json</artifactId>
- <version>2.0.6</version>
- </dependency>
- <dependency>
- <groupId>org.apache.sling</groupId>
<artifactId>org.apache.sling.mailarchive.server</artifactId>
<version>0.1.0-SNAPSHOT</version>
<scope>provided</scope>
Modified: sling/trunk/samples/mail-archive/stats/src/main/java/org/apache/sling/mailarchive/stats/impl/StatsDataServlet.java
URL: http://svn.apache.org/viewvc/sling/trunk/samples/mail-archive/stats/src/main/java/org/apache/sling/mailarchive/stats/impl/StatsDataServlet.java?rev=1797261&r1=1797260&r2=1797261&view=diff
==============================================================================
--- sling/trunk/samples/mail-archive/stats/src/main/java/org/apache/sling/mailarchive/stats/impl/StatsDataServlet.java (original)
+++ sling/trunk/samples/mail-archive/stats/src/main/java/org/apache/sling/mailarchive/stats/impl/StatsDataServlet.java Thu Jun 1 20:14:40 2017
@@ -27,13 +27,12 @@ import java.util.TreeSet;
import javax.servlet.ServletException;
import org.apache.felix.scr.annotations.sling.SlingServlet;
+import org.apache.felix.utils.json.JSONWriter;
import org.apache.sling.api.SlingHttpServletRequest;
import org.apache.sling.api.SlingHttpServletResponse;
import org.apache.sling.api.resource.Resource;
import org.apache.sling.api.resource.ValueMap;
import org.apache.sling.api.servlets.SlingSafeMethodsServlet;
-import org.apache.sling.commons.json.JSONException;
-import org.apache.sling.commons.json.io.JSONWriter;
@SuppressWarnings("serial")
@SlingServlet(
@@ -85,7 +84,6 @@ public class StatsDataServlet extends Sl
// from those that have the stats data resource type
{
final JSONWriter w = new JSONWriter(response.getWriter());
- w.setTidy(true);
out.write("var statsData = ");
w.array();
dumpStatsData(request.getResource(), w, layers);
@@ -97,7 +95,6 @@ public class StatsDataServlet extends Sl
// Output the layers array in JSON
{
final JSONWriter w = new JSONWriter(response.getWriter());
- w.setTidy(true);
out.write("var layers = ");
w.array();
for(String layer : layers) {
@@ -107,7 +104,7 @@ public class StatsDataServlet extends Sl
out.write(";");
}
- } catch(JSONException je) {
+ } catch(IOException je) {
throw new ServletException("JSONException in doGet", je);
}
}
@@ -115,7 +112,7 @@ public class StatsDataServlet extends Sl
/** Dump stats data to w if r is a stats data resource,
* and recurse into children
*/
- private void dumpStatsData(Resource r, JSONWriter w, Set<String> layers) throws JSONException {
+ private void dumpStatsData(Resource r, JSONWriter w, Set<String> layers) throws IOException {
if(MailStatsProcessorImpl.DATA_RESOURCE_TYPE.equals(r.getResourceType())) {
final ValueMap m = r.adaptTo(ValueMap.class);
if(m != null) {