Skip to content

Updating the translation entries from Brazilian Portuguese (pt_BR)#60

Merged
simonharrer merged 3 commits into
JabRef:masterfrom
igorsteinmacher:master
Jul 2, 2015
Merged

Updating the translation entries from Brazilian Portuguese (pt_BR)#60
simonharrer merged 3 commits into
JabRef:masterfrom
igorsteinmacher:master

Conversation

@igorsteinmacher

Copy link
Copy Markdown
Member

No description provided.

simonharrer added a commit that referenced this pull request Jul 2, 2015
Updating the translation entries from Brazilian Portuguese (pt_BR)
@simonharrer simonharrer merged commit 6d6e228 into JabRef:master Jul 2, 2015
koppor added a commit that referenced this pull request Jan 29, 2024
Uncaught exception occurred in Thread[#60,JavaFX Application Thread,5,main]
java.lang.NullPointerException: Cannot invoke "org.apache.lucene.index.IndexWriter.close()" because "this.indexWriter" is null
	at org.jabref@100.0.0/org.jabref.logic.pdf.search.PdfIndexer.close(PdfIndexer.java:299)
	at org.jabref@100.0.0/org.jabref.logic.pdf.search.PdfIndexerManager.shutdownIndexer(PdfIndexerManager.java:74)
	at org.jabref@100.0.0/org.jabref.gui.LibraryTab.onClosed(LibraryTab.java:792)
github-merge-queue Bot pushed a commit that referenced this pull request Feb 3, 2024
* Fix closing of JabRef in case of issues with index writer

Uncaught exception occurred in Thread[#60,JavaFX Application Thread,5,main]
java.lang.NullPointerException: Cannot invoke "org.apache.lucene.index.IndexWriter.close()" because "this.indexWriter" is null
	at org.jabref@100.0.0/org.jabref.logic.pdf.search.PdfIndexer.close(PdfIndexer.java:299)
	at org.jabref@100.0.0/org.jabref.logic.pdf.search.PdfIndexerManager.shutdownIndexer(PdfIndexerManager.java:74)
	at org.jabref@100.0.0/org.jabref.gui.LibraryTab.onClosed(LibraryTab.java:792)

* NPEs are caught at onClosed

* Handle absent index writer

* Try to handle null directory

* Discard changes to src/main/java/org/jabref/gui/LibraryTab.java

* Use java.io.tempDir

* Add comments and annotations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants