Include PMID in PubMed imports

[ryan-carpenter]

The MEDLINE importer does not include PubMed IDs (PMID) when importing from PubMed text files.

This is problematic for two reasons:

1. PMID is an important identifier
2. JabRef includes PMID when importing via API, so the behavior is inconsistent

PubMed records also have an NLM unique identifier that JabRef imports, but this is not the same as the PMID and does not substitute for it. MEDLINE records produced by sources other than PubMed tend to include the PMID and not the NLM identifier, so the PMID is more useful for merging and deduplicating entries in JabRef. The PMID can also be used to visit the record on PubMed, fetch additional data, and so on.

The MEDLINE importer should include the PMID element from pubmed/medline files.

See MEDLINE/PubMed Data Element (Field) Descriptions for information about the elements included in PubMed text files.

[ryan-carpenter]

jabref/src/main/java/org/jabref/logic/importer/fileformat/MedlinePlainImporter.java

Lines 145 to 168 in 7e4e7d3

	Map<String, Field> hashMap = new HashMap<>();
	hashMap.put("PG", StandardField.PAGES);
	hashMap.put("PL", StandardField.ADDRESS);
	hashMap.put("PHST", new UnknownField("history"));
	hashMap.put("PST", new UnknownField("publication-status"));
	hashMap.put("VI", StandardField.VOLUME);
	hashMap.put("LA", StandardField.LANGUAGE);
	hashMap.put("PUBM", new UnknownField("model"));
	hashMap.put("RN", new UnknownField("registry-number"));
	hashMap.put("NM", new UnknownField("substance-name"));
	hashMap.put("OCI", new UnknownField("copyright-owner"));
	hashMap.put("CN", new UnknownField("corporate"));
	hashMap.put("IP", StandardField.ISSUE);
	hashMap.put("EN", StandardField.EDITION);
	hashMap.put("GS", new UnknownField("gene-symbol"));
	hashMap.put("GN", StandardField.NOTE);
	hashMap.put("GR", new UnknownField("grantno"));
	hashMap.put("SO", new UnknownField("source"));
	hashMap.put("NR", new UnknownField("number-of-references"));
	hashMap.put("SFM", new UnknownField("space-flight-mission"));
	hashMap.put("STAT", new UnknownField("status"));
	hashMap.put("SB", new UnknownField("subset"));
	hashMap.put("OTO", new UnknownField("termowner"));
	hashMap.put("OWN", StandardField.OWNER);

[ryan-carpenter]

Is this all it would take to resolve the issue?

hashMap.put("PMID", new UnknownField("pmid")); 

[Siedlerchr]

Yes, that should be enough. You can do it through the github UI (just click on Edit File). You might need to adjust the MedlinePlainImporterTest as well if it fails but CI will tell you that :)