-
-
Notifications
You must be signed in to change notification settings - Fork 229
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
incorrect sync: some bookmarks moved into another folder #1395
Comments
Hello 👋 Thank you for taking the time to open this issue with floccus. I know it's frustrating when software I look forward to working with you on this issue |
Hello also from my present self 👋 this is a tough one. Some remarks, mostly for my future self: I've looked at the logs and traced the code and so far 2 possible causes for this presented themselves:
Both seem unlikely. no 2 even more so than no 1. But I couldn't find a reason why a mapping would be created for these two, even though the have the same title, they are on a different level, haven't been moved... Next step for me is to try and reproduce this. Also: What happens when you move the bookmarks in question back to where they belong and pull sync again? |
Hello, I did another test: First I pushed up in edge. I was thinking that pull down was like a "full replace" of old bookmarks, but it seems it's not the case. Here is the log from edge: Here are the logs from chrome: |
Yes, that would have been easier for cases like this, perhaps, but my goal was for normal merge-sync to not be disrupted by the occasional pushes and pulls, which requires applying those as a diff not as a full tree replacement... |
yes, that makes sense. |
any news? bug is still happening. |
I'm sorry for the big delay on my part, I haven't had a lot of time and energy the last months. Can you try this with v5.1.1? There's a good chance it might have been fixed by a change I've done recently. |
Which version of floccus are you using?
4.19.1
Sync method
Google Drive
Which browser are you using? In case you are using the phone App, specify the Android or iOS version and device please.
Firefox 113.0.2, Edge 113.0.1774.57, Chrome 113.0.5672.127
Which version of Nextcloud Bookmarks are you using? (if relevant)
No response
Which version of Nextcloud? (if relevant)
No response
What kind of WebDAV server are you using? (if relevant)
No response
Describe the Bug
I have found an error in the synchronizarion.
I have synced from Firefox (push up) to Edge (pull down) and to Chrome (pull down).
Then I exported bookmarks to HTML files and compared them using WinMerge (after removing useless data like icon data and timestamps).
It seems that the XBEL file created during the push up from Firefox is correct.
The pull down process in Edge has read the XBEL file and then generated almost correct bookmarks.
There are differences that seems related mainly to special bookmarks like chrome:// and chrome-extension://
There are also some missing bookmarks but at least some of them were duplicated in the same folder, so I think the sync process ignored them on purpose.
In any case I didn't have time to fully analyze these differences (this bug report is about the other errors, in particular it's about the differences found in bookmarks in Edge and Chrome after the sync from Firefox.)
The pull down process in Chrome has read the XBEL file and then generated wrong bookmarks, and this is the issue I have tried to analyze and I'm reporting here. Since the XBEL source file is the same for Edge and Chrome I was expecting to get exactly the same bookmarks on the two broesers, instead there are differences.
This is the log of push up in Firefox (redacted):
floccus-4.19.1-2023-05-30-firefox-redacted.log
This is the log of pull down in Edge (redacted):
floccus-4.19.1-2023-05-30-edge-redacted.log
This is the log of pull down in Chrome (redacted):
floccus-4.19.1-2023-05-30-chrome-redacted.log
The bookmarks of Edge and Chrome are almost the same, except seven bookmarks that are moved from one folder to another one.
This is the list of moved bookmarks (taken from the exported bookmarks in the HTML file):
list of moved bookmarks.txt
In Edge these seven bookmarks are located in a particular folder (1) and in Chrome they were located into a different folder (2).
The two folders have the same name "browser" but are located into two different subfolders: the first one is in the toolbar, and the second one is in another folder (3) that is in the toolbar:
folder (1) located in -> toolbar -> browser -> bookmarks
folder (2) located in -> toolbar -> folder (3) -> browser -> bookmarks
Maybe it is just a coincidence that the name of the two folders is the same.
I have tried to understand the Chrome pull down log, to identify the folders:
folder (1) is #11370 (parent folder 1) -> d4c3e8a11256ab82a4fc72560eb4a2b0e87bad820c290dd9b03616de240aa6db
folder (2) is #11230 (parent folder 9050) -> d4c3e8a11256ab82a4fc72560eb4a2b0e87bad820c290dd9b03616de240aa6db
folder (3) is #9050 (parent id 1) -> ef260e9aa3c673af240d17a2660480361a8e081d1ffeca2a5ed0e3219fc18567
I'm not sure these infos are totally correct thought.
Please can you fix the issue?
Thanks.
Expected Behavior
bookmarks in edge and chrome should be exactly the same.
To Reproduce
see previous description.
Debug log provided
The text was updated successfully, but these errors were encountered: