Export pdf/linked files

CHANGELOG.md

@@ -23,6 +23,7 @@ We refer to [GitHub issues](https://github.com/JabRef/jabref/issues) by using `#
 - We now set the WM_CLASS of the UI to org-jabref-JabRefMain to allow certain Un*x window managers to properly identify its windows
 - We changed the default paths for the OpenOffice/LibreOffice binaries to the default path for LibreOffice
 - We no longer create a new entry editor when selecting a new entry to increase performance. [#3187](https://github.com/JabRef/jabref/pull/3187)
+- We added the possibility to copy linked files from entries to a single output folder [#2539](https://github.com/JabRef/jabref/pull/2593)
 - We increased performance and decreased the memory footprint of the entry editor drastically. [#3331](https://github.com/JabRef/jabref/pull/3331)
 
 

src/main/java/org/jabref/gui/AbstractDialogView.java

@@ -1,10 +1,15 @@
 package org.jabref.gui;
 
+import java.util.function.Function;
+
 public abstract class AbstractDialogView extends AbstractView {
 
     public AbstractDialogView() {
         super();
+    }
 
+    public AbstractDialogView(Function<String, Object> injectionContext) {
+        super(injectionContext);
     }
 
     public abstract void show();

src/main/java/org/jabref/gui/DialogService.java

@@ -4,6 +4,7 @@
 import java.util.List;
 import java.util.Optional;
 
+import javafx.concurrent.Task;
 import javafx.scene.control.Alert;
 import javafx.scene.control.ButtonType;
 import javafx.scene.control.DialogPane;
@@ -13,6 +14,8 @@
 import org.jabref.gui.util.FileDialogConfiguration;
 import org.jabref.logic.l10n.Localization;
 
+import org.controlsfx.dialog.ProgressDialog;
+
 /**
  * This interface provides methods to create dialogs and show them to the user.
  */
@@ -121,6 +124,12 @@ Optional<ButtonType> showCustomButtonDialogAndWait(Alert.AlertType type, String
      */
     <R> Optional<R> showCustomDialogAndWait(Dialog<R> dialog);
 
+    /**
+     * Constructs and shows a canceable {@link ProgressDialog}. Clicking cancel will cancel the underlying service and close the dialog
+     * @param task The {@link Task} which executes the work and for which to show the dialog
+     */
+    <V> void showCanceableProgressDialogAndWait(Task<V> task);
+
     /**
      * Notify the user in an non-blocking way (i.e., update status message instead of showing a dialog).
      * @param message the message to show.
@@ -173,4 +182,5 @@ Optional<ButtonType> showCustomButtonDialogAndWait(Alert.AlertType type, String
      * @return A configured instance of the {@link FileChooser}
      */
     FileChooser getConfiguredFileChooser(FileDialogConfiguration fileDialogConfiguration);
+
 }

src/main/java/org/jabref/gui/FXDialogService.java

@@ -7,7 +7,9 @@
 import java.util.Optional;
 import java.util.stream.Collectors;
 
+import javafx.concurrent.Task;
 import javafx.scene.control.Alert.AlertType;
+import javafx.scene.control.Button;
 import javafx.scene.control.ButtonBar;
 import javafx.scene.control.ButtonType;
 import javafx.scene.control.DialogPane;
@@ -22,6 +24,7 @@
 import org.jabref.logic.l10n.Localization;
 
 import org.controlsfx.dialog.ExceptionDialog;
+import org.controlsfx.dialog.ProgressDialog;
 
 /**
  * This class provides methods to create default
@@ -127,6 +130,21 @@ public <R> Optional<R> showCustomDialogAndWait(Dialog<R> dialog) {
         return dialog.showAndWait();
     }
 
+    @Override
+    public <V> void showCanceableProgressDialogAndWait(Task<V> task) {
+        ProgressDialog progressDialog = new ProgressDialog(task);
+        progressDialog.setOnCloseRequest(evt -> task.cancel());
+        DialogPane dialogPane = progressDialog.getDialogPane();
+        dialogPane.getButtonTypes().add(ButtonType.CANCEL);
+        Button cancelButton = (Button) dialogPane.lookupButton(ButtonType.CANCEL);
+        cancelButton.setOnAction(evt -> {
+            task.cancel();
+            progressDialog.close();
+        });
+        progressDialog.showAndWait();
+
+    }
+
     @Override
     public void notify(String message) {
         JabRefGUI.getMainFrame().output(message);

src/main/java/org/jabref/gui/JabRefFrame.java

@@ -80,6 +80,7 @@
 import org.jabref.gui.actions.SortTabsAction;
 import org.jabref.gui.autosaveandbackup.AutosaveUIManager;
 import org.jabref.gui.bibtexkeypattern.BibtexKeyPatternDialog;
+import org.jabref.gui.copyfiles.ExportLinkedFilesAction;
 import org.jabref.gui.customentrytypes.EntryCustomizationDialog;
 import org.jabref.gui.dbproperties.DatabasePropertiesDialog;
 import org.jabref.gui.documentviewer.ShowDocumentViewerAction;
@@ -389,6 +390,7 @@ public void actionPerformed(ActionEvent e) {
             Localization.menuTitle("Unabbreviate journal names"),
             Localization.lang("Unabbreviate journal names of the selected entries"),
             Globals.getKeyPrefs().getKey(KeyBinding.UNABBREVIATE));
+    private final AbstractAction exportLinkedFiles = new ExportLinkedFilesAction();
     private final AbstractAction manageJournals = new ManageJournalsAction();
     private final AbstractAction databaseProperties = new DatabasePropertiesAction();
     private final AbstractAction bibtexKeyPattern = new BibtexKeyPatternAction();
@@ -1059,6 +1061,7 @@ private void fillMenu() {
         file.add(importCurrent);
         file.add(exportAll);
         file.add(exportSelected);
+        file.add(exportLinkedFiles);
         file.addSeparator();
         file.add(connectToSharedDatabaseAction);
         file.add(pullChangesFromSharedDatabase);
@@ -1440,7 +1443,7 @@ dupliCheck, autoSetFile, newEntryAction, newSpec, customizeAction, plainTextImpo
         twoEntriesOnlyActions.addAll(Arrays.asList(mergeEntries));
 
         atLeastOneEntryActions.clear();
-        atLeastOneEntryActions.addAll(Arrays.asList(downloadFullText, lookupIdentifiers));
+        atLeastOneEntryActions.addAll(Arrays.asList(downloadFullText, lookupIdentifiers, exportLinkedFiles));
 
         tabbedPane.addChangeListener(event -> updateEnabledState());
 

src/main/java/org/jabref/gui/copyfiles/CopyFilesDialogController.java

@@ -0,0 +1,58 @@
+package org.jabref.gui.copyfiles;
+
+import javax.inject.Inject;
+
+import javafx.event.ActionEvent;
+import javafx.fxml.FXML;
+import javafx.scene.control.TableColumn;
+import javafx.scene.control.TableView;
+import javafx.scene.paint.Color;
+import javafx.scene.text.Text;
+
+import org.jabref.gui.AbstractController;
+import org.jabref.gui.util.ValueTableCellFactory;
+
+import de.jensd.fx.glyphs.materialdesignicons.MaterialDesignIcon;
+import de.jensd.fx.glyphs.materialdesignicons.utils.MaterialDesignIconFactory;
+
+public class CopyFilesDialogController extends AbstractController<CopyFilesDialogViewModel> {
+
+    @FXML private TableView<CopyFilesResultItemViewModel> tvResult;
+    @FXML private TableColumn<CopyFilesResultItemViewModel, MaterialDesignIcon> colStatus;
+    @FXML private TableColumn<CopyFilesResultItemViewModel, String> colMessage;
+    @FXML private TableColumn<CopyFilesResultItemViewModel, String> colFile;
+
+    @Inject private CopyFilesResultListDependency copyfilesresultlistDependency; //This var must have the same name as the key in the View
+
+    @FXML
+    private void close(@SuppressWarnings("unused") ActionEvent event) {
+        getStage().close();
+    }
+
+    @FXML
+    private void initialize() {
+        viewModel = new CopyFilesDialogViewModel(copyfilesresultlistDependency);
+        setupTable();
+    }
+
+    private void setupTable() {
+        colFile.setCellValueFactory(cellData -> cellData.getValue().getFile());
+        colMessage.setCellValueFactory(cellData -> cellData.getValue().getMessage());
+        colStatus.setCellValueFactory(cellData -> cellData.getValue().getIcon());
+
+        colFile.setCellFactory(new ValueTableCellFactory<CopyFilesResultItemViewModel, String>().withText(item -> item).withTooltip(item -> item));
+        colStatus.setCellFactory(new ValueTableCellFactory<CopyFilesResultItemViewModel, MaterialDesignIcon>().withGraphic(item -> {
+
+            Text icon = MaterialDesignIconFactory.get().createIcon(item);
+            if (item == MaterialDesignIcon.CHECK) {
+                icon.setFill(Color.GREEN);
+            }
+            if (item == MaterialDesignIcon.ALERT) {
+                icon.setFill(Color.RED);
+            }
+            return icon;
+        }));
+
+        tvResult.setItems(viewModel.copyFilesResultListProperty());
+    }
+}

src/main/java/org/jabref/gui/copyfiles/CopyFilesDialogView.java

@@ -0,0 +1,36 @@
+package org.jabref.gui.copyfiles;
+
+import java.util.HashMap;
+import java.util.Map;
+import java.util.function.Function;
+
+import javafx.scene.control.Alert.AlertType;
+import javafx.scene.control.DialogPane;
+
+import org.jabref.gui.AbstractDialogView;
+import org.jabref.gui.FXDialog;
+import org.jabref.logic.l10n.Localization;
+import org.jabref.model.database.BibDatabaseContext;
+
+public class CopyFilesDialogView extends AbstractDialogView {
+
+    public CopyFilesDialogView(BibDatabaseContext bibDatabaseContext, CopyFilesResultListDependency results) {
+        super(createContext(bibDatabaseContext, results));
+    }
+
+    @Override
+    public void show() {
+        FXDialog copyFilesResultDlg = new FXDialog(AlertType.INFORMATION, Localization.lang("Result"));
+        copyFilesResultDlg.setResizable(true);
+        copyFilesResultDlg.setDialogPane((DialogPane) this.getView());
+        copyFilesResultDlg.show();
+    }
+
+    private static Function<String, Object> createContext(BibDatabaseContext bibDatabaseContext, CopyFilesResultListDependency copyfilesresultlistDependency) {
+        Map<String, Object> context = new HashMap<>();
+        //The "keys" of the HashMap must have the same name as the with @inject annotated field in the controller
+        context.put("bibdatabasecontext", bibDatabaseContext);
+        context.put("copyfilesresultlistDependency", copyfilesresultlistDependency);
+        return context::get;
+    }
+}

src/main/java/org/jabref/gui/copyfiles/CopyFilesDialogViewModel.java

@@ -0,0 +1,21 @@
+package org.jabref.gui.copyfiles;
+
+import javafx.beans.property.SimpleListProperty;
+import javafx.collections.FXCollections;
+
+import org.jabref.gui.AbstractViewModel;
+
+public class CopyFilesDialogViewModel extends AbstractViewModel {
+
+    private final SimpleListProperty<CopyFilesResultItemViewModel> copyFilesResultItems = new SimpleListProperty<>(
+            FXCollections.observableArrayList());
+
+    public CopyFilesDialogViewModel(CopyFilesResultListDependency results) {
+        copyFilesResultItems.addAll(results.getResults());
+    }
+
+    public SimpleListProperty<CopyFilesResultItemViewModel> copyFilesResultListProperty() {
+        return this.copyFilesResultItems;
+    }
+
+}

src/main/java/org/jabref/gui/copyfiles/CopyFilesResultItemViewModel.java

@@ -0,0 +1,42 @@
+package org.jabref.gui.copyfiles;
+
+import java.nio.file.Path;
+
+import javafx.beans.property.ObjectProperty;
+import javafx.beans.property.SimpleObjectProperty;
+import javafx.beans.property.SimpleStringProperty;
+import javafx.beans.property.StringProperty;
+
+import de.jensd.fx.glyphs.materialdesignicons.MaterialDesignIcon;
+
+public class CopyFilesResultItemViewModel {
+
+    private final StringProperty file = new SimpleStringProperty("");
+    private final ObjectProperty<MaterialDesignIcon> icon = new SimpleObjectProperty<>(MaterialDesignIcon.ALERT);
+    private final StringProperty message = new SimpleStringProperty("");
+
+    public CopyFilesResultItemViewModel(Path file, boolean success, String message) {
+        this.file.setValue(file.toString());
+        this.message.setValue(message);
+        if (success) {
+            this.icon.setValue(MaterialDesignIcon.CHECK);
+        }
+    }
+
+    public StringProperty getFile() {
+        return file;
+    }
+
+    public StringProperty getMessage() {
+        return message;
+    }
+
+    public ObjectProperty<MaterialDesignIcon> getIcon() {
+        return icon;
+    }
+
+    @Override
+    public String toString() {
+        return "CopyFilesResultItemViewModel [file=" + file.get() + ", message=" + message.get() + "]";
+    }
+}

src/main/java/org/jabref/gui/copyfiles/CopyFilesResultListDependency.java

@@ -0,0 +1,31 @@
+package org.jabref.gui.copyfiles;
+
+import java.util.ArrayList;
+import java.util.List;
+
+/**
+ * This class is a wrapper class for the containing list as it is currently not possible to inject complex object types into FXML controller
+ *
+ */
+public class CopyFilesResultListDependency {
+
+    private List<CopyFilesResultItemViewModel> results = new ArrayList<>();
+
+    public CopyFilesResultListDependency() {
+        //empty, workaround for injection into FXML controller
+    }
+
+    public CopyFilesResultListDependency(List<CopyFilesResultItemViewModel> results) {
+        this.results = results;
+    }
+
+    public List<CopyFilesResultItemViewModel> getResults() {
+        return results;
+    }
+
+    @Override
+    public String toString() {
+        return "CopyFilesResultListDependency [results=" + results + "]";
+    }
+
+}