From 65245999ba988d81e1c1e39e738445f2dbcf4eaf Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" Date: Mon, 18 Mar 2019 08:32:21 +0100 Subject: [PATCH 01/61] Bump mockito-core from 2.25.0 to 2.25.1 (#4771) Bumps [mockito-core](https://github.com/mockito/mockito) from 2.25.0 to 2.25.1. - [Release notes](https://github.com/mockito/mockito/releases) - [Commits](https://github.com/mockito/mockito/compare/v2.25.0...v2.25.1) Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 0a45aab3885..599dd191bfe 100644 --- a/build.gradle +++ b/build.gradle @@ -167,7 +167,7 @@ dependencies { testRuntime 'org.apache.logging.log4j:log4j-core:2.11.1' testRuntime 'org.apache.logging.log4j:log4j-jul:2.11.2' - testCompile 'org.mockito:mockito-core:2.25.0' + testCompile 'org.mockito:mockito-core:2.25.1' testCompile 'com.github.tomakehurst:wiremock:2.21.0' testCompile 'org.reflections:reflections:0.9.11' testCompile 'org.xmlunit:xmlunit-core:2.6.2' From d64494e517893e71e2f1cdafa9ed826c0c1935b9 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" Date: Mon, 18 Mar 2019 08:32:44 +0100 Subject: [PATCH 02/61] Bump junit-vintage-engine from 5.4.0 to 5.4.1 (#4772) Bumps [junit-vintage-engine](https://github.com/junit-team/junit5) from 5.4.0 to 5.4.1. - [Release notes](https://github.com/junit-team/junit5/releases) - [Commits](https://github.com/junit-team/junit5/compare/r5.4.0...r5.4.1) Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 599dd191bfe..c329ea2d3c5 100644 --- a/build.gradle +++ b/build.gradle @@ -162,7 +162,7 @@ dependencies { compile group: 'com.microsoft.azure', name: 'applicationinsights-logging-log4j2', version: '2.3.1' testImplementation 'org.junit.jupiter:junit-jupiter:5.4.0' - testRuntimeOnly 'org.junit.vintage:junit-vintage-engine:5.4.0' + testRuntimeOnly 'org.junit.vintage:junit-vintage-engine:5.4.1' testCompile 'org.junit.platform:junit-platform-launcher:1.4.0' testRuntime 'org.apache.logging.log4j:log4j-core:2.11.1' From 8a1fbe1ee6f1ed6aa47ebe989081fd7e3825ecb8 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" Date: Mon, 18 Mar 2019 08:34:05 +0100 Subject: [PATCH 03/61] Bump archunit-junit5-api from 0.9.3 to 0.10.1 (#4773) Bumps [archunit-junit5-api](https://github.com/TNG/ArchUnit) from 0.9.3 to 0.10.1. - [Release notes](https://github.com/TNG/ArchUnit/releases) - [Commits](https://github.com/TNG/ArchUnit/compare/v0.9.3...v0.10.1) Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index c329ea2d3c5..9b75a95267f 100644 --- a/build.gradle +++ b/build.gradle @@ -173,7 +173,7 @@ dependencies { testCompile 'org.xmlunit:xmlunit-core:2.6.2' testCompile 'org.xmlunit:xmlunit-matchers:2.6.2' testRuntime 'com.tngtech.archunit:archunit-junit5-engine:0.9.3' - testCompile 'com.tngtech.archunit:archunit-junit5-api:0.9.3' + testCompile 'com.tngtech.archunit:archunit-junit5-api:0.10.1' testCompile "org.testfx:testfx-core:4.0.+" testCompile "org.testfx:testfx-junit5:4.0.+" From 52696f04b518f4093fea32aff0c2f80b540dab67 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" Date: Mon, 18 Mar 2019 10:50:00 +0100 Subject: [PATCH 04/61] Bump wiremock from 2.21.0 to 2.22.0 (#4776) Bumps [wiremock](https://github.com/tomakehurst/wiremock) from 2.21.0 to 2.22.0. - [Release notes](https://github.com/tomakehurst/wiremock/releases) - [Changelog](https://github.com/tomakehurst/wiremock/blob/master/release-settings.gradle) - [Commits](https://github.com/tomakehurst/wiremock/compare/2.21.0...2.22.0) Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 9b75a95267f..0b8f14c6198 100644 --- a/build.gradle +++ b/build.gradle @@ -168,7 +168,7 @@ dependencies { testRuntime 'org.apache.logging.log4j:log4j-core:2.11.1' testRuntime 'org.apache.logging.log4j:log4j-jul:2.11.2' testCompile 'org.mockito:mockito-core:2.25.1' - testCompile 'com.github.tomakehurst:wiremock:2.21.0' + testCompile 'com.github.tomakehurst:wiremock:2.22.0' testCompile 'org.reflections:reflections:0.9.11' testCompile 'org.xmlunit:xmlunit-core:2.6.2' testCompile 'org.xmlunit:xmlunit-matchers:2.6.2' From 39f7690c2c3ec76a9747b2e65e77d31d1c741dba Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" Date: Mon, 18 Mar 2019 10:50:18 +0100 Subject: [PATCH 05/61] Bump junit-jupiter from 5.4.0 to 5.4.1 (#4775) Bumps [junit-jupiter](https://github.com/junit-team/junit5) from 5.4.0 to 5.4.1. - [Release notes](https://github.com/junit-team/junit5/releases) - [Commits](https://github.com/junit-team/junit5/compare/r5.4.0...r5.4.1) Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 0b8f14c6198..1bee7bc1732 100644 --- a/build.gradle +++ b/build.gradle @@ -161,7 +161,7 @@ dependencies { compile group: 'com.microsoft.azure', name: 'applicationinsights-core', version: '2.3.1' compile group: 'com.microsoft.azure', name: 'applicationinsights-logging-log4j2', version: '2.3.1' - testImplementation 'org.junit.jupiter:junit-jupiter:5.4.0' + testImplementation 'org.junit.jupiter:junit-jupiter:5.4.1' testRuntimeOnly 'org.junit.vintage:junit-vintage-engine:5.4.1' testCompile 'org.junit.platform:junit-platform-launcher:1.4.0' From 2512657ea02908057787dfb124ed0bcb84df07e9 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" Date: Mon, 18 Mar 2019 10:50:38 +0100 Subject: [PATCH 06/61] Bump junit-platform-launcher from 1.4.0 to 1.4.1 (#4774) Bumps [junit-platform-launcher](https://github.com/junit-team/junit5) from 1.4.0 to 1.4.1. - [Release notes](https://github.com/junit-team/junit5/releases) - [Commits](https://github.com/junit-team/junit5/commits) Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 1bee7bc1732..70d2bc30c2b 100644 --- a/build.gradle +++ b/build.gradle @@ -163,7 +163,7 @@ dependencies { testImplementation 'org.junit.jupiter:junit-jupiter:5.4.1' testRuntimeOnly 'org.junit.vintage:junit-vintage-engine:5.4.1' - testCompile 'org.junit.platform:junit-platform-launcher:1.4.0' + testCompile 'org.junit.platform:junit-platform-launcher:1.4.1' testRuntime 'org.apache.logging.log4j:log4j-core:2.11.1' testRuntime 'org.apache.logging.log4j:log4j-jul:2.11.2' From 2b791c306070f08e31b5218b4de1f818fb25a49e Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" Date: Mon, 18 Mar 2019 10:51:02 +0100 Subject: [PATCH 07/61] Bump archunit-junit5-engine from 0.9.3 to 0.10.1 (#4777) Bumps [archunit-junit5-engine](https://github.com/TNG/ArchUnit) from 0.9.3 to 0.10.1. - [Release notes](https://github.com/TNG/ArchUnit/releases) - [Commits](https://github.com/TNG/ArchUnit/compare/v0.9.3...v0.10.1) Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 70d2bc30c2b..9673626d496 100644 --- a/build.gradle +++ b/build.gradle @@ -172,7 +172,7 @@ dependencies { testCompile 'org.reflections:reflections:0.9.11' testCompile 'org.xmlunit:xmlunit-core:2.6.2' testCompile 'org.xmlunit:xmlunit-matchers:2.6.2' - testRuntime 'com.tngtech.archunit:archunit-junit5-engine:0.9.3' + testRuntime 'com.tngtech.archunit:archunit-junit5-engine:0.10.1' testCompile 'com.tngtech.archunit:archunit-junit5-api:0.10.1' testCompile "org.testfx:testfx-core:4.0.+" testCompile "org.testfx:testfx-junit5:4.0.+" From 90376f40fe0acc9b4b85c5574292e3997d3975f5 Mon Sep 17 00:00:00 2001 From: Tobias Diez Date: Tue, 19 Mar 2019 23:29:48 +0100 Subject: [PATCH 08/61] Fix opening of entry editor (#4784) Fixes #4762. The problem was that the NotificationPane was hiding the entry editor/preview. --- src/main/java/org/jabref/gui/BasePanel.java | 40 ++++++--------------- 1 file changed, 11 insertions(+), 29 deletions(-) diff --git a/src/main/java/org/jabref/gui/BasePanel.java b/src/main/java/org/jabref/gui/BasePanel.java index ea88315f304..96d68e870dd 100644 --- a/src/main/java/org/jabref/gui/BasePanel.java +++ b/src/main/java/org/jabref/gui/BasePanel.java @@ -25,7 +25,6 @@ import javafx.scene.Node; import javafx.scene.control.ScrollPane; import javafx.scene.control.SplitPane; -import javafx.scene.layout.AnchorPane; import javafx.scene.layout.StackPane; import org.jabref.Globals; @@ -191,7 +190,7 @@ public BasePanel(JabRefFrame frame, BasePanelPreferences preferences, BibDatabas if (file.isPresent()) { // Register so we get notifications about outside changes to the file. changeMonitor = Optional.of(new DatabaseChangeMonitor(bibDatabaseContext, Globals.getFileUpdateMonitor(), Globals.TASK_EXECUTOR)); - changePane = new DatabaseChangePane(mainTable.getPane(), bibDatabaseContext, changeMonitor.get()); + changePane = new DatabaseChangePane(splitPane, bibDatabaseContext, changeMonitor.get()); getChildren().add(changePane); } else { if (bibDatabaseContext.getDatabase().hasEntries()) { @@ -394,7 +393,7 @@ private void setupActions() { .withPreviewPanelEnabled(enabled) .build(); Globals.prefs.storePreviewPreferences(newPreviewPreferences); - DefaultTaskExecutor.runInJavaFXThread(() -> setPreviewActiveBasePanels(enabled)); + setPreviewActive(enabled); }); actions.put(Actions.NEXT_PREVIEW_STYLE, this::nextPreviewStyle); @@ -776,13 +775,9 @@ public void setupMainPanel() { createMainTable(); ScrollPane pane = mainTable.getPane(); - AnchorPane anchorPane = new AnchorPane(pane); - AnchorPane.setBottomAnchor(pane, 0.0); - AnchorPane.setTopAnchor(pane, 0.0); - AnchorPane.setLeftAnchor(pane, 0.0); - AnchorPane.setRightAnchor(pane, 0.0); - splitPane.getItems().add(anchorPane); - this.getChildren().setAll(splitPane); + pane.setFitToHeight(true); + pane.setFitToWidth(true); + splitPane.getItems().add(pane); // Set up name autocompleter for search: instantiateSearchAutoCompleter(); @@ -845,17 +840,13 @@ public EntryEditor getEntryEditor() { * @param entry The entry to edit. */ public void showAndEdit(BibEntry entry) { - DefaultTaskExecutor.runInJavaFXThread(() -> { + showBottomPane(BasePanelMode.SHOWING_EDITOR); - showBottomPane(BasePanelMode.SHOWING_EDITOR); - - if (entry != getShowing()) { - entryEditor.setEntry(entry); - showing = entry; - } - entryEditor.requestFocus(); - - }); + if (entry != getShowing()) { + entryEditor.setEntry(entry); + showing = entry; + } + entryEditor.requestFocus(); } private void showBottomPane(BasePanelMode newMode) { @@ -1150,15 +1141,6 @@ public String formatOutputMessage(String start, int count) { return String.format("%s %d %s.", start, count, (count > 1 ? Localization.lang("entries") : Localization.lang("entry"))); } - /** - * Set the preview active state for all BasePanel instances. - */ - private void setPreviewActiveBasePanels(boolean enabled) { - for (int i = 0; i < frame.getTabbedPane().getTabs().size(); i++) { - frame.getBasePanelAt(i).setPreviewActive(enabled); - } - } - private void setPreviewActive(boolean enabled) { if (enabled) { showPreview(); From 6c61664a8f715e4011064ddca5cb0ace0b803131 Mon Sep 17 00:00:00 2001 From: Christoph Date: Wed, 20 Mar 2019 20:48:13 +0100 Subject: [PATCH 09/61] Fix NPE in Duplicate Resolver Dialog (#4791) * Fix NPE in Duplicate Resolver Dialog Fixes #4785 * Fix NPE in review dialog Fixes #4786 --- src/main/java/org/jabref/gui/DuplicateResolverDialog.java | 2 +- src/main/java/org/jabref/gui/PreviewPanel.java | 2 +- .../java/org/jabref/gui/collab/EntryAddChangeViewModel.java | 2 +- src/main/java/org/jabref/gui/search/SearchResultFrame.java | 3 ++- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/jabref/gui/DuplicateResolverDialog.java b/src/main/java/org/jabref/gui/DuplicateResolverDialog.java index 858a7c7e7f2..fa1b2b92b96 100644 --- a/src/main/java/org/jabref/gui/DuplicateResolverDialog.java +++ b/src/main/java/org/jabref/gui/DuplicateResolverDialog.java @@ -39,8 +39,8 @@ public enum DuplicateResolverResult { public DuplicateResolverDialog(BibEntry one, BibEntry two, DuplicateResolverType type, BibDatabaseContext database) { this.setTitle(Localization.lang("Possible duplicate entries")); - init(one, two, type); this.database = database; + init(one, two, type); } private void init(BibEntry one, BibEntry two, DuplicateResolverType type) { diff --git a/src/main/java/org/jabref/gui/PreviewPanel.java b/src/main/java/org/jabref/gui/PreviewPanel.java index 796fafdd016..ff26ee6765b 100644 --- a/src/main/java/org/jabref/gui/PreviewPanel.java +++ b/src/main/java/org/jabref/gui/PreviewPanel.java @@ -84,7 +84,7 @@ public class PreviewPanel extends ScrollPane implements SearchQueryHighlightList /** * @param panel (may be null) Only set this if the preview is associated to the main window. - * @param databaseContext Used for resolving pdf directories for links. Must not be null. + * @param databaseContext Used for resolving pdf directories for links. Must not be null, just pass a new empty BibDatabaseContext() */ public PreviewPanel(BasePanel panel, BibDatabaseContext databaseContext, KeyBindingRepository keyBindingRepository, PreviewPreferences preferences, DialogService dialogService, ExternalFileTypes externalFileTypes) { this.databaseContext = Objects.requireNonNull(databaseContext); diff --git a/src/main/java/org/jabref/gui/collab/EntryAddChangeViewModel.java b/src/main/java/org/jabref/gui/collab/EntryAddChangeViewModel.java index 8284f544cab..c5849decfef 100644 --- a/src/main/java/org/jabref/gui/collab/EntryAddChangeViewModel.java +++ b/src/main/java/org/jabref/gui/collab/EntryAddChangeViewModel.java @@ -29,7 +29,7 @@ public void makeChange(BibDatabaseContext database, NamedCompound undoEdit) { @Override public Node description() { - PreviewPanel previewPanel = new PreviewPanel(null, null, Globals.getKeyPrefs(), Globals.prefs.getPreviewPreferences(), new FXDialogService(), ExternalFileTypes.getInstance()); + PreviewPanel previewPanel = new PreviewPanel(null, new BibDatabaseContext(), Globals.getKeyPrefs(), Globals.prefs.getPreviewPreferences(), new FXDialogService(), ExternalFileTypes.getInstance()); previewPanel.setEntry(diskEntry); return previewPanel; } diff --git a/src/main/java/org/jabref/gui/search/SearchResultFrame.java b/src/main/java/org/jabref/gui/search/SearchResultFrame.java index 9977866efad..376de69855e 100644 --- a/src/main/java/org/jabref/gui/search/SearchResultFrame.java +++ b/src/main/java/org/jabref/gui/search/SearchResultFrame.java @@ -55,6 +55,7 @@ import org.jabref.logic.bibtex.comparator.FieldComparator; import org.jabref.logic.l10n.Localization; import org.jabref.logic.search.SearchQuery; +import org.jabref.model.database.BibDatabaseContext; import org.jabref.model.entry.BibEntry; import org.jabref.model.entry.FieldName; import org.jabref.model.entry.FieldProperty; @@ -122,7 +123,7 @@ private void init(String title) { searchResultFrame.setTitle(title); searchResultFrame.setIconImages(IconTheme.getLogoSet()); - preview = new PreviewPanel(null, null, Globals.getKeyPrefs(), Globals.prefs.getPreviewPreferences(), frame.getDialogService(), ExternalFileTypes.getInstance()); + preview = new PreviewPanel(null, new BibDatabaseContext(), Globals.getKeyPrefs(), Globals.prefs.getPreviewPreferences(), frame.getDialogService(), ExternalFileTypes.getInstance()); sortedEntries = new SortedList<>(entries, new EntryComparator(false, true, FieldName.AUTHOR)); model = (DefaultEventTableModel) GlazedListsSwing.eventTableModelWithThreadProxyList(sortedEntries, From 29e9d4971954a80510134800499fd65563e3b280 Mon Sep 17 00:00:00 2001 From: samiyac Date: Fri, 22 Mar 2019 01:57:11 +0530 Subject: [PATCH 10/61] Shortens text fields in Required Fields Tab --- src/main/java/org/jabref/gui/entryeditor/FieldsEditorTab.java | 2 +- src/main/java/org/jabref/gui/fieldeditors/BibtexKeyEditor.fxml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/jabref/gui/entryeditor/FieldsEditorTab.java b/src/main/java/org/jabref/gui/entryeditor/FieldsEditorTab.java index bbd23b29d97..26f22eb2b6f 100644 --- a/src/main/java/org/jabref/gui/entryeditor/FieldsEditorTab.java +++ b/src/main/java/org/jabref/gui/entryeditor/FieldsEditorTab.java @@ -103,7 +103,7 @@ private void setupPanel(BibEntry entry, boolean compressed, SuggestionProviders } ColumnConstraints columnExpand = new ColumnConstraints(); - columnExpand.setHgrow(Priority.ALWAYS); + columnExpand.setPercentWidth(60); ColumnConstraints columnDoNotContract = new ColumnConstraints(); columnDoNotContract.setMinWidth(Region.USE_PREF_SIZE); diff --git a/src/main/java/org/jabref/gui/fieldeditors/BibtexKeyEditor.fxml b/src/main/java/org/jabref/gui/fieldeditors/BibtexKeyEditor.fxml index 99d6d3f7601..32160f0c544 100644 --- a/src/main/java/org/jabref/gui/fieldeditors/BibtexKeyEditor.fxml +++ b/src/main/java/org/jabref/gui/fieldeditors/BibtexKeyEditor.fxml @@ -6,5 +6,5 @@ -