retorquere / zotero-better-bibtex

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

Stuck on export #2972

Closed matthiasfabry closed 1 day ago

matthiasfabry commented 2 weeks ago

Debug log ID

SQWH5QKE-euc/6.7.226-7

What happened?

much like #2954 and/or #2940 and/or #2942, I cannot export to bib files. This worked in Zotero 6 (although sometimes I had to enter the settings and manually "export now" instead of BBT automatically exporting after a change), but now it's stuck on "preparing Better BibTex +1"

retorquere commented 2 weeks ago

I don't see export activity in SQWH5QKE-euc/6.7.226-7. What were you trying to export?

matthiasfabry commented 2 weeks ago

I clicked on a collection and hit export. It opens a dialog with options, clicked on keep updated, gave the filename and save. Then it’s stuck on this message. Is this supposed to take more than 2 mins?

retorquere commented 2 weeks ago

Is this supposed to take more than 2 mins?

No. A build will drop here soon that has more logging, please send a new log from that.

much like #2954 and/or #2940 and/or #2942

it's not likely there is a relation.

github-actions[bot] commented 2 weeks ago

:robot: this is your friendly neighborhood build bot announcing test build 6.7.226.6920 ("logging")

This update may name other issues, but the build just dropped here is for you; it just means problems already fixed in other issues have been folded into the work we are doing here. Install in Zotero by downloading test build 6.7.226.6920, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

matthiasfabry commented 1 week ago

YQF9PDU9-euc/6.7.226.6920-7

It appears as if zotero as a whole is not hanging though. While the "preparing BBT" message shows, graying out the main zotero window, I can still access the "File, Edit, Tools, etc." menus.

matthiasfabry commented 1 week ago

Ok this is really strange. I tried exporting another collection, at it went without problems. Then tried exporting the problematic collection again, and now it exports as normal.

X224MX2B-euc/6.7.226.6920-7

github-actions[bot] commented 1 week ago

:robot: this is your friendly neighborhood build bot announcing test build 6.7.227.6932 ("test both")

This update may name other issues, but the build just dropped here is for you; it just means problems already fixed in other issues have been folded into the work we are doing here. Install in Zotero by downloading test build 6.7.227.6932, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

retorquere commented 1 week ago

please try with build 6932

matthiasfabry commented 1 week ago

W3SEIVRK-euc/6.7.227.6932-7

github-actions[bot] commented 1 week ago

:robot: this is your friendly neighborhood build bot announcing test build 6.7.227.6938 ("more logging")

This update may name other issues, but the build just dropped here is for you; it just means problems already fixed in other issues have been folded into the work we are doing here. Install in Zotero by downloading test build 6.7.227.6938, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

matthiasfabry commented 1 week ago

HQKVE6NA-euc/6.7.227.6938-7

retorquere commented 1 week ago

So the translate starts and succeeds but there's still a recursion error. I'm looking into this but it may well take me a few days to come up with something.

github-actions[bot] commented 5 days ago

:robot: this is your friendly neighborhood build bot announcing test build 6.7.229.6971 ("Merge branch 'master' into gh-2972")

This update may name other issues, but the build just dropped here is for you; it just means problems already fixed in other issues have been folded into the work we are doing here. Install in Zotero by downloading test build 6.7.229.6971, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

retorquere commented 1 day ago

Does this build fix the problem?

matthiasfabry commented 1 day ago

SBWG6MJ4-euc/6.7.229.6971-7

BBT now seems to correctly export (I see it in my files), but the dialog with "Exporting items..." does not disappear, needing a restart of Zotero to interact again with the UI.

github-actions[bot] commented 1 day ago

:robot: this is your friendly neighborhood build bot announcing test build 6.7.229.6989 ("merge master")

This update may name other issues, but the build just dropped here is for you; it just means problems already fixed in other issues have been folded into the work we are doing here. Install in Zotero by downloading test build 6.7.229.6989, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

matthiasfabry commented 1 day ago

This looks to work as intended! Thank you so much Emiliano!

PC8JBD2G-euc/6.7.229.6989-7 for your reference.

retorquere commented 1 day ago

awesome. thanks for sticking this out with me.

matthiasfabry commented 1 day ago

Just out of curiosity, what turned out to be the problem?

retorquere commented 1 day ago

To be totally honest -- I think I know what triggered the problem, but I don't know what the actual problem was.

It looked like my database writes for the auto-exports triggered a recursive invocation of Zotero's dateparser; I have no idea why that happened, as I could find nothing about SQL writes through the Zotero API that did date parsing, so in the end I have just removed the trigger; I have taken the auto-exports out of the SQL database and have stuck them in preferences. I could not create a focused testcase (a so-called MWE) for the problem so a discussion about this on the dev list would not have gotten much traction.