Comments (19)
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 :-(
At the moment, this means I have to fix the bug locally from time to time :-(
from notes.
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.
from notes.
Yes, I can verify this is happening to me as well. This error message repeatedly appears in the Nextcloud logs.
LockedException "Notes" is locked
Controller failed with OCP\Lock\LockedException
from notes.
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.
from notes.
Can confirm.. This is a regression for me :/
from notes.
Thanks for reporting it, got same problem, and did use the same revert of a5bec1a to fix it
I do share my "Notes" folder with my wife, and it was not working anymore, it was creating endlessly "Notes (2)", "Notes (3)", ... , up to "Notes (19)"
FYI, we are mainly using the "Nextcloud Notes" Android app where we had the problem using the "Steps To Reproduce" reported above
Best Regards
from notes.
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.
The program flow goes like this:
If the chosen notes folder exists but is shared, it may not be used and a new one is created (with an incrementing number appended).
Once it has been created, the existence check confirms that it exists, which now makes it forbidden since only non-existing folders are allowed, that's the point of getNonExistingName().
Therefore it fails to use the just created folder and a new (higher numbered) folder is created, and the cycle repeats forever.
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?
from notes.
Indeed, it's still broken, I've tried both options,
with and without the code
if ($folder->isShared()) {
$folderName = $this->root->getNonExistingName($path);
$folder = $this->root->newFolder($folderName);
}
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.
from notes.
Related Issues (20)
- Notes are not loading (user has ~5k notes). HTTP 504 Gateway Timeout.
- Notes keep scrolling up HOT 14
- Code/quote text is copied with markdown symbols HOT 2
- Edit Timestamp HOT 1
- Changing the storage location isn't applied HOT 2
- Rich Text Editor: Input field for hyperlinks clears its input at seemingly random intervals.
- Dark theme breaks highlighting
- Bug - category selection menu is cut off
- Hide hidden directory HOT 2
- Notes incompatible with NC 28 HOT 3
- Getting a few errors and very slow performance on nc28 HOT 7
- "Uncaught (in promise) TypeError: t is undefined" after updating to nextcloud 28.0.3 and latest notes HOT 5
- [Bug]: Nextcloud Notes not respecting spelling HOT 1
- Filerun compatible HOT 1
- Checkbox reordering like google keep HOT 2
- Manage concurrent edits of notes part 2
- Option to download/export a note file.
- Sort on Note title and Category
- Artefacts in the settings window
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from notes.