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

Deleting notes inside notebooks crashes tomboy-ng #312

Open
uwemock opened this issue Jul 12, 2024 · 4 comments
Open

Deleting notes inside notebooks crashes tomboy-ng #312

uwemock opened this issue Jul 12, 2024 · 4 comments
Labels
fixed in next release Fixed commited and, maybe even a pre-release available.

Comments

@uwemock
Copy link

uwemock commented Jul 12, 2024

tomboy-ng gets stuck upon deleting notes that are inside a notebook.

To reproduce:
Create a notebook, create a note inside the notebook. Open the new note and klick the delete button ("X" in a circle). Confirm "Yes". tomboy-ng hangs. However, the note will be gone when you restart tomboy-ng.

When I delete notes that are not connected to a notebook, tomboy-ng might crash a little later.

@davidbannon
Copy link
Member

Yep, I can replicate the main issue. Its clearly been there since I made some substantial changes to the way notes are listed in the right hand list box (to speed up that listing). I personally don't use the notebook model that much and apparently just did not stumble across that bug. Not very good !

So, not that hard to fix but I found two more, closely related bugs ! One when adding and immediately deleting a Notebook and the other, a newly created Notebook was not appearing in the notebook list immediately. This area of the code is quite complicated so I need to run some extensive tests before pushing out even a beta.

Thanks uwemock, if you had not spotted this, who know how long it would be in there for !

Now, the second part of your report,

When I delete notes that are not connected to a notebook, tomboy-ng might crash a little later.

That I am afraid I cannot reproduce. I have deleted over a 100 notes, mostly not members of a notebook and I cannot produce any instability. Can you tell me a bit more about what you do that might be the trigger for for this problem ?
(just groping in the dark here ...)
Operating system ?
Approx number of notes in your system, hundreds, thousands ?
Do you have a lot of notes open at the time ?
Are these notes that you delete, old, existing notes or brand new ones ?
Anything else you can think of ?

Thanks for an important bug report !

Davo

@uwemock
Copy link
Author

uwemock commented Jul 21, 2024

I cannot fully reproduce the crash when deleting notes that are not member of a notbook. Once I can, I will file another bug.

@davidbannon
Copy link
Member

OK, I have just released a beta that addresses this issue. Reasonably well tested and probably quite safe to use.
https://github.com/tomboy-notes/tomboy-ng/releases/tag/v0.40a

While I have addressed the first part of your report, I am not so sure about the second part. I suspect it might relate to a situation where an empty note was previously deleted and then an existing one opened, I found potential issues in that space but could not be sure. Feel free to let me know your experiences !

Thanks for your report and helping to make tomboy-ng a better product.

Davo

@uwemock
Copy link
Author

uwemock commented Aug 21, 2024

Thanks. I downloaded and installed it. I'll come back once I have something to report.

@davidbannon davidbannon added the fixed in next release Fixed commited and, maybe even a pre-release available. label Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed in next release Fixed commited and, maybe even a pre-release available.
Projects
None yet
Development

No branches or pull requests

2 participants