You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ro...@apache.org on 2017/10/18 23:26:57 UTC

[sling-org-apache-sling-scripting-console] 22/23: SLING-6902: Remove commons.json from Apache Sling Script Console

This is an automated email from the ASF dual-hosted git repository.

rombert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-scripting-console.git

commit 938f8a741685806d003bd6752ec39918bf658aa3
Author: Karl Pauls <pa...@apache.org>
AuthorDate: Thu Jun 1 07:02:02 2017 +0000

    SLING-6902: Remove commons.json from Apache Sling Script Console
    
    git-svn-id: https://svn.apache.org/repos/asf/sling/trunk@1797143 13f79535-47bb-0310-9956-ffa450edef68
---
 pom.xml                                            | 27 ++++++++++++++--------
 .../console/internal/ScriptConsolePlugin.java      |  8 +++----
 2 files changed, 20 insertions(+), 15 deletions(-)

diff --git a/pom.xml b/pom.xml
index 6e97129..0d3345a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -56,6 +56,13 @@
                 <groupId>org.apache.felix</groupId>
                 <artifactId>maven-bundle-plugin</artifactId>
                 <extensions>true</extensions>
+                <configuration>
+                    <instructions>
+                        <Embed-Dependency>
+                                org.apache.felix.utils;inline=org/apache/felix/utils/json/JSONWriter.class
+                        </Embed-Dependency>
+                    </instructions>
+                </configuration>
             </plugin>
             <plugin>
                 <groupId>org.apache.rat</groupId>
@@ -88,15 +95,21 @@
     </reporting>
 
     <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>
             <version>4.2.0</version>
         </dependency>
- 		<dependency>
-		    <groupId>org.apache.felix</groupId>
-		    <artifactId>org.apache.felix.scr.annotations</artifactId>
-		</dependency>
+         <dependency>
+            <groupId>org.apache.felix</groupId>
+            <artifactId>org.apache.felix.scr.annotations</artifactId>
+        </dependency>
         <dependency>
             <groupId>javax.servlet</groupId>
             <artifactId>servlet-api</artifactId>
@@ -141,12 +154,6 @@
             <version>2.0.4</version>
         </dependency>
         <dependency>
-            <groupId>org.apache.sling</groupId>
-            <artifactId>org.apache.sling.commons.json</artifactId>
-            <version>2.0.6</version>
-            <scope>provided</scope>
-        </dependency>
-        <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-api</artifactId>
         </dependency>
diff --git a/src/main/java/org/apache/sling/scripting/console/internal/ScriptConsolePlugin.java b/src/main/java/org/apache/sling/scripting/console/internal/ScriptConsolePlugin.java
index ff2df9b..44294cc 100644
--- a/src/main/java/org/apache/sling/scripting/console/internal/ScriptConsolePlugin.java
+++ b/src/main/java/org/apache/sling/scripting/console/internal/ScriptConsolePlugin.java
@@ -39,6 +39,7 @@ import org.apache.felix.scr.annotations.Deactivate;
 import org.apache.felix.scr.annotations.Property;
 import org.apache.felix.scr.annotations.Reference;
 import org.apache.felix.scr.annotations.Service;
+import org.apache.felix.utils.json.JSONWriter;
 import org.apache.felix.webconsole.AbstractWebConsolePlugin;
 import org.apache.felix.webconsole.DefaultVariableResolver;
 import org.apache.felix.webconsole.SimpleWebConsolePlugin;
@@ -46,8 +47,6 @@ import org.apache.felix.webconsole.WebConsoleUtil;
 import org.apache.sling.api.resource.Resource;
 import org.apache.sling.api.scripting.SlingBindings;
 import org.apache.sling.api.scripting.SlingScript;
-import org.apache.sling.commons.json.JSONException;
-import org.apache.sling.commons.json.io.JSONWriter;
 import org.osgi.framework.BundleContext;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -156,15 +155,14 @@ public class ScriptConsolePlugin extends SimpleWebConsolePlugin {
     private String getScriptConfig() {
         try {
             return getScriptConfig0();
-        } catch (JSONException e) {
+        } catch (IOException e) {
             throw new RuntimeException(e);
         }
     }
 
-    private String getScriptConfig0() throws JSONException {
+    private String getScriptConfig0() throws IOException {
         StringWriter sw = new StringWriter();
         JSONWriter jw = new JSONWriter(sw);
-        jw.setTidy(true);
         jw.array();
 
         for(ScriptEngineFactory sef : scriptEngineManager.getEngineFactories()){

-- 
To stop receiving notification emails like this one, please contact
"commits@sling.apache.org" <co...@sling.apache.org>.