Bug #624
closed
eml-methods/methodsType needs clarification on choice/sequence
Added by David Blankman about 22 years ago.
Updated about 22 years ago.
Category:
eml - general bugs
Description
The current model for methodsType is a repetable choice of methodSteps (1 or
more) or sampling (0 or 1) or qualityControl (0 or more). This should be either:
1. a repeteable choice of methodStep or qualityControl or sampling
or
2. Sequence of methodSteps (1 or more), sampling (0 or 1?), qualityControl (0 or
more?).
We need to get some agreement on which model to use.
I recomend that we go with option 1.
Files
Proposed fix of MethodsType without name change.
Proposed changed for MethodsType with name changes
1. ProcedureStepType needs to be fixed. See attached ProcedureStepTypeFix.png
for proposed changes.
2. MethodsType should be changed. While technicly correct the current model is:
methodStep+ | sampling? | qualityControl*. This should probably be changed to
either:
A. methodStep+ | sampling | qualityControl+ (see MethodsType_option1.png)
or
B. methodStep | sampling | qualityControl
I recommend B since the choice is repetable and since both methodStep and
qualityControl are of type ProcedureStepType which allows for subSteps.
I also recommend that methodStep be renamed to method and subStep be renamed to
step (see MethodsType_option3.png). Both methodStep and qualityControl are of
type ProcedureStepType. For consistency we should have either:
A. methodStep and qualityControlStep
or
B. method and qualityControl.
I recommend option B.
David Blankman
Proposed change for MethodsType with name changes.
Im on irc right now with david and have realized that I made an error in my
files that i never caught. The intended design was to be
methodsType(methodStep+,sampling?,qualityControl*)
Made changes to ProcedureStepType and MethodsType. MethodsType model changed
from choice to sequence as indicated in this previous comment:
Additional Comment #6 From Peter McCartney 2002-10-11 12:50 -------
I made an error in my files that i never caught. The intended design was to be
methodsType(methodStep+,sampling?,qualityControl*)
Original Bugzilla ID was 624
Also available in: Atom
PDF
