From f90cbb1850d5c6f52f8fb07b74bab4b32eab7727 Mon Sep 17 00:00:00 2001
From: Beate Jost <beate.jost@iais.fraunhofer.de>
Date: Wed, 21 Aug 2024 10:34:54 +0200
Subject: [PATCH] Issue #1685: fix for hotfix for old calliope programs with
 calli:bot

---
 .../roberta/util/XsltAndJavaTransformer.java    | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/OpenRobertaRobot/src/main/java/de/fhg/iais/roberta/util/XsltAndJavaTransformer.java b/OpenRobertaRobot/src/main/java/de/fhg/iais/roberta/util/XsltAndJavaTransformer.java
index 53bf79e859..350d5aaf76 100644
--- a/OpenRobertaRobot/src/main/java/de/fhg/iais/roberta/util/XsltAndJavaTransformer.java
+++ b/OpenRobertaRobot/src/main/java/de/fhg/iais/roberta/util/XsltAndJavaTransformer.java
@@ -76,12 +76,15 @@ public Pair<String, String> transform(RobotFactory factory, String programText,
         if ( programText.contains(HIGHEST_XML_VERSION) ) {
             return Pair.of(programText, configText);
         }
+        boolean configMissing;
         String xmlVersion;
         try {
-            if ( configText != null ) {
-                configText = transformXslt(configText);
-            } else {
+            if ( configText == null ) {
+                configMissing = true;
                 configText = factory.getConfigurationDefault();
+            } else {
+                configMissing = false;
+                configText = transformXslt(configText);
             }
             programText = transformXslt(programText);
             BlockSet jaxbProgram = JaxbHelper.xml2Element(programText, BlockSet.class);
@@ -97,14 +100,14 @@ public Pair<String, String> transform(RobotFactory factory, String programText,
             Pair<String, String> progConfPair = transformBetweenVersions(factory, xmlVersion, programText, configText);
             programText = progConfPair == null ? programText : progConfPair.getFirst();
             configText = progConfPair == null ? configText : progConfPair.getSecond();
-            configText = hasXMLChanged(configText, factory.getConfigurationDefault()) ? configText : null;
+            configText = configMissing && isXMLEqual(configText, factory.getConfigurationDefault()) ? null : configText;
             return Pair.of(programText, configText);
         } catch ( Exception e ) {
             throw new DbcException("Java transformation failed", e);
         }
     }
 
-    private boolean hasXMLChanged(String xml1, String xml2) {
+    private boolean isXMLEqual(String xml1, String xml2) {
         DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance();
         dbf.setNamespaceAware(true);
         dbf.setCoalescing(true);
@@ -118,9 +121,9 @@ private boolean hasXMLChanged(String xml1, String xml2) {
             inputStream = new ByteArrayInputStream(xml2.getBytes());
             Document doc2 = db.parse(inputStream);
             doc2.normalizeDocument();
-            return !doc1.isEqualNode(doc2);
+            return doc1.isEqualNode(doc2);
         } catch ( IOException | SAXException | ParserConfigurationException e ) {
-            return false; // if the comparison fails, we assume it has not changed
+            return true; // if the comparison fails, we assume they are equal to provide the valid default config
         }
     }