bookmarks icon indicating copy to clipboard operation
bookmarks copied to clipboard

Bookmarks counter wrong

Open InfamousUser opened this issue 2 years ago • 1 comments

Describe the bug Just deleted my bookmarks and now the bookmark counter is wrong, showing 13 bookmarks and the main screen telling me there are no bookmarks here.

To Reproduce Steps to reproduce the behavior:

  1. Previously use bookmarks.
  2. Deleted bookmarks.
  3. Be unable to delete them all as counter shows non-zero value with zero bookmarks.

Expected behavior Counter should indicate zero or be gone.

Screenshots image

Desktop (please complete the following information):

  • OS: Linux Mint 21.3
  • Browser: Firefox
  • Version: 122.0.1

Server (please complete the following information):

  • OS: Ubuntu
  • HTTP server: Apache
  • Database: Postgresql
  • PHP version: 8.1
  • Nextcloud version: 28.0.2
  • Bookmarks app version: 13.1.3
  • Activated Nextcloud Apps: [the output of running $ sudo -u www-data php occ app:list in your command line]
  • Nextcloud configuration: [the output of running sudo -u www-data php occ config:list system]
  • Nextcloud external user backend: [e.g. ldap, or none]

Additional context Add any other context about the problem here.

Web server error log

Insert your webserver log here

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) ...

InfamousUser avatar Feb 18 '24 18:02 InfamousUser

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:

github-actions[bot] avatar Feb 18 '24 18:02 github-actions[bot]

Turns out when I resynced the bookmarks from my device, there were 13 bookmarks that became visible...

InfamousUser avatar Feb 25 '24 23:02 InfamousUser

Yeah, it's possible this is an old remnant from an old migration when folders were first introduced. The bookmarks would not have been in any folder then, so are not visible from the "All bookmarks" view, but may have been visible from the "Recent bookmarks" view for example.

marcelklehr avatar Apr 28 '24 17:04 marcelklehr

So this should not reoccur? Although I do not think it is that old...

InfamousUser avatar Apr 29 '24 06:04 InfamousUser

I'm not sure. It's unlikely to occur again, I think. If it does reoccur, could you send me a dump of the oc_bookmarks* tables? That would be great.

marcelklehr avatar Apr 29 '24 14:04 marcelklehr

Alright, I'll close this for now then.

InfamousUser avatar Apr 29 '24 14:04 InfamousUser