feat: support open reference at google scholar #13153

brandon-lau0 · 2025-05-20T09:20:26Z

Summary of Changes

This PR adds a new "Search Google Scholar" feature to JabRef, similar to the existing "Search ShortScience" functionality. The feature allows users to quickly search for a selected entry's title in Google Scholar directly from the main table's context menu. The search engines are moved under the "Search..." tab in the right click menu for better organization.

In addition, we make Google Scholar and Short Science links configurable in the Preferences tab, under a new section named "Search Engine URL Templates". One of the comments mentioned, in the issue, that "the URL should be configurable. Reason: We cannot maintain JabRef for all site URLs; and the list of sites might be oppionated." This allows users to customize their URL templates, such as adding additional query parameters like author or strings.

The following shows the link the user is directed to after clicking "Search Google Scholar".

Closes #12268.

Bullet Pointed List of Changes:

Added new search Google Scholar option
Moved search engines to a "Search" sub-menu in right click menu
Allow "Search Google Scholar" and "Search Short Science" URLs to be configurable in Preferences under the "Search Engine URL Templates" section

Collaborators

@lydia-yan @yoasaaa @brandon-lau0 @FlyJoanne

Mandatory checks

I own the copyright of the code submitted and I license it under the MIT license
Change in CHANGELOG.md described in a way that is understandable for the average user (if change is visible to the user)
[/] Tests created for changes (if applicable)
Manually tested changed features in running JabRef (always required)
Screenshots added in PR description (if change is visible to the user)
Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

jabgui/src/main/java/org/jabref/gui/actions/StandardActions.java

koppor · 2025-05-20T11:20:47Z

Update following are wrong comments; the feature is about opening using the web browser

~~Please investigate org.jabref.logic.importer.fetcher.GoogleScholar and read on at https://devdocs.jabref.org/code-howtos/fetchers.html.~~

~~Also look at #7075~~

~~You can try to work on JabRef#695.~~

jabgui/src/main/java/org/jabref/gui/maintable/SearchGoogleScholarAction.java

jablib/src/test/java/org/jabref/logic/util/ExternalLinkCreatorTest.java

jabref-machine · 2025-05-30T23:47:48Z

Your code currently does not meet JabRef's code guidelines. We use Checkstyle to identify issues. You can see which checks are failing by locating the box "Some checks were not successful" on the pull request page. To see the test output, locate "Tests / Checkstyle (pull_request)" and click on it.

In case of issues with the import order, double check that you activated Auto Import. You can trigger fixing imports by pressing Ctrl+Alt+O to trigger Optimize Imports.

Please carefully follow the setup guide for the codestyle. Afterwards, please run checkstyle locally and fix the issues, commit, and push.

trag-bot · 2025-05-30T23:52:18Z

jabgui/src/main/java/org/jabref/gui/actions/StandardActions.java

@@ -37,6 +37,8 @@ public enum StandardActions implements Action {
    EXTRACT_FILE_REFERENCES_OFFLINE(Localization.lang("Extract references from file (offline)"), IconTheme.JabRefIcons.FILE_STAR),
    OPEN_URL(Localization.lang("Open URL or DOI"), IconTheme.JabRefIcons.WWW, KeyBinding.OPEN_URL_OR_DOI),
    SEARCH_SHORTSCIENCE(Localization.lang("Search ShortScience")),
+    SEARCH_GOOGLE_SCHOLAR(Localization.lang("Search Google Scholar")),
+    SEARCH(Localization.lang("Search...")),


The 'SEARCH' action should be grouped with other search-related actions to maintain semantic consistency and improve code organization.

jabgui/src/main/java/org/jabref/gui/maintable/SearchGoogleScholarAction.java

jabgui/src/main/java/org/jabref/gui/maintable/SearchShortScienceAction.java

jablib/src/main/java/org/jabref/logic/importer/ImporterPreferences.java

koppor

Small comment on the tests

(I currently don't have much time to dive into this PR, but wanted to give a little pice of work to move this PR forward)

koppor · 2025-05-31T08:44:49Z

jablib/src/test/java/org/jabref/logic/util/ExternalLinkCreatorTest.java

@@ -29,25 +50,34 @@ private boolean urlIsValid(String url) {

    @Test
    void getShortScienceSearchURLEncodesSpecialCharacters() {
+        ImporterPreferences stubPreferences = new StubImporterPreferences();


Typically, this is done with mock and when. See other classes. Can you try this?

Yep, I removed the StubImporterPreferences and replaced it with mock.

koppor · 2025-06-01T19:17:53Z

jabgui/src/main/resources/org/jabref/gui/preferences/websearch/WebSearchTab.fxml

+                         fx:id="catalogEnabledColumn"
+                         text="%Enabled"


this is no off lines 50/51.

Why the additoinal indent / reformat?

It is hard to review code which makes formatting changes.

I'm not too sure why it shows up as a diff in GitHub. Locally, I didn't make any changes to Lines 50 and 51, but it shows up as an indent formatting change here. I've tried a couple times to resync with the original, but it didn't seem to work. Do you have any ideas for why this may be the case?

koppor · 2025-06-05T02:34:42Z

@brandon-lau0 May I ask if you are going to continue to work on this? - There are merge conflicts to resolve and tests made properly. If not, it is allright for us; we can then ask another student to continue.

trag-bot · 2025-06-08T01:14:26Z

jablib/src/main/java/org/jabref/logic/util/ExternalLinkCreator.java

+                    author.ifPresent(a -> uriBuilder.addParameter("author", a));
+                    return uriBuilder.toString();
+                } catch (URISyntaxException ex) {
+                    throw new AssertionError("ShortScience URL is invalid.", ex);


Using exceptions for control flow is not recommended. Exceptions should be used for exceptional states, not normal control flow.

trag-bot · 2025-06-08T01:14:27Z

jablib/src/main/java/org/jabref/logic/util/ExternalLinkCreator.java

+                    author.ifPresent(a -> uriBuilder.addParameter("author", a));
+                    return uriBuilder.toString();
+                } catch (URISyntaxException ex) {
+                    throw new AssertionError("Default Google Scholar URL is invalid.", ex);


Using exceptions for control flow is not recommended. Exceptions should be used for exceptional states, not normal control flow.

brandon-lau0 · 2025-06-08T02:08:12Z

@brandon-lau0 May I ask if you are going to continue to work on this? - There are merge conflicts to resolve and tests made properly. If not, it is allright for us; we can then ask another student to continue.

Thank you for your patience. I just resolved the merge conflicts and the tests to use mocks.

brandon-lau0 · 2025-06-08T02:10:32Z

jabgui/src/main/resources/org/jabref/gui/preferences/websearch/WebSearchTab.fxml

@@ -78,12 +101,12 @@
                         text="%Catalog"
            />
            <TableColumn minWidth="120"
-                    fx:id="customApiKey"
-                    text="Key"/>
+                         fx:id="customApiKey"


Similar formatting issue as previous comment. Unsure why a diff shows here. I'm not able to see the changes locally.

jabref-machine · 2025-06-08T03:02:14Z

JUnit tests of jablib are failing. You can see which checks are failing by locating the box "Some checks were not successful" on the pull request page. To see the test output, locate "Tests / Unit tests (pull_request)" and click on it.

You can then run these tests in IntelliJ to reproduce the failing tests locally. We offer a quick test running howto in the section Final build system checks in our setup guide.

jablib/src/test/java/org/jabref/logic/util/ExternalLinkCreatorTest.java

trag-bot bot reviewed May 20, 2025

View reviewed changes

jabgui/src/main/java/org/jabref/gui/actions/StandardActions.java Show resolved Hide resolved

koppor changed the title ~~feat: support searching via google scholar~~ feat: support open reference at google scholar May 21, 2025