From bc38350453f27142fb3bf9cc4e4ff3921f723d03 Mon Sep 17 00:00:00 2001
From: nik <Nikolay.Chashnikov@jetbrains.com>
Date: Mon, 11 Apr 2011 11:25:06 +0400
Subject: [PATCH] fixed building of plugin modules

---
 .../compiler/ArtifactsCompilerInstance.java   | 26 +++++++++----------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/java/compiler/impl/src/com/intellij/packaging/impl/compiler/ArtifactsCompilerInstance.java b/java/compiler/impl/src/com/intellij/packaging/impl/compiler/ArtifactsCompilerInstance.java
index 2eccc6bd92e1..376a1e56b1c7 100644
--- a/java/compiler/impl/src/com/intellij/packaging/impl/compiler/ArtifactsCompilerInstance.java
+++ b/java/compiler/impl/src/com/intellij/packaging/impl/compiler/ArtifactsCompilerInstance.java
@@ -95,19 +95,6 @@ public class ArtifactsCompilerInstance extends GenericCompilerInstance<ArtifactB
         else {
           artifacts = new HashSet<Artifact>(Arrays.asList(ArtifactManager.getInstance(getProject()).getArtifacts()));
         }
-        List<Artifact> additionalArtifacts = new ArrayList<Artifact>();
-        for (BuildParticipantProvider provider : BuildParticipantProvider.EXTENSION_POINT_NAME.getExtensions()) {
-          for (Module module : ModuleManager.getInstance(getProject()).getModules()) {
-            final Collection<? extends BuildParticipant> participants = provider.getParticipants(module);
-            for (BuildParticipant participant : participants) {
-              ContainerUtil.addIfNotNull(participant.createArtifact(myContext), additionalArtifacts);
-            }
-          }
-        }
-        if (LOG.isDebugEnabled() && !additionalArtifacts.isEmpty()) {
-          LOG.debug("additional artifacts to build: " + additionalArtifacts);
-        }
-        artifacts.addAll(additionalArtifacts);
 
         Map<String, Artifact> artifactsMap = new HashMap<String, Artifact>();
         for (Artifact artifact : artifacts) {
@@ -119,6 +106,19 @@ public class ArtifactsCompilerInstance extends GenericCompilerInstance<ArtifactB
             targets.add(new ArtifactBuildTarget(artifact));
           }
         }
+
+        for (BuildParticipantProvider provider : BuildParticipantProvider.EXTENSION_POINT_NAME.getExtensions()) {
+          for (Module module : ModuleManager.getInstance(getProject()).getModules()) {
+            final Collection<? extends BuildParticipant> participants = provider.getParticipants(module);
+            for (BuildParticipant participant : participants) {
+              Artifact artifact = participant.createArtifact(myContext);
+              if (artifact != null) {
+                LOG.debug("additional artifact to build: " + artifact);
+                targets.add(new ArtifactBuildTarget(artifact));
+              }
+            }
+          }
+        }
       }
     }.execute();
     return targets;