merge honors header

jpos · Mar 27, 2024 · f49ebf1 · f49ebf1
1 parent 0c26276
commit f49ebf1
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 7 deletions.
diff --git a/jpos/src/main/java/org/jpos/iso/ISOMsg.java b/jpos/src/main/java/org/jpos/iso/ISOMsg.java
@@ -836,11 +836,18 @@ public ISOMsg clone(String ... fpaths) {
     }
 
     /**
-     * add all fields present on received parameter to this ISOMsg<br>
-     * please note that received fields take precedence over
-     * existing ones (simplifying card agent message creation
-     * and template handling)
-     * @param m ISOMsg to merge
+     * Merges the content of the specified ISOMsg into this ISOMsg instance.
+     * It iterates over the fields of the input message and, for each field that is present,
+     * sets the corresponding component in this message to the value from the input message.
+     * This operation includes all fields that are present in the input message, but does not remove
+     * any existing fields from this message unless they are explicitly overwritten by the input message.
+     * <p>
+     * If the input message contains a header (non-null), this method also clones the header
+     * and sets it as the header of this message.
+     *
+     * @param m The ISOMsg to merge into this ISOMsg. It must not be {@code null}.
+     *          The method does nothing if {@code m} is {@code null}.
+     *
      */
     @SuppressWarnings("PMD.EmptyCatchBlock")
     public void merge (ISOMsg m) {
@@ -852,6 +859,8 @@ public void merge (ISOMsg m) {
                 // should never happen
             }
         }
+        if (m.header != null)
+            header = (ISOHeader) m.header.clone();
     }
 
     /**

diff --git a/jpos/src/test/java/org/jpos/iso/ISOMsg2Test.java b/jpos/src/test/java/org/jpos/iso/ISOMsg2Test.java
@@ -629,12 +629,14 @@ public void testMerge1() throws Throwable {
         ISOMsg iSOMsg = new ISOMsg("testISOMsgMti");
         ISOMsg m = new ISOMsg();
         m.recalcBitMap();
-        byte[] value = new byte[0];
-        m.set(3, value);
+        m.set(3, "000000");
+        m.setHeader("ISOHEADER".getBytes());
         iSOMsg.merge(m);
         assertEquals(2, iSOMsg.fields.size(), "iSOMsg.fields.size()");
         assertEquals(3, iSOMsg.maxField, "iSOMsg.maxField");
         assertTrue(iSOMsg.dirty, "iSOMsg.dirty");
+        assertEquals("000000", iSOMsg.getString(3));
+        assertArrayEquals("ISOHEADER".getBytes(), iSOMsg.getHeader());
     }
 
     @Test