search

dsnopek / anki-sync-server

A personal Anki sync server (so you can sync against your own server rather than AnkiWeb)
GNU Affero General Public License v3.0
744 stars 95 forks source link

Not only add, but also remove files when adopting changes in SyncMediaHandler.uploadChanges() #34

Closed cdpm closed 8 years ago

cdpm commented 8 years ago

Added logic to not only add, but also remove files when adopting changes to media files in SyncMediaHandler.uploadChanges(), and to count deletions when updating the media usn. Also refactored SyncMediaHandler.uploadChanges().

Akendo commented 8 years ago

+1 Thank you for the swiftly response. Was able to upload the new 'changes'. Didn't verify the entry deck about this, but I'll give it later a shot.

So far it seems good to me. Thank you very much!

Best regards akendo

dsnopek commented 8 years ago

Wow, these changes are really extension. Since I've been committing stuff without test it (yet), I'll probably just commit this one too :-) once I have an answer about the print() vs logging.

cdpm commented 8 years ago

Hi, thanks. I replaced the print statements with logging calls. Logging for sync app wasn't set up on my system, so I put prints in but forgot to remove them.

dsnopek commented 8 years ago

Thanks!