Change <apply>'s "overwrite" attribute name to "force". Also commit missing

Java changes.  Oops.


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

WHATSNEW

@@ -86,7 +86,7 @@ Other changes:
 * Junit task -- display suite first.
   Bugzilla report 31962.
 
-* <apply> has a new "overwrite" attribute that, when true, disables
+* <apply> has a new "force" attribute that, when true, disables
   checking of target files.
 
 Changes from Ant 1.6.2 to current Ant 1.6 CVS version

docs/manual/CoreTasks/apply.html

@@ -256,7 +256,7 @@ to send input to it is via the input and inputstring attributes.</p>
     <td align="center" valign="top">No, default is <i>true</i></td>
   </tr>
   <tr>
-    <td valign="top">overwrite</td>
+    <td valign="top">force</td>
     <td valign="top">Whether to bypass timestamp comparisons
       for target files.  <em>Since Ant 1.7.</em></td>
     <td align="center" valign="top">No, default is <i>false</i></td>

src/etc/testcases/taskdefs/exec/apply.xml

@@ -378,7 +378,7 @@
         </fail>
     </target>
 
-    <target name="overwrite" depends="init,xyz,pad" if="echo.can.run">
+    <target name="force" depends="init,xyz,pad" if="echo.can.run">
         <presetdef name="ekko">
             <apply executable="echo" append="true" dest="${basedir}">
                 <filelist refid="xyzlist" />
@@ -393,7 +393,7 @@
         </pathconvert>
 
         <ekko outputproperty="foo" />
-        <ekko outputproperty="bar" overwrite="true" />
+        <ekko outputproperty="bar" force="true" />
         <fail>
             <condition>
                 <not>

src/main/org/apache/tools/ant/taskdefs/ExecuteOn.java

@@ -60,6 +60,7 @@ public class ExecuteOn extends ExecTask {
     private boolean addSourceFile = true;
     private boolean verbose = false;
     private boolean ignoreMissing = true;
+    private boolean force = false;
 
     /**
      * Has <srcfile> been specified before <targetfile>
@@ -180,6 +181,15 @@ public class ExecuteOn extends ExecTask {
         ignoreMissing = b;
     }
 
+    /**
+     * Whether to bypass timestamp comparisons for target files.
+     *
+     * @since Ant 1.7
+     */
+    public void setForce(boolean b) {
+        force = b;
+    }
+
     /**
      * Marker that indicates where the name of the source file should
      * be put on the command line.
@@ -563,13 +573,7 @@ public class ExecuteOn extends ExecTask {
      * be included on the command line.
      */
     protected String[] getFiles(File baseDir, DirectoryScanner ds) {
-        if (mapper != null) {
-            SourceFileScanner sfs = new SourceFileScanner(this);
-            return sfs.restrict(ds.getIncludedFiles(), baseDir, destDir,
-                                mapper);
-        } else {
-            return ds.getIncludedFiles();
-        }
+        return restrict(ds.getIncludedFiles(), baseDir);
     }
 
     /**
@@ -577,13 +581,7 @@ public class ExecuteOn extends ExecTask {
      * should be included on the command line.
      */
     protected String[] getDirs(File baseDir, DirectoryScanner ds) {
-        if (mapper != null) {
-            SourceFileScanner sfs = new SourceFileScanner(this);
-            return sfs.restrict(ds.getIncludedDirectories(), baseDir, destDir,
-                                mapper);
-        } else {
-            return ds.getIncludedDirectories();
-        }
+        return restrict(ds.getIncludedDirectories(), baseDir);
     }
 
     /**
@@ -593,14 +591,12 @@ public class ExecuteOn extends ExecTask {
      * @since Ant 1.6.2
      */
     protected String[] getFilesAndDirs(FileList list) {
-        if (mapper != null) {
-            SourceFileScanner sfs = new SourceFileScanner(this);
-            return sfs.restrict(list.getFiles(getProject()),
-                                list.getDir(getProject()), destDir,
-                                mapper);
-        } else {
-            return list.getFiles(getProject());
-        }
+        return restrict(list.getFiles(getProject()), list.getDir(getProject()));
+    }
+
+    private String[] restrict(String[] s, File baseDir) {
+        return (mapper == null || force) ? s
+            : new SourceFileScanner(this).restrict(s, baseDir, destDir, mapper);
     }
 
     /**

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

@@ -554,8 +554,8 @@ public class ExecuteOnTest extends BuildFileTest {
         executeTarget("ignoremissing");
     }
 
-    public void testOverwrite() {
-        executeTarget("overwrite");
+    public void testForce() {
+        executeTarget("force");
     }
 
     //borrowed from TokenFilterTest