v1.3.2 pom

pom.xml

@@ -7,7 +7,7 @@
 
 	<groupId>com.inteligr8</groupId>
 	<artifactId>ban-maven-plugin</artifactId>
-	<version>1.3.0</version>
+	<version>1.3.2</version>
 	<packaging>maven-plugin</packaging>
 
 	<name>Ban Dependencies Maven Plugin</name>

src/main/java/com/inteligr8/maven/ban/AbstractBanConfiguration.java

@@ -90,7 +90,7 @@ public abstract class AbstractBanConfiguration implements BanConfiguration {
                         this.logger.error("The artifact version range could not be resolved; skipping: {}", child.getValue());
                     } else {
                         Version version = vrresult.getHighestVersion();
-                        artifact.setVersion(version.toString());
+                        artifact = artifact.setVersion(version.toString());
                         
                         ArtifactRequest arequest = new ArtifactRequest(artifact, this.session.getCurrentProject().getRemoteProjectRepositories(), null);
                         try {
@@ -98,7 +98,7 @@ public abstract class AbstractBanConfiguration implements BanConfiguration {
                             File file = aresult.getArtifact().getFile();
                             downloader = new BanConfigurationDownloader(this.session, this.artifactResolver, this.versionRangeResolver, file);
                         } catch (ArtifactResolutionException are) {
-                            this.logger.warn("The artifact version could not be resolved; skipping: {} | {}", child.getValue(), version);
+                            this.logger.warn("The artifact version could not be resolved; skipping: {}", artifact, version);
                         }
                     }
                 } catch (VersionRangeResolutionException vrre) {

src/main/java/com/inteligr8/maven/ban/PurgeRepoMojo.java

@@ -219,14 +219,27 @@ public class PurgeRepoMojo extends AbstractMojo {
         
         Files.list(repoPath.resolve(groupPath)).forEach(new Consumer<Path>() {
             @Override
-            public void accept(Path t) {
+            public void accept(Path fullArtifactPath) {
                 if (artifactPattern == null) {
-                    paths.add(repoPath.relativize(t));
+                    // these may include sub-groups and not just artifacts
+                    // which will lead to paths with artifacts as versions
+                    // so we are looping through versions to see if it is indeed an artifact
+                    try {
+                        Files.list(fullArtifactPath).findFirst().ifPresent(new Consumer<Path>() {
+                            @Override
+                            public void accept(Path fullVersionPath) {
+                                if (Files.exists(fullVersionPath.resolve("_remote.repositories")))
+                                    paths.add(repoPath.relativize(fullArtifactPath));
+                            }
+                        });
+                    } catch (IOException ie) {
+                        getLog().error(ie.getMessage(), ie);
+                    }
                 } else {
-                    Matcher matcher = artifactPattern.matcher(t.getFileName().toString());
+                    Matcher matcher = artifactPattern.matcher(fullArtifactPath.getFileName().toString());
                     if (matcher.matches()) {
-                        getLog().debug("The artifact directory '" + t.getFileName() + "' qualifies as included");
-                        paths.add(repoPath.relativize(t));
+                        getLog().debug("The artifact directory '" + fullArtifactPath.getFileName() + "' qualifies as included");
+                        paths.add(repoPath.relativize(fullArtifactPath));
                     }
                 }
             }