Sync feature make the category unable to delete

Cologler commented 7 months ago

Steps to reproduce

Enable the sync feature and sync
Delete some categories
Sync

Expected behavior

Deleted categories do not come back

Actual behavior

Deleted categories come back

Crash logs

No response

TachiyomiSY version

latest preview

Android version

14

Device

phone

Other details

The code shows it always adds the non-existent categories:

https://github.com/jobobby04/TachiyomiSY/blob/fb1649125cd4fce6843ede61862de195b393f1d1/app/src/main/java/eu/kanade/tachiyomi/data/sync/service/SyncService.kt#L294-L299

I suggest caching the content (or hash) of the previously synced data and comparing it with the current sync, if the two are equal, overwrite remote data with local data instead of doing the merging operation.

@kaiserbh

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 1.10.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Cologler commented 7 months ago

Or include a device ID in the synced data. If the latest synced device is the current device, overwrite remote data.

kaiserbh commented 7 months ago

That could work but you will still need to delete the category manually in your other devices, as far as I know I don't think the backup restore process account for the deleted category.

Cologler commented 7 months ago

Another benefit is that this approach can bypass some of the restore processes, improving synchronization speed and reducing synchronization conflicts. To give an example, when synchronizing while reading, sometimes already-read chapters may become unread after restoration.

kaiserbh commented 7 months ago

Opened #1154 feel free to use that and test it, the latest master version or develop branch in syncyomi (https://github.com/SyncYomi/SyncYomi) have the feature but you will need to build it manually I haven't pushed the release yet. Also probably need to delete the current sync data or api key and generate a new one. @Cologler

kaiserbh commented 7 months ago

This should be fixed with the latest changes right @Cologler ?

Cologler commented 7 months ago

This should be fixed with the latest changes right @Cologler ?

The issue is still there.

jacfalcon0 commented 3 months ago

I've still got it.

jobobby04 / TachiyomiSY