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

Unable to set the folder path #1091

Closed
morceaudebois opened this issue Sep 20, 2023 · 9 comments
Closed

Unable to set the folder path #1091

morceaudebois opened this issue Sep 20, 2023 · 9 comments
Labels

Comments

@morceaudebois
Copy link

Hi! I'm trying to configure the Music app but I can't seem to set the library's path. It's lacking a button to confirm the selection and there's no way to save the page when I type it in manually.

Not sure if it's a real bug or if I'm just being an idiot, but some help would be appreciated!
Thank you 😊

nextcloud.mp4
@paulijar
Copy link
Collaborator

Thanks for the report. Which version of Nextcloud is this? The folder selection dialog seems slightly different than any of the versions which I have tested.

@morceaudebois
Copy link
Author

Thanks for the quick reply! It's version 27.0.1. I tried to reinstall it but it didn't change anything.

I have some errors in the console but I'm not sure if they're related or not:

Screenshot 2023-09-21 at 08 25 56

@paulijar
Copy link
Collaborator

Thanks, I need to install that version and check. The logging in the console doesn't look like it would be relevant here.

@Rob-Geebivi
Copy link

Same problem here with NC 27.1.0. Please fix soon or post if there is a workaround. Thank you!

@paulijar
Copy link
Collaborator

paulijar commented Sep 23, 2023 via email

@paulijar
Copy link
Collaborator

The workaround is to set the path from the command line using the occ tool. The command to use is like
sudo -u www-data php occ user:setting <user_name> music path <path_relative_to_home>
e.g.
sudo -u www-data php occ user:setting pauli music path /Music/MP3/.

When done like this, any previously scanned files outside the new path are not removed automatically. To remove them, you may use the command
sudo -u www-data php occ music:scan --clean-obsolete <user_name>.

@paulijar
Copy link
Collaborator

It seems that this happens because Nextcloud has accidentally broken their API compatibility and it has been fixed here: nextcloud/server#40557. Hence, the issue should be gone in the NC version 27.1.2, which is scheduled to be released on 2023-10-19. I may be able to release a Music-app-side workaround before that but this remains to be seen.

@paulijar paulijar added the Bug label Sep 25, 2023
paulijar added a commit that referenced this issue Sep 27, 2023
In the mentioned NC versions, the legacy filepicker API doesn't work
without passing the dialog type (choose/copy/move/copy-and-move)
explicitly while the "choose" has previously been the default type.
This is supposed to get fixed in NC 27.1.2. In the meantime, we may
pass the type "choose" explicitly.

All the access to the OC filepicker API now happens through our own
wrapper class. This should make our life easier if the deprecated
legacy API gets removed for good at some point.

refs #1091
@paulijar
Copy link
Collaborator

paulijar commented Oct 9, 2023

The app-side workaround has now been released in Music v1.9.1.

@paulijar paulijar closed this as completed Oct 9, 2023
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

4 participants
@Rob-Geebivi @paulijar @morceaudebois and others