dsifford / academic-bloggers-toolkit

WordPress plugin providing an all-in-one solution for effective academic blogging.
https://wordpress.org/plugins/academic-bloggers-toolkit/
GNU General Public License v3.0
120 stars 38 forks source link

Some fields not importing in Book type #567

Closed glorious1 closed 5 years ago

glorious1 commented 5 years ago

So I'm getting back to production, and found that three fields aren't importing to ABT in the Book reference type. The fields are:

Zotero UI CSL Field RIS ABT UI
Series collection-title T2 Series Title
Series Number collection-number M1 Series Number
# of Pages number-of-pages SP # of Pages

Here is a RIS file with a few books where those fields don't import (only one has series) test.ris.txt

dsifford commented 5 years ago

I'm having a hard time understanding your table.

Glancing at it, it shows that both Zotero and ABT UI match? So we're good?

glorious1 commented 5 years ago

No, unfortunately. The disconnect is between the RIS fields and ABT. None of those fields show up in ABT after import.

dsifford commented 5 years ago

I see.

Well unfortunately, "fixing" for book would mean breaking for other types. T2 and T3 are used for Series Title, so changing it to support book, would break in others.

M1 is an arbitrary metadata field which is linked to number to best support most cases.

SP is not number-of-pages. It is to be used only and exclisively for Start Page.

So gonna close this due it being just the quirks of RIS. Hopefully I'll have time soon to work on #568. Keep an eye on that.

glorious1 commented 5 years ago

Not to beat a dead horse, and I know you just want to forget RIS and move on, and that's fine, but just FYI the meaning of the RIS tags varies among the reference types. Here is the tag reference for BOOK type, from the RIS standard issued by Thomson Reuters (now available at the bottom of this page).

Note that T2 is Series Title (as you said, but it doesn't import as that in a BOOK) M1 is Series Volume SP is number of pages

TY  - BOOK
A3  - Editor
A2  - Series Editor
A4  - Translator
AB  - Abstract
AD  - Author Address
AN  - Accession Number
AU  - Author
C3  - Title Prefix
C4  - Reviewer
CA  - Caption
CN  - Call Number
CY  - City
DA  - Date
DB  - Name of Database
DO  - DOI
DP  - Database Provider
ET  - Edition
J2  - Abbreviation
KW  - Keywords
L1  - File Attachments
L4  - Figure
LA  - Language
LB  - Label
M1  - Series Volume
M3  - Type of Work
N1  - Notes
NV  - Number of Volumes
OP  - Original Publication
PB  - Publisher
PY  - Year
RN  - Research Notes
RP  - Reprint Edition
SE  - Pages
SN  - ISBN
SP  - Number of Pages
ST  - Short Title
T2  - Series Title
TA  - Translated Author
TI  - Title
TT  - Translated Title
UR  - URL
VL  - Volume
Y2  - Access Date
ER  - {IGNORE}
dsifford commented 5 years ago

I'm aware.

Unfortunately, as stated previously...

  1. If I change T2, I'd break journal title for citing academic journals.
  2. If I change SP, I'd break all sorts of other types of citations that use it when paired with EP for start and end pages.

I might be able to be convinced to change M1, but that should be done in the appropriate repo, not here. See: https://github.com/dsifford/astrocite