search

AAVLD-USAHA-ITStandards / eCVI

eCVI Data Exchange Standard (Starting with version 2)
12 stars 9 forks source link

Blank OtherOfficialID #30

Closed mkm1879 closed 3 years ago

mkm1879 commented 5 years ago

Just noticed that "" validates as a value for OtherOfficialID. We probably should follow the NAHLN and define a NonBlankString type for this and other required string elements and attributes.

alexturnerdvm commented 5 years ago

This should be such that no one leaves it blank on purpose so that if it is blank, then it was a mistake rather than omitted to get around the system somehow.

mkm1879 commented 5 years ago

We could copy the approach used by NAHLN for "NonNullString". It requires at least one non-whitespace character.

mkm1879 commented 4 years ago

Will add this solution to a draft in pending changes. What other fields should be defined as NonNullString?

YdaMitzyTorres commented 4 years ago

This blank issue is on SpeciesOther as well

mkm1879 commented 4 years ago

And ManagementID, EquineDescription, almost any required attribute now using xs:string. Any exceptions?

mmcgrath commented 4 years ago

see https://github.com/AAVLD-USAHA-ITStandards/eCVI/commit/92fa9e5bbf525a3a4132068efa5f4f95c033b0ae for the proposed implementation

mkm1879 commented 3 years ago

In Jan 2021 release v2.3