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

"Normalize to BibTeX name format" should help when manually importing entries. #1181

Closed
koppor opened this issue Apr 12, 2016 · 6 comments
Closed
Assignees

Comments

@koppor
Copy link
Member

koppor commented Apr 12, 2016

JabRef as of 2016-04-12, 39f7ed7

Steps:

  1. Use ISBN to BibTeX fetcher with 978-3-642-02373-6
  2. Go to required fields
  3. Editor is empty
  4. Go to http://www.springer.com/us/book/9783642023736
  5. Copy the editors string Ali Babar, M., Dingsøyr, T., Lago, P., van der Vliet, H.
  6. Right click in the editors field and select "Normalize to BibTeX name format"
  7. Result: Ali Babar, M., Dingsøyr T. Lago P. van der Vliet H.

Expected: Babar, Ali and M. and Dingsøyr, T. and Lago, P. and van der Vliet, H.

JabRef 3.2: Babar, Ali and M. and Dingsøyr and T. and Lago and P. and van der Vliet and H.

Refs #318 and #950

@tobiasdiez
Copy link
Member

The comma as a separator of author names was removed in #1149. It is just not clear how to consistently guess the right naming schema. Just replace the commas separating authors by semicolons and everything should work as expected.

@koppor
Copy link
Member Author

koppor commented Apr 12, 2016

Ah, great hint. Then, the implementation is easy:

Check

  • only , (no ;, no and)
  • even number of ,

Then, convert every second , to ; and then run the implemented algorithm.

@tobiasdiez
Copy link
Member

But then Name, della, first would be recognized as two names although it is valid BibTeX syntax for one name with vonPart = della.

@koppor
Copy link
Member Author

koppor commented Apr 13, 2016

Oh, I see it at http://maverick.inria.fr/~Xavier.Decoret/resources/xdkbibtex/bibtex_summary.html, section "Names specifications". According to that explanation della would be the Jr part, not the von part. Reading that and also assuming that I never saw three commas, I would accept that this "helper" formats 1% of the cases wrong.

@koppor koppor added the stupro label Apr 13, 2016
bruehldev added a commit to bruehldev/jabref that referenced this issue Jun 2, 2016
…ted names with colons only work now

- Added test for odd count of commas
- Additionally added test for correct caching of getAsLastFirstNamesWithAnd
@matthiasgeiger
Copy link
Member

matthiasgeiger commented Jun 20, 2016

-- sorry comment should be placed in the PR

bruehldev added a commit to bruehldev/jabref that referenced this issue Jun 27, 2016
… format"

Added the jr, sr,... special cases for semicolon partition.
Fixed to avoid the "and", "{", ";" cases.
Added Test for every case.
bruehldev added a commit to bruehldev/jabref that referenced this issue Aug 15, 2016
… format"

Added the jr, sr,... special cases for semicolon partition.
Fixed to avoid the "and", "{", ";" cases.
Added Test for every case.
bruehldev added a commit to bruehldev/jabref that referenced this issue Aug 22, 2016
… format"

Added the jr, sr,... special cases for semicolon partition.
Fixed to avoid the "and", "{", ";" cases.
Added Test for every case.
bruehldev added a commit to bruehldev/jabref that referenced this issue Sep 1, 2016
… format"

Added the jr, sr,... special cases for semicolon partition.
Fixed to avoid the "and", "{", ";" cases.
Added Test for every case.
bruehldev added a commit to bruehldev/jabref that referenced this issue Sep 6, 2016
… format"

Added the jr, sr,... special cases for semicolon partition.
Fixed to avoid the "and", "{", ";" cases.
Added Test for every case.
bruehldev added a commit to bruehldev/jabref that referenced this issue Sep 8, 2016
… format"

Added the jr, sr,... special cases for semicolon partition.
Fixed to avoid the "and", "{", ";" cases.
Added Test for every case.
@koppor
Copy link
Member Author

koppor commented Sep 12, 2016

Fixed by #1470

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants