You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2009/06/30 18:13:58 UTC

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

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));
-                        }
                     }
                 }
             }