Uploaded image for project: 'QuickFIX/J'
  1. QuickFIX/J
  2. QFJ-158

CxlType field should only show up in FIX40.xml file - it doesn't exist in subsequent FIX versions

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Default
    • Resolution: Fixed
    • Affects Version/s: 1.2.1
    • Fix Version/s: 1.2.1
    • Component/s: Build
    • Labels:
      None

      Description

      According to the FIX specs at http://www.btobits.com/fixopaedia/, the CxlType (125) field only exists in FIX.4.0
      That field is no longer present in anything from 4.1-4.4

      However, the field does show up in FIX41-43.xml files, and as a result the CxlType field that is created uses the definition from FIX43.xml file which doesn't contain the 2 constants that the CxlType field should contain (see http://www.btobits.com/fixopaedia/fixdic40/fd0e-125-3132.html).

      I propose to remove the CxlType field definition from the FIX41-43.xml files to comply with the spec.

      Perhaps there's some other dependency or backwards compatibility issues that i may be missing?

      thoughts?

        Attachments

          Activity

          Hide
          toli Toli Kuznets added a comment -

          fixed in revision 604

          Show
          toli Toli Kuznets added a comment - fixed in revision 604
          Hide
          jthoennes Jörg Thönnes added a comment -

          Hi Toli,

          just checked FPL Fiximate. They confirm this.
          So removing is OK, I see no issues except somebody uses this
          field where he should not.

          Cheers, Jörg

          Show
          jthoennes Jörg Thönnes added a comment - Hi Toli, just checked FPL Fiximate. They confirm this. So removing is OK, I see no issues except somebody uses this field where he should not. Cheers, Jörg

            People

            • Assignee:
              toli Toli Kuznets
              Reporter:
              toli Toli Kuznets
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: