wileyyugioh/zotmoov

More Options for Handling Duplicates

Opened this issue · 14 comments

Currently, it creates a new pdf with -1 at the end. Would it be possible to have more customization, where:

  1. It would create a new file with " - 1" at the end as it does now
  2. Do not copy the file if it already exists
  3. Overwrite the previous copy, and show an error if overwrite fails

Please let me know if this sounds like a good feature to implement :)

It's definitely a possibility. Can you explain why specifically you want features 2 and 3?

The reason duplicates are handled the way they are now is to prevent files from accidentally being deleted when moved. For example Item A and Item B have pdfs that share the same filename, but contain different content. In the case of 2, Item B cannot be moved unless the filename is manually changed, and in the case of 3 Item A will be pointing to the wrong file.

I think @heymaad is referring to a duplicate behavior in zotmoov when attaching files from the source folder. Specifically, when attaching a file in Zotero using the menu option Zotmoov: Attach New File, the extension attaches two identical files to the reference.

This is really annoying. It makes the whole effort meaningless.

Oh wow. That is definitely a bug. What version of ZotMoov are you running? In the latest version (1.1.8) I made it so files are instantly linked when using the add menu option.

Also, the linked files are only deleted when you empty the trash.

I'm using the most recent version (1.1.8).

Oh wow. That is definitely a bug. What version of ZotMoov are you running? In the latest version (1.1.8) I made it so files are instantly linked when using the add menu option.

Also, the linked files are only deleted when you empty the trash.

Try 1.1.9. I think I identified the bug (a race condition between adding the file and the auto functionality).

Try 1.1.9. I think I identified the bug (a race condition between adding the file and the auto functionality).

It's working fine now! Thank you for all your hard work @wileyyugioh ! 🎉

Can you please try the latest prerelease and see if the problem is also fixed in this version? I changed how the bug was fixed under the hood and want to make sure it still works.

Can you please try the latest prerelease and see if the problem is also fixed in this version? I changed how the bug was fixed under the hood and want to make sure it still works.

Work like a charm. :)

Hi @wileyyugioh ,

I found the same duplicate behavior when attaching an EPUB to a reference.

EDIT: It's happening with PDFs too.

How to reproduce the error

  • Create a reference on Zotero 7.
  • Manually attach an EPUB to the reference by dragging the file to it using the mouse.

System Info

  • Windows 11 Home Single Language 23H2
  • Zotero 7.0.5 (64-bit)
  • ZotMoov 1.1.12

Is this happening when you drag-drop an item? So it's not an issue with the add button is that right?

Is this happening when you drag-drop an item? So it's not an issue with the add button is that right?

Yes. It's not an issue with the add button.

@danielvartan Can you try 1.1.14. I think I found a race condition in the drag-drop code.

Just tried with the 1.1.14, as you suggested. All good!

Thank you for the quick response!