From 6ed54839c9a6f757f9f023b3c9170fbd455f1e56 Mon Sep 17 00:00:00 2001
From: Conor MacNeill <conor@apache.org>
Date: Mon, 30 Jul 2001 13:35:23 +0000
Subject: [PATCH] Produce a summary, on request, of the actions taken by the
 replace task

PR:	956


git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@269409 13f79535-47bb-0310-9956-ffa450edef68
---
 docs/manual/CoreTasks/replace.html            |  8 ++++++
 .../apache/tools/ant/taskdefs/Replace.java    | 26 +++++++++++++++++--
 2 files changed, 32 insertions(+), 2 deletions(-)

diff --git a/docs/manual/CoreTasks/replace.html b/docs/manual/CoreTasks/replace.html
index e84336b20..6191f238b 100644
--- a/docs/manual/CoreTasks/replace.html
+++ b/docs/manual/CoreTasks/replace.html
@@ -42,6 +42,14 @@ must use a nested <code>&lt;replacetoken&gt;</code> element.</p>
       (&quot;&quot;) is used.</td>
     <td valign="top" align="center">No</td>
   </tr>
+  <tr>
+    <td valign="top">summary</td>
+    <td valign="top">Indicates whether a summary of the replace operation 
+                     should be produced, detailing how many token occurrences 
+                     and files were processed
+                     </td>
+    <td valign="top" align="center">No, by default no summary is produced</td>
+  </tr>
   <tr>
     <td valign="top">propertyFile</td>
     <td valign="top">valid property file from which properties specified using nested <code>&lt;replacefilter&gt;</code> elements are drawn.</td>
diff --git a/src/main/org/apache/tools/ant/taskdefs/Replace.java b/src/main/org/apache/tools/ant/taskdefs/Replace.java
index ff0cd0f29..ba049f2d9 100644
--- a/src/main/org/apache/tools/ant/taskdefs/Replace.java
+++ b/src/main/org/apache/tools/ant/taskdefs/Replace.java
@@ -78,7 +78,11 @@ public class Replace extends MatchingTask {
     private Vector replacefilters = new Vector();
 
     private File dir = null;
-   
+
+    private int fileCount;
+    private int replaceCount;    
+    private boolean summary = false;
+    
     //Inner class
     public class NestedString {
 
@@ -187,13 +191,15 @@ public class Replace extends MatchingTask {
         }
 
         validateReplacefilters();
+        fileCount = 0;
+        replaceCount = 0;
 
         if (src != null) {
             processFile(src);
         }
 
         if (dir != null) {
-        DirectoryScanner ds = super.getDirectoryScanner(dir);
+            DirectoryScanner ds = super.getDirectoryScanner(dir);
             String[] srcs = ds.getIncludedFiles();
 
             for(int i=0; i<srcs.length; i++) {
@@ -201,6 +207,10 @@ public class Replace extends MatchingTask {
                 processFile(file);
             }
         }
+        
+        if (summary) {
+            log("Replaced " + replaceCount + " occurrences in " + fileCount + " files.", Project.MSG_INFO);
+        }
     }
     
     /**
@@ -337,6 +347,7 @@ public class Replace extends MatchingTask {
             // If there were changes, move the new one to the old one;
             // otherwise, delete the new one
             if (changes) {
+                ++fileCount;
                 src.delete();
                 temp.renameTo(src);
             } else {
@@ -370,6 +381,16 @@ public class Replace extends MatchingTask {
         this.src = file;
     }
 
+    /**
+     * Request a summary
+     *
+     * @param summary true if you would like a summary logged of the replace operation
+     */
+    public void setSummary(boolean summary) {
+        this.summary = summary;
+    }
+    
+    
     /**
      * Set the source files path when using matching tasks.
      */
@@ -445,6 +466,7 @@ public class Replace extends MatchingTask {
             // search again
             start = found + str1.length();
             found = str.indexOf(str1,start);
+            ++replaceCount;
         }
 
         // write the remaining characters