Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No Shutdown possible after external modification of file #2449

Closed
matthiasgeiger opened this issue Jan 6, 2017 · 3 comments
Closed

No Shutdown possible after external modification of file #2449

matthiasgeiger opened this issue Jan 6, 2017 · 3 comments
Labels
component: ui [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs status: duplicate

Comments

@matthiasgeiger
Copy link
Member

JabRef version at least 3.8 and master

Steps to reproduce:

  1. Open a file
  2. modify it externally
  3. Accept changes
  4. Try to close JabRef
java.lang.IllegalArgumentException: missing event subscriber for an annotated method. Is net.sf.jabref.logic.autosaveandbackup.BackupManager@5147902d registered?
	at com.google.common.eventbus.SubscriberRegistry.unregister(SubscriberRegistry.java:111) ~[guava-20.0.jar:?]
	at com.google.common.eventbus.EventBus.unregister(EventBus.java:201) ~[guava-20.0.jar:?]
	at net.sf.jabref.model.metadata.MetaData.unregisterListener(MetaData.java:257) ~[main/:?]
	at net.sf.jabref.logic.autosaveandbackup.BackupManager.shutdown(BackupManager.java:149) ~[main/:?]
	at java.util.stream.ForEachOps$ForEachOp$OfRef.accept(ForEachOps.java:184) ~[?:1.8.0_60]
	at java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:175) ~[?:1.8.0_60]
	at java.util.HashMap$KeySpliterator.forEachRemaining(HashMap.java:1540) ~[?:1.8.0_60]
	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481) ~[?:1.8.0_60]
	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471) ~[?:1.8.0_60]
	at java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:151) ~[?:1.8.0_60]
	at java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:174) ~[?:1.8.0_60]
	at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) ~[?:1.8.0_60]
	at java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:418) ~[?:1.8.0_60]
	at net.sf.jabref.logic.autosaveandbackup.BackupManager.shutdown(BackupManager.java:85) ~[main/:?]
	at net.sf.jabref.gui.JabRefFrame.quit(JabRefFrame.java:859) ~[main/:?]
	at net.sf.jabref.gui.JabRefFrame$CloseAction.actionPerformed(JabRefFrame.java:1778) ~[main/:?]
	at net.sf.jabref.gui.JabRefFrame$2.windowClosing(JabRefFrame.java:604) ~[main/:?]
	at java.awt.AWTEventMulticaster.windowClosing(AWTEventMulticaster.java:349) ~[?:1.8.0_60]`

....

@matthiasgeiger matthiasgeiger added [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs component: ui labels Jan 6, 2017
@rfgiusti
Copy link

rfgiusti commented Jan 16, 2017

Additional info: this only happens if you review the changes (regardless of whether you keep them checked as accepted or uncheck). If you cancel the review or don't click "Review changes", nothing happens.

This bug can be work-arounded by closing the database before attempting to close JabRef (still raises an exception).

@matthiasgeiger
Copy link
Member Author

also covered in #2486

as @lenhard is working on a fix I'll close this as a duplicate

@lenhard
Copy link
Member

lenhard commented Jan 24, 2017

Blind me! Sorry for not discovering that this issue already existed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: ui [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs status: duplicate
Projects
None yet
Development

No branches or pull requests

3 participants