Added a custom Export for academicpages.github.io #14463

GreLucie · 2025-11-29T10:50:03Z

This PR is to add a custom exporter for academicpages.github.io by adding the corresponding layouts.
It also implements a way to export multiple files to this format.

Steps to test

When jabref is running, go to file->Export->Export selected entries ( or all entries) and choose "Academic Pages Markdowns" for the export type.

You can then add the markdown file(s) to your academic page to see it on your own page.

More information about academic pages on the page : https://academicpages.github.io/

Mandatory checks

I own the copyright of the code submitted and I license it under the MIT license
I manually tested my changes in running JabRef (always required)
I added JUnit tests for changes (if applicable)
I added screenshots in the PR description (if change is visible to the user)
I described the change in CHANGELOG.md in a way that is understandable for the average user (if change is visible to the user)
I checked the user 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 updating file(s) in https://github.com/JabRef/user-documentation/tree/main/en.

github-actions · 2025-11-29T10:50:25Z

Hey @GreLucie!

Thank you for contributing to JabRef! Your help is truly appreciated ❤️

We have automated checks in place, based on which you will soon get feedback if any of them are failing. In a while, maintainers will also review your contribution. Once that happens, you can go through their comments in the "Files changed" tab and act on them, or reply to the conversation if you have further inputs.

Please re-check our contribution guide in case of any other doubts related to our contribution workflow.

Siedlerchr · 2025-11-29T14:21:39Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

+        }
+        try {
+            Integer iterator = 1;
+            for (BibEntry entry : entries) {


You could simply switch this to a normal for loop with for (int i =1; i< entiries.length; i++) so you don't need the iterator variable.

Siedlerchr · 2025-11-29T14:22:58Z

jablib/src/test/java/org/jabref/logic/exporter/AcademicPagesExporterTest.java

+
+        String content = Files.readString(expectedFile);
+
+        // Verify YAML front matter fields


Please follow the pattern of other tests in that directory, do a full comparison. of the content

Siedlerchr · 2025-11-29T14:23:28Z

CHANGELOG.md

 ### Removed

-## [6.0-alpha.3] – 2025-10-30
+## [6.0-alpha.3] – 2025-11-29


Please revert this change

…uits avec le format exigé de academic pages

…r-issue-12727 # Conflicts: # jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

…Exporter.java

Siedlerchr · 2025-12-01T20:22:44Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

 import org.jabref.model.entry.BibEntry;
 import org.jabref.model.metadata.SelfContainedSaveOrder;

+import org.jetbrains.annotations.NotNull;


Use jspecify nonnull

Siedlerchr · 2025-12-01T20:23:07Z

CHANGELOG.md

 ### Removed

-## [6.0-alpha.3] – 2025-11-29
+## [6.0-alpha.3] – 2025-10-30


is this some automatic tool that always changes the date?

Hello, you told me to revert this change so I went back to the unmodified CHANGELOG for now. I know we have to change it eventually but while this PR is just a draft we decided to change it once all the issues have been fixed.

You can change it at any time - the earlier the better :) -- BUT - do NOT CHANGE RELEASE DATES

Add your change to the "Unreleased" section.

Note: Current diff shows CHANGELOG.md unmodified: https://github.com/JabRef/jabref/pull/14463/files (which is OK at this stage).

…r-issue-12727

into fix-for-issue-12727

sadok-lajmi · 2025-12-02T19:08:32Z

Hi, we just wanted to clarify the mechanism that we chose to implement for this new AcademicPagesExporter so that we can have your opinion on it and know whether or not it meets your expectations :
This exporter actually creates a folder at first (even when there's only one entry selected) with the name typed by the user in the export dialog and then exports all the entries selected one by one inside this folder, giving them names that match exactly what is defined by the academicpages layout as their file names in the permalink field of the markdown (that way it facilitates instant integration in the personal academic pages repository).
So is this logic fine with you or is there something we need to change especially about the creation of a folder... ?

Siedlerchr · 2025-12-02T19:52:36Z

Yep that sound reasonable. Go ahead

koppor · 2025-12-04T13:45:13Z

@sadok-lajmi Please merge upstream/main. You can do it with the button "Update branch" here on this page or using your GitHub CLI.

koppor · 2025-12-05T23:00:20Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

+    /**
+     * Initialize another export format based on templates stored in dir with layoutFile lfFilename.
+     *
+     */


Comment is strange. What is lfFilename? Where does it come from? I think, this is not something a user of the class should know. You can just delete whole JavaDoc comment.

koppor · 2025-12-05T23:00:49Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

+
+    private static final Logger LOGGER = LoggerFactory.getLogger(AcademicPagesExporter.class);
+
+    private final String lfFileName;


Do not abbreviate. Use layoutFileFileName or short layoutFile (because you don't have layoutFileContent.

koppor · 2025-12-05T23:02:08Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

+     */
+    @Override
+    public void export(@NonNull final BibDatabaseContext databaseContext,
+                       final Path file,


Are you sure that Path may be null?

Annoate all or nothing.

Proposal: Add @NullMarked to the class and mark only things which might be null - I think, there aren't any

koppor · 2025-12-05T23:02:33Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

+            Files.createDirectories(exportDirectory);
+
+            for (BibEntry entry : entries) {
+                if (entry.getType() == null) {


Wait, what? When can this happen?

koppor · 2025-12-05T23:03:02Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

+                List<BibEntry> individual_entry = new ArrayList<>();
+                individual_entry.add(entry);


Wait, what? Use List.of(entry) directly as parameter in the next line.

koppor · 2025-12-05T23:04:08Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

+    }
+
+    private static @NonNull Path getPath(BibEntry entry, Path exportDirectory) {
+        Replace replace_formatter = new Replace();


Why underscores in varable names? In Java, it is camel case.

koppor · 2025-12-05T23:04:24Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

+
+    private static @NonNull Path getPath(BibEntry entry, Path exportDirectory) {
+        Replace replace_formatter = new Replace();
+        replace_formatter.setArgument(" ,-");


Thsi is a magic constant - please add a comment why this strange argument.

koppor · 2025-12-05T23:04:37Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

+        Replace replace_formatter = new Replace();
+        replace_formatter.setArgument(" ,-");
+        RemoveLatexCommandsFormatter commands_formatter = new RemoveLatexCommandsFormatter();
+        HTMLChars html_formatter = new HTMLChars();


Add all formatters as class variable

koppor · 2025-12-05T23:04:55Z

jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

+        HTMLChars html_formatter = new HTMLChars();
+        String title = entry.getTitle().get();
+        String formatted_title = commands_formatter.format(html_formatter.format(replace_formatter.format(title)));
+        SafeFileName safe_formatter = new SafeFileName(); // added custom formatter to remove all characters that are not allowed in filenames


Why? It is obvious - then remove the comment.

koppor · 2025-12-05T23:06:19Z

jablib/src/main/java/org/jabref/logic/layout/format/NumberMonthFormatter.java

+        Optional<Month> month = Month.parse(fieldText);
+        return month.map(Month::getTwoDigitNumber).orElse("01");


Merge this into one line plese.

…porter.java Co-authored-by: Oliver Kopp <kopp.dev@gmail.com>

jabref-machine · 2025-12-06T18:40:31Z

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 "Source Code 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.

sadok-lajmi and others added 11 commits November 5, 2025 15:51

modification des fichiers qui font la configuration des exporters

48450d4

Adding the empty layouts in resources/resource/layout/academicpages

65bbc44

création du fichier academicpages.article.layout

a4e2fea

utilisation du fichier article.layout pour le new TemplateExporter

3e34b92

generalisation du layout en utilisant academicpages.layout

8e76f57

améliorations au academicpages layout

2799f1d

ajout d'un custom formatter to help with academic layout

be96c94

AcademicPagesExporter

1421e98

test: add AcademicPagesExporter tests

e4a987c

Delete unecessary imports

df7aa05

Changelog

9b12666

github-actions bot added the first contrib label Nov 29, 2025

GreLucie changed the title ~~Fix for issue 12727~~ Added a custom Export for academicpages.github.io Nov 29, 2025

GreLucie added 3 commits November 29, 2025 12:00

Changed CHANGELOG.md

f588da4

trying to fix the CHANGELOG.md

de65527

Adding blankspace in CHANGELOG.md

2d8afcf

Siedlerchr reviewed Nov 29, 2025

View reviewed changes

CHANGELOG.md Outdated

### Removed

## [6.0-alpha.3] – 2025-10-30

## [6.0-alpha.3] – 2025-11-29

Copy link

Member

Siedlerchr Nov 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert this change

sadok-lajmi added 2 commits November 30, 2025 00:05

Correction du nouveau exporter + Cohérence des noms des fichiers prod…

4ea372d

…uits avec le format exigé de academic pages

Merge remote-tracking branch 'origin/fix-for-issue-12727' into fix-fo…

3972b34

…r-issue-12727 # Conflicts: # jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesExporter.java

github-actions bot added the status: changes-required Pull requests that are not yet complete label Nov 30, 2025

GreLucie added 2 commits December 1, 2025 20:34

Reverted the date on the CHANGELOG.md and one change on AcademicPages…

9b8a86d

…Exporter.java

revert all changes to Changelog.md

9087390

Siedlerchr reviewed Dec 1, 2025

View reviewed changes

sadok-lajmi and others added 3 commits December 2, 2025 08:39

format correction for the file : LayoutEntry

655afe3

Merge remote-tracking branch 'origin/fix-for-issue-12727' into fix-fo…

4d7dac1

…r-issue-12727

trying to fix Open Rewrite problem

30e7626

GreLucie added 2 commits December 2, 2025 09:57

deleted useless import in AcademicPagesExporter.java

aaa5855

Merge branch 'fix-for-issue-12727' of https://github.com/GreLucie/jabref

3547cd0

into fix-for-issue-12727

github-actions bot added the status: changes-required Pull requests that are not yet complete label Dec 2, 2025

correction of the error given by the localizationconsistancy test

7b55f27

github-actions bot removed the status: changes-required Pull requests that are not yet complete label Dec 2, 2025

sadok-lajmi and others added 4 commits December 4, 2025 15:42

Merge branch 'main' into fix-for-issue-12727

ec19bb9

Merge branch 'main' into fix-for-issue-12727

05792cb

Updated CHANGELOG.md

7ccbc6f

added the link to issue 12727

6f25e5c

GreLucie marked this pull request as ready for review December 5, 2025 08:44

koppor requested changes Dec 5, 2025

View reviewed changes

github-actions bot added the status: changes-required Pull requests that are not yet complete label Dec 5, 2025

Update jablib/src/main/java/org/jabref/logic/exporter/AcademicPagesEx…

3c24b32

…porter.java Co-authored-by: Oliver Kopp <kopp.dev@gmail.com>

github-actions bot removed the status: changes-required Pull requests that are not yet complete label Dec 6, 2025

GreLucie and others added 4 commits December 6, 2025 14:00

Adressing some of the comments regarding the PR

b38bed2

Merge branch 'main' into fix-for-issue-12727

fe03fc4

Comment added for replaceFormatter.setArgument(' ,-')

c2a06e0

tests: added 4 test cases and specified the SaveException

280a1fb

github-actions bot added the status: changes-required Pull requests that are not yet complete label Dec 6, 2025

AloisHasNeurons and others added 7 commits December 6, 2025 16:36

Fixed submodules?

6a4c5b0

add all formatters as class variable

100fe95

changed a comment

07088a8

Trying to fix indentation in AcademicPagesExporterTest

fcd3db4

Revert change on AcademicPagesExporter

1efe4a2

Fixed some indentation in AcademicPagesExporterTest

488620d

add all formatters as class variable

5d6e3a2


		String content = Files.readString(expectedFile);

		// Verify YAML front matter fields


		private static final Logger LOGGER = LoggerFactory.getLogger(AcademicPagesExporter.class);

		private final String lfFileName;

		List<BibEntry> individual_entry = new ArrayList<>();
		individual_entry.add(entry);

		Optional<Month> month = Month.parse(fieldText);
		return month.map(Month::getTwoDigitNumber).orElse("01");

Uh oh!

Added a custom Export for academicpages.github.io #14463

Are you sure you want to change the base?

Added a custom Export for academicpages.github.io #14463

Uh oh!

Conversation

GreLucie commented Nov 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Steps to test

Mandatory checks

Uh oh!

github-actions bot commented Nov 29, 2025

Hey @GreLucie!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sadok-lajmi commented Dec 2, 2025

Uh oh!

Siedlerchr commented Dec 2, 2025

Uh oh!

koppor commented Dec 4, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jabref-machine commented Dec 6, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

GreLucie commented Nov 29, 2025 •

edited

Loading