remove any duplicates in nested groupings when generating MessageFact…

…ory#create factory method some vendors use the same repeating groups in different parts of a message conditional on the value of other fields e.g. different product types
jonfreedman · Jan 31, 2018 · 5f45a39 · 5f45a39
1 parent 9b46d28
commit 5f45a39
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/quickfixj-codegenerator/src/main/resources/org/quickfixj/codegenerator/MessageFactory.xsl b/quickfixj-codegenerator/src/main/resources/org/quickfixj/codegenerator/MessageFactory.xsl
@@ -75,7 +75,7 @@ public class MessageFactory implements quickfix.MessageFactory {
 
 <xsl:template name="group-switch-statement">
 		switch (msgType) {
-	<xsl:for-each select="//fix/messages/message[group or component]">
+	<xsl:for-each-group select="//fix/messages/message[group or component]" group-by="@name">
 			case <xsl:value-of select="$messagePackage"/>.<xsl:value-of select="@name"/>.MSGTYPE:
 				switch (correspondingFieldID) {
 			<xsl:apply-templates mode="group-factories" select="group">
@@ -86,7 +86,7 @@ public class MessageFactory implements quickfix.MessageFactory {
 			</xsl:apply-templates>
 				}
 				break;
-	</xsl:for-each>
+	</xsl:for-each-group>
 		}
 </xsl:template>