Replace old image

[quaidesbalises]

Great package ! Curently if I upload an image for my client logo, it saves to S3 like a charm. But, if my client update his logo, it works but it doesn't delete the old one in the folder. Any improvements for this ?

[saqueib]

Glad you liked it, please share config so I can try to reproduce it especially upload_disk & path and your model image field definition.

[quaidesbalises]

yes for sure :

config/imageup.php :

return [
    'upload_disk' => 's3',
    'upload_directory' => 'uploads',
    'auto_upload_images' => true,
    'auto_delete_images' => true,
    'resize_image_quality' => 80
];

app/Client.php :

protected static $imageFields = [
        'logo' => [
            'width' => 200,
            'height' => 100,
            'crop' => true,
            'path' => 'avatars'
        ]
    ];

ClientsController.php

public function update(Request $request)
    {
    	auth()->user()->update($request->all());
    }

[saqueib]

run composer update, fix has been released as 1.0.1

[quaidesbalises]

Hello,

Sorry but I always have the same problem with the same configuration. Any help about this ?

[saqueib]

Are you still unable to delete old files on s3?

Please update the package, it should be version 1.0.2, delete composer.lock file and run composer update

[quaidesbalises]

Yes, i upload the logo of my client in the logo field. That's work.
I update the logo, it works again but the old one is still on the s3 folder.
I'm on the last version of the package.

[saqueib]

It could be due to IAM permission, make sure you have delete privilege on the bucket you are working with.

[quaidesbalises]

This point is OK. I hope we could find a solution . :)

[saqueib]

It's hard to debug since I am unable to reproduce it. If you can share your code on some repo then I can try to figure it out.

[saqueib]

this is the image i uploaded https://acheter-un-appartement-neuf.s3.eu-west-1.amazonaws.com/avatars/4VYOiX5ow6XKRFtKfBX3MUCYMakjxXLvTJiFnk6c.jpeg

and now when I re-upload I am unable to access old image, which mean It was deleted.

[saqueib]

Now new image is this
https://acheter-un-appartement-neuf.s3.eu-west-1.amazonaws.com/avatars/bdV4V6hyKDezgvIl7gg0cN3Zf6JQDtyF5jSy8ikU.jpeg

You cant access old one.

what version on laravel you are using, I am on 5.7

[quaidesbalises]

Yes that’s right, I can see it. Very strange, I’m on the last version 5.7.8. I test again with : auth()->user()->update($request->all()); in my Controller because my initial is : auth()->user()->updateProfile($request) which is called directly in the model but that the same. François Dusautoir Co-dirigeant, Quai des Balises 06.49.41.48.71 <tel:06.49.41.48.71> | [email protected] <mailto:[email protected]> www.quaidesbalises.com <http://www.quaidesbalises.com/> | Skype: fdusautoir <https://webapp.wisestamp.com/sig_iframe?origin=mac-mail&signature_id=5489047395631104&t=0.5071669318293202#> <http://facebook.com/quaidesbalises> <http://us.linkedin.com/in/quaidesbalises> <http://twitter.com/quaidesbalises>

…

Le 4 oct. 2018 à 18:29, Mohd. Saqueib Ansari ***@***.***> a écrit : Now new image is this https://acheter-un-appartement-neuf.s3.eu-west-1.amazonaws.com/avatars/bdV4V6hyKDezgvIl7gg0cN3Zf6JQDtyF5jSy8ikU.jpeg <https://acheter-un-appartement-neuf.s3.eu-west-1.amazonaws.com/avatars/bdV4V6hyKDezgvIl7gg0cN3Zf6JQDtyF5jSy8ikU.jpeg> You cant access old one. what version on laravel you are using, I am on 5.7 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#5 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AX2LQ4tIR9lK9X78mrQryx-yQN-iYwvUks5uhjdhgaJpZM4W-E94>.