facilityregistry / ihe

Issue and discussion tracking for IHE profiles related to facility registries (CSD, etc)
0 stars 0 forks source link

IHE: Make Language attribute optional for facilities, specify content #3

Open edjez opened 11 years ago

edjez commented 11 years ago

Summary

Currently CSD requires a mandatory Languages element as part of facilities. Based on the requirements of existing Facility Registry implementations, it is not a required field. In addition, the current spec does not describe the Languages entity and its sub fields.

Issue

The CSD Facility object has the following element:

<?xml version="1.0"?>
<xs:element
maxOccurs="unbounded"
name="facilityLanguage"
type="Languages"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xmlns:xs="http://www.w3.org/2001/XMLSchema"
xmlns:hfp="http://www.w3.org/2001/XMLSchema-hasFacetAndProperty"
>
</xs:element>

where Languages is defined as:

<?xml version="1.0"?>
<xs:complexType
   name="Languages"
   xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
   xmlns:xs="http://www.w3.org/2001/XMLSchema"
   xmlns:hfp="http://www.w3.org/2001/XMLSchema-hasFacetAndProperty">
    <xs:sequence>
        <xs:element name="entityID" type="xs:ID"></xs:element>
        <xs:element name="languageCode" type="xs:string" ></xs:element>
        <xs:element minOccurs="0" name="languageDescription" type="xs:string"></xs:element>
        <xs:element minOccurs="0" name="type" type="xs:string"></xs:element>
        <xs:element minOccurs="0" name="status" type="xs:string"></xs:element>
    </xs:sequence>
</xs:complexType>

Languages is a useful field but has not been identified as a required field in any Facility Registry implementation so far.

http://www.ietf.org/rfc/rfc3066.txt

djritz commented 11 years ago

This was discussed on the 2013-05-31 call. The agreed action will be to relax the facilityLanguage to minOccurs=0 and to add to the front matter of the spec an "open issue" regarding the use cases that will be broken by not having a mandatory language attribute. It will be open to public comment whether it is better to have a mandatory attribute to which one can specify "unknown" as the language or to leave the attribute off and have to figure out how to execute the use case(s) using client side workarounds.

edjez commented 11 years ago

Cool when the new XSD rolls around I'll close this issue too.