bz-63259 Don't check the FTP file for being a symlink, more than once

Patch contributed by Eugène Adell, as an attachment in the bugzilla issue.

Signed-off-by: Jaikiran Pai <jaikiran@apache.org>

2 files changed, 9 insertions, 1 deletions

diff --git a/WHATSNEW b/WHATSNEW
index b88b95f52..86f2e4c26 100644
--- a/WHATSNEW
+++ b/WHATSNEW
@@ -1,6 +1,12 @@
 Changes from Ant 1.10.6 TO Ant 1.10.7
 =====================================
 
+Fixed bugs:
+-----------
+
+ * FTP task no longer duplicates a check for a file being a symlink.
+   Bugzilla Report 63259
+
 
 Changes from Ant 1.10.5 TO Ant 1.10.6
 =====================================
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java b/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java
index fe757e265..5838e1e31 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java
@@ -537,7 +537,9 @@ public class FTP extends Task implements FTPTaskConfig {
                         } else {
                             if (!isFollowSymlinks() && file.isSymbolicLink()) {
                                 filesExcluded.addElement(name);
-                            } else if (isFunctioningAsFile(ftp, dir, file)) {
+                            } else {
+                                // at this point, it's either a symbolic link or a file, but not a directory.
+                                // so we include it
                                 accountForIncludedFile(name);
                             }
                         }