Merge branch 'develop' into stable

.gitignore

@@ -7,3 +7,6 @@ pom.xml.versionsBackup
 .project
 .classpath
 
+# Visual Studio Code
+.factorypath
+

pom.xml

@@ -8,7 +8,7 @@
 
 	<groupId>com.inteligr8.alfresco</groupId>
 	<artifactId>amp-plugin</artifactId>
-	<version>1.0.4</version>
+	<version>1.0.5</version>
 	<packaging>maven-plugin</packaging>
 
 	<name>A Maven plugin to generate AMP files</name>

src/main/java/com/inteligr8/alfresco/amp/AmpDependencyFilter.java

@@ -6,7 +6,9 @@ import java.io.FileInputStream;
 import java.io.IOException;
 import java.nio.charset.Charset;
 import java.util.Arrays;
+import java.util.HashSet;
 import java.util.List;
+import java.util.Set;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 import java.util.zip.ZipEntry;
@@ -27,11 +29,23 @@ public class AmpDependencyFilter implements DependencyFilter {
 	private final Log log;
 	private final Charset charset;
 	private final ArtifactResolutionCallback callback;
+	private final String projectModuleId;
+	private Set<String> moduleIds = new HashSet<String>();
 	
 	public AmpDependencyFilter(Log log, String charsetName, ArtifactResolutionCallback callback) {
 		this.log = log;
 		this.charset = Charset.forName(charsetName);
 		this.callback = callback;
+
+		this.projectModuleId = this.getModuleId(this.callback.getProject().getArtifact());
+	}
+	
+	private String getModuleId(org.apache.maven.artifact.Artifact artifact) {
+		return artifact.getGroupId() + "." + artifact.getArtifactId();
+	}
+	
+	private String getModuleId(Artifact artifact) {
+		return artifact.getGroupId() + "." + artifact.getArtifactId();
 	}
 	
 	@Override
@@ -39,6 +53,29 @@ public class AmpDependencyFilter implements DependencyFilter {
 		Artifact artifact = node.getArtifact();
 		if (this.log.isDebugEnabled())
 			this.log.debug("Checking dependency: " + artifact.getArtifactId());
+		
+		String moduleId = this.getModuleId(artifact);
+		if (this.projectModuleId.equals(moduleId)) {
+			// always include project itself, even if it is a module
+			return true;
+		}
+		
+		if (this.moduleIds.contains(moduleId)) {
+			if (this.log.isDebugEnabled())
+				this.log.debug("Not packaging library; detected as Alfresco Module: " + node.getArtifact().getArtifactId());
+			return false;
+		} else if (parents != null) {
+			for (DependencyNode parent : parents) {
+				String parentModuleId = this.getModuleId(parent.getArtifact());
+				if (this.moduleIds.contains(parentModuleId)) {
+					this.moduleIds.add(moduleId);
+					if (this.log.isDebugEnabled())
+						this.log.debug("Not packaging library; detected as dependency to other Alfresco Module: " + node.getArtifact().getArtifactId());
+					return false;
+				}
+			}
+		}
+		
 		File file = artifact.getFile();
 		try {
 			if (file == null) {
@@ -54,8 +91,9 @@ public class AmpDependencyFilter implements DependencyFilter {
 				this.log.debug("Checking dependency file: " + file);
 			
 			if (this.isAlfrescoModule(file)) {
+				this.moduleIds.add(moduleId);
 				if (this.log.isInfoEnabled())
-					this.log.info("Not packaging JAR; detected as Alfresco JAR Module: " + node.getArtifact().getArtifactId());
+					this.log.info("Not packaging library; detected as Alfresco Module: " + node.getArtifact().getArtifactId());
 				return false;
 			}
 			

src/main/java/com/inteligr8/alfresco/amp/AmpMojo.java

@@ -90,6 +90,11 @@ public class AmpMojo extends AbstractMojo implements ArtifactResolutionCallback
 	
 	@Parameter( property = "skip", required = true, defaultValue = "false" )
 	protected boolean skip;
+	
+	@Override
+	public MavenProject getProject() {
+		return this.project;
+	}
 
     public void execute() throws MojoExecutionException {
     	if (this.skip) {

src/main/java/com/inteligr8/alfresco/amp/ArtifactResolutionCallback.java

@@ -1,11 +1,14 @@
 package com.inteligr8.alfresco.amp;
 
+import org.apache.maven.project.MavenProject;
 import org.eclipse.aether.resolution.ArtifactRequest;
 import org.eclipse.aether.resolution.ArtifactResolutionException;
 import org.eclipse.aether.resolution.ArtifactResult;
 
 public interface ArtifactResolutionCallback {
 	
+	MavenProject getProject();
+	
 	ArtifactResult resolveArtifact(ArtifactRequest artifactRequest) throws ArtifactResolutionException;
 
 }