Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't delete original mime icons #21

Closed
FlorianMoser opened this issue Oct 17, 2019 · 5 comments
Closed

Don't delete original mime icons #21

FlorianMoser opened this issue Oct 17, 2019 · 5 comments

Comments

@FlorianMoser
Copy link

When adding a product image and the deleting it, the image is (as expected) also deleted.

However when adding a non-image file, a mime icon is displayed. Deleting that media item also deletes the original Aimeos mime icon.

How to reproduce:

  1. Open a product
  2. add a media file (in my case a .zip file)
  3. save -> Aimeos now displays the icon public/packages/aimeos/shop/mimeicons/application/zip.png
  4. remove the media file
  5. save

After step 5, the file public/packages/aimeos/shop/mimeicons/application/zip.png is gone.

@aimeos
Copy link
Owner

aimeos commented Oct 17, 2019

We will release the new 2019.10 version based on Laravel 6 that shouldn't have that problem any more next Thusday or Wednesday

@aimeos
Copy link
Owner

aimeos commented Oct 17, 2019

Here's the fix: aimeos/aimeos-core@676f10f

@FlorianMoser
Copy link
Author

Cool, thanks.
I currently also have the problem that I can't reset a password. A remember_token is generated and sent by email, but not stored in the DB. Is this also related with Laravel 6 or another issue?

@aimeos
Copy link
Owner

aimeos commented Oct 17, 2019

The column remember_token is for something different ("remember me" without login feature) then a password reset (new password is set in password column) in Laravel AFAIK

@FlorianMoser
Copy link
Author

Yeah, you are right. I checked the wrong table. The reset token is stored in password_resets table.
I had an issue with my routes. Nothing to do with Aimeos, sorry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants