retorquere / zotero-better-bibtex

Make Zotero effective for us LaTeX holdouts
https://retorque.re/zotero-better-bibtex/
MIT License
5.21k stars 285 forks source link

Error exporting #576

Closed fedtpelsen closed 7 years ago

fedtpelsen commented 7 years ago

I have Zotero standalone, chrome ext and BBT installed. I can export BibTeX and BibLaTeX fine but exporting the Better-versions give me a javascript error.

Here's the ID: EAFUV3B3.

retorquere commented 7 years ago

@fedtpelsen, @stfl: could you please enable "extended debugging" in the BBT preferences, try another export, and submit a new error report? The current logs have only the error, but not the activity leading up to the error.

stfl commented 7 years ago

report id: P3SCK9RW

fedtpelsen commented 7 years ago

New ID with extended debugging: WS9ZJDIB

retorquere commented 7 years ago

Same problem in both cases. The translator is being handed an empty object, but I can't replicate that on my end. Can you both try https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.77-circle-3110.xpi and subit a new debug log?

stfl commented 7 years ago

UHGR3G3A

retorquere commented 7 years ago

This is strange -- I'm getting reference objects that haven't passed through my pipeline. Can you try again with https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.77-circle-3113.xpi ?

I can't yet reproduce this on my end alas, or I wouldn't be bothering you.

stfl commented 7 years ago

ID37J4A4

stfl commented 7 years ago

I just downgraded to 1.6.75 and it works..

retorquere commented 7 years ago

Dammit, that rules out one possible cause. You're on Linux, right? Could you get me a real-time debug log? They say to copy to the clipboard, but just start script before you start Zotero. After you've closed Zotero, if you type exit, the whole log will be in a file called typescript.

retorquere commented 7 years ago

While a real-time log will have the most information, this version has been instrumented to disable itself if it encounters an initialization error (which almost certainly must be the case) and will display that error in the BBT preferences. It may have enough information, and it'll be easier for you (the realtime debug log is not really an easy process).

stfl commented 7 years ago

I will give it a go in a minute

retorquere commented 7 years ago

60... 59... 58...

(jk, I'm starting my commute right now so I won't be able to do anything for the next 30 minutes or so)

If it is indeed an init error:

  1. It'll probably be fairly easy to fix, and
  2. I have no idea why I'm not experiencing it.
stfl commented 7 years ago

when I try to open the preferences:

Initialize failed: TypeError: Zotero.BetterBibTeX.CSL.DateParser.addDateParserMonths is not a function
stfl commented 7 years ago

with version 3115: typescript.txt

3113.txt

retorquere commented 7 years ago

Well that localizes the issue but doesn't explain why you're getting an error and I'm not. So frustrating.

Could you try the same (a realtime debug log) with both this version (will still fail but hopefully with informative logging) and this version (there is a decent chance this version will just work, but I'd be a little disappointed if this were the problem).

I think I've isolated the change between .77 and .75 that could yield this particular error; .75 bundled a private copy of the Zotero citation processor, .77 uses the citation processor that's already built into Zotero, but that doesn't explain why there is apparently a difference between the bundled processor you have and the one I have.

stfl commented 7 years ago

Could it be related to the underlying firefox version? I am using the firefox-esr from debian stable (don't know the version now)

And i haven't tried version 1.6.76

retorquere commented 7 years ago

Should really more have to do with the Zotero version rather than the Firefox version. If I have identified the problem correctly, .76 should have the same issue as .77. The two versions I linked to above (variants on .77) will tell me more.

That profile picture of yours, is that from a Holi celebration?

stfl commented 7 years ago

Yes. From holi in Kathmandu :D I semd you the report tomorrow

On Tue, Oct 18, 2016, 21:40 retorquere notifications@github.com wrote:

Should really more have to do with the Zotero version rather than the Firefox version. If I have identified the problem correctly, .76 should have the same issue as .77. The two versions I linked to above (variants on .77) will tell me more.

That profile picture of yours, is that from a Holi celebration?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/retorquere/zotero-better-bibtex/issues/576#issuecomment-254616423, or mute the thread https://github.com/notifications/unsubscribe-auth/ABQqRkhiLdka2yZcHnRCSnn2HnvIpGuoks5q1SCRgaJpZM4KZob- .

retorquere commented 7 years ago

Looking forward to that report, keen to get this solved. I haven't been able to reproduce on my end.

Where I was, Holi got out of hand really rapidly. Colors, yes, but also egg and waste motor oil.

stfl commented 7 years ago

3116.txt

3117 works.. 3117.txt

fedtpelsen commented 7 years ago

I still get an error with .77

V6PRRVM2

On Wed, Oct 19, 2016 at 10:16 AM, Stefan Lendl notifications@github.com wrote:

3116.txt https://github.com/retorquere/zotero-better-bibtex/files/538289/3116.txt

3117 works.. 3117.txt https://github.com/retorquere/zotero-better-bibtex/files/538290/3117.txt

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/retorquere/zotero-better-bibtex/issues/576#issuecomment-254745384, or mute the thread https://github.com/notifications/unsubscribe-auth/ACVZH0NDNfjoqqrm84s9pbNgdBg0usySks5q1dH4gaJpZM4KZob- .

fedtpelsen commented 7 years ago

3177 works for me too!

On Wed, Oct 19, 2016 at 10:40 AM, Søren Wahlgren swahlgren@gmail.com wrote:

I still get an error with .77

V6PRRVM2

On Wed, Oct 19, 2016 at 10:16 AM, Stefan Lendl notifications@github.com wrote:

3116.txt https://github.com/retorquere/zotero-better-bibtex/files/538289/3116.txt

3117 works.. 3117.txt https://github.com/retorquere/zotero-better-bibtex/files/538290/3117.txt

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/retorquere/zotero-better-bibtex/issues/576#issuecomment-254745384, or mute the thread https://github.com/notifications/unsubscribe-auth/ACVZH0NDNfjoqqrm84s9pbNgdBg0usySks5q1dH4gaJpZM4KZob- .

fedtpelsen commented 7 years ago

3117 ofc.

On Wed, Oct 19, 2016 at 10:59 AM, Søren Wahlgren swahlgren@gmail.com wrote:

3177 works for me too!

On Wed, Oct 19, 2016 at 10:40 AM, Søren Wahlgren swahlgren@gmail.com wrote:

I still get an error with .77

V6PRRVM2

On Wed, Oct 19, 2016 at 10:16 AM, Stefan Lendl notifications@github.com wrote:

3116.txt https://github.com/retorquere/zotero-better-bibtex/files/538289/3116.txt

3117 works.. 3117.txt https://github.com/retorquere/zotero-better-bibtex/files/538290/3117.txt

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/retorquere/zotero-better-bibtex/issues/576#issuecomment-254745384, or mute the thread https://github.com/notifications/unsubscribe-auth/ACVZH0NDNfjoqqrm84s9pbNgdBg0usySks5q1dH4gaJpZM4KZob- .

retorquere commented 7 years ago

Thanks for the confirmation, tests are running. A little disappointing, but so it goes.

retorquere commented 7 years ago

This has been merged to master; when #579 clears I'll cut a new release.

retorquere commented 7 years ago

new release is out.

github-actions[bot] commented 3 years ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.