mistakes in XSD in transitionCondition and joinCondition syntax description

[Alex Yiu <alex.yiu@oracle.com>]

Hi, all editors,

When Yaron and I were cooking the formal proposal for Issue 103, we found a syntax mistake about joinCondition in Section 6.2:

========================
<targets>?           
   <target linkName="ncname">+
      <joinCondition expressionLanguage="anyURI"?>?
        ... bool-expr ...
      </joinCondition>
   </target>
</targets>
<sources>?
   <source linkName="ncname">+
      <transitionCondition expressionLanguage="anyURI"?>?
        ... bool-expr ...
      </transitionCondition>
   </source>
</sources>
========================

It is not in sync with the rest of the spec.

I guess it should become:
========================
<targets>?           
   <joinCondition expressionLanguage="anyURI"?>?
        ... bool-expr ...
   </joinCondition>
   <target linkName="ncname"/>+
</targets>
<sources>?
   <source linkName="ncname">+
      <transitionCondition expressionLanguage="anyURI"?>?
        ... bool-expr ...
      </transitionCondition>
   </source>
</sources>
========================

Also, in the XSD, I believe the "minOccurs" is missing in XSD for joinCondition and transitionCondition:
========================
    <complexType name="tTargets">
        <complexContent>
            <extension base="bpws:tExtensibleElements">
                <sequence>
                    <element name="joinCondition" type="bpws:tCondition" minOccurs="0"/>
                    <element name="target" type="bpws:tTarget" maxOccurs="unbounded"/>
                </sequence>
            </extension>
        </complexContent>
    </complexType>
========================
========================
    <complexType name="tSource">
        <complexContent>
            <extension base="bpws:tExtensibleElements">
                <sequence>
                    <element name="transitionCondition" type="bpws:tCondition" minOccurs="0"/>
                </sequence>
                <attribute name="linkName" type="NCName" use="required"/>
            </extension>
        </complexContent>
========================


Is this change big enough to file a bug issue ?


Thanks!


Regards,
Alex Yiu