nextcloud / bookmarks

🔖 Bookmark app for Nextcloud
https://apps.nextcloud.com/apps/bookmarks
GNU Affero General Public License v3.0
1k stars 173 forks source link

Loaded bookmarks (only viewed through the normal view) are added also to Trash Bin section #2167

Closed estux closed 3 months ago

estux commented 3 months ago

Describe the bug I opened Bookmarks app and started navigating through folders I created (only viewing what bookmarks I have). Then I went to Trash Bin and seen the "viewed" items added also there. So I exited the app and then opened again (I have 3 bookmarks that aren't in any folder). Opened the Trash Bin and saw the 3 loaded bookmarks (the one seen in the previous view, that aren't in any folder, loaded in the main view) added also there. So every time you go out and come back the view in the Trash Bin is reset and shows only the "last viewed-navigated items" from the main view.

To Reproduce Steps to reproduce the behavior:

  1. Open Bookmarks app
  2. Have some bookmarks that aren't in any folder (e.g. 3 of them)
  3. Click to see what bookmarks you have inside a folder (e.g. 8)
  4. Open the Trash Bin
  5. You can see 3+8 items "viewed" before in the Trash Bin view.
  6. It seems that they aren't really trashed because if you come back to the main view you can still see e.g. the first 3 bookmarks (the ones that aren't in any folder)
  7. Now you can try exiting Bookmarks app
  8. open again Bookmarks app
  9. In the main view the 3 bookmarks that aren't in any folder are automatically "loaded"
  10. Go again to the Trash Bin to see only this 3 bookmarks in the Trash Bin view.

Expected behavior No bookmarks seen in the Trash Bin as I didn't trashed any bookmark.

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Server (please complete the following information):

Additional context Add any other context about the problem here.

Web server error log

No errors related to Bookmarks seem to be recorded.

Nextcloud log (nextcloud/data/nextcloud.log)

Insert your Nextcloud log here

Browser log

Insert your browser log here, this could for example include:

a) The javascript console log
b) The network log 
c) ...
github-actions[bot] commented 3 months ago

Hello :wave:

Thank you for taking the time to open this issue with the bookmarks app. I know it's frustrating when software causes problems. You have made the right choice to come here and open an issue to make sure your problem gets looked at and if possible solved. I'm Marcel and have been maintaining this software the last few years. I currently work for Nextcloud but maintain this app in my free time, because it is not an official Nextcloud product. My day job at Nextcloud is pretty awesome but sadly leaves me with less time for side projects like this one than I used to have. I still try to answer all issues and if possible fix all bugs here, but it sometimes takes a while until I get to it. Until then, please be patient. Note also that GitHub is a place where people meet to make software better together. Nobody here is under any obligation to help you, solve your problems or deliver on any expectations or demands you may have, but if enough people come together we can collaborate to make this software better. For everyone. Thus, if you can, you could also look at other issues to see whether you can help other people with your knowledge and experience. If you have coding experience it would also be awesome if you could step up to dive into the code and try to fix the odd bug yourself. Everyone will be thankful for extra helping hands! One last word: If you feel, at any point, like you need to vent, this is not the place for it; you can go to the forum, to twitter or somewhere else. But this is a technical issue tracker, so please make sure to focus on the tech and keep your opinions to yourself. (Also see our Code of Conduct. Really.)

I look forward to working with you on this issue Cheers :blue_heart:

marcelklehr commented 3 months ago

Confirmed.

marcelklehr commented 3 months ago

Workaround for now is to reload the page after accessing the trash bin

estux commented 3 months ago

Workaround for now is to reload the page after accessing the trash bin

@marcelklehr Thanks for looking into this and confirming! I'm fine with the workaround ...... I only hope it doesn't automatically trash some bookmarks!

marcelklehr commented 3 months ago

Yeah, I understand that it's disconcerting to see stuff in the bin that shouldn't be there. I'll fix this soon.

estux commented 3 months ago

Thank you so much for working so fast on this! :)