Author: jbonofre
Date: Thu May  2 17:07:35 2013
New Revision: 1478447

URL: http://svn.apache.org/r1478447
Log:
[KARAF-2285] Fixed Admin WebConsole plugin with Equinox

Modified:
    
karaf/branches/karaf-2.3.x/webconsole/admin/src/main/java/org/apache/karaf/webconsole/admin/AdminPlugin.java
    
karaf/branches/karaf-2.3.x/webconsole/admin/src/main/resources/OSGI-INF/blueprint/webconsole-admin.xml

Modified: 
karaf/branches/karaf-2.3.x/webconsole/admin/src/main/java/org/apache/karaf/webconsole/admin/AdminPlugin.java
URL: 
http://svn.apache.org/viewvc/karaf/branches/karaf-2.3.x/webconsole/admin/src/main/java/org/apache/karaf/webconsole/admin/AdminPlugin.java?rev=1478447&r1=1478446&r2=1478447&view=diff
==============================================================================
--- 
karaf/branches/karaf-2.3.x/webconsole/admin/src/main/java/org/apache/karaf/webconsole/admin/AdminPlugin.java
 (original)
+++ 
karaf/branches/karaf-2.3.x/webconsole/admin/src/main/java/org/apache/karaf/webconsole/admin/AdminPlugin.java
 Thu May  2 17:07:35 2013
@@ -39,7 +39,7 @@ import org.osgi.framework.BundleContext;
 import org.slf4j.LoggerFactory;
 
 /**
- * Felix Web Console plugin for interacting with the {@link AdminService}
+ * WebConsole plugin for the {@link AdminService}.
  */
 public class AdminPlugin extends AbstractWebConsolePlugin {
 
@@ -52,9 +52,6 @@ public class AdminPlugin extends Abstrac
     private AdminService adminService;
     private ClassLoader classLoader;
 
-    /**
-     * Blueprint lifecycle callback methods
-     */
     public void start() {
         super.activate(bundleContext);
         this.classLoader = this.getClass().getClassLoader();
@@ -134,22 +131,19 @@ public class AdminPlugin extends Abstrac
                 Thread.sleep(1000);
             } catch (InterruptedException e) {
             }
-            this.renderJSON(res, null);
+            this.renderJSON(res);
         } else {
             super.doPost(req, res);
         }
     }
 
-    /*
-     * Parse the String value, returning <code>null</code> if the String is 
empty 
-     */
     private String parseString(String value) {
         if (value != null && value.trim().length() == 0) {
             value = null;
         }
         return value;
     }
-    
+
     private List<String> parseStringList(String value) {
         List<String> list = new ArrayList<String>();
         if (value != null) {
@@ -159,14 +153,11 @@ public class AdminPlugin extends Abstrac
                     continue;
                 }
                 list.add(trimmed);
-            }            
+            }
         }
         return list;
     }
 
-    /*
-     * Parse the port number for the String given, returning 0 if the String 
does not represent an integer 
-     */
     private int parsePortNumber(String port) {
         try {
             return Integer.parseInt(port);
@@ -177,6 +168,9 @@ public class AdminPlugin extends Abstrac
 
     protected URL getResource(String path) {
         path = path.substring(NAME.length() + 1);
+        if (path == null || path.isEmpty()) {
+            return null;
+        }
         URL url = this.classLoader.getResource(path);
         if (url != null) {
             InputStream ins = null;
@@ -198,11 +192,11 @@ public class AdminPlugin extends Abstrac
                     }
                 }
             }
-        } 
+        }
         return url;
     }
 
-    private void renderJSON(final HttpServletResponse response, final String 
feature) throws IOException {
+    private void renderJSON(final HttpServletResponse response) throws 
IOException {
         response.setContentType("application/json");
         response.setCharacterEncoding("UTF-8");
 
@@ -220,9 +214,7 @@ public class AdminPlugin extends Abstrac
             jw.key("instances");
             jw.array();
             for (Instance i : instances) {
-//                if (!i.isRoot()) {
-                    instanceInfo(jw, i);
-//                }
+                instanceInfo(jw, i);
             }
             jw.endArray();
             jw.endObject();
@@ -364,17 +356,12 @@ public class AdminPlugin extends Abstrac
         return false;
     }
 
-    /**
-     * @param adminService the adminService to set
-     */
     public void setAdminService(AdminService adminService) {
         this.adminService = adminService;
     }
 
-    /**
-     * @param bundleContext the bundleContext to set
-     */
     public void setBundleContext(BundleContext bundleContext) {
         this.bundleContext = bundleContext;
     }
+
 }

Modified: 
karaf/branches/karaf-2.3.x/webconsole/admin/src/main/resources/OSGI-INF/blueprint/webconsole-admin.xml
URL: 
http://svn.apache.org/viewvc/karaf/branches/karaf-2.3.x/webconsole/admin/src/main/resources/OSGI-INF/blueprint/webconsole-admin.xml?rev=1478447&r1=1478446&r2=1478447&view=diff
==============================================================================
--- 
karaf/branches/karaf-2.3.x/webconsole/admin/src/main/resources/OSGI-INF/blueprint/webconsole-admin.xml
 (original)
+++ 
karaf/branches/karaf-2.3.x/webconsole/admin/src/main/resources/OSGI-INF/blueprint/webconsole-admin.xml
 Thu May  2 17:07:35 2013
@@ -25,7 +25,6 @@
         <property name="adminService" ref="adminService" />
         <property name="bundleContext" ref="blueprintBundleContext" />
     </bean>
-
     <service ref="adminPlugin" interface="javax.servlet.Servlet" >
         <service-properties>
             <entry key="felix.webconsole.label" value="admin"/>


Reply via email to