Add a new field for citation count

CHANGELOG.md

@@ -11,6 +11,7 @@ Note that this project **does not** adhere to [Semantic Versioning](https://semv
 
 ### Added
 
+- We added a new citationCount field on the General tab, in order to track this value for a given paper. [#13477](https://github.com/JabRef/jabref/issues/13477)
 - We introduced a settings parameter to manage citations' relations local storage time-to-live with a default value set to 30 days. [#11189](https://github.com/JabRef/jabref/issues/11189)
 - We distribute arm64 images for Linux. [#10842](https://github.com/JabRef/jabref/issues/10842)
 - When adding an entry to a library, a warning is displayed if said entry already exists in an active library. [#13261](https://github.com/JabRef/jabref/issues/13261)

jabgui/src/main/java/org/jabref/gui/fieldeditors/CitationCountEditor.java

@@ -0,0 +1,88 @@
+package org.jabref.gui.fieldeditors;
+
+import java.util.Optional;
+
+import javax.swing.undo.UndoManager;
+
+import javafx.fxml.FXML;
+import javafx.scene.Parent;
+import javafx.scene.control.Button;
+import javafx.scene.control.Tooltip;
+import javafx.scene.layout.HBox;
+
+import org.jabref.gui.DialogService;
+import org.jabref.gui.StateManager;
+import org.jabref.gui.autocompleter.SuggestionProvider;
+import org.jabref.gui.fieldeditors.contextmenu.DefaultMenu;
+import org.jabref.gui.preferences.GuiPreferences;
+import org.jabref.logic.citation.SearchCitationsRelationsService;
+import org.jabref.logic.integrity.FieldCheckers;
+import org.jabref.logic.l10n.Localization;
+import org.jabref.logic.util.TaskExecutor;
+import org.jabref.model.entry.BibEntry;
+import org.jabref.model.entry.field.Field;
+
+import com.airhacks.afterburner.injection.Injector;
+import com.airhacks.afterburner.views.ViewLoader;
+import jakarta.inject.Inject;
+
+public class CitationCountEditor extends HBox implements FieldEditorFX {
+    @FXML private CitationCountEditorViewModel viewModel;
+    @FXML private EditorTextField textField;
+    @FXML private Button fetchCitationCountButton;
+
+    @Inject private DialogService dialogService;
+    @Inject private GuiPreferences preferences;
+    @Inject private UndoManager undoManager;
+    @Inject private TaskExecutor taskExecutor;
+    @Inject private StateManager stateManager;
+    @Inject private SearchCitationsRelationsService searchCitationsRelationsService;
+
+    private Optional<BibEntry> entry = Optional.empty();
+    public CitationCountEditor(Field field,
+                             SuggestionProvider<?> suggestionProvider,
+                             FieldCheckers fieldCheckers) {
+        Injector.registerExistingAndInject(this);
+        this.viewModel = new CitationCountEditorViewModel(
+                field,
+                suggestionProvider,
+                fieldCheckers,
+                taskExecutor,
+                dialogService,
+                undoManager,
+                stateManager,
+                preferences,
+                searchCitationsRelationsService);
+
+        ViewLoader.view(this)
+                  .root(this)
+                  .load();
+
+        textField.textProperty().bindBidirectional(viewModel.textProperty());
+
+        fetchCitationCountButton.setTooltip(
+                new Tooltip(Localization.lang("Look up %0", field.getDisplayName())));
+        textField.initContextMenu(new DefaultMenu(textField), preferences.getKeyBindingRepository());
+        new EditorValidator(preferences).configureValidation(viewModel.getFieldValidator().getValidationStatus(), textField);
+    }
+
+    @FXML
+    private void fetchCitationCount() {
+        this.entry.ifPresent(viewModel::getCitationCount);
+    }
+
+    public CitationCountEditorViewModel getViewModel() {
+        return viewModel;
+    }
+
+    @Override
+    public void bindToEntry(BibEntry entry) {
+        this.entry = Optional.of(entry);
+        viewModel.bindToEntry(entry);
+    }
+
+    @Override
+    public Parent getNode() {
+        return this;
+    }
+}

jabgui/src/main/java/org/jabref/gui/fieldeditors/CitationCountEditorViewModel.java

@@ -0,0 +1,65 @@
+package org.jabref.gui.fieldeditors;
+
+import java.util.Optional;
+
+import javax.swing.undo.UndoManager;
+
+import javafx.beans.property.BooleanProperty;
+import javafx.beans.property.SimpleBooleanProperty;
+
+import org.jabref.gui.DialogService;
+import org.jabref.gui.StateManager;
+import org.jabref.gui.autocompleter.SuggestionProvider;
+import org.jabref.gui.preferences.GuiPreferences;
+import org.jabref.logic.citation.SearchCitationsRelationsService;
+import org.jabref.logic.integrity.FieldCheckers;
+import org.jabref.logic.util.BackgroundTask;
+import org.jabref.logic.util.TaskExecutor;
+import org.jabref.model.entry.BibEntry;
+import org.jabref.model.entry.field.Field;
+
+public class CitationCountEditorViewModel extends AbstractEditorViewModel {
+    protected final BooleanProperty fetchCitationCountInProgress = new SimpleBooleanProperty(false);
+    private final TaskExecutor taskExecutor;
+    private final DialogService dialogService;
+    private final UndoManager undoManager;
+    private final StateManager stateManager;
+    private final GuiPreferences preferences;
+    private final SearchCitationsRelationsService searchCitationsRelationsService;
+    public CitationCountEditorViewModel(
+            Field field,
+            SuggestionProvider<?> suggestionProvider,
+            FieldCheckers fieldCheckers,
+            TaskExecutor taskExecutor,
+            DialogService dialogService,
+            UndoManager undoManager,
+            StateManager stateManager,
+            GuiPreferences preferences,
+            SearchCitationsRelationsService searchCitationsRelationsService) {
+        super(field, suggestionProvider, fieldCheckers, undoManager);
+        this.taskExecutor = taskExecutor;
+        this.dialogService = dialogService;
+        this.undoManager = undoManager;
+        this.stateManager = stateManager;
+        this.preferences = preferences;
+        this.searchCitationsRelationsService = searchCitationsRelationsService;
+    }
+
+    public BooleanProperty fetchCitationCountInProgressProperty() {
+        return fetchCitationCountInProgress;
+    }
+
+    public boolean getFetchCitationCountInProgress() {
+        return fetchCitationCountInProgress.get();
+    }
+
+    public void getCitationCount(BibEntry bibEntry) {
+        Optional<String> fieldAux = entry.getField(field);
+        BackgroundTask.wrap(() -> searchCitationsRelationsService.getCitationCount(bibEntry, fieldAux))
+                      .onRunning(() -> fetchCitationCountInProgress.setValue(true))
+                      .onFinished(() -> fetchCitationCountInProgress.setValue(false))
+                      .onSuccess(identifier -> {
+                          entry.setField(field, String.valueOf(identifier));
+                      }).executeWith(taskExecutor);
+    }
+}

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

@@ -75,6 +75,8 @@ public static FieldEditorFX getForField(final Field field,
         } else if (fieldProperties.contains(FieldProperty.IDENTIFIER) && field != StandardField.PMID || field == StandardField.ISBN) {
             // Identifier editor does not support PMID, therefore excluded at the condition above
             return new IdentifierEditor(field, suggestionProvider, fieldCheckers);
+        } else if (field == StandardField.CITATIONCOUNT) {
+            return new CitationCountEditor(field, suggestionProvider, fieldCheckers);
         } else if (field == StandardField.ISSN) {
             return new ISSNEditor(field, suggestionProvider, fieldCheckers, undoAction, redoAction);
         } else if (field == StandardField.OWNER) {

jabgui/src/main/java/org/jabref/gui/fieldeditors/identifier/IdentifierEditor.java

@@ -45,7 +45,6 @@ public class IdentifierEditor extends HBox implements FieldEditorFX {
     @Inject private GuiPreferences preferences;
     @Inject private UndoManager undoManager;
     @Inject private StateManager stateManager;
-
     private Optional<BibEntry> entry = Optional.empty();
 
     public IdentifierEditor(Field field,
@@ -63,6 +62,7 @@ public IdentifierEditor(Field field,
                     this.viewModel = new ISBNIdentifierEditorViewModel(suggestionProvider, fieldCheckers, dialogService, taskExecutor, preferences, undoManager, stateManager);
             case EPRINT ->
                     this.viewModel = new EprintIdentifierEditorViewModel(suggestionProvider, fieldCheckers, dialogService, taskExecutor, preferences, undoManager);
+
             // TODO: Add support for PMID
             case null, default -> {
                 assert field != null;

jabgui/src/main/resources/org/jabref/gui/fieldeditors/CitationCountEditor.fxml

@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<?import javafx.scene.control.Button?>
+<?import javafx.scene.control.ProgressIndicator?>
+<?import javafx.scene.control.Tooltip?>
+<?import javafx.scene.layout.HBox?>
+<?import javafx.scene.layout.StackPane?>
+<?import org.jabref.gui.fieldeditors.EditorTextField?>
+<?import org.jabref.gui.icon.JabRefIconView?>
+<fx:root xmlns:fx="http://javafx.com/fxml/1" type="HBox" xmlns="http://javafx.com/javafx/8.0.112"
+         fx:controller="org.jabref.gui.fieldeditors.CitationCountEditor">
+    <EditorTextField fx:id="textField" prefHeight="0.0" HBox.hgrow="ALWAYS"/>
+    <Button fx:id="fetchCitationCountButton" onAction="#fetchCitationCount" styleClass="icon-button">
+        <graphic>
+            <StackPane>
+                <JabRefIconView glyph="LOOKUP_IDENTIFIER"
+                                visible="${controller.viewModel.fetchCitationCountInProgress == false}"/>
+                <ProgressIndicator maxHeight="12.0" maxWidth="12.0"
+                                   visible="${controller.viewModel.fetchCitationCountInProgress}"/>
+            </StackPane>
+        </graphic>
+    </Button>
+</fx:root>

jablib/build.gradle.kts

@@ -421,7 +421,8 @@ tasks.test {
     jvmArgs = listOf(
         "-javaagent:${mockitoAgent.asPath}",
         "--add-opens", "java.base/jdk.internal.ref=org.apache.pdfbox.io",
-        "--add-opens", "java.base/java.nio=org.apache.pdfbox.io"
+        "--add-opens", "java.base/java.nio=org.apache.pdfbox.io",
+        "--add-reads", "org.xmlunit.matchers=java.xml"
     )
 }
 

jablib/src/main/java/org/jabref/logic/citation/SearchCitationsRelationsService.java

@@ -1,6 +1,7 @@
 package org.jabref.logic.citation;
 
 import java.util.List;
+import java.util.Optional;
 
 import org.jabref.logic.bibtex.FieldPreferences;
 import org.jabref.logic.citation.repository.BibEntryCitationsAndReferencesRepository;
@@ -9,6 +10,7 @@
 import org.jabref.logic.importer.ImportFormatPreferences;
 import org.jabref.logic.importer.ImporterPreferences;
 import org.jabref.logic.importer.fetcher.citation.CitationFetcher;
+import org.jabref.logic.importer.fetcher.citation.semanticscholar.PaperDetails;
 import org.jabref.logic.importer.fetcher.citation.semanticscholar.SemanticScholarCitationFetcher;
 import org.jabref.logic.util.Directories;
 import org.jabref.model.entry.BibEntry;
@@ -83,6 +85,21 @@ public List<BibEntry> searchCitations(BibEntry cited) {
         return relationsRepository.readCitations(cited);
     }
 
+    public int getCitationCount(BibEntry citationCounted, Optional<String> actualFieldValue) {
+        Optional<PaperDetails> citationCountResult = null;
+        boolean isFetchingAllowed = relationsRepository.isCitationsUpdatable(citationCounted)
+                || actualFieldValue.isEmpty();
+        if (isFetchingAllowed) {
+            try {
+                citationCountResult = citationFetcher.searchCitationCount(citationCounted);
+            } catch (FetcherException e) {
+                LOGGER.error("Error while fetching citation count for entry", e);
+            }
+            return citationCountResult.map(PaperDetails::getCitationCount).orElse(0);
+        }
+        return Integer.parseInt(actualFieldValue.get());
+    }
+
     public void close() {
         relationsRepository.close();
     }

jablib/src/main/java/org/jabref/logic/importer/fetcher/citation/CitationFetcher.java

@@ -1,8 +1,10 @@
 package org.jabref.logic.importer.fetcher.citation;
 
 import java.util.List;
+import java.util.Optional;
 
 import org.jabref.logic.importer.FetcherException;
+import org.jabref.logic.importer.fetcher.citation.semanticscholar.PaperDetails;
 import org.jabref.model.entry.BibEntry;
 
 /**
@@ -40,6 +42,14 @@ enum SearchType {
      */
     List<BibEntry> searchCiting(BibEntry entry) throws FetcherException;
 
+    /**
+     * Get the paper details that includes citation count field for a given {@link BibEntry}.
+     *
+     * @param entry entry to search citation count field
+     * @return returns a {@link PaperDetails}, that includes citation count field (may be empty)
+     */
+    Optional<PaperDetails> searchCitationCount(BibEntry entry) throws FetcherException;
+
     /**
      * Returns the localized name of this fetcher.
      * The title can be used to display the fetcher in the menu and in the side pane.

jablib/src/main/java/org/jabref/logic/importer/fetcher/citation/semanticscholar/SemanticScholarCitationFetcher.java

@@ -3,6 +3,7 @@
 import java.net.MalformedURLException;
 import java.net.URL;
 import java.util.List;
+import java.util.Optional;
 
 import org.jabref.logic.importer.FetcherException;
 import org.jabref.logic.importer.ImporterPreferences;
@@ -33,6 +34,12 @@ public String getAPIUrl(String entry_point, BibEntry entry) {
                 + "&limit=1000";
     }
 
+    public String getAPIUrl(BibEntry entry) {
+        return SEMANTIC_SCHOLAR_API + "paper/" + "DOI:" + entry.getDOI().orElseThrow().asString()
+                + "?fields=" + "title,authors,year,citationCount,referenceCount,externalIds,publicationTypes,abstract,url"
+                + "&limit=1000";
+    }
+
     @Override
     public List<BibEntry> searchCitedBy(BibEntry entry) throws FetcherException {
         if (entry.getDOI().isEmpty()) {
@@ -84,6 +91,27 @@ public List<BibEntry> searchCitedBy(BibEntry entry) throws FetcherException {
                                  .map(referenceDataItem -> referenceDataItem.getCitedPaper().toBibEntry()).toList();
     }
 
+    @Override
+    public Optional<PaperDetails> searchCitationCount(BibEntry entry) throws FetcherException {
+        if (entry.getDOI().isEmpty()) {
+            return Optional.empty();
+        }
+        URL referencesUrl;
+        try {
+            referencesUrl = URLUtil.create(getAPIUrl(entry));
+        } catch (MalformedURLException e) {
+            throw new FetcherException("Malformed URL", e);
+        }
+        URLDownload urlDownload = new URLDownload(referencesUrl);
+        importerPreferences.getApiKey(getName()).ifPresent(apiKey -> urlDownload.addHeader("x-api-key", apiKey));
+        PaperDetails referencesResponse = GSON.fromJson(urlDownload.asString(), PaperDetails.class);
+
+        if (referencesResponse == null) {
+            return Optional.empty();
+        }
+        return Optional.of(referencesResponse);
+    }
+
     @Override
     public String getName() {
         return FETCHER_NAME;

jablib/src/main/java/org/jabref/model/entry/field/FieldFactory.java

@@ -231,7 +231,7 @@ private static Set<Field> getAllFields() {
      * separate preferences object
      */
     public static List<Field> getDefaultGeneralFields() {
-        List<Field> defaultGeneralFields = new ArrayList<>(Arrays.asList(StandardField.DOI, StandardField.CROSSREF, StandardField.KEYWORDS, StandardField.EPRINT, StandardField.URL, StandardField.FILE, StandardField.GROUPS, StandardField.OWNER, StandardField.TIMESTAMP));
+        List<Field> defaultGeneralFields = new ArrayList<>(Arrays.asList(StandardField.DOI, StandardField.CITATIONCOUNT, StandardField.CROSSREF, StandardField.KEYWORDS, StandardField.EPRINT, StandardField.URL, StandardField.FILE, StandardField.GROUPS, StandardField.OWNER, StandardField.TIMESTAMP));
         defaultGeneralFields.addAll(EnumSet.allOf(SpecialField.class));
         return defaultGeneralFields;
     }

jablib/src/main/java/org/jabref/model/entry/field/FieldProperty.java

@@ -35,6 +35,7 @@ public enum FieldProperty {
 
     // Field content should be treated as data
     VERBATIM,
+    CITATION_COUNT,
 
     YES_NO
 }

jablib/src/main/java/org/jabref/model/entry/field/StandardField.java

@@ -138,8 +138,10 @@ public enum StandardField implements Field {
     OWNER("owner"),
     TIMESTAMP("timestamp", FieldProperty.DATE),
     CREATIONDATE("creationdate", FieldProperty.DATE),
+    CITATIONCOUNT("Citation count", FieldProperty.CITATION_COUNT),
     MODIFICATIONDATE("modificationdate", FieldProperty.DATE);
 
+
     public static final Set<Field> AUTOMATIC_FIELDS = Set.of(OWNER, TIMESTAMP, CREATIONDATE, MODIFICATIONDATE);
 
     private static final Map<String, StandardField> NAME_TO_STANDARD_FIELD = new HashMap<>();