Skip to content

Fix the identifiers (the rest of cases that were not caught before) #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 33 commits into
base: issue-85
Choose a base branch
from
Open

Fix the identifiers (the rest of cases that were not caught before) #2

wants to merge 33 commits into from

Conversation

jdvorak001
Copy link

Hello @abollini , there were still some naked numeric IDs left.
I am sorry, the checking command should have been

egrep -o 'id="[^"]*"' samples/openaire_*.xml | sort | uniq -c

So I am suggesting the rest of the changes in this PR.
Referential integrity will be satisfied after applying this patch.

Dragan Ivanovic and others added 30 commits November 29, 2019 12:18
orgunit identifiers added: ROR, GRID, FundRefRegistry
e5240d7 
ISNI identifier moved to cerif-base-identifiers.xsd, used in
person-identifiers.xsd and orgunit-identifiers.xsd
Application of identifier naming convention
41cf208 
ROR -> RORID
FundRefRegistry -> FundRefID
Small type corrections
4eba5fc
change gender urls
0d02074
update funding originator urls
c56a8b8
update organisation funding role url
6da6975
update language urls
7a76abe
update inter output relations urls
9895098
correct InterFundingRelations url (undisplayed character)
c7d2bed
connect url for output-infrastructure relation url
c21c9c9
update vocabulary to use for OpenAccess uri fon funding
90e1596
@jdvorak001
Fix the identifiers (some case were not caught before)
d0d6eb8
@jdvorak001
Merge pull request openaire#82 from lremy/urls-correction
108febf 
@lremy, many thanks for this!
@chenejac
Update orgunit-identifiers.xsd
7dedc94
@jdvorak001
switch the order of imports (openaire#60)
7f8d129
@jdvorak001
add the gender vocabulary label and its terms
900c5ff
@jdvorak001
Corrections openaire#73
68b0e5f 
https://w3id.org/cerif/vocab/OutputFundingRoles#Originator
https://w3id.org/cerif/vocab/OrganisationFundingRoles#Financier
https://w3id.org/cerif/vocab/InterOutputRelations#Reference
https://w3id.org/cerif/vocab/OutputResearchInfrastructureRelations#Generation
@jdvorak001
backport the ROARMAP link (openaire#94)
034b925
@jdvorak001
use InterOutputRelations#Reference for <References>
f0946f2 
instead of InterOutputRelations#Relation
@jdvorak001
openaire#94 translates the effects of the backports on the documentation
9b354e9
@jdvorak001
Merge remote-tracking branch 'chenejac/master'
19e4014
@jdvorak001
openaire#94 backport the regexp change into documentation
8e8464a
@jdvorak001
openaire#76 support multiple xs:documentation/@source
df5b2a2
@jdvorak001
openaire#76 plug in the specific orgunit identifer elements
8775636
@jdvorak001
openaire#76 adapt the examples (FundRefID)
0089a89
@jdvorak001
openaire#94 backport openaire#65
5f41535
@jdvorak001
openaire#78 add the ./tools/compile.sh script and call it from .travi…
c2e8395 
…s.yml
@jdvorak001
openaire#78 checkout also the necessary modules
1655004 
- CERIF-TG-Tools
- openaire-cris-validator
@jdvorak001
openaire#78 introduce an error to be discovered by the validator
716eb6d
@jdvorak001
openaire#78 corrects the error
45938f4 
And uses the more fitting ElectronicAddress for organisations' websites.
jdvorak001 and others added 3 commits November 20, 2020 14:06
@jdvorak001
Make a total compile script (openaire#96)
be7f44b 
* openaire#78 add the ./tools/compile.sh script and call it from .travis.yml
* openaire#78 checkout also the necessary modules (CERIF-TG-Tools, openaire-cris-validator)
* openaire#78 introduce an error to be discovered by the validator
* openaire#78 corrects the error (and uses the more fitting ElectronicAddress for organisations' websites)
@jdvorak001
Merge branch 'issue-78-patch-1'
257abaf
@jdvorak001
Merge branch 'master' into issue-85-patch-2
38e8557 
Conflicts:
	samples/openaire_cerif_xml_example_publications.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jdvorak001 @chenejac