PHP :: Bug #47534 :: RecursiveDirectoryIterator::current() always returns SplFileInfo
| Bug #47534 | RecursiveDirectoryIterator::current() always returns SplFileInfo | ||||
|---|---|---|---|---|---|
| Submitted: | 2009-03-01 01:12 UTC | Modified: | 2010-04-27 06:28 UTC | ||
| From: | david at ethos-consulting dot com | Assigned: | colder (profile) | ||
| Status: | Not a bug | Package: | SPL related | ||
| PHP Version: | 5.2.9 | OS: | MacOS X 10.5.6 | ||
| Private report: | No | CVE-ID: | None | ||
[2009-03-01 01:12 UTC] david at ethos-consulting dot com
Description:
------------
The return value for RecursiveDirectoryIterator::current() is _always_ SplFileInfo, which isn't as expected and is less useful than the behavior in previous versions within the 5.2.x branch. At least one other programmer has submitted this as a bug, as well (see #37530), however he was dismissed with the standard RTFM boilerplate.
This is a flaw in the behavior of the class, IMO, however, since there's no way to _disable_ the class from returning SplFileInfo. The SPL documentation briefly refers to a flag that can be passed to __construct() named "CURRENT_AS_FILEINFO", which would indicate to me that the default behavior for the class is to return something _other_ than SplFileInfo for current().
The docs seem to back that up, listing the variable behavior of current() to alternate between getFilename() and getFileInfo(). However, the class consistently returns SplFileInfo, regardless of the flags provided to the constructor.
Reproduce code:
---------------
<?php
function testReturnTypeOfCurrent ( $Directory, $assertion )
{
$assertion = sprintf($assertion, '$Node');
echo 'Asserting: ' . $assertion, "\n";
foreach ( $Directory as $Node )
{
echo ( @assert($assertion) ?
'Assertion Passed!' : 'Assertion Failed!' ), "\n";
}
} // END testReturnTypeOfCurrent
testReturnTypeOfCurrent(
new RecursiveDirectoryIterator('/path/to/walk'),
'gettype(%s) == "string"'
);
testReturnTypeOfCurrent(
new RecursiveDirectoryIterator(
'/path/to/walk'
RecursiveDirectoryIterator::CURRENT_AS_FILE
),
'%s instanceof SplFileInfo'
);
Expected result:
----------------
Asserting: gettype($Node) == 'string'
Assertion Passed! x number of items in /path/to/walk
Asserting: $Node instanceof SplFileInfo
Assertion Passed! x number of items in /path/to/walk
Actual result:
--------------
Asserting: gettype($Node) == 'string'
Assertion Failed! x number of items in /path/to/walk
Asserting: $Node instanceof SplFileInfo
Assertion Passed! x number of items in /path/to/walk
Patches
Pull Requests
History
AllCommentsChangesGit/SVN commits
[2010-04-27 06:28 UTC] colder@php.net