Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.4.1

Bug #14513 Timestamp when using the LS function
Submitted: 2008-08-18 06:58 UTC
From: ursel Assigned: jorrit
Status: Closed Package: Net_FTP (version 1.3.7)
PHP Version: 5.2.6 OS: IIS 6 / Win2003
Roadmaps: 1.3.8, 1.4.0a3    
Subscription  


 [2008-08-18 06:58 UTC] ursel (Ursel Schie)
Description: ------------ In the function _listAndParse on LINE 2166 the $entry['date'] is passed true $this->_parseDate en the return value is stored in the $entry['stamp']. Unfortunately, in version 1.3.7 the $entry['date'] now only contains :[date] => 08-08-07 The time is splitted from the date and is now in $entry['time'] :[time] => 10:56AM In version 1.3.2 (our previous one) the entry['date'] contained: [date] => 08-08-07 10:56AM As only the date is passed true the parseDate , the stamp will have the timestamp of the date only. Test script: --------------- Can simply been seen using a dump of the value of $entry when using version 1.3.2 and version 1.3.7 Expected result: ---------------- The timestamp of the date, including the time. Actual result: -------------- The timestamp of the date.

Comments

 [2008-08-23 18:16 UTC] jorrit (Jorrit Schippers)
I think I'll reverse the change that split date into date and time. Would that be sufficient? It was an unintentional breaking of backwards compatibility anyway.
 [2008-08-28 07:12 UTC] ursel (Ursel Schie)
For the time being I editted this pear class by adding an extra variable where both date and time are combined and put this new variable into $this->_parseDate. I'm not sure what the reason was for splitting both. If there was no real intention for this then reversing is fine with me :)
 [2008-09-13 19:28 UTC] jorrit (Jorrit Schippers)
Does the change I just committed fix the problem?
 [2008-10-15 13:27 UTC] jorrit (Jorrit Schippers)
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.