-
Notifications
You must be signed in to change notification settings - Fork 30
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
CSL-JSON dates are wrong #53
Comments
@larsgw do you have a projected completion date for v.0.3.0-11? |
@lexnat187 Hopefully later today. Surprisingly, the most basic fix already works, but only at output level, so not for |
@larsgw that's great news. Let me know when your done and I can test it with my implementation. Let me know if I can help. |
Delay and dismay: trouble because of browser distributions. Very annoying. I hope to be able to release it anyway, somewhere before 01:00 UTC. |
|
Completely fixed now, loose years (and months) as well. |
@larsgw, one of my users reported an issue with BibTeX output missing date information (with citation.js v0.4.0-7). My input is CSL JSON. With CSL JSON as output format, the output is: [
{
"id": 454,
"type": "article-journal",
"title": "Abortion in the lives of women struggling financially: why insurance coverage matters",
"container-title": "Guttmacher Policy Review",
"page": "46–52",
"volume": "19",
"source": "Google Scholar",
"shortTitle": "Abortion in the lives of women struggling financially",
"author": [
{
"family": "Boonstra",
"given": "Heather D."
}
],
"issued": {
"date-parts": [
[
2016
]
]
}
}
] With BibTeX as output format, the output is:
As you can see, the "issued" date is missing from the BibTeX output. I do see dates in the BibTeX output for full dates (with year, month, and day), e.g.: CSL JSON as output: {
"id": 459,
"type": "report",
"title": "State Funding of Abortion Under Medicaid",
"publisher": "Guttmacher Institute",
"URL": "https://www.guttmacher.org",
"issued": {
"date-parts": [
[
2018,
2,
1
]
]
}
}
] BibTeX as output:
|
Previously, empty parts in the date were filled in with |
Fix handling of incomplete dates (i.e. date-parts with only a year or a year and a month) by output formatters like BibTeX and RIS. See #127, #138, #139 See #53 (comment)
Fixed in
|
Thanks for the very quick fix! |
Fix handling of incomplete dates (i.e. date-parts with only a year or a year and a month) by output formatters like BibTeX and RIS. See #127, #138, #139 See larsgw/citation.js#53 (comment)
Fix handling of incomplete dates (i.e. date-parts with only a year or a year and a month) by output formatters like BibTeX and RIS. See #127, #138, #139 See larsgw/citation.js#53 (comment)
Problem
The CSL-JSON date format used in Citation.js is wrong, and therefore not picked up by citeproc-js.
Consequence
All dates are omitted in all formatted Citation.js output.
Cause
Miscommunication. When designing this version of Citation.js, a different date format was taken into account, as that was the documented format back then.
Expected solution
Solution isn't trivial, but not really complex either. Just a lot of time making sure everything is updated. Problem expected to be fixed in
v0.3.0-11
.Thanks to @austinjp for uncovering this issue in #52 and to @fbennett for helping with the citeproc-js side of things.
The text was updated successfully, but these errors were encountered: