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

Adding from DOI does not generate timestamp #3074

Closed
a-torgovitsky opened this issue Aug 3, 2017 · 3 comments
Closed

Adding from DOI does not generate timestamp #3074

a-torgovitsky opened this issue Aug 3, 2017 · 3 comments
Labels
component: fetcher [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs status: waiting-for-feedback The submitter or other users need to provide more information about the issue

Comments

@a-torgovitsky
Copy link

JabRef version
JabRef 4.0-dev--snapshot--2017-08-03--master--e9fbf1963
Linux 4.4.0-21-generic amd64
Java 1.8.0_144
on
Ubuntu 14.04

Steps to reproduce:

Check on Options --> Preferences --> General --> Mark new entries with addition date
Check on Options --> Preferences --> General --> Overwrite
Field name: timestamp

  1. Add an entry by DOI. I have tried several and the same behavior happens. Here is a sample: 10.1002/jae.2557

  2. Entry is added, but timestamp is not populated.

I have confirmed that adding an entry manually still creates a timestamp.

@tobiasdiez tobiasdiez added [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs component: fetcher labels Aug 3, 2017
@stefan-kolb
Copy link
Member

Thank you for your report 👍
This should be fixed in current master. Please try the latest build from http://builds.jabref.org/fetcher-timestamp.

@stefan-kolb stefan-kolb added the status: waiting-for-feedback The submitter or other users need to provide more information about the issue label Aug 10, 2017
tobiasdiez pushed a commit that referenced this issue Aug 10, 2017
* Remove redundancy

* Update timestamp when entry is fetched #3074

* Unused import

* Move to own preferences class

* Checkstyle is a pain in the ***
@tobiasdiez
Copy link
Member

This should be fixed in the latest development version. Could you please check the build from http://builds.jabref.org/master/. Thanks!

@a-torgovitsky
Copy link
Author

I checked the latest master build and the behavior is as expected now. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: fetcher [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs status: waiting-for-feedback The submitter or other users need to provide more information about the issue
Projects
None yet
Development

No branches or pull requests

3 participants