Bug in BibTeX-importer

The importer code currently reads:
else if(field == "fjournal") {
if(item.publicationTitle) {
// move publicationTitle to abbreviation
item.journalAbbreviation = value;
}
item.publicationTitle = value;
}
but the comment indicates the more sensible:
else if(field == "fjournal") {
if(item.publicationTitle) {
// move publicationTitle to abbreviation
item.journalAbbreviation = item.publicationTitle;
}
item.publicationTitle = value;
}
Still, this doesn't work if the input for some reason has duplicate fields (two JOURNAL doesn't mean the first one is an abbreviation). AFAIC, BibTeX ignores any duplicate entries, like:
Warning--I'm ignoring ABC's extra "journal" field.
Hence, I'd suggest changing the assignment of fields that can't be repeated (including, but not limited to JOURNAL) so that any duplicate field is ignored with a warning; first field survives. JOURNAL would then always go to journalAbbreviation, FJOURNAL to publicationTitle. The possible swapping would then go after all fields have been parsed. (This issue is less important though: the input is invalid.)

An entirely different issue, but I don't know where to report it: it's a pretty bad idea to have the forum's password-reset form show the user's email address, such a thing only makes spammers happy. (Sure, it might be useful if you have a million different email accounts, but which is more important?)
  • dstillman Zotero Team
    An entirely different issue, but I don't know where to report it: it's a pretty bad idea to have the forum's password-reset form show the user's email address, such a thing only makes spammers happy. (Sure, it might be useful if you have a million different email accounts, but which is more important?)
    Good call. I've modified the forum software not to do this.

This is an old discussion that has not been active in a long time. Instead of commenting here, you should start a new discussion. If you think the content of this discussion is still relevant, you can link to it from your new discussion.