Project

General

Profile

bug #8628

Add ORCID as person identifier

Added by Andreas Müller 23 days ago. Updated 9 days ago.

Status:
Closed
Priority:
New
Category:
cdm
Target version:
Start date:
10/29/2019
Due date:
% Done:

50%

Severity:
normal
Found in Version:
Tags:

Description

WGB:

wir sollten darüber nachdenken, wie wir ORCID Identifier mit Personen verbinden - das wäre ein zusätzliches Attribut, aber wie können wir das auswerten? Siehe z.B. https://bloodhound-tracker.net/

AM:

We need to discuss if can handle ORCIDS as simple Identifier with Identifiertype "ORCID" such as other identifiers or if we want to integrate it into the Person class as explicit field.


Related issues

Related to Edit - feature request #8668: Show Person.orcid in dataportal New 11/08/2019
Precedes Edit - feature request #8667: Make Person.orcid editable in TaxEditor Closed 10/30/2019

Associated revisions

Revision 6e1b0ff9 (diff)
Added by Andreas Müller 13 days ago

fix #8628 add ORCID as person identifier

Revision 58a6ce02 (diff)
Added by Andreas Müller 13 days ago

fix #8628 update TaxEditor test DB to Person.orcid

Revision 9d0322b2 (diff)
Added by Andreas Müller 13 days ago

fix #8628 update Vaadin test DB to Person.orcid

Revision 535525fc (diff)
Added by Andreas Müller 13 days ago

fix #8628 update cdm.map.ser to Person.orcid

Revision e82373d1 (diff)
Added by Andreas Müller 9 days ago

ref #8628 Improve error messages for ORCID parsing

History

#1 Updated by Andreas Müller 13 days ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50

#2 Updated by Andreas Müller 13 days ago

  • Assignee changed from Andreas Müller to Katja Luther
  • Target version changed from CDM UML 5.13 to Release 5.11

This should be fixed now. Please review.

#3 Updated by Andreas Müller 13 days ago

#4 Updated by Andreas Müller 13 days ago

#5 Updated by Katja Luther 10 days ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Katja Luther to Andreas Müller

Could you add an example in the exception message to show how it should look like. And that an x at the end is possible, too.

#6 Updated by Andreas Müller 10 days ago

This is difficult because multiple formats are possible (with http, https, only orcid.org or completely missing all prefix - https://orcid.org/, with hyphenw between numbers and without, with checksum or without). This all is recognized by the parser so users may think if giving an example that only one exact format is supported.

But I may try to improve the error messages.

#7 Updated by Andreas Müller 9 days ago

  • Status changed from Feedback to Resolved
  • Assignee changed from Andreas Müller to Katja Luther

I adapted the error messages a bit. Can you please review again?

#8 Updated by Katja Luther 9 days ago

  • Status changed from Resolved to Closed
  • Assignee changed from Katja Luther to Andreas Müller

this message is easier to understand how the identifier should look like

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)