svn commit: r789816 - /sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/JcrResourceResolverFactoryImpl.java

[cziegeler]

Author: cziegeler
Date: Tue Jun 30 16:13:57 2009
New Revision: 789816

URL: http://svn.apache.org/viewvc?rev=789816&view=rev
Log:
SLING-944 : Only fire event if binding has been successful.

Modified:
    
sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/JcrResourceResolverFactoryImpl.java

Modified: 
sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/JcrResourceResolverFactoryImpl.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/JcrResourceResolverFactoryImpl.java?rev=789816&r1=789815&r2=789816&view=diff
==============================================================================
--- 
sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/JcrResourceResolverFactoryImpl.java
 (original)
+++ 
sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/JcrResourceResolverFactoryImpl.java
 Tue Jun 30 16:13:57 2009
@@ -471,18 +471,18 @@
 
                             log.debug("bindResourceProvider: {}={} ({})",
                                 new Object[] { root, provider, serviceName });
+                            if ( localEA != null ) {
+                                final Dictionary<String, Object> props = new 
Hashtable<String, Object>();
+                                props.put(SlingConstants.PROPERTY_PATH, root);
+                                localEA.postEvent(new 
Event(SlingConstants.TOPIC_RESOURCE_PROVIDER_ADDED,
+                                        props));
+                            }
                         } catch (ResourceProviderEntryException rpee) {
                             log.error(
                                 "bindResourceProvider: Cannot register 
ResourceProvider {} for {}: ResourceProvider {} is already registered",
                                 new Object[] { provider, root,
                                     rpee.getExisting().getResourceProvider() 
});
                         }
-                        if ( localEA != null ) {
-                            final Dictionary<String, Object> props = new 
Hashtable<String, Object>();
-                            props.put(SlingConstants.PROPERTY_PATH, root);
-                            localEA.postEvent(new 
Event(SlingConstants.TOPIC_RESOURCE_PROVIDER_ADDED,
-                                    props));
-                        }
                     }
                 }
             }