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
Cannot use shared folder for notes anymore. #1266
Comments
Yes, I can verify this is happening to me as well. This error message repeatedly appears in the Nextcloud logs.
|
This is happening to me as well using the same setup as above. The bug was introduced in commit a5bec1a. After commenting out the faulty code in the source code of the installed Notes App, everything works as expected again. Unfortunately it is unclear why this fix was introduced and why no tests failed. |
Can confirm, reverting a5bec1a does help, just in my case the shared-to user needed to temporarily set a different folder before I could re-share the original. Might be normal behavior caused by me deleting the share before, in an attempt to fix the issue. Similar setup as OP, Notes 4.9.4. |
Can confirm.. This is a regression for me :/ |
Thanks for reporting it, got same problem, and did use the same revert of a5bec1a to fix it Best Regards |
All of my storage is external since my nextcloud docker container is ephemeral, so all storage is "shared" by group rules, even the user's home directory. So I can't use Notes any more. The current implementation using Folder->getNonExistingName() couldn't have passed even superficial testing in a shared folder. So whenever NoteUtil->getOrCreateFolder is called (apparently quite frequently, on every note file save attempt), it creates a new empty copy of the forbidden existing folder. This goes on forever and creates hundreds of empty folders in minutes. Obviously, getNonExistingName is being used incorrectly. Is there a reason for the apparent intention not to use existing shared folders? |
Indeed, it's still broken, I've tried both options,
it simply stays broken. I can't use the Notes app anymore together with my partner. At the legitimate user, who is sharing the folder, everything works fine. The one who is using the shared folder, gets these other folders 'Notes (2)' etc. |
Any response on this problem? Looks like a lot of people have the same problem and nobody knows, what the code should do, which causes the problem. It is a little bit annoying, yesterday was a new release and there is still no response from a developer :-( |
I have the same problem. First my parter had an issue, then I've changed the permission, she started to share with me, and now I cannot use Notes app on my phone. |
Updated to 4.10.0. The issue is still the same and the workaround (reverting a5bec1a) still works. |
I share the
Notes
between two accounts. The first account is my main account. The second account is a dedicated account for my smartphone. Up to version 4.9.3, everything works fine. I could create and edit markdown files on my PC and sync it via Nextcloud client to the main account, use the web interface of my main account for editing notices and read and write notices on my smartphone with the nextcloud notes app syncing over the dedicated android account.Since version 4.9.4. The android app and the notice app of the android account is not working anymore. The notice app on the main account is still working.
I think, this is related to this PR: #1263
Steps to reproduce
Notes
folder of the account A with account BExpected behaviour
The notices should be shared and the web interface and the android app should work with account A and B in the same way.
Actual behaviour
Account B creates an new notes folder (
Notes (2)
) and stores the notes there.Server
Please complete the following information.
Nextcloud configuration:
Client
Please complete the following information.
The text was updated successfully, but these errors were encountered: