Skip to content

Fix fail on testPerformExportForSingleEntry from DocBook5ExporterTest#5168

Merged
Siedlerchr merged 2 commits into
JabRef:masterfrom
davidemdot:fix-DocBook5ExporterTest
Aug 5, 2019
Merged

Fix fail on testPerformExportForSingleEntry from DocBook5ExporterTest#5168
Siedlerchr merged 2 commits into
JabRef:masterfrom
davidemdot:fix-DocBook5ExporterTest

Conversation

@davidemdot

Copy link
Copy Markdown
Member

Fix a bug that appeared on DocBook5ExporterTest (testPerformExportForSingleEntry) since the last commit in the master branch (4b39b78).

The point is that the <biblioset> node is not contained in the export file. Also, I have used normalizeWhitespace() and ignoreComments() methods.

I have added the [WIP] mark because I still want to check if there is any trouble with the exporter. Any help about this?

@davidemdot

Copy link
Copy Markdown
Member Author

I think the problem is that the exporter does not detect which format (Book, in this case) it has to export.

@davidemdot

Copy link
Copy Markdown
Member Author

Template Explorer was looking for layouts with wrong file names (docbook5.Book.layout instead of docbook5.book.layout). This patch should solve related issues as well.

@davidemdot davidemdot changed the title [WIP] Fix fail on testPerformExportForSingleEntry from DocBook5ExporterTest Fix fail on testPerformExportForSingleEntry from DocBook5ExporterTest Aug 2, 2019
@davidemdot davidemdot added [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs component: export-or-save status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers labels Aug 2, 2019

@tobiasdiez tobiasdiez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks a lot!

@Siedlerchr Siedlerchr merged commit 76f9ac5 into JabRef:master Aug 5, 2019
@Siedlerchr

Copy link
Copy Markdown
Member

I merged this now. You could ask @Bernhard Kleine (if I remember correctly), if his exports are still working.

@davidemdot davidemdot deleted the fix-DocBook5ExporterTest branch August 6, 2019 11:43
Siedlerchr added a commit that referenced this pull request Aug 9, 2019
…rter

# By David Méndez (47) and others
# Via GitHub (5) and David Méndez (3)
* upstream/master: (57 commits)
  fix wrong package (#5181)
  Remove logging message for non-existing nested files
  Bump applicationinsights-core from 2.4.0 to 2.4.1 (#5171)
  Bump archunit-junit5-engine from 0.10.2 to 0.11.0 (#5157)
  Bump applicationinsights-logging-log4j2 from 2.4.0 to 2.4.1 (#5172)
  Bump tika-core from 1.21 to 1.22 (#5166)
  Fix fail on testPerformExportForSingleEntry from DocBook5ExporterTest (#5168)
  Add a check for nested files and improve the code to skip lines (DefaultTexParser)
  Add latest changes to latexintegration (#5170)
  LaTeX integration latest changes (#5167)
  Move to extended enums for fields and entry types (#5148)
  Bump archunit-junit5-api from 0.10.2 to 0.11.0 (#5158)
  Revert temporal change
  Fix all issues from reviews of #5137
  Bump com.simonharrer.modernizer from 1.6.0-1 to 1.8.0-1 (#5154)
  Bump checkstyle from 8.22 to 8.23 (#5153)
  Add a new JabRefIcons.LATEX_CITATIONS
  Change toString() methods
  Update DefaultTexParser for explaining when and why it skips the citation matching
  Update TexParserResult for avoiding 'orElse(null)'
  ...

# Conflicts:
#	src/main/java/org/jabref/logic/importer/fileformat/PdfContentImporter.java
#	src/test/java/org/jabref/logic/importer/fileformat/PdfContentImporterTest.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: export-or-save [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants