andmer / quickfix-messenger

Automatically exported from code.google.com/p/quickfix-messenger
0 stars 1 forks source link

Wrong sequence of repeating group fields #44

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Select a FIX 4.2 session
2. Create a NewOrderList (E)
3. Populate all required fields
4. Send

What is the expected output? What do you see instead?
Receiver should accept message. Receiver rejected message due to incorrect 
sequence.
Check parser.

Please use labels and text to provide additional information.

Original issue reported on code.google.com by jramoyo on 7 Oct 2012 at 5:10

GoogleCodeExporter commented 9 years ago
Attached fix -- Underlying model does not consider field sequence at all.
Added a new Class called MemberOrder which will be used to as a key to the 
sorted map.

Original comment by jramoyo on 8 Oct 2012 at 10:04

Attachments:

GoogleCodeExporter commented 9 years ago
Applied changes

Original comment by jramoyo on 8 Oct 2012 at 12:34

GoogleCodeExporter commented 9 years ago
Reopened -- actual message send is still in the wrong order

Original comment by jramoyo on 8 Oct 2012 at 1:48

GoogleCodeExporter commented 9 years ago
Used Goup.setField(int i, Field<?> field) instead.

Original comment by jramoyo on 8 Oct 2012 at 4:18