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

ME misreporting failed save as successful #1701

Open
2 tasks done
mihawk90 opened this issue Jan 8, 2024 · 1 comment
Open
2 tasks done

ME misreporting failed save as successful #1701

mihawk90 opened this issue Jan 8, 2024 · 1 comment
Labels
Milestone

Comments

@mihawk90
Copy link

mihawk90 commented Jan 8, 2024

Describe the bug
I just ran into this by accident. It seems that the directory I'm trying to save into has a permissions issue (which I did not realise before), and so ME doesn't have write-access. When saving the movie, it reports a successful save, but as you can imagine, it didn't actually save anything.

To Reproduce
Steps to reproduce the behavior:

  1. Have a directory with missing write permissions
  2. Scrape directory for a movie and save
  3. Check the upper right corner and/or console logs

Expected behavior
ME should notify the user about the failed save, if possible with an appropriate error message (as seen in the console log warning).

Screenshots
image

MediaElch Version:

  • 2.10.6 (stable)

Operating System:

  • Linux (distribution: Fedora 39)

Additional context
Movie is on a NAS, mounted on an NFS share. Seems I messed up my permissions somewhere.

@bugwelle bugwelle added the bug label Jan 13, 2024
@bugwelle
Copy link
Collaborator

Thanks for reporting! Yes, that's a bug. In fact, the log entry is my first approach to log something. A few versions ago, nothing was printed at all. However, I totally forgot about that. It should definitely be shown to the user!

@bugwelle bugwelle added this to the v2.10.8 milestone Jan 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants