Support storing file on s3

[tantam]

I have changed the code a little bit, so now when you change driver in filesystems.php file from local to s3, it is still working.
So now you can use local or s3 to store files.

Can you check them and accept my PR. If you don't understand anything, please let me know.

Thanks in advance.

[pratiksahu]

@tantam Can you please resolve the conflicts ? I will merge it.

[ido567]

Looking forward for this change
@tantam can you please help with the conflict?

[tantam]

I just resolved on behavior of my code and tested it.
Everything for me is ok. Let me know if there is any issue, I will fix it

[ido567]

@pratiksahu Can you please merge and tag it?

[ido567]

@tantam I tried to pull your branch. but I get the following error when I try to upload a file:

The error is from SahusoftCom\EloquentImageMutator\Dist\ImageService:

[tantam]

@ido567 That function should exist by default in Laravel, you can check that function here: https://laravel.com/docs/5.4/filesystem#storing-files

Can you tell me what is Default Filesystem Disk in your filesystems.php file, your laravel version ?

[ido567]

@tantam My .env configurations are:

FILESYSTEM_DRIVER=s3
FILESYSTEM_CLOUD=s3

But I think I found the problem, this function was added in 5.3, but the package supports Laravel 5.1 and above (I use it with version 5.1).

[tantam]

Yes, you are right.

Can you able to update your laravel version ? I will change the code to fix that version, but can not do it now

[ido567]

@tantam I can't.. upgrading from 5.1 to 5.4 is too complicated and it's gonna break some other packages.
When do you think you will be able to fix this issue?

[samcambridge]

@TamTam any update on this? or have you shelved it?