Fix for execution of top level tasks getting delayed by targets.

PR: 31487


git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@276905 13f79535-47bb-0310-9956-ffa450edef68

WHATSNEW

@@ -26,6 +26,9 @@ Fixed bugs:
 * Incorrect task name with invalid "javac" task after a "presetdef.
   Bugzilla reports 31389 and 29499.
 
+* Execution of top level tasks in imported files get delayed by targets.
+  Bugzilla report 31487.
+
 Other changes:
 --------------
 

src/etc/testcases/taskdefs/import/importtargetfirst.xml

@@ -0,0 +1,5 @@
+<project>
+  <echo>Importing targetfirst</echo>
+  <import file="targetfirst.xml"/>
+  <echo>After importing</echo>
+</project>

src/etc/testcases/taskdefs/import/targetfirst.xml

@@ -0,0 +1,4 @@
+<project>
+  <target name="first"/>
+  <echo>After target first</echo>
+</project>

src/main/org/apache/tools/ant/helper/ProjectHelper2.java

@@ -116,15 +116,18 @@ public class ProjectHelper2 extends ProjectHelper {
             // we are in an imported file.
             context.setIgnoreProjectTag(true);
             Target currentTarget = context.getCurrentTarget();
+            Target currentImplicit = context.getImplicitTarget();
             try {
                 Target newCurrent = new Target();
                 newCurrent.setProject(project);
                 newCurrent.setName("");
                 context.setCurrentTarget(newCurrent);
+                context.setImplicitTarget(newCurrent);
                 parse(project, source, new RootHandler(context, mainHandler));
                 newCurrent.execute();
             } finally {
                 context.setCurrentTarget(currentTarget);
+                context.setImplicitTarget(currentImplicit);
             }
         } else {
             // top level file

src/testcases/org/apache/tools/ant/taskdefs/ImportTest.java

@@ -142,4 +142,8 @@ public class ImportTest extends BuildFileTest {
         }
     }
 
+    public void testTargetFirst() {
+        configureProject("src/etc/testcases/taskdefs/import/importtargetfirst.xml");
+        assertLogContaining("Importing targetfirstAfter target firstAfter importing");
+    }
 }