spatie / flysystem-dropbox

A flysystem driver for Dropbox that uses the v2 API
https://freek.dev/734-dropbox-will-turn-off-v1-of-their-api-soon-its-time-to-update-your-php-application
MIT License
342 stars 50 forks source link

Avoid listing the base directory itself in listContents calls. #73

Closed BusterNeece closed 3 years ago

BusterNeece commented 3 years ago

Another minor fix that brings this adapter in line with the other Flysystem adapters by ensuring that, when calling listContents, the base directory itself isn't included in the results, which it otherwise would be without this little safety check.

BusterNeece commented 3 years ago

@freekmurze We're green on the tests for this one; I believe the test for this one was mocking up a response that wasn't actually the response this call would give, so once filtering was added to that response it looked empty.

freekmurze commented 3 years ago

Thank you!