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

"Cannot generate preview" #8381

Closed
2 tasks done
wujastyk opened this issue Jan 3, 2022 · 3 comments · Fixed by #8532
Closed
2 tasks done

"Cannot generate preview" #8381

wujastyk opened this issue Jan 3, 2022 · 3 comments · Fixed by #8532
Labels
entry-preview status: depends-on-external A bug or issue that depends on an update of an external library

Comments

@wujastyk
Copy link

wujastyk commented Jan 3, 2022

JabRef version

Latest development branch build (please note build date below)

Operating system

GNU / Linux

Details on version and operating system

JabRef 5.5--2022-01-03--aac66f5 Linux 5.13.0-22-generic amd64 Java 16.0.2 JavaFX 17.0.1+1

Checked with the latest development build

  • I made a backup of my libraries before testing the latest development version.
  • I have tested the latest development version and the problem persists

Steps to reproduce the behaviour

  1. I pulled in the following new article entry using its DOI
@Article{baks-2021,
  author           = {Sandipan Baksi},
  date             = {2021},
  journaltitle     = {The Indian Economic {\&} Social History Review},
  pages            = {001946462110645},
  title            = {Science journalism in Hindi in pre-independence India: A study of Hindi periodicals},
  doi              = {10.1177/00194646211064586},
  creationdate     = {2022-01-03T11:59:38},
  modificationdate = {2022-01-03T12:01:49},
  publisher        = {{SAGE} Publications},
}
  1. I got this error message from the preview tab:
    image
  2. I tried several preview styles: it made no difference.
  3. Some testing showed that if I removed that long number from the pages field, everything behaved normally and the preview was fine.

Appendix

No response

@ThiloteE
Copy link
Member

ThiloteE commented Jan 3, 2022

The long number seems part of the DOI as most of it overlaps. Imagine a document having so many pages. It actually is good that the error emerged and you noticed that the data is not realistic. The author of the article should be noticed.

@wujastyk
Copy link
Author

wujastyk commented Jan 3, 2022

I disagree that this is a good error. One should be able to put any numerical string in the page field. There are some truly weird publications out there.

If JabRef really wants to flag a problem field value, then flag it. But throwing a preview error is just confusing for everyone.

The error is in the DOI data delivery, and I've no idea who to talk to about that (doi.org? SAGE, the journal publishers?). Nor do I have the resources to take up that task. I'm sure the author has no idea how to fix this either.

@Siedlerchr
Copy link
Member

Thanks for opening the issue. I checked the DOI, it's coming directly from the publisher's site. Some publishers seem to abuse the pages field for other information.
The error is not really from JabRef, but from the underlying citation styles, where it indeed created an error.
Details of errors are always logged in the error console.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
entry-preview status: depends-on-external A bug or issue that depends on an update of an external library
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants