rufuspollock-okfn / bibserver

BibServer is open-source software what makes it easy to publish, manage and find bibliographies. BibServer is RESTful and web-friendly.
MIT License
126 stars 34 forks source link

Bugfix: ,\n in author list #247

Closed sciunto closed 9 years ago

sciunto commented 12 years ago

Hi,

First of all, many thanks for your bibtex parser, it's awesome and I use it for my own project.

Nevertheless, I found a bug.

Consider the following bibtex snippet

 @ARTICLE{Vigil1994,
author = {Vigil, Gene and Xu, Zhenghe and Steinberg, Suzi and Israelachvili,
 Jacob},
title = {Interactions of Silica Surfaces},
journal = {Journal of Colloid and Interface Science},
year = {1994},
volume = {165},
pages = {367--385},
}

The parser fails. The last author name is 'IsraelachviliJacob'. The coma is gone.

This is because in line 96, ',\n' is splitted

kvs = [i.strip() for i in record.split(',\n')]

I suggest to add the coma back in the case 'inkey' because we know that in such case, the comma was not a separator, but a real coma.

François.

rufuspollock commented 10 years ago

@markmacgillivray ping :-)