Add autowiring+AOP support to injected components

kelveny · kelveny · commit 0e9924fceee2 · 2012-11-07T15:03:23.000-08:00
diff --git a/utils/src/com/cloud/utils/component/ComponentContext.java b/utils/src/com/cloud/utils/component/ComponentContext.java
@@ -20,6 +20,7 @@
 import org.springframework.aop.Advisor;
 import org.springframework.aop.framework.ProxyFactory;
 import org.springframework.aop.support.DefaultPointcutAdvisor;
+import org.springframework.beans.factory.config.AutowireCapableBeanFactory;
 import org.springframework.context.ApplicationContext;
 import org.springframework.context.ApplicationContextAware;
 import org.springframework.stereotype.Component;
@@ -53,15 +54,24 @@ public static <T> T getCompanent(Class<T> beanType) {
     	assert(s_appContext != null);
     	return (T)s_appContext.getBean(beanType);
     }
+
+    public static<T> T inject(Class<T> clz) {
+    	T instance = s_appContext.getAutowireCapableBeanFactory().createBean(clz);
+    	return inject(instance);
+    }
     
     public static<T> T inject(Object instance) {
+    	// autowire dynamically loaded object
+    	AutowireCapableBeanFactory  beanFactory = s_appContext.getAutowireCapableBeanFactory();
+    	beanFactory.autowireBean(instance);
+
     	Advisor advisor = new DefaultPointcutAdvisor(new MatchAnyMethodPointcut(),
     			new TransactionContextBuilder());
+    
     	ProxyFactory pf = new ProxyFactory();
-    	
         pf.setTarget(instance);
         pf.addAdvisor(advisor);
         
-        return (T)pf.getProxy();
+        return (T)pf.getProxy();        
     }
 }
diff --git a/utils/test/com/cloud/utils/db/DbAnnotatedBase.java b/utils/test/com/cloud/utils/db/DbAnnotatedBase.java
@@ -18,6 +18,7 @@
 
 
 import javax.annotation.PostConstruct;
+import javax.inject.Inject;
 
 import junit.framework.Assert;
 
@@ -29,12 +30,15 @@
 public class DbAnnotatedBase {
     private static final Logger s_logger = Logger.getLogger(DbAnnotatedBase.class);
 
+    @Inject DummyComponent _dummy;
+    
     @PostConstruct
     public void initTest() {
     	Assert.assertTrue(true);
     }
     
     public void MethodWithClassDbAnnotated() {
     	s_logger.info("called");
+    	_dummy.sayHello();
     }
 }
diff --git a/utils/test/com/cloud/utils/db/DummyComponent.java b/utils/test/com/cloud/utils/db/DummyComponent.java
@@ -0,0 +1,27 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.  See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership.  The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// the License.  You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied.  See the License for the
+// specific language governing permissions and limitations
+// under the License.
+package com.cloud.utils.db;
+
+import org.springframework.stereotype.Component;
+
+@Component
+public class DummyComponent {
+
+	public void sayHello() {
+		System.out.println("Hello, world");
+	}
+}

Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,7 @@`
`18`	`18`
`19`	`19`
`20`	`20`	`import javax.annotation.PostConstruct;`
	`21`	`+import javax.inject.Inject;`
`21`	`22`
`22`	`23`	`import junit.framework.Assert;`
`23`	`24`
`@@ -29,12 +30,15 @@`
`29`	`30`	`public class DbAnnotatedBase {`
`30`	`31`	`private static final Logger s_logger = Logger.getLogger(DbAnnotatedBase.class);`
`31`	`32`
	`33`	`+ @Inject DummyComponent _dummy;`
	`34`	`+`
`32`	`35`	`@PostConstruct`
`33`	`36`	`public void initTest() {`
`34`	`37`	`Assert.assertTrue(true);`
`35`	`38`	`}`
`36`	`39`
`37`	`40`	`public void MethodWithClassDbAnnotated() {`
`38`	`41`	`s_logger.info("called");`
	`42`	`+ _dummy.sayHello();`
`39`	`43`	`}`
`40`	`44`	`}`