Fix multiple entries allowed in crossref (issue #5284)

AUTHORS

@@ -158,6 +158,7 @@ Jörg Lenhard
 Jörg Wegner
 Jörg Zieren
 Jørgen Kvalsvik
+Julien Benard
 Jürgen Lange
 Kai Mindermann
 Kai Takac
@@ -181,6 +182,7 @@ Li Zhilin
 Ling Wang
 Linus Dietz
 Lorenzo Genta
+Lucas Beretti
 Luciana de Melo e Abud
 Mairieli Wessel
 Malik Atalla
@@ -317,6 +319,7 @@ Ulrik Stervbo
 UltimaBGD
 Uwe Kuehn
 Valentin Pons
+Venceslas Roullier
 Victor Figueira
 Victor Michelan
 Vincent W. Yang

CHANGELOG.md

@@ -64,6 +64,7 @@ We refer to [GitHub issues](https://github.com/JabRef/jabref/issues) by using `#
 - We fixed an exception which occurred when an invalid jstyle was loaded. [#5452](https://github.com/JabRef/jabref/issues/5452)
 - We fixed an issue where the command line arguments `importBibtex` and `importToOpen` did not import into the currently open library, but opened a new one. [#5537](https://github.com/JabRef/jabref/issues/5537)
 - We fixed an error where the preview theme did not adapt to the "Dark" mode [#5463](https://github.com/JabRef/jabref/issues/5463)
+- We fixed an issue where multiple entries were allowed in the "crossref" field [#5284](https://github.com/JabRef/jabref/issues/5284)
 - We fixed an issue where the merge dialog showed the wrong text colour in "Dark" mode [#5516](https://github.com/JabRef/jabref/issues/5516)
 - We fixed visibility issues with the scrollbar and group selection highlight in "Dark" mode, and enabled "Dark" mode for the OpenOffice preview in the style selection window. [#5522](https://github.com/JabRef/jabref/issues/5522)
 - We fixed an issue where the author field was not correctly parsed during bibtex key-generation. [#5551](https://github.com/JabRef/jabref/issues/5551)

src/main/java/org/jabref/gui/fieldeditors/FieldEditors.java

@@ -84,8 +84,10 @@ public static FieldEditorFX getForField(final Field field,
             } else {
                 return new OptionEditor<>(new TypeEditorViewModel(field, suggestionProvider, fieldCheckers));
             }
-        } else if (fieldProperties.contains(FieldProperty.SINGLE_ENTRY_LINK) || fieldProperties.contains(FieldProperty.MULTIPLE_ENTRY_LINK)) {
-            return new LinkedEntriesEditor(field, databaseContext, suggestionProvider, fieldCheckers);
+        } else if (fieldProperties.contains(FieldProperty.SINGLE_ENTRY_LINK)) {
+            return new LinkedEntriesEditor(field, databaseContext, suggestionProvider, fieldCheckers,false);
+        } else if (fieldProperties.contains(FieldProperty.MULTIPLE_ENTRY_LINK)) {
+            return new LinkedEntriesEditor(field, databaseContext, suggestionProvider, fieldCheckers,true);
         } else if (fieldProperties.contains(FieldProperty.PERSON_NAMES)) {
             return new PersonsEditor(field, suggestionProvider, preferences, fieldCheckers, isSingleLine);
         } else if (StandardField.KEYWORDS.equals(field)) {

src/main/java/org/jabref/gui/fieldeditors/LinkedEntriesEditor.java

@@ -21,13 +21,14 @@ public class LinkedEntriesEditor extends HBox implements FieldEditorFX {
     @FXML private final LinkedEntriesEditorViewModel viewModel;
     @FXML private TagBar<ParsedEntryLink> linkedEntriesBar;
 
-    public LinkedEntriesEditor(Field field, BibDatabaseContext databaseContext, AutoCompleteSuggestionProvider<?> suggestionProvider, FieldCheckers fieldCheckers) {
+    public LinkedEntriesEditor(Field field, BibDatabaseContext databaseContext, AutoCompleteSuggestionProvider<?> suggestionProvider, FieldCheckers fieldCheckers, Boolean isMultiple) {
         this.viewModel = new LinkedEntriesEditorViewModel(field, suggestionProvider, databaseContext, fieldCheckers);
 
         ViewLoader.view(this)
                   .root(this)
                   .load();
 
+        linkedEntriesBar.allowsMultipleEntries(isMultiple);
         linkedEntriesBar.setStringConverter(viewModel.getStringConverter());
         linkedEntriesBar.setOnTagClicked((parsedEntryLink, mouseEvent) -> viewModel.jumpToEntry(parsedEntryLink));
 

src/main/java/org/jabref/gui/util/component/TagBar.java

@@ -30,6 +30,7 @@ public class TagBar<T> extends HBox {
     @FXML private TextField inputTextField;
     @FXML private HBox tagList;
     private BiConsumer<T, MouseEvent> onTagClicked;
+    private Boolean allowsMultiple=true;
 
     public TagBar() {
         tags = new SimpleListProperty<>(FXCollections.observableArrayList());
@@ -62,8 +63,10 @@ private void onTagsChanged(ListChangeListener.Change<? extends T> change) {
         while (change.next()) {
             if (change.wasRemoved()) {
                 tagList.getChildren().subList(change.getFrom(), change.getFrom() + change.getRemovedSize()).clear();
+                if(!allowsMultiple) { inputTextField.setDisable(false);}
             } else if (change.wasAdded()) {
                 tagList.getChildren().addAll(change.getFrom(), change.getAddedSubList().stream().map(this::createTag).collect(Collectors.toList()));
+                if(!allowsMultiple) { inputTextField.setDisable(true);}
             }
         }
     }
@@ -83,7 +86,7 @@ private void addTextAsNewTag(ActionEvent event) {
         String inputText = inputTextField.getText();
         if (StringUtil.isNotBlank(inputText)) {
             T newTag = stringConverter.fromString(inputText);
-            if ((newTag != null) && !tags.contains(newTag)) {
+            if ((newTag != null) && !tags.contains(newTag)&&(tags.isEmpty()||this.allowsMultiple)) {
                 tags.add(newTag);
                 inputTextField.clear();
             }
@@ -97,4 +100,6 @@ public void setStringConverter(StringConverter<T> stringConverter) {
     public void setOnTagClicked(BiConsumer<T, MouseEvent> onTagClicked) {
         this.onTagClicked = onTagClicked;
     }
+
+    public void allowsMultipleEntries(Boolean isMultiple) { this.allowsMultiple=isMultiple; }
 }