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

xBrowserSync not compatible with bookmark sorters #120

Open
boomshankerx opened this issue Aug 10, 2019 · 4 comments

Comments

@boomshankerx
Copy link

commented Aug 10, 2019

I am using a chrome extension called SuperSorter to properly sort my bookmarks and folders. Every time I perform a sort on my bookmarks xBookmarkSync throws an error and reverts my bookmarks back to before the sort.

I've attached log file and screen cap of the error.

SNAG-2019-08-10-0000

xBrowserSyncLog_20190810110519568.txt

@boomshankerx

This comment has been minimized.

Copy link
Author

commented Aug 12, 2019

Additional Info:

It appears that the extension "Recursive Bookmark Sorter" seems to work fine with xBrowserSync. At least when making small changes.

SuperSorter has some extra valuable features that are valuable like deleting duplicates and removing empty folders.

@nero120

This comment has been minimized.

Copy link
Member

commented Aug 12, 2019

@boomshankerx in order for me to fix this, can you please provide me with an xbrowsersync backup file that when SuperSorter is applied causes the error? It doesn't have to be your bookmarks data, anything generic that recreates the issue will work.

@maephisto666

This comment has been minimized.

Copy link

commented Aug 12, 2019

Hello @nero120, I think I was able to narrow the set of boomarks that in my case is causing the issue described by @boomshankerx (i.e. when I run SuperSorter, I got the same error message). Now as you can see (for some reasons, don't ask my why) I have 2 folders with the same name and this is causing the crash: when I moved every bookmark from one folder to the other one and I ran again SuperSorter, the sync went fine.

xBrowserSyncBackup_20190812205656287.txt

@nero120

This comment has been minimized.

Copy link
Member

commented Aug 20, 2019

@maephisto666 thanks for this. It appears that the process of moving bookmarks either simultaneously or in quick succession (as does SuperSorter) can sometimes lead to conflicts. This is because xBrowserSync uses indexes when processing syncs. I have some ideas on how to resolve this, but it will take some experimentation, it's good that you can work around this issue for the time being.

@nero120 nero120 added bug and removed needs triage labels Aug 20, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.