From a6241394df1ed4db6674d69717078f14e3e831e5 Mon Sep 17 00:00:00 2001
From: Gustavo Martin Morcuende <gu.martinm@gmail.com>
Date: Sun, 17 Apr 2016 13:35:09 +0200
Subject: [PATCH] AOP: LTW and maven plguin, improvements

---
 .../src/main/java/de/spring/example/TransactionManager.java          | 2 --
 SpringJava/AOP/SpringLTW/src/main/resources/META-INF/aop.xml         | 2 ++
 SpringJava/AOP/SpringMavenWeaver/pom.xml                             | 5 ++++-
 .../src/main/java/de/spring/example/TransactionManager.java          | 2 --
 4 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/SpringJava/AOP/SpringLTW/src/main/java/de/spring/example/TransactionManager.java b/SpringJava/AOP/SpringLTW/src/main/java/de/spring/example/TransactionManager.java
index 53f3c59..338c82a 100644
--- a/SpringJava/AOP/SpringLTW/src/main/java/de/spring/example/TransactionManager.java
+++ b/SpringJava/AOP/SpringLTW/src/main/java/de/spring/example/TransactionManager.java
@@ -1,12 +1,10 @@
 package de.spring.example;
 
-import org.aspectj.lang.annotation.Aspect;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.jdbc.datasource.DataSourceTransactionManager;
 import org.springframework.transaction.TransactionStatus;
 
-@Aspect
 public class TransactionManager {
 	private static final Logger LOGGER = LoggerFactory.getLogger(TransactionManager.class);
     private static TransactionManager instance = new TransactionManager();
diff --git a/SpringJava/AOP/SpringLTW/src/main/resources/META-INF/aop.xml b/SpringJava/AOP/SpringLTW/src/main/resources/META-INF/aop.xml
index a5e58ae..186f621 100644
--- a/SpringJava/AOP/SpringLTW/src/main/resources/META-INF/aop.xml
+++ b/SpringJava/AOP/SpringLTW/src/main/resources/META-INF/aop.xml
@@ -1,6 +1,8 @@
 <!DOCTYPE aspectj PUBLIC
         "-//AspectJ//DTD//EN" "http://www.eclipse.org/aspectj/dtd/aspectj.dtd">
 
+	<!-- REQUIRED FILE WHEN USING LOAD TIME WEAVING!!!! -->
+
 <aspectj>
 	<!--
 		With or without logs
diff --git a/SpringJava/AOP/SpringMavenWeaver/pom.xml b/SpringJava/AOP/SpringMavenWeaver/pom.xml
index 1e3dac6..dd5a42e 100644
--- a/SpringJava/AOP/SpringMavenWeaver/pom.xml
+++ b/SpringJava/AOP/SpringMavenWeaver/pom.xml
@@ -204,15 +204,18 @@
                 <encoding>${project.build.sourceEncoding}</encoding>
                 <complianceLevel>1.8</complianceLevel>                    
                 <verbose>true</verbose>
+                <warn>constructorName,packageDefaultMethod,deprecation,maskedCatchBlocks,unusedLocals,unusedArguments,unusedImports</warn>
+                <crossrefs>true</crossrefs>
 				<weaveDirectories>
             		<weaveDirectory>${project.build.directory}/classes</weaveDirectory>
           		</weaveDirectories>
           		<showWeaveInfo>true</showWeaveInfo>
+          		<outxml>true</outxml>
 				<sources>
 					<source>
 						<basedir>src/main/java</basedir>
 						<includes>
-							<include>**/aspects/*.java</include>
+							<include>**/*.java</include>
 						</includes>
 						<excludes>
 							<exclude>**/logging/*.aj</exclude>
diff --git a/SpringJava/AOP/SpringMavenWeaver/src/main/java/de/spring/example/TransactionManager.java b/SpringJava/AOP/SpringMavenWeaver/src/main/java/de/spring/example/TransactionManager.java
index 53f3c59..338c82a 100644
--- a/SpringJava/AOP/SpringMavenWeaver/src/main/java/de/spring/example/TransactionManager.java
+++ b/SpringJava/AOP/SpringMavenWeaver/src/main/java/de/spring/example/TransactionManager.java
@@ -1,12 +1,10 @@
 package de.spring.example;
 
-import org.aspectj.lang.annotation.Aspect;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.jdbc.datasource.DataSourceTransactionManager;
 import org.springframework.transaction.TransactionStatus;
 
-@Aspect
 public class TransactionManager {
 	private static final Logger LOGGER = LoggerFactory.getLogger(TransactionManager.class);
     private static TransactionManager instance = new TransactionManager();
-- 
2.1.4