Been looking into an error I found with this adapter, only to discover that while it's okay for 90% of use cases to return an array on the listContents call, as arrays are indeed iterable things, if you do things like $filesystem->listContents()->filter(function() { }), then it requires that it returns an iterator.
I also fixed a very silly error I discovered (likely my own fault from previous iterations) that mistakenly passed $path as the second parameter ($deep for recursive searches).
freekmurze
commented
3 years ago
Been looking into an error I found with this adapter, only to discover that while it's okay for 90% of use cases to return an array on the
listContentscall, as arrays are indeed iterable things, if you do things like$filesystem->listContents()->filter(function() { }), then it requires that it returns an iterator.I also fixed a very silly error I discovered (likely my own fault from previous iterations) that mistakenly passed
$pathas the second parameter ($deepfor recursive searches).