diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/config/MessageBrokerBeanDefinitionParser.java b/spring-websocket/src/main/java/org/springframework/web/socket/config/MessageBrokerBeanDefinitionParser.java index 015d305559..206d3dfb74 100644 --- a/spring-websocket/src/main/java/org/springframework/web/socket/config/MessageBrokerBeanDefinitionParser.java +++ b/spring-websocket/src/main/java/org/springframework/web/socket/config/MessageBrokerBeanDefinitionParser.java @@ -104,6 +104,10 @@ class MessageBrokerBeanDefinitionParser implements BeanDefinitionParser { public static final String SOCKJS_SCHEDULER_BEAN_NAME = "messageBrokerSockJsScheduler"; + public static final String MESSAGING_TEMPLATE_BEAN_NAME = "brokerMessagingTemplate"; + + public static final String MESSAGE_CONVERTER_BEAN_NAME = "brokerMessageConverter"; + private static final int DEFAULT_MAPPING_ORDER = 1; private static final boolean jackson2Present = ClassUtils.isPresent( @@ -482,7 +486,9 @@ class MessageBrokerBeanDefinitionParser implements BeanDefinitionParser { ConstructorArgumentValues cavs = new ConstructorArgumentValues(); cavs.addIndexedArgumentValue(0, converters); RootBeanDefinition messageConverterDef = new RootBeanDefinition(CompositeMessageConverter.class, cavs, null); - return new RuntimeBeanReference(registerBeanDef(messageConverterDef, context, source)); + String name = MESSAGE_CONVERTER_BEAN_NAME; + registerBeanDefByName(name, messageConverterDef, context, source); + return new RuntimeBeanReference(name); } private RuntimeBeanReference registerMessagingTemplate(Element element, RuntimeBeanReference brokerChannel, @@ -495,7 +501,9 @@ class MessageBrokerBeanDefinitionParser implements BeanDefinitionParser { beanDef.getPropertyValues().add("userDestinationPrefix", element.getAttribute("user-destination-prefix")); } beanDef.getPropertyValues().add("messageConverter", messageConverter); - return new RuntimeBeanReference(registerBeanDef(beanDef,context, source)); + String name = MESSAGING_TEMPLATE_BEAN_NAME; + registerBeanDefByName(name, beanDef, context, source); + return new RuntimeBeanReference(name); } private void registerAnnotationMethodMessageHandler(Element messageBrokerElement, diff --git a/spring-websocket/src/test/java/org/springframework/web/socket/config/MessageBrokerBeanDefinitionParserTests.java b/spring-websocket/src/test/java/org/springframework/web/socket/config/MessageBrokerBeanDefinitionParserTests.java index 0d9c93ac71..27024c139b 100644 --- a/spring-websocket/src/test/java/org/springframework/web/socket/config/MessageBrokerBeanDefinitionParserTests.java +++ b/spring-websocket/src/test/java/org/springframework/web/socket/config/MessageBrokerBeanDefinitionParserTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2016 the original author or authors. + * Copyright 2002-2017 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -333,10 +333,12 @@ public class MessageBrokerBeanDefinitionParserTests { assertNotNull(messageConverter); assertTrue(messageConverter instanceof CompositeMessageConverter); - CompositeMessageConverter compositeMessageConverter = this.appContext.getBean(CompositeMessageConverter.class); + String name = MessageBrokerBeanDefinitionParser.MESSAGE_CONVERTER_BEAN_NAME; + CompositeMessageConverter compositeMessageConverter = this.appContext.getBean(name, CompositeMessageConverter.class); assertNotNull(compositeMessageConverter); - SimpMessagingTemplate simpMessagingTemplate = this.appContext.getBean(SimpMessagingTemplate.class); + name = MessageBrokerBeanDefinitionParser.MESSAGING_TEMPLATE_BEAN_NAME; + SimpMessagingTemplate simpMessagingTemplate = this.appContext.getBean(name, SimpMessagingTemplate.class); assertNotNull(simpMessagingTemplate); assertEquals("/personal/", simpMessagingTemplate.getUserDestinationPrefix());