Added warnings dialog if 10 or more entries are selected for full tex…

…t download
JabRef · Aug 27, 2016 · 90a3f72 · 90a3f72
1 parent 287b388
commit 90a3f72
Show file tree

Hide file tree

Showing 18 changed files with 100 additions and 37 deletions.
diff --git a/src/main/java/net/sf/jabref/external/FindFullTextAction.java b/src/main/java/net/sf/jabref/external/FindFullTextAction.java
@@ -44,6 +44,20 @@ public void init() throws Throwable {
 
     @Override
     public void run() {
+        if (basePanel.getSelectedEntries().size() >= 10) {
+            String[] options = new String[]{Localization.lang("Look up full text documents"), Localization.lang("Cancel")};
+            int answer = JOptionPane.showOptionDialog(basePanel.frame(),
+                    Localization.lang(
+                            "You are about to look up full text documents for %0 entries.",
+                            String.valueOf(basePanel.getSelectedEntries().size())) + "\n"
+                            + Localization.lang("Do you still want to continue?"),
+                    Localization.lang("Look up full text documents"), JOptionPane.OK_CANCEL_OPTION,
+                    JOptionPane.QUESTION_MESSAGE, null, options, options[0]);
+            if (answer != JOptionPane.OK_OPTION) {
+                basePanel.output(Localization.lang("Operation canceled."));
+                return;
+            }
+        }
         for (BibEntry entry : basePanel.getSelectedEntries()) {
             FulltextFetchers fft = new FulltextFetchers();
             downloads.put(fft.findFullTextPDF(entry), entry);
@@ -66,7 +80,6 @@ public void update() {
                             Localization.lang("Directory not found"), JOptionPane.ERROR_MESSAGE);
                     return;
                 }
-                // TODO: this needs its own thread as it blocks the UI!
                 DownloadExternalFile def = new DownloadExternalFile(basePanel.frame(),
                         basePanel.getBibDatabaseContext(), entry);
                 try {
@@ -84,11 +97,11 @@ public void update() {
                 } catch (IOException e) {
                     LOGGER.warn("Problem downloading file", e);
                 }
-                basePanel.output(Localization.lang("Finished downloading full text document for entry %0",
+                basePanel.output(Localization.lang("Finished downloading full text document for entry %0.",
                         entry.getCiteKeyOptional().orElse(Localization.lang("undefined"))));
             } else {
                 String title = Localization.lang("Full text document download failed");
-                String message = Localization.lang("Full text document download failed for entry %0",
+                String message = Localization.lang("Full text document download failed for entry %0.",
                         entry.getCiteKeyOptional().orElse(Localization.lang("undefined")));
 
                 basePanel.output(message);

diff --git a/src/main/resources/l10n/JabRef_da.properties b/src/main/resources/l10n/JabRef_da.properties
@@ -1772,5 +1772,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_de.properties b/src/main/resources/l10n/JabRef_de.properties
@@ -2484,5 +2484,8 @@ See_what_has_been_changed_in_the_JabRef_versions=Beschreibt_was_in_den_verschied
 
 Referenced_BibTeX_key_does_not_exist=Referenzierter_BibTeX-Key_existiert_nicht
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_en.properties b/src/main/resources/l10n/JabRef_en.properties
@@ -2303,5 +2303,7 @@ Opens_JabRef's_website=Opens_JabRef's_website
 Opens_a_link_where_the_current_development_version_can_be_downloaded=Opens_a_link_where_the_current_development_version_can_be_downloaded
 See_what_has_been_changed_in_the_JabRef_versions=See_what_has_been_changed_in_the_JabRef_versions
 Referenced_BibTeX_key_does_not_exist=Referenced_BibTeX_key_does_not_exist
-Finished_downloading_full_text_document_for_entry_%0=Finished_downloading_full_text_document_for_entry_%0
-Full_text_document_download_failed_for_entry_%0=Full_text_document_download_failed_for_entry_%0
+Finished_downloading_full_text_document_for_entry_%0.=Finished_downloading_full_text_document_for_entry_%0.
+Full_text_document_download_failed_for_entry_%0.=Full_text_document_download_failed_for_entry_%0.
+Look_up_full_text_documents=Look_up_full_text_documents
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=You_are_about_to_look_up_full_text_documents_for_%0_entries.
diff --git a/src/main/resources/l10n/JabRef_es.properties b/src/main/resources/l10n/JabRef_es.properties
@@ -1673,5 +1673,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_fa.properties b/src/main/resources/l10n/JabRef_fa.properties
@@ -2453,5 +2453,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_fr.properties b/src/main/resources/l10n/JabRef_fr.properties
@@ -1722,5 +1722,8 @@ See_what_has_been_changed_in_the_JabRef_versions=Afficher_les_changements_dans_l
 
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_in.properties b/src/main/resources/l10n/JabRef_in.properties
@@ -1689,5 +1689,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_it.properties b/src/main/resources/l10n/JabRef_it.properties
@@ -1790,5 +1790,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_ja.properties b/src/main/resources/l10n/JabRef_ja.properties
@@ -2430,5 +2430,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_nl.properties b/src/main/resources/l10n/JabRef_nl.properties
@@ -2462,5 +2462,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_no.properties b/src/main/resources/l10n/JabRef_no.properties
@@ -2854,5 +2854,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_pt_BR.properties b/src/main/resources/l10n/JabRef_pt_BR.properties
@@ -1686,5 +1686,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_ru.properties b/src/main/resources/l10n/JabRef_ru.properties
@@ -2431,5 +2431,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_sv.properties b/src/main/resources/l10n/JabRef_sv.properties
@@ -1634,5 +1634,8 @@ See_what_has_been_changed_in_the_JabRef_versions=Se_vad_som_har_ändrats_i_de_ol
 
 Referenced_BibTeX_key_does_not_exist=Refererad_BibTeX-nyckel_finns_ej
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_tr.properties b/src/main/resources/l10n/JabRef_tr.properties
@@ -1707,5 +1707,8 @@ See_what_has_been_changed_in_the_JabRef_versions=
 
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_vi.properties b/src/main/resources/l10n/JabRef_vi.properties
@@ -2457,5 +2457,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=
diff --git a/src/main/resources/l10n/JabRef_zh.properties b/src/main/resources/l10n/JabRef_zh.properties
@@ -1698,5 +1698,8 @@ Opens_a_link_where_the_current_development_version_can_be_downloaded=
 See_what_has_been_changed_in_the_JabRef_versions=
 Referenced_BibTeX_key_does_not_exist=
 
-Finished_downloading_full_text_document_for_entry_%0=
-Full_text_document_download_failed_for_entry_%0=
+
+Finished_downloading_full_text_document_for_entry_%0.=
+Full_text_document_download_failed_for_entry_%0.=
+Look_up_full_text_documents=
+You_are_about_to_look_up_full_text_documents_for_%0_entries.=