From df52d57a1d9b05d78afb88c6e72cbf72e2ce59da Mon Sep 17 00:00:00 2001
From: Paulo Gustavo Veiga <pveiga@wisemapping.com>
Date: Thu, 30 Aug 2012 20:43:56 -0300
Subject: [PATCH] Bug  java.lang.NullPointerException 	at
 com.wisemapping.exporter.FreemindExporter.addNote(FreemindExporter.java:170)
 fixed.

---
 wise-editor/src/main/webapp/samples/issue.xml | 120 ++++++++++++++++++
 .../src/main/webapp/samples/sample3.xml       |  41 ++----
 .../src/main/webapp/samples/sample4.xml       |  10 ++
 .../exporter/FreemindExporter.java            |   5 +
 .../src/test/resources/data/wisemaps/bug2.xml |  49 +++++++
 .../resources/data/wisemaps/cdata-support.mmr |   8 ++
 .../data/wisemaps/cdata-support.wxml          |   4 +-
 7 files changed, 202 insertions(+), 35 deletions(-)
 create mode 100644 wise-webapp/src/test/resources/data/wisemaps/cdata-support.mmr

diff --git a/wise-editor/src/main/webapp/samples/issue.xml b/wise-editor/src/main/webapp/samples/issue.xml
index e69de29b..e957fa3d 100644
--- a/wise-editor/src/main/webapp/samples/issue.xml
+++ b/wise-editor/src/main/webapp/samples/issue.xml
@@ -0,0 +1,120 @@
+<map name="70866" version="tango">
+    <topic central="true" text="La computadora" shape="rounded rectagle" id="1" fontStyle="Verdana;15;#feffff;bold;;"
+           bgColor="#cc0000" brColor="#660000">
+        <topic position="220,0" order="0" shape="rounded rectagle" id="21" fontStyle=";10;#feffff;bold;;"
+               bgColor="#a64d79" brColor="#4c1130">
+            <text><![CDATA[Hardware
+(componentes físicos)]]></text>
+            <topic position="386,-29" order="0" shape="rounded rectagle" id="25" fontStyle=";8;#feffff;;;"
+                   bgColor="#c27ba0" brColor="#4c1130">
+                <text><![CDATA[Entrada de datos
+]]></text>
+                <topic position="558,-34" order="0" shape="rounded rectagle" id="28" fontStyle=";8;#000000;;;"
+                       bgColor="#ead1dc" brColor="#4c1130">
+                    <text><![CDATA[ Ratón, Teclado, Joystick,
+Cámara digital, Micrófono, Escáner.]]></text>
+                </topic>
+            </topic>
+            <topic position="383,0" order="1" text="Salida de datos" shape="rounded rectagle" id="29"
+                   fontStyle=";8;#feffff;;;" bgColor="#c27ba0" brColor="#4c1130">
+                <topic position="554,0" order="0" shape="rounded rectagle" id="30" fontStyle=";8;#000000;;;"
+                       bgColor="#ead1dc" brColor="#4c1130">
+                    <text><![CDATA[Monitor, Impresora, Bocinas, Plóter.
+]]></text>
+                </topic>
+            </topic>
+            <topic position="387,40" order="2" text="Almacenamiento" shape="rounded rectagle" id="31"
+                   fontStyle=";8;#feffff;;;" bgColor="#c27ba0" brColor="#4c1130">
+                <topic position="553,35" order="0" shape="rounded rectagle" id="32" fontStyle=";8;#000000;;;"
+                       bgColor="#ead1dc" brColor="#4c1130">
+                    <text><![CDATA[Disquete, Disco compacto, DVD,
+BD, Disco duro, Memoria flash.]]></text>
+                </topic>
+            </topic>
+        </topic>
+        <topic position="-314,-72" order="1" shape="rectagle" id="59" fontStyle=";8;#000000;bold;;" bgColor="#bf9000"
+               brColor="#7f6000">
+            <text><![CDATA[Software
+(Programas y datos con los que funciona la computadora)
+]]></text>
+            <topic position="-664,-145" order="0" shape="rectagle" id="92" fontStyle=";8;#000000;normal;;"
+                   bgColor="#f1c232" brColor="#7f6000">
+                <text><![CDATA[Software de Sistema:Permite el entendimiento
+entre el usuario y la maquina.]]></text>
+                <topic position="-883,-174" order="0" text="Microsoft Windows" shape="rectagle" id="101"
+                       fontStyle=";8;#000000;;;" bgColor="#ffd966" brColor="#7f6000"/>
+                <topic position="-864,-145" order="1" text="GNU/LINUX" shape="rectagle" id="106"
+                       fontStyle=";8;#000000;;;" bgColor="#ffd966" brColor="#7f6000"/>
+                <topic position="-846,-116" order="2" text="MAC " shape="rectagle" id="107" fontStyle=";8;#000000;;;"
+                       bgColor="#ffd966" brColor="#7f6000"/>
+            </topic>
+            <topic position="-667,-43" order="1" shape="rectagle" id="93" fontStyle=";8;#000000;;;" bgColor="#f1c232"
+                   brColor="#7f6000">
+                <text><![CDATA[Software de Aplicación: Permite hacer hojas de
+calculo navegar en internet, base de datos, etc.]]></text>
+                <topic position="-855,-87" order="0" text="Office" shape="rectagle" id="108" fontStyle=";8;#000000;;;"
+                       bgColor="#ffd966" brColor="#783f04"/>
+                <topic position="-869,-58" order="1" text="Libre Office" shape="rectagle" id="109"
+                       fontStyle=";8;#000000;;;" bgColor="#ffd966" brColor="#7f6000"/>
+                <topic position="-873,-29" order="2" text="Navegadores" shape="rectagle" id="110"
+                       fontStyle=";8;#000000;;;" bgColor="#ffd966" brColor="#7f6000"/>
+                <topic position="-851,0" order="3" text="Msn" shape="rectagle" id="111" fontStyle=";8;#000000;;;"
+                       bgColor="#ffd966" brColor="#783f04"/>
+            </topic>
+            <topic position="-590,29" order="2" shape="rectagle" id="94" fontStyle=";8;#000000;;;" bgColor="#f1c232"
+                   brColor="#7f6000">
+                <text><![CDATA[Software de Desarrollo
+]]></text>
+            </topic>
+        </topic>
+        <topic position="-218,116" order="3" text="Tipos de computadora" shape="elipse" id="3" fontStyle=";10;;bold;;">
+            <topic position="-476,58" order="0" text="Computadora personal de escritorio o Desktop" shape="elipse"
+                   id="8" fontStyle=";8;;bold;;"/>
+            <topic position="-352,87" order="1" shape="elipse" id="10" fontStyle=";8;;bold;;">
+                <text><![CDATA[PDA
+]]></text>
+            </topic>
+            <topic position="-360,116" order="2" text="Laptop" shape="elipse" id="11" fontStyle=";8;;bold;;"/>
+            <topic position="-365,145" order="3" text="Servidor" shape="elipse" id="12" fontStyle=";8;;bold;;"/>
+            <topic position="-368,174" order="4" text="Tablet PC" shape="elipse" id="13" fontStyle=";8;;bold;;"/>
+        </topic>
+    </topic>
+    <topic position="283,192" text="CPU y sus partes internas" shape="rounded rectagle" id="35"
+           fontStyle=";10;#feffff;;;" bgColor="#c27ba0" brColor="#4c1130">
+        <topic position="493,120" order="0" text="Ranuras de expansión o PCI" shape="rounded rectagle" id="36"
+               fontStyle=";8;#000000;;;" bgColor="#ead1dc" brColor="#4c1130"/>
+        <topic position="458,149" order="1" shape="rounded rectagle" id="38" fontStyle=";8;#000000;;;" bgColor="#ead1dc"
+               brColor="#4c1130">
+            <text><![CDATA[Memoria RAM
+]]></text>
+        </topic>
+        <topic position="466,178" order="2" shape="rounded rectagle" id="40" fontStyle=";8;#000000;;;" bgColor="#ead1dc"
+               brColor="#4c1130">
+            <text><![CDATA[Unidades ópticas
+]]></text>
+        </topic>
+        <topic position="457,207" order="3" shape="rounded rectagle" id="41" fontStyle=";8;#000000;;;" bgColor="#ead1dc"
+               brColor="#4c1130">
+            <text><![CDATA[Tarjeta Madre
+]]></text>
+        </topic>
+        <topic position="465,236" order="4" shape="rounded rectagle" id="42" fontStyle=";8;#000000;;;" bgColor="#ead1dc"
+               brColor="#4c1130">
+            <text><![CDATA[Microprocesador
+]]></text>
+        </topic>
+        <topic position="450,265" order="5" shape="rounded rectagle" id="43" fontStyle=";8;#000000;;;" bgColor="#ead1dc"
+               brColor="#4c1130">
+            <text><![CDATA[Disco Duro
+]]></text>
+        </topic>
+    </topic>
+    <topic position="80,-215" shape="rounded rectagle" id="118" fontStyle=";10;#feffff;;;" bgColor="#cc0000"
+           brColor="#660000">
+        <text><![CDATA[Máquina electrónica que sirve para:  escribir,  dibujar,  pintar,
+escuchar música, ver videos, calcular, comunicarnos con otras personas, etc-]]></text>
+    </topic>
+    <relationship srcTopicId="35" destTopicId="21" lineType="3" endArrow="true" startArrow="false"/>
+    <relationship srcTopicId="118" destTopicId="1" lineType="3" endArrow="true" startArrow="false"/>
+    <relationship srcTopicId="118" destTopicId="1" lineType="3" endArrow="true" startArrow="false"/>
+</map>
\ No newline at end of file
diff --git a/wise-editor/src/main/webapp/samples/sample3.xml b/wise-editor/src/main/webapp/samples/sample3.xml
index 41060d97..a5a72ca4 100644
--- a/wise-editor/src/main/webapp/samples/sample3.xml
+++ b/wise-editor/src/main/webapp/samples/sample3.xml
@@ -1,34 +1,9 @@
-<map name="5327">
-    <topic central="true" text="Guia Ezeiza" shape="rounded rectagle" brColor="#1b2863">
-        <topic position="237,-50" text="Directorio" bgColor="#cfe8d4">
-            <icon id="chart_organisation"/>
-            <topic order="0" text="Empresas">
-                <topic order="0" text="Rubro"/>
-                <topic order="1" text="Nombre"/>
-            </topic>
-            <topic order="1" text="Personas">
-                <topic order="0" text="Presidentes"/>
-                <topic order="1" text="Ventas"/>
-                <topic order="2" text="Otros"/>
-            </topic>
-        </topic>
-        <topic position="152,100" text="Social" bgColor="#cfe8d4">
-            <icon id="bulb_light_on"/>
-            <topic order="0" text="Red de contactos (linkedin)"/>
-            <topic order="1" text="Preguntas y respuestas"/>
-            <topic order="2" text="Negocios"/>
-            <topic order="3" text="Compartir intereses"/>
-        </topic>
-        <topic position="-168,100" text="Noticias" bgColor="#cfe8d4">
-            <icon id="money_coins"/>
-            <topic order="0" text="Subscripcion"/>
-            <topic order="1" text="Municipales"/>
-            <topic order="2" text="De interes general"/>
-        </topic>
-        <topic position="-162,-100" text="Eventos" bgColor="#cfe8d4">
-            <icon id="thumb_thumb_up"/>
-            <topic order="0" text="De la comunidad"/>
-            <topic order="1" text="Municipales"/>
-        </topic>
-    </topic>
+<map name="70838" version="tango">
+    <topic central="true" text="Clickview Overview" shape="elipse" id="1"/>
+    <topic position="-260,107" text="Subscription Library" shape="elipse" id="3" fontStyle=";10;#ff0000;bold;;"/>
+    <topic position="1,135" order="0" text="Clickview 24/7" shape="elipse" id="6" fontStyle=";10;#38761d;bold;;"/>
+    <topic position="179,133" order="0" text="Clickview Exchange" shape="elipse" id="9" fontStyle=";10;#741b47;bold;;"/>
+    <topic position="1,65" order="1" text="Components/Services" shape="rectagle" id="2" fontStyle=";10;#0000ff;bold;;"/>
+    <relationship srcTopicId="3" destTopicId="2" lineType="3" endArrow="true" startArrow="false"/>
+    <relationship srcTopicId="6" destTopicId="2" lineType="3" endArrow="true" startArrow="false"/>
 </map>
\ No newline at end of file
diff --git a/wise-editor/src/main/webapp/samples/sample4.xml b/wise-editor/src/main/webapp/samples/sample4.xml
index e69de29b..8053577a 100644
--- a/wise-editor/src/main/webapp/samples/sample4.xml
+++ b/wise-editor/src/main/webapp/samples/sample4.xml
@@ -0,0 +1,10 @@
+<map name="sample3" version="tango">
+    <topic central="true" text="Clickview Overview" shape="elipse" id="1"/>
+    <topic position="-260,107" text="Subscription Library" shape="elipse" id="3" fontStyle=";10;#ff0000;bold;;"/>
+    <topic position="1,135" text="Clickview 24/7" shape="elipse" id="6" fontStyle=";10;#38761d;bold;;"/>
+    <topic position="179,133" text="Clickview Exchange" shape="elipse" id="9" fontStyle=";10;#741b47;bold;;"/>
+    <topic position="1,65" text="Components/Services" shape="rectagle" id="2" fontStyle=";10;#0000ff;bold;;"/>
+    <relationship srcTopicId="3" destTopicId="2" lineType="3" endArrow="true" startArrow="false"/>
+    <relationship srcTopicId="6" destTopicId="2" lineType="3" srcCtrlPoint="-1,-12" destCtrlPoint="-30,25"
+                  endArrow="true" startArrow="false"/>
+</map>
\ No newline at end of file
diff --git a/wise-webapp/src/main/java/com/wisemapping/exporter/FreemindExporter.java b/wise-webapp/src/main/java/com/wisemapping/exporter/FreemindExporter.java
index 06ed9647..1bd0f470 100755
--- a/wise-webapp/src/main/java/com/wisemapping/exporter/FreemindExporter.java
+++ b/wise-webapp/src/main/java/com/wisemapping/exporter/FreemindExporter.java
@@ -174,6 +174,11 @@ public class FreemindExporter
             if (textNote == null || textNote.isEmpty()) {
                 textNote = mindmapTopic.getNote().getText();
             }
+
+            // @Todo: For some reason central topic nodes with CDATA seems not to be loaded in the JAXB model.
+            // Temporally excluding and continue ..
+            textNote = (textNote != null) ? textNote : "";
+
             textNote = textNote.replaceAll("%0A", "\n");
             note.setNAME("accessories/plugins/NodeNote.properties");
             note.setText(textNote);
diff --git a/wise-webapp/src/test/resources/data/wisemaps/bug2.xml b/wise-webapp/src/test/resources/data/wisemaps/bug2.xml
index e69de29b..713e55e2 100644
--- a/wise-webapp/src/test/resources/data/wisemaps/bug2.xml
+++ b/wise-webapp/src/test/resources/data/wisemaps/bug2.xml
@@ -0,0 +1,49 @@
+<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>
+            <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>
+            <topic position="248,165" order="4" text="Paleontología" 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">
+            <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"/>
+        </topic>
+    </topic>
+</map>
\ No newline at end of file
diff --git a/wise-webapp/src/test/resources/data/wisemaps/cdata-support.mmr b/wise-webapp/src/test/resources/data/wisemaps/cdata-support.mmr
new file mode 100644
index 00000000..ff0d661b
--- /dev/null
+++ b/wise-webapp/src/test/resources/data/wisemaps/cdata-support.mmr
@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<map version="0.9.0">
+    <node TEXT="Observation" ID="ID_1">
+        <hook NAME="accessories/plugins/NodeNote.properties">
+            <text></text>
+        </hook>
+    </node>
+</map>
diff --git a/wise-webapp/src/test/resources/data/wisemaps/cdata-support.wxml b/wise-webapp/src/test/resources/data/wisemaps/cdata-support.wxml
index e02150bb..4d262220 100644
--- a/wise-webapp/src/test/resources/data/wisemaps/cdata-support.wxml
+++ b/wise-webapp/src/test/resources/data/wisemaps/cdata-support.wxml
@@ -1,6 +1,6 @@
 <map name="71137" version="tango">
-    <topic central="true" text="Observation " shape="rectagle" id="1">
-        <note><![CDATA[Always ask any questions when you make your observation to find what you are observing or trying to find out]]></note>
+    <topic central="true" text="Observation" shape="rectagle" id="1">
+        <note><![CDATA[Always ask ]]></note>
     </topic>
     <topic position="-4,-107" text="Data Analysis" id="29">
         <note>