jabref icon indicating copy to clipboard operation
jabref copied to clipboard

Fix moving of file into library

Open koppor opened this issue 1 year ago • 7 comments

Fixes https://github.com/JabRef/jabref-issue-melting-pot/issues/630

Follow-up to #11576

Additionally:

  • We added support for modifier keys when dropping a file on an entry in the main table.
  • When dropping a file into the main table, after copy or move, the file is now renamed according to the configured patterns.

Mandatory checks

  • [x] I own the copyright of the code submitted and I licence it under the MIT license
  • [x] Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
  • [ ] Tests created for changes (if applicable)
  • [x] Manually tested changed features in running JabRef (always required)
  • [ ] Screenshots added in PR description (for UI changes)
  • [ ] Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • [ ] Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

koppor avatar Oct 16 '24 22:10 koppor

Need to test under Mac, there was something about modifiers not working

Siedlerchr avatar Oct 17 '24 06:10 Siedlerchr

Need to test under Mac, there was something about modifiers not working

Test is drag and drop with and without modifier keys>

  • outside file directory
  • inside file directory

Which means: 6 different manual test cases.

You can also add drag'n'drop from another volume, which makes then 9 test cases.

koppor avatar Oct 17 '24 06:10 koppor

The build of this PR is available at https://builds.jabref.org/pull/12001/merge.

github-actions[bot] avatar Oct 17 '24 06:10 github-actions[bot]

The build of this PR is available at https://builds.jabref.org/pull/12001/merge.

github-actions[bot] avatar Oct 17 '24 12:10 github-actions[bot]

The build of this PR is available at https://builds.jabref.org/pull/12001/merge.

github-actions[bot] avatar Oct 17 '24 18:10 github-actions[bot]

The build of this PR is available at https://builds.jabref.org/pull/12001/merge.

github-actions[bot] avatar Oct 17 '24 21:10 github-actions[bot]

Mac only supports copy https://bugs.openjdk.org/browse/JDK-8264172

Siedlerchr avatar Oct 20 '24 17:10 Siedlerchr

I've only done 3 test cases with a .bib file outside of the PDFs files directory.

The PR works!

(Linux Mint)

InAnYan avatar Oct 23 '24 09:10 InAnYan

One needs to check whether the file indexing works - or causes trouble... I would bet it works, because I first handle the file on the file system and then add it to jabref which then indexes it...

koppor avatar Oct 23 '24 12:10 koppor

Fixed compilation errors, removed IndexManager from ImportHandler, and removed the index blocker. Tested, the indexing works without issues.

LoayGhreeb avatar Oct 23 '24 13:10 LoayGhreeb

The build for this PR is no longer available. Please visit https://builds.jabref.org/main/ for the latest build.

github-actions[bot] avatar Oct 23 '24 15:10 github-actions[bot]

@LoayGhreeb Thank you for fixing! Then we can go ahead!

koppor avatar Oct 23 '24 15:10 koppor