cvs: pear /Net_FTP generate_package_xml.php /Net_FTP/Net FTP.php

jschippers		Tue Apr 22 21:01:32 2008 UTC

  Modified files:              
    /pear/Net_FTP/Net	FTP.php 
    /pear/Net_FTP	generate_package_xml.php 
  Log:
  Fixed Bug #13114: Error check in $FTP->getRecursive(...)
  
http://cvs.php.net/viewvc.cgi/pear/Net_FTP/Net/FTP.php?r1=1.64&r2=1.65&diff_format=u
Index: pear/Net_FTP/Net/FTP.php
diff -u pear/Net_FTP/Net/FTP.php:1.64 pear/Net_FTP/Net/FTP.php:1.65
--- pear/Net_FTP/Net/FTP.php:1.64	Tue Apr 22 20:51:33 2008
+++ pear/Net_FTP/Net/FTP.php	Tue Apr 22 21:01:31 2008
@@ -20,7 +20,7 @@
  * @author    Jorrit Schippers <[email protected]>
  * @copyright 1997-2008 The PHP Group
  * @license   http://www.php.net/license/3_0.txt PHP License 3.0
- * @version   CVS: $Id: FTP.php,v 1.64 2008/04/22 20:51:33 jschippers Exp $
+ * @version   CVS: $Id: FTP.php,v 1.65 2008/04/22 21:01:31 jschippers Exp $
  * @link      http://pear.php.net/package/Net_FTP
  * @since     File available since Release 0.0.1
  */
@@ -1506,6 +1506,9 @@
         
         $dir_list = array();
         $dir_list = $this->ls($remote_path, NET_FTP_DIRS_ONLY);
+        if (PEAR::isError($dir_list)) {
+            return $dir_list;
+        }
         foreach ($dir_list as $dir_entry) {
             if ($dir_entry['name'] != '.' && $dir_entry['name'] != '..') {
                 $remote_path_new = $remote_path.$dir_entry["name"]."/";
@@ -1519,6 +1522,9 @@
         }
         $file_list = array();
         $file_list = $this->ls($remote_path, NET_FTP_FILES_ONLY);
+        if (PEAR::isError($file_list)) {
+            return $file_list;
+        }
         foreach ($file_list as $file_entry) {
             $remote_file = $remote_path.$file_entry["name"];
             $local_file  = $local_path.$file_entry["name"];
http://cvs.php.net/viewvc.cgi/pear/Net_FTP/generate_package_xml.php?r1=1.21&r2=1.22&diff_format=u
Index: pear/Net_FTP/generate_package_xml.php
diff -u pear/Net_FTP/generate_package_xml.php:1.21 pear/Net_FTP/generate_package_xml.php:1.22
--- pear/Net_FTP/generate_package_xml.php:1.21	Tue Apr 22 20:51:34 2008
+++ pear/Net_FTP/generate_package_xml.php	Tue Apr 22 21:01:32 2008
@@ -17,7 +17,7 @@
  * @author    Tobias Schlitt <[email protected]>
  * @copyright 1997-2005 The PHP Group
  * @license   http://www.php.net/license/3_0.txt  PHP License 3.0
- * @version   CVS: $Id: generate_package_xml.php,v 1.21 2008/04/22 20:51:34 jschippers Exp $
+ * @version   CVS: $Id: generate_package_xml.php,v 1.22 2008/04/22 21:01:32 jschippers Exp $
  * @link      http://pear.php.net/package/Net_FTP
  * @since     File available since Release 1.3.0
  */
@@ -59,7 +59,8 @@
 '* Fixed Bug #13496: set bit not supported
 * Fixed Bug #13689: . in file owner or group name breaks _ls_match
 * Fixed Bug #13690: getRecursive does not work because of \'.\' and \'..\' '.
-'directories';
+'directories
+* Fixed Bug #13114: Error check in $FTP->getRecursive(...)';
 
 PEAR::setErrorHandling(PEAR_ERROR_CALLBACK, 'dumpError');