[Quickfix-developers] bug in parsing groups

[Alok L. <al...@ra...>]

All,

	I think I discovered a bug in parsing groups.  In
Message::setGroup, there's a call to DataDictionary::getGroup in which the
delimiter is set to the first field that is within the group.
However, this assumes the first field in the group will always be
present, which is false:

<group name="NoMyGroup" required="N" >
<field name="Foo1" required="N" />
<field name="Foo2" required="N" />
</group>

In the above, the delimiter is set to whatever tag # is associated with
"Foo1" but since it clearly isn't required, there's no guarantee it'll
appear within a message.  If it doesn't appear, then "Foo2" won't be
parsed as part of the group and that causes the body length to be
incorrect resulting in an invalid FIX.  Please let me know if you
agree/disagree or are unclear about what I've just mentioned.  Thanks.

Alok