From 41211110447adc0da99f062ad4ab5b3350266c05 Mon Sep 17 00:00:00 2001
From: Ezequiel-Vega <vegaezequiel51@gmail.com>
Date: Thu, 17 Mar 2022 10:40:11 -0300
Subject: [PATCH] Corrections in files expect

---
 .../src/components/import/FreemindImporter.ts | 17 +++--
 .../test/unit/import/expected/bug2.wxml       | 75 +++++++++----------
 2 files changed, 44 insertions(+), 48 deletions(-)

diff --git a/packages/mindplot/src/components/import/FreemindImporter.ts b/packages/mindplot/src/components/import/FreemindImporter.ts
index ece2cd7c..476578eb 100644
--- a/packages/mindplot/src/components/import/FreemindImporter.ts
+++ b/packages/mindplot/src/components/import/FreemindImporter.ts
@@ -17,6 +17,8 @@ import FreemindArrowLink from '../export/freemind/Arrowlink';
 import VersionNumber from '../export/freemind/importer/VersionNumber';
 import FreemindIconConverter from './FreemindIconConverter';
 import NoteModel from '../model/NoteModel';
+import FeatureModelFactory from '../model/FeatureModelFactory';
+import FeatureModel from '../model/FeatureModel';
 
 export default class FreemindImporter extends Importer {
   private mindmap: Mindmap;
@@ -61,7 +63,7 @@ export default class FreemindImporter extends Importer {
 
     this.nodesmap.set(freeNode.getId(), wiseTopic);
 
-    this.convertChildNodes(freeNode, wiseTopic, this.mindmap, 1, wiseTopicId);
+    this.convertChildNodes(freeNode, wiseTopic, this.mindmap, 1);
     this.addRelationship(this.mindmap);
 
     this.mindmap.setDescription(description);
@@ -153,7 +155,7 @@ export default class FreemindImporter extends Importer {
     if (folded) wiseTopic.setChildrenShrunken(folded);
   }
 
-  private convertChildNodes(freeParent: FreemindNode, wiseParent: NodeModel, mindmap: Mindmap, depth: number, nodeId: number): void {
+  private convertChildNodes(freeParent: FreemindNode, wiseParent: NodeModel, mindmap: Mindmap, depth: number): void {
     const freeChilden = freeParent.getArrowlinkOrCloudOrEdge();
     let currentWiseTopic: NodeModel = wiseParent;
     let order = 0;
@@ -188,9 +190,7 @@ export default class FreemindImporter extends Importer {
         // Convert the rest of the node properties...
         this.convertNodeProperties(child, wiseChild, false);
 
-        if (child.getArrowlinkOrCloudOrEdge().length > 0) {
-          this.convertChildNodes(child, wiseChild, mindmap, depth++, nodeId);
-        }
+        this.convertChildNodes(child, wiseChild, mindmap, depth++);
 
         if (wiseChild !== wiseParent) {
           wiseParent.append(wiseChild);
@@ -239,7 +239,7 @@ export default class FreemindImporter extends Importer {
         if (type === 'NOTE') {
           // Formating text
           const text = this.html2Text(child.getHtml());
-          const noteModel: NoteModel = new NoteModel({ text: text || FreemindConstant.EMPTY_NOTE });
+          const noteModel: FeatureModel = FeatureModelFactory.createModel('note', { text: text || FreemindConstant.EMPTY_NOTE });
           currentWiseTopic.addFeature(noteModel);
         }
       }
@@ -309,6 +309,8 @@ export default class FreemindImporter extends Importer {
       result = TopicShape.ROUNDED_RECT;
     } else if (node.getBackgorundColor()) {
       result = TopicShape.RECTANGLE;
+    } else {
+      result = TopicShape.LINE;
     }
     return result;
   }
@@ -378,7 +380,7 @@ export default class FreemindImporter extends Importer {
       }
     } else {
       const position = wiseParent.getPosition();
-      y = position.y - ((childrenCount / 2) * FreemindConstant.SECOND_LEVEL_TOPIC_HEIGHT - (order * FreemindConstant.SECOND_LEVEL_TOPIC_HEIGHT));
+      y = Math.round(position.y - ((childrenCount / 2) * FreemindConstant.SECOND_LEVEL_TOPIC_HEIGHT - (order * FreemindConstant.SECOND_LEVEL_TOPIC_HEIGHT)));
     }
 
     return {
@@ -389,7 +391,6 @@ export default class FreemindImporter extends Importer {
 
   private html2Text(content: string): string {
     const contentConvert = content.replace(/(<([^>]+)>)/gi, '');
-    contentConvert.replace('\t', '');
     return contentConvert.trim();
   }
 }
diff --git a/packages/mindplot/test/unit/import/expected/bug2.wxml b/packages/mindplot/test/unit/import/expected/bug2.wxml
index 713e55e2..4e7394c4 100644
--- a/packages/mindplot/test/unit/import/expected/bug2.wxml
+++ b/packages/mindplot/test/unit/import/expected/bug2.wxml
@@ -1,49 +1,44 @@
 <map name="58984" version="tango">
     <topic central="true" text="SaberMás" id="1">
-        <topic position="271,-39" order="8" text="Utilización de medios de expresión artística, digitales y analógicos"
-               id="5"/>
-        <topic position="-181,-17" order="5" text="Precio también limitado: 100-120?" id="9"/>
-        <topic position="132,165" order="14" text="Talleres temáticos" id="2">
-            <topic position="242,57" order="0" text="Naturaleza" id="13">
-                <topic position="362,57" order="0" text="Animales, Plantas, Piedras" id="17"/>
+        <topic position="200,-350" order="0" text="Utilización de medios de expresión artística, digitales y analógicos" shape="line" id="5"/>
+        <topic position="-290,-75" order="0" text="Precio también limitado: 100-120?" shape="line" id="9"/>
+        <topic position="-380,-75" order="1" text="Talleres temáticos" shape="line" id="2">
+            <topic position="380,-187" order="1" text="Naturaleza" shape="line" id="13">
+                <topic position="-380,-199" order="1" text="Animales, Plantas, Piedras" shape="line" id="17"/>
             </topic>
-            <topic position="245,84" order="1" text="Arqueología" id="21"/>
-            <topic position="236,138" order="3" text="Energía" id="18"/>
-            <topic position="244,192" order="5" text="Astronomía" id="16"/>
-            <topic position="245,219" order="6" text="Arquitectura" id="20"/>
-            <topic position="234,246" order="7" text="Cocina" id="11"/>
-            <topic position="234,273" order="8" text="Poesía" id="24"/>
-            <topic position="256,111" order="2" text="Culturas Antiguas" id="25">
-                <topic position="378,111" order="0" text="Egipto, Grecia, China..." id="26"/>
+            <topic position="470,-162" order="2" text="Arqueología" shape="line" id="21"/>
+            <topic position="560,-137" order="3" text="Energía" shape="line" id="18"/>
+            <topic position="650,-112" order="4" text="Astronomía" shape="line" id="16"/>
+            <topic position="740,-87" order="5" text="Arquitectura" shape="line" id="20"/>
+            <topic position="830,-62" order="6" text="Cocina" shape="line" id="11"/>
+            <topic position="920,-37" order="7" text="Poesía" shape="line" id="24"/>
+            <topic position="1010,-12" order="8" text="Culturas Antiguas" shape="line" id="25">
+                <topic position="-1010,-24" order="1" text="Egipto, Grecia, China..." shape="line" id="26"/>
             </topic>
-            <topic position="248,165" order="4" text="Paleontología" id="38"/>
+            <topic position="1100,13" order="9" text="Paleontología" shape="line" id="38"/>
         </topic>
-        <topic position="-168,-49" order="3" text="Duración limitada: 5-6 semanas" id="6"/>
-        <topic position="-181,16" order="7" text="Niños y niñas que quieren saber más" id="7"/>
-        <topic position="-184,-81" order="1" text="Alternativa a otras actividades de ocio" id="8"/>
-        <topic position="255,-6" order="10" text="Uso de la tecnología durante todo el proceso de aprendizaje" id="23"/>
-        <topic position="336,-137" order="2"
-               text="Estructura PBL: aprendemos cuando buscamos respuestas a nuestras propias preguntas " id="3"/>
-        <topic position="238,-105" order="4" text="Trabajo basado en la experimentación y en la investigación" id="4"/>
-        <topic position="-201,48" order="9" text="De 8 a 12 años, sin separación por edades" id="10"/>
-        <topic position="-146,81" order="11" text="Máximo 10/1 por taller" id="19"/>
-        <topic position="211,-72" order="6" text="Actividades centradas en el contexto cercano" id="37"/>
-        <topic position="303,27" order="12"
-               text="Flexibilidad en el uso de las lenguas de trabajo (inglés, castellano, esukara?)" id="22"/>
-        <topic position="206,-220" order="0" text="Complementamos el trabajo de la escuela" shape="rounded rectagle"
-               id="27">
+        <topic position="-470,-25" order="2" text="Duración limitada: 5-6 semanas" shape="line" id="6"/>
+        <topic position="-560,0" order="3" text="Niños y niñas que quieren saber más" shape="line" id="7"/>
+        <topic position="-650,25" order="4" text="Alternativa a otras actividades de ocio" shape="line" id="8"/>
+        <topic position="-740,25" order="5" text="Uso de la tecnología durante todo el proceso de aprendizaje" shape="line" id="23"/>
+        <topic position="-830,50" order="6" text="Estructura PBL: aprendemos cuando buscamos respuestas a nuestras propias preguntas " shape="line" id="3"/>
+        <topic position="-920,75" order="7" text="Trabajo basado en la experimentación y en la investigación" shape="line" id="4"/>
+        <topic position="-1010,125" order="8" text="De 8 a 12 años, sin separación por edades" shape="line" id="10"/>
+        <topic position="-1100,150" order="9" text="Máximo 10/1 por taller" shape="line" id="19"/>
+        <topic position="-1190,150" order="10" text="Actividades centradas en el contexto cercano" shape="line" id="37"/>
+        <topic position="-1280,175" order="11" text="Flexibilidad en el uso de las lenguas de trabajo (inglés, castellano, esukara?)" shape="line" id="22"/>
+        <topic position="-1370,200" order="12" text="Complementamos el trabajo de la escuela" shape="rounded rectagle" id="27">
             <note><![CDATA[Todos los contenidos de los talleres están relacionados con el currículo de la enseñanza básica.
-A diferencia de la práctica tradicional, pretendemos ahondar en el conocimiento partiendo de lo que realmente interesa al niño o niña,
-ayudándole a que encuentre respuesta a las preguntas que él o ella se plantea.
-
-Por ese motivo, SaberMás proyecta estar al lado de los niños que necesitan una motivación extra para entender la escuela y fluir en ella,
- y también al lado de aquellos a quienes la curiosidad y las ganas de saber les lleva más allá.]]></note>
-            <topic position="477,-220" order="2" text="Cada uno va a su ritmo, y cada cual pone sus límites" id="30"/>
-            <topic position="425,-193" order="3" text="Aprendemos todos de todos" id="31"/>
-            <topic position="440,-167" order="4" text="Valoramos lo que hemos aprendido" id="33"/>
-            <topic position="468,-273" order="0" text="SaberMás trabaja con, desde y para la motivación" shape="line"
-                   id="28"/>
-            <topic position="458,-247" order="1" text="Trabajamos en equipo en nuestros proyectos " id="32"/>
+                        A diferencia de la práctica tradicional, pretendemos ahondar en el conocimiento partiendo de lo que realmente interesa al niño o niña,
+                        ayudándole a que encuentre respuesta a las preguntas que él o ella se plantea.
+                        
+                        Por ese motivo, SaberMás proyecta estar al lado de los niños que necesitan una motivación extra para entender la escuela y fluir en ella,
+                        y también al lado de aquellos a quienes la curiosidad y las ganas de saber les lleva más allá.]]></note>
+            <topic position="1370,138" order="1" text="Cada uno va a su ritmo, y cada cual pone sus límites" shape="line" id="30"/>
+            <topic position="1460,163" order="2" text="Aprendemos todos de todos" shape="line" id="31"/>
+            <topic position="1550,188" order="3" text="Valoramos lo que hemos aprendido" shape="line" id="33"/>
+            <topic position="1640,213" order="4" text="SaberMás trabaja con, desde y para la motivación" shape="line" id="28"/>
+            <topic position="1730,238" order="5" text="Trabajamos en equipo en nuestros proyectos " shape="line" id="32"/>
         </topic>
     </topic>
 </map>
\ No newline at end of file