Author: bodewig
Date: Tue Aug 18 15:44:32 2009
New Revision: 805463

URL: http://svn.apache.org/viewvc?rev=805463&view=rev
Log:
the update attribute of the modified selector was ignored.  PR 32597

Added:
    ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml   (with 
props)
Modified:
    ant/core/trunk/WHATSNEW
    
ant/core/trunk/src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java

Modified: ant/core/trunk/WHATSNEW
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/WHATSNEW?rev=805463&r1=805462&r2=805463&view=diff
==============================================================================
--- ant/core/trunk/WHATSNEW (original)
+++ ant/core/trunk/WHATSNEW Tue Aug 18 15:44:32 2009
@@ -436,6 +436,9 @@
    separator other than "/".
    Bugzilla Report 35649.
 
+ * The update attribute of the modified selector was ignored.
+   Bugzilla Report 32597.
+
 Other changes:
 --------------
  * The get task now also follows redirects from http to https

Modified: 
ant/core/trunk/src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java?rev=805463&r1=805462&r2=805463&view=diff
==============================================================================
--- 
ant/core/trunk/src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java
 (original)
+++ 
ant/core/trunk/src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java
 Tue Aug 18 15:44:32 2009
@@ -298,8 +298,6 @@
         Cache      defaultCache      = new PropertiesfileCache(cachefile);
         Algorithm  defaultAlgorithm  = new DigestAlgorithm();
         Comparator defaultComparator = new EqualComparator();
-        update = true;
-        selectDirectories = true;
 
         //
         // -----  Set the main attributes, pattern '*'  -----

Added: ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml?rev=805463&view=auto
==============================================================================
--- ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml (added)
+++ ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml Tue Aug 
18 15:44:32 2009
@@ -0,0 +1,76 @@
+<?xml version="1.0"?>
+<!--
+  Licensed to the Apache Software Foundation (ASF) under one or more
+  contributor license agreements.  See the NOTICE file distributed with
+  this work for additional information regarding copyright ownership.
+  The ASF licenses this file to You under the Apache License, Version 2.0
+  (the "License"); you may not use this file except in compliance with
+  the License.  You may obtain a copy of the License at
+
+      http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitations under the License.
+-->
+<project xmlns:au="antlib:org.apache.ant.antunit" default="antunit">
+
+  <import file="../../antunit-base.xml" />
+
+  <target name="tearDown" depends="antunit-base.tearDown">
+    <delete file="cache.properties" quiet="true"/>
+  </target>
+
+  <target name="testUpdate"
+          
description="https://issues.apache.org/bugzilla/show_bug.cgi?id=32597";>
+    <mkdir dir="${input}"/>
+    <touch file="${input}/A"/>
+    <mkdir dir="${output}"/>
+    <pathconvert>
+      <fileset dir="${input}">
+        <modified/>
+      </fileset>
+    </pathconvert>
+    <au:assertFileExists file="cache.properties"/>
+    <copy todir="${output}" file="cache.properties"/>
+    <echo file="${input}/A">Hello</echo>
+    <pathconvert>
+      <fileset dir="${input}">
+        <modified update="false"/>
+      </fileset>
+    </pathconvert>
+    <au:assertFilesMatch
+       expected="${output}/cache.properties"
+       actual="cache.properties"/>
+    <pathconvert>
+      <fileset dir="${input}">
+        <modified>
+          <param name="update" value="false"/>
+        </modified>
+      </fileset>
+    </pathconvert>
+    <au:assertFilesMatch
+       expected="${output}/cache.properties"
+       actual="cache.properties"/>
+    <pathconvert>
+      <fileset dir="${input}">
+        <modified update="true"/>
+      </fileset>
+    </pathconvert>
+    <au:assertFilesDiffer
+       expected="${output}/cache.properties"
+       actual="cache.properties"/>
+    <copy todir="${output}" file="cache.properties"/>
+    <echo file="${input}/A">world</echo>
+    <pathconvert>
+      <fileset dir="${input}">
+        <modified update="true"/>
+      </fileset>
+    </pathconvert>
+    <au:assertFilesDiffer
+       expected="${output}/cache.properties"
+       actual="cache.properties"/>
+  </target>
+</project>

Propchange: ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml
------------------------------------------------------------------------------
    svn:eol-style = native


Reply via email to