elifesciences / elife-pubmed-feed

code to support uploading feeds to pubmed for POA articles and VOR articles
1 stars 4 forks source link

CoiStatement with inline formatting tags #71

Closed gnott closed 6 years ago

gnott commented 6 years ago

First run of the new PubMed library today, and looking at the files produced. All seem to be valid.

One for the PubMed deposit for https://cdn.elifesciences.org/articles/31065/elife-31065-v2.xml, in the <CoiStatement> I noticed an escaped tag.

The XML has this

<fn fn-type="COI-statement" id="conf2">
<p>
Senior editor,
<italic>eLife</italic>
</p>
</fn>

and the PubMed deposit CoiStatement ends up like this:

<CoiStatement>DK, NS, JL No competing interests declared, DS Senior editor, &lt;italic&gt;eLife&lt;/italic&gt;</CoiStatement>

The escaped italic tag turns into the "&lt;italic&gt;" part.

@Melissa37 do you think we should strip all tags from the CoiStatement? Or, should <italic> be turned into <i> tags? I tried the <i> tag and it is still valid according to the PubMed citation checker. I do not see the CoiStatement displayed on the pubmed page, so I'm not sure what the allowed formats are regarding inline tagging.

Melissa37 commented 6 years ago

screen shot 2018-01-19 at 08 43 56 This is what displays on the pubmed site. I suggest we strip all italics and escaped tags?