Why this commit not reflected to the latest branch. Any reason for that? https://svn.wso2.org/repos/wso2/carbon/platform/branches/turing/components/event/org.wso2.carbon.event.core/4.2.1/src/main/java/org/wso2/carbon/event/core/internal/topic/registry/RegistryTopicManager.java
Thanks, Chamara Silva On Fri, Sep 27, 2013 at 8:58 AM, <[email protected]> wrote: > Author: [email protected] > Date: Fri Sep 27 18:28:26 2013 > New Revision: 186583 > URL: http://wso2.org/svn/browse/wso2?view=rev&revision=186583 > > Log: > Fix for ESBJAVA-2506 > > Modified: > > > carbon/platform/trunk/components/event/org.wso2.carbon.event.core/src/main/java/org/wso2/carbon/event/core/internal/topic/registry/RegistryTopicManager.java > > Modified: > carbon/platform/trunk/components/event/org.wso2.carbon.event.core/src/main/java/org/wso2/carbon/event/core/internal/topic/registry/RegistryTopicManager.java > URL: > http://wso2.org/svn/browse/wso2/carbon/platform/trunk/components/event/org.wso2.carbon.event.core/src/main/java/org/wso2/carbon/event/core/internal/topic/registry/RegistryTopicManager.java?rev=186583&r1=186582&r2=186583&view=diff > > ============================================================================== > --- > carbon/platform/trunk/components/event/org.wso2.carbon.event.core/src/main/java/org/wso2/carbon/event/core/internal/topic/registry/RegistryTopicManager.java > (original) > +++ > carbon/platform/trunk/components/event/org.wso2.carbon.event.core/src/main/java/org/wso2/carbon/event/core/internal/topic/registry/RegistryTopicManager.java > Fri Sep 27 18:28:26 2013 > @@ -121,12 +121,16 @@ > Collection collection = userRegistry.newCollection(); > userRegistry.put(resourcePath, collection); > > - // Grant this user (owner) rights to update permission on > newly created topic > - UserRealm userRealm = > EventBrokerHolder.getInstance().getRealmService().getTenantUserRealm( > - > CarbonContext.getCurrentContext().getTenantId()); > + // Grant this user (owner) rights to update permission on > newly created topic. Ideally, loggedInUser > + // cannot be null but sometimes components like rule > mediator creates queues for internal use. So at > + // that time username can be null. > + if (loggedInUser != null) { > + UserRealm userRealm = > EventBrokerHolder.getInstance().getRealmService().getTenantUserRealm( > + > CarbonContext.getCurrentContext().getTenantId()); > > - userRealm.getAuthorizationManager().authorizeUser( > - loggedInUser, resourcePath, > EventBrokerConstants.EB_PERMISSION_CHANGE_PERMISSION); > + userRealm.getAuthorizationManager().authorizeUser( > + loggedInUser, resourcePath, > EventBrokerConstants.EB_PERMISSION_CHANGE_PERMISSION); > + } > } > } catch (RegistryException e) { > throw new EventBrokerException("Can not access the config > registry", e); > _______________________________________________ > Commits mailing list > [email protected] > http://wso2.org/cgi-bin/mailman/listinfo/commits > -- A. Suminda Chamara Silva Senior Software Engineer WSO2 Inc. Mobile: +94718302858 blog: http://chamaras.blogspot.com
_______________________________________________ Dev mailing list [email protected] http://wso2.org/cgi-bin/mailman/listinfo/dev
