Could someone test it with s3?

I will also write some documentation on how to use it, if accepted

Tested with

• Local
• Dropbox
• Orphange

Hi @lsv ! Thanks for your PR, looks really good to me. And thanks for testing! My bad, that I can't test it with s3...

The failed tests are PHP 5.3 as flysystem requires PHP 5.4
I would suggest not support 5.3 anymore?

Yes, I saw it. I would say we can skip 5.3 due to the fact that 5.3 is EOL since 14th of august 2014.

And the reason for using flysystem imo gaufrette is somekind of dead, and lot of other bundles are using flysystem instead.

Can you drop the 5.3 support in travis.yml within this PR?

Is there an estimate when this fork will be merged? I'm looking forward in using flysystem with this bundle

@rvmourik 1.5.0 is out :)

@bytehead Thanks, that was pretty quick :-)

It was on my list for at least 8 days ;-)

@lsv: @sheeep is pretty happy about your PR and much thanks from him as well! 🎉

Wow, this is very good news! It might need some documentation though.

Is it as straightforward as replacing gaufrette with flysystem? Does it support chunked uploading?

When I have a few moments i will write a proper documentation.

But for short, change gaufrette to flysystem in storage type.
And then setup flysystem bundle

• Orpange is tested
• Dropbox is tested (so maybe other external ones works - havent tested with others, please do if you have access to other external storages)
• Local also tested :)
• streamwrapper is not fully tested, but I will do this one of the following days, as I just started with a project based on flysystem (hence this PR ;-)

@bytehead well it actually took some time for me to find out that 1up-lab also have flysystem bundle, so I was a bit confused why you have chosen gaufrette :)

Anyways, it was actually pretty easy to create this, when I figured out that FileInterface missing a few methods, like move() and so on - I will properly make a PR for this.

And also a PR for documentation will come in a near future

@lsv I can totally understand your confusion 😄
The uploader bundle came out way before our flysystem bundle - that's probably why it ended up like this...

Sounds great - your PRs are always welcome!

I get a OutOfMemoryException on oneup/uploader-bundle/Oneup/UploaderBundle/Uploader/Storage/FlysystemStorage.php line 47 when uploading a larger file (~400MB). The chunked upload works, but the last step seems to cause the exception.

Any ideas?

UPDATE:
It seems the file_get_contents($file) is the cause. I replaced it with
$stream = fopen($file->getPathname(), 'r+');
$this->filesystem->putStream($name, $stream);
if (is_resource($stream)) {
fclose($stream);
}
and it works. Anyone interested in a PR?