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

occ scan errors when scan folder is not available anymore #539

Closed
G33rt-hub opened this issue Jun 2, 2021 · 2 comments
Closed

occ scan errors when scan folder is not available anymore #539

G33rt-hub opened this issue Jun 2, 2021 · 2 comments
Assignees
Milestone

Comments

@G33rt-hub
Copy link

Describe the bug
An .occ audioplayer:scan -all stops scanning when a path is encountered which can't be accessed anymore. For example this can happen when the access to an external storage (in my case a local storage) has been removed for a user.

To Reproduce
Steps to reproduce the behavior:

  1. User A gets access to a local external storage location from an admin
  2. User A sets this location as prefered scan location on the Scanner Settings page
  3. Admin has set .occ audioplayer:scan -all in a cron job
  4. Admin removes access to the external storage location for User A
  5. .occ audioplayer:scan -all gives a PHP error and doesn't continue

Expected behavior
skip the error and continue the scan for other users

Versions:

  • Nextcloud: 21.0.1
  • Audioplayer 3.1.0
@G33rt-hub G33rt-hub changed the title [BUG] [BUG] scan -all errors when preset path to scan is not available anymore Jun 2, 2021
@G33rt-hub G33rt-hub changed the title [BUG] scan -all errors when preset path to scan is not available anymore [BUG] occ audioplayer:scan --all errors when preset path to scan is not available anymore Jun 2, 2021
@Rello Rello self-assigned this Jun 2, 2021
@Rello
Copy link
Owner

Rello commented Jun 2, 2021

Hello,
thank you for the report. I will need to look at the error handling. thats not good.

@Rello Rello added this to the 3.2.0 milestone Jul 10, 2021
@Rello Rello changed the title [BUG] occ audioplayer:scan --all errors when preset path to scan is not available anymore occ scan errors when scan folder is not available anymore Aug 17, 2021
@Rello
Copy link
Owner

Rello commented Aug 17, 2021

issue is solved with the next update. sorry for the delay

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

No branches or pull requests

2 participants