* Rafael Laboissiere <[EMAIL PROTECTED]> [2008-11-26 17:22]:

> * David Bremner <[EMAIL PROTECTED]> [2008-11-26 10:47]:
> 
> > Thanks very much for your report. At the moment I am holding off on
> > uploads to unstable of non-RC fixes because the Lenny-freeze. Let me
> > know if you think this merits an upload to experimental (I normally
> > hesitate to bother sponsors with experimental uploads).
> 
> No, it is not that urgent, although the bug is annoying (at list for me, who
> download frequently publication records from Pubmed and translate them into
> BibTeX using bibutils). At any rate, if you wish I would gladly upload the
> package for you.

Actually, I came across some XML records downloaded from the Pubmed site
which use the ForeName name instead of FirstName.  I do not remember now how
I was getting records with the FirstName tag.  

At any rate, the changed patch attached below can cope with both tags.

-- 
Rafael
Use FirstName instead of ForeName tag in the Author record, which is the
correct SGML tag name according to the NCBI Medline format specification
(see http://xml.coverpages.org/ncbiPubFormat.html).

 -- Rafael Laboissiere <[EMAIL PROTECTED]>  Wed, 26 Nov 2008 13:34:04 +0100

--- a/lib/medin.c
+++ b/lib/medin.c
@@ -201,8 +201,8 @@
 /* <AuthorList CompleteYN="Y">
  *    <Author>
  *        <LastName>Barondeau</LastName>
- *        <ForeName>David P</ForeName>
- *        <Initials>DP</Initials>
+ *        <FirstName>David P</FirstName>
+ *        (or <ForeName>David P</ForeName>)
  *    </Author>
  * </AuthorList>
  */
@@ -216,7 +216,7 @@
 			newstr_prepend( name, node->value->data );
 		}
 		else newstr_strcat( name, node->value->data );
-	} else if ( xml_tagexact( node, "ForeName" ) ) {
+	} else if ( xml_tagexact( node, "FirstName" ) || xml_tagexact( node, "ForeName" ) ) {
 		p = node->value->data;
 		while ( p && *p ) {
 			if ( name->len ) newstr_addchar( name, '|' );

Reply via email to