You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by cz...@apache.org on 2010/10/14 12:04:02 UTC

svn commit: r1022448 - /felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/misc/ConfigurationRender.java

Author: cziegeler
Date: Thu Oct 14 10:04:02 2010
New Revision: 1022448

URL: http://svn.apache.org/viewvc?rev=1022448&view=rev
Log:
FELIX-2659 : ConfigurationRender.searchMethod must catch problems more broadly

Modified:
    felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/misc/ConfigurationRender.java

Modified: felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/misc/ConfigurationRender.java
URL: http://svn.apache.org/viewvc/felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/misc/ConfigurationRender.java?rev=1022448&r1=1022447&r2=1022448&view=diff
==============================================================================
--- felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/misc/ConfigurationRender.java (original)
+++ felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/misc/ConfigurationRender.java Thu Oct 14 10:04:02 2010
@@ -288,9 +288,10 @@ public class ConfigurationRender extends
         {
             return obj.getClass().getDeclaredMethod(mName, params);
         }
-        catch (NoSuchMethodException nsme)
+        catch (Throwable nsme)
         {
-            // ignore
+            // ignore, we catch Throwable above to not only catch NoSuchMethodException
+            // but also other ones like ClassDefNotFoundError etc.
         }
         return null;
     }