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,7 +84,9 @@ 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)) {
+        } else if (fieldProperties.contains(FieldProperty.SINGLE_ENTRY_LINK)) {
+            return new LinkedEntriesEditor(field, databaseContext, suggestionProvider, fieldCheckers);
+        } else if (fieldProperties.contains(FieldProperty.MULTIPLE_ENTRY_LINK)) {
             return new LinkedEntriesEditor(field, databaseContext, suggestionProvider, fieldCheckers);
         } else if (fieldProperties.contains(FieldProperty.PERSON_NAMES)) {
             return new PersonsEditor(field, suggestionProvider, preferences, fieldCheckers, isSingleLine);

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

@@ -15,19 +15,23 @@
 import org.jabref.model.entry.field.Field;
 
 import com.airhacks.afterburner.views.ViewLoader;
+import org.jabref.model.entry.field.FieldProperty;
 
 public class LinkedEntriesEditor extends HBox implements FieldEditorFX {
 
-    @FXML private final LinkedEntriesEditorViewModel viewModel;
-    @FXML private TagBar<ParsedEntryLink> linkedEntriesBar;
+    @FXML
+    private final LinkedEntriesEditorViewModel viewModel;
+    @FXML
+    private TagBar<ParsedEntryLink> linkedEntriesBar;
 
     public LinkedEntriesEditor(Field field, BibDatabaseContext databaseContext, AutoCompleteSuggestionProvider<?> suggestionProvider, FieldCheckers fieldCheckers) {
         this.viewModel = new LinkedEntriesEditorViewModel(field, suggestionProvider, databaseContext, fieldCheckers);
 
         ViewLoader.view(this)
-                  .root(this)
-                  .load();
+                .root(this)
+                .load();
 
+        linkedEntriesBar.allowsMultipleEntries(field.getProperties().contains(FieldProperty.MULTIPLE_ENTRY_LINK));
         linkedEntriesBar.setStringConverter(viewModel.getStringConverter());
         linkedEntriesBar.setOnTagClicked((parsedEntryLink, mouseEvent) -> viewModel.jumpToEntry(parsedEntryLink));
 

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

@@ -27,18 +27,21 @@ public class TagBar<T> extends HBox {
 
     private final ListProperty<T> tags;
     private StringConverter<T> stringConverter;
-    @FXML private TextField inputTextField;
-    @FXML private HBox tagList;
+    @FXML
+    private TextField inputTextField;
+    @FXML
+    private HBox tagList;
     private BiConsumer<T, MouseEvent> onTagClicked;
+    private boolean allowsMultiple = true;
 
     public TagBar() {
         tags = new SimpleListProperty<>(FXCollections.observableArrayList());
         tags.addListener(this::onTagsChanged);
 
         // Load FXML
         ViewLoader.view(this)
-                  .root(this)
-                  .load();
+                .root(this)
+                .load();
         getStylesheets().add(0, TagBar.class.getResource("TagBar.css").toExternalForm());
     }
 
@@ -66,6 +69,9 @@ private void onTagsChanged(ListChangeListener.Change<? extends T> change) {
                 tagList.getChildren().addAll(change.getFrom(), change.getAddedSubList().stream().map(this::createTag).collect(Collectors.toList()));
             }
         }
+        if (!allowsMultiple) {
+            inputTextField.setDisable(!tags.isEmpty());
+        }
     }
 
     private Tag<T> createTag(T item) {
@@ -83,7 +89,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 +103,8 @@ public void setStringConverter(StringConverter<T> stringConverter) {
     public void setOnTagClicked(BiConsumer<T, MouseEvent> onTagClicked) {
         this.onTagClicked = onTagClicked;
     }
+
+    public void allowsMultipleEntries(boolean isMultiple) {
+        this.allowsMultiple = isMultiple;
+    }
 }