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

Error while exporting collection to Zotero-RDF #2336

Closed
Prometheos2 opened this issue Dec 13, 2022 · 21 comments
Closed

Error while exporting collection to Zotero-RDF #2336

Prometheos2 opened this issue Dec 13, 2022 · 21 comments

Comments

@Prometheos2
Copy link

Prometheos2 commented Dec 13, 2022

Hello,

I've got errors trying to export some of my collections to Zotero RDF files while BBT is enabled.

Debug log ID:

  • YJMUWV36-euc
  • 6VTRNZ3N-euc (second collection)
  • G52R6DHP-euc (with cache disabled)

It seems that it only concerns elements with the connector's timestamps (2013-07-11T07:00:00.000Z), as one of my collections without one is exported without error.

There are some errors given in the debug logs, which seem to imply it's from Zotero or one of the files (?):

[JavaScript Error: "Component returned failure code: 0x80520001 (NS_ERROR_FILE_UNRECOGNIZED_PATH) [nsIFile.initWithPath]" {file: "chrome://zotero/content/xpcom/file.js" line: 47}]
[JavaScript Error: "Error: Unexpected path value 'undefined'" {file: "chrome://zotero/content/xpcom/file.js" line: 56}]
[JavaScript Error: "value must be defined in Zotero.RDF.addStatement" {file: "chrome://zotero/content/xpcom/translate/src/translation/translate.js" line: 2995}]
[JavaScript Error: "value must be defined in Zotero.RDF.addStatement" {file: "chrome://zotero/content/xpcom/translate/src/translation/translate.js" line: 2995}]
[JavaScript Error: "XML Parsing Error: no root element found
Location: http://s3.eu-central-1.amazonaws.com/ping
Line Number 1, Column 1:" {file: "http://s3.eu-central-1.amazonaws.com/ping" line: 1}]
[JavaScript Error: "XML Parsing Error: no root element found
Location: http://s3.ap-southeast-1.amazonaws.com/ping
Line Number 1, Column 1:" {file: "http://s3.ap-southeast-1.amazonaws.com/ping" line: 1}]
[JavaScript Error: "value must be defined in Zotero.RDF.addStatement" {file: "chrome://zotero/content/xpcom/translate/src/translation/translate.js" line: 2995}]
[JavaScript Error: "value must be defined in Zotero.RDF.addStatement" {file: "chrome://zotero/content/xpcom/translate/src/translation/translate.js" line: 2995}]

Thank you for your attention

@Prometheos2
Copy link
Author

Prometheos2 commented Dec 13, 2022

and I just noticed #2315, let me try that.

Edit: Nope, it wasn't the cache

G52R6DHP-euc

@retorquere
Copy link
Owner

And this is reliably tied to whether BBT is active? I don't interfere in any way with RDF exports, so I have no idea how to begin to address this.

it only concerns elements with the connector's timestamps

I don't know what this means.

@Prometheos2
Copy link
Author

Prometheos2 commented Dec 14, 2022

Yes, it errors out when BBT is active, and works properly when it's not.
I was able to find the faulty elements for the first collection: TJNGAX34-euc (5JTKWIF4-euc includes three of them).

It seems it might be due to titles and author names with special characters and the "timestamps"

I don't know what this means.

Oops, sorry about that.

It's the time of the last update/publication, appending to the date. Like this: 2013-07-11T07:00:00.000Z

@retorquere
Copy link
Owner

Can you select one or more of the items that exhibit this behavior, right-click, and send a debug log from the BBT submenu? The ID will have -refs- in it.

@Prometheos2
Copy link
Author

Here F8D5896L-refs-euc

@retorquere
Copy link
Owner

I cannot reproduce the problem using F8D5896L-refs-euc, can you try disabling all 3rd party plugins except BBT, restart Zotero, and try again?

@retorquere
Copy link
Owner

I have tried on Windows and on MacOS, both export these 3 items without issue to Zotero RDF.

@Prometheos2
Copy link
Author

I've disabled everything else, and there is still an error: 7JKPK8PV-refs-euc

@retorquere
Copy link
Owner

Can you drop this Zotero RDF.js into your translators dir, restart Zotero, reproduce the problem, and send a new log?

@Prometheos2
Copy link
Author

Sadly, I will be stuck on Ubuntu until tomorrow; I will try on Windows tomorrow, hopefully in the morning.

I was able to reproduce the problem, and I tried this translator, but it didn't work: WD5FKTAR-refs-euc

@retorquere
Copy link
Owner

It wasn't expected to work, just give me more debug info. Can you try the new version I just published at the previous URL? I don't see any extra logging.

@Prometheos2
Copy link
Author

Weird; I probably didn't install it properly

This one should be good to go: A56UAGGQ-refs-euc

@github-actions
Copy link

🤖 this is your friendly neighborhood build bot announcing test build 6.7.42.3534 ("don't leave null properties on serialized object")

Install in Zotero by downloading test build 6.7.42.3534, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

@retorquere
Copy link
Owner

please try build 3534

@Prometheos2
Copy link
Author

Still erroring out: ICPASXML-refs-euc

[JavaScript Error: "value must be defined in Zotero.RDF.addStatement" {file: "chrome://zotero/content/xpcom/translate/src/translation/translate.js" line: 2995}]
[JavaScript Error: "value must be defined in Zotero.RDF.addStatement" {file: "chrome://zotero/content/xpcom/translate/src/translation/translate.js" line: 2995}]

@retorquere
Copy link
Owner

All of these items have at least one single-part name. Can you test what happens for one item if you switch that name to two-part?

@github-actions
Copy link

🤖 this is your friendly neighborhood build bot announcing test build 6.7.42.3539 ("fixes #2336")

Install in Zotero by downloading test build 6.7.42.3539, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

@retorquere
Copy link
Owner

Can you test 3539?

@Prometheos2
Copy link
Author

Both of those solutions work
Thank you very much!

Still, what a weird bug ^^'

I wish you happy holidays

@github-actions
Copy link

Thanks for the feedback; there's no way you could have known, but @retorquere prefers to keep bugreports/enhancements open as a reminder to merge the changes into a new release.

@retorquere
Copy link
Owner

The change to two-part names wasn't intended to be a solution, just a diagnosis. The new build is the solution. Thanks for reporting this, I'll roll it out in a new release.

retorquere added a commit that referenced this issue Dec 16, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants