ApplicationListenerDetector ignores non-managed bean instances

Issue: SPR-14879

Author: jhoeller

spring-context/src/main/java/org/springframework/context/support/ApplicationListenerDetector.java

@@ -27,6 +27,7 @@
 import org.springframework.beans.factory.support.RootBeanDefinition;
 import org.springframework.context.ApplicationListener;
 import org.springframework.context.event.ApplicationEventMulticaster;
+import org.springframework.util.ObjectUtils;
 
 /**
  * {@code BeanPostProcessor} that detects beans which implement the {@code ApplicationListener}
@@ -57,8 +58,8 @@ public ApplicationListenerDetector(AbstractApplicationContext applicationContext
 
 	@Override
 	public void postProcessMergedBeanDefinition(RootBeanDefinition beanDefinition, Class<?> beanType, String beanName) {
-		if (this.applicationContext != null && beanDefinition.isSingleton()) {
-			this.singletonNames.put(beanName, Boolean.TRUE);
+		if (this.applicationContext != null) {
+			this.singletonNames.put(beanName, beanDefinition.isSingleton());
 		}
 	}
 
@@ -76,23 +77,23 @@ public Object postProcessAfterInitialization(Object bean, String beanName) {
 				// singleton bean (top-level or inner): register on the fly
 				this.applicationContext.addApplicationListener((ApplicationListener<?>) bean);
 			}
-			else if (flag == null) {
+			else if (Boolean.FALSE.equals(flag)) {
 				if (logger.isWarnEnabled() && !this.applicationContext.containsBean(beanName)) {
 					// inner bean with other scope - can't reliably process events
 					logger.warn("Inner bean '" + beanName + "' implements ApplicationListener interface " +
 							"but is not reachable for event multicasting by its containing ApplicationContext " +
 							"because it does not have singleton scope. Only top-level listener beans are allowed " +
 							"to be of non-singleton scope.");
 				}
-				this.singletonNames.put(beanName, Boolean.FALSE);
+				this.singletonNames.remove(beanName);
 			}
 		}
 		return bean;
 	}
 
 	@Override
 	public void postProcessBeforeDestruction(Object bean, String beanName) {
-		if (bean instanceof ApplicationListener) {
+		if (this.applicationContext != null && bean instanceof ApplicationListener) {
 			ApplicationEventMulticaster multicaster = this.applicationContext.getApplicationEventMulticaster();
 			multicaster.removeApplicationListener((ApplicationListener<?>) bean);
 			multicaster.removeApplicationListenerBean(beanName);
@@ -113,7 +114,7 @@ public boolean equals(Object other) {
 
 	@Override
 	public int hashCode() {
-		return this.applicationContext.hashCode();
+		return ObjectUtils.nullSafeHashCode(this.applicationContext);
 	}
 
 }