file_scan_directory incorrect test of readdir()'s return value

need to fix this in HEAD and then backport. made false upper case but this looks good to go.

setup some test files:

amorton@minivac:~/Sites/dh/sites/default/test% mkdir 0
amorton@minivac:~/Sites/dh/sites/default/test% cd 0
amorton@minivac:~/Sites/dh/sites/default/test/0% touch foo
amorton@minivac:~/Sites/dh/sites/default/test/0% ll
total 0
-rw-r--r--  1 amorton  staff  0 Apr 15 23:38 foo

then, without the patch tried running:
print_r(file_scan_directory('sites/default/test', '.*'));

Array
(
)

after the patch it returned:

Array
(
    [sites/default/test/0/foo] => stdClass Object
        (
            [filename] => sites/default/test/0/foo
            [basename] => foo
            [name] => 
        )

)

Committed to CVS HEAD and DRUPAL-6. Let's start writing tests for these things ... thanks.

Dries, of course. if simpletest has been in core when i tested it i would have.

Back ported for D5.

Capitalized false and committed to 5.x.

FALSE needs to be capitalized in HEAD and 6.x. Drewish mentioned this in his review.

ah, it looks like i thought i was was attaching a patch at the time.

here it is for HEAD.

Thanks. :)

Oops. 6.x too.

Landed in 6.x as well. Thanks!