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 2012/01/27 09:01:09 UTC

svn commit: r1236562 - /felix/sandbox/cziegeler/eventadmin/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java

Author: cziegeler
Date: Fri Jan 27 08:01:08 2012
New Revision: 1236562

URL: http://svn.apache.org/viewvc?rev=1236562&view=rev
Log:
Improve warning msg if property is missing or has wrong type

Modified:
    felix/sandbox/cziegeler/eventadmin/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java

Modified: felix/sandbox/cziegeler/eventadmin/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java
URL: http://svn.apache.org/viewvc/felix/sandbox/cziegeler/eventadmin/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java?rev=1236562&r1=1236561&r2=1236562&view=diff
==============================================================================
--- felix/sandbox/cziegeler/eventadmin/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java (original)
+++ felix/sandbox/cziegeler/eventadmin/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java Fri Jan 27 08:01:08 2012
@@ -118,10 +118,19 @@ public class EventHandlerProxy {
         }
         else
         {
+            final String reason;
+            if ( topicObj == null )
+            {
+                reason = "Missing";
+            }
+            else
+            {
+                reason = "Neither of type String nor String[] : " + topicObj.getClass().getName();
+            }
             LogWrapper.getLogger().log(
                     this.reference,
                     LogWrapper.LOG_WARNING,
-                    "Invalid EVENT_TOPICS - Ignoring ServiceReference ["
+                    "Invalid EVENT_TOPICS : " + reason + " - Ignoring ServiceReference ["
                         + this.reference + " | Bundle("
                         + this.reference.getBundle() + ")]");
             this.topics = null;