Fix #13269 : support multi-file import across different formats

[070jhz]

Closes #13269

this PR improves the import command to properly handle importing multiple files with different supported formats in a single operation.

ImportCommand::execute now calls dialogService.showFileOpenDialogAndGetMultipleFiles instead of dialogService.showFileOpenDialog and importSingleFile() has been refactored to importMultipleFiles() to handle one or more files

Steps to test

1. File menu > Import > Import into current/new library
2. Select one or more files
3. ???
4. Profit

Mandatory checks

• I own the copyright of the code submitted and I license it under the MIT license
• [/] Change in CHANGELOG.md described in a way that is understandable for the average user (if change is visible to the user)
• Tests created for changes (if applicable)
• Manually tested changed features in running JabRef (always required)
• [/] Screenshots added in PR description (if change is visible to the user)
• 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.

[Siedlerchr]

PLease add a changelog entry, otherwise it's good to go!

[jabref-machine]

You modified Markdown (*.md) files and did not meet JabRef's rules for consistently formatted Markdown files. To ensure consistent styling, we have markdown-lint in place. Markdown lint's rules help to keep our Markdown files consistent within this repository and consistent with the Markdown files outside here.

You can check the detailed error output by navigating to your pull request, selecting the tab "Checks", section "Tests" (on the left), subsection "Markdown".

[subhramit]

I have some questions

[subhramit]

Why should it be null for cases selectedExtensionFilter == FileFilterConverter.ANY_FILE || "Available import formats".equals(selectedExtensionFilter.getDescription()) ? The logic in the else case is more general - shouldn't that work here as well?

[070jhz]

Well I just realized that it worked because showFileOpenDialogAndGetMultipleFiles doesn't update selectedExtensionFilter which makes sense, but then it's always null and always imports with no format specified, whether its a singular or multiple files.

I'll update showFileOpenDialogAndGetMultipleFiles and refactor importMultipleFiles to infer format only when there's only one file to be imported and no specific filter

[subhramit]

Please add a comment why the second condition works, as otherwise just by reading code this seems like a logic proxy/hacky way.

[070jhz]

"Import into current library" can still be enabled if there are no libraries open. Is that intentional ? If not, then comment should mention a bug report ?

[trag-bot]

@trag-bot didn't find any issues in the code! ✅✨