You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by cl...@apache.org on 2010/07/12 19:45:53 UTC

svn commit: r963396 - /felix/trunk/ipojo/webconsole-plugin/src/main/java/org/apache/felix/ipojo/webconsole/IPOJOPlugin.java

Author: clement
Date: Mon Jul 12 17:45:53 2010
New Revision: 963396

URL: http://svn.apache.org/viewvc?rev=963396&view=rev
Log:
Improve error handling

Modified:
    felix/trunk/ipojo/webconsole-plugin/src/main/java/org/apache/felix/ipojo/webconsole/IPOJOPlugin.java

Modified: felix/trunk/ipojo/webconsole-plugin/src/main/java/org/apache/felix/ipojo/webconsole/IPOJOPlugin.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/webconsole-plugin/src/main/java/org/apache/felix/ipojo/webconsole/IPOJOPlugin.java?rev=963396&r1=963395&r2=963396&view=diff
==============================================================================
--- felix/trunk/ipojo/webconsole-plugin/src/main/java/org/apache/felix/ipojo/webconsole/IPOJOPlugin.java (original)
+++ felix/trunk/ipojo/webconsole-plugin/src/main/java/org/apache/felix/ipojo/webconsole/IPOJOPlugin.java Mon Jul 12 17:45:53 2010
@@ -270,7 +270,7 @@ public class IPOJOPlugin extends Abstrac
         }
         
         if (factory == null) {
-            // TODO Error management
+            pw.println("The factory " + name + " does not exist or is private");
             System.err.println("factory " + name + "  not found");
             return;
         }
@@ -353,7 +353,7 @@ public class IPOJOPlugin extends Abstrac
         }
         
         if (instance == null) {
-            // TODO Error management
+            pw.println("The instance " + name + " does not exist or does not exposed its architecture");
             System.err.println("instance " + name + "  not found");
             return;
         }