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

Fix bug seriesTitle and series #1348

Closed
wants to merge 1 commit into from
Closed

Conversation

helieh
Copy link

@helieh helieh commented Jul 2, 2017

seriesTitle and series do not show after export item and then import that.

seriesTitle and series do not show after export item and then import that.
@@ -244,7 +244,7 @@ function generateItem(item, zoteroType, resource) {
// set series type
Zotero.RDF.addStatement(series, rdf+"type", n.bib+"Series", false);
// add relationship to resource
Zotero.RDF.addStatement((containerElement ? containerElement : resource), n.dcterms+"isPartOf", series, false);
Zotero.RDF.addStatement(resource, n.dcterms+"isPartOf", series, false);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks but I don't think this is the right fix -- if there is a container (i.e. a book or a journal) the series term should be applied to that with the isPartOf term, not to the article/chapter. This needs to get fixed on import, not on export.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ticket created to track this: #1350

@adam3smith
Copy link
Collaborator

I'm going to close this here, since import is in a different translator, but it'd be great if you'd take that one, obviously (I believe that should be RDF.js)

@adam3smith adam3smith closed this Jul 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants