Repository: camel
Updated Branches:
  refs/heads/camel-2.18.x ba7fa060d -> 6cb200c09
  refs/heads/master 0da97136b -> 371a22e39


CAMEL-10509: Fixed model helper to work without camel context which was 
optiona. Such as camel-swagger-java uses that.


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/371a22e3
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/371a22e3
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/371a22e3

Branch: refs/heads/master
Commit: 371a22e39c7ed7f78444d46a77b8d0768d3bcd88
Parents: 9e685b1
Author: Claus Ibsen <davscl...@apache.org>
Authored: Thu Dec 1 11:35:36 2016 +0100
Committer: Claus Ibsen <davscl...@apache.org>
Committed: Thu Dec 1 12:23:22 2016 +0100

----------------------------------------------------------------------
 .../org/apache/camel/model/ModelHelper.java     | 58 ++++++++++++++++----
 1 file changed, 48 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/371a22e3/camel-core/src/main/java/org/apache/camel/model/ModelHelper.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/model/ModelHelper.java 
b/camel-core/src/main/java/org/apache/camel/model/ModelHelper.java
index a1d30fd..db40f63 100644
--- a/camel-core/src/main/java/org/apache/camel/model/ModelHelper.java
+++ b/camel-core/src/main/java/org/apache/camel/model/ModelHelper.java
@@ -38,6 +38,7 @@ import org.w3c.dom.Node;
 import org.apache.camel.CamelContext;
 import org.apache.camel.Expression;
 import org.apache.camel.NamedNode;
+import org.apache.camel.TypeConversionException;
 import org.apache.camel.converter.jaxp.XmlConverter;
 import org.apache.camel.model.language.ExpressionDefinition;
 import org.apache.camel.spi.NamespaceAware;
@@ -83,7 +84,14 @@ public final class ModelHelper {
         StringWriter buffer = new StringWriter();
         marshaller.marshal(definition, buffer);
 
-        Document dom = context.getTypeConverter().convertTo(Document.class, 
buffer.toString());
+        XmlConverter xmlConverter = newXmlConverter(context);
+        String xml = buffer.toString();
+        Document dom;
+        try {
+            dom = xmlConverter.toDOMDocument(xml, null);
+        } catch (Exception e) {
+            throw new TypeConversionException(xml, Document.class, e);
+        }
 
         // Add additional namespaces to the document root element
         Element documentElement = dom.getDocumentElement();
@@ -92,9 +100,6 @@ public final class ModelHelper {
         }
 
         // We invoke the type converter directly because we need to pass some 
custom XML output options
-        TypeConverterRegistry registry = context.getTypeConverterRegistry();
-        XmlConverter xmlConverter = 
registry.getInjector().newInstance(XmlConverter.class);
-
         Properties outputProperties = new Properties();
         outputProperties.put(OutputKeys.INDENT, "yes");
         outputProperties.put(OutputKeys.STANDALONE, "yes");
@@ -103,7 +108,6 @@ public final class ModelHelper {
         } catch (TransformerException e) {
             throw new IllegalStateException("Failed converting document object 
to string", e);
         }
-
     }
 
     /**
@@ -116,7 +120,7 @@ public final class ModelHelper {
      * @throws javax.xml.bind.JAXBException is thrown if error unmarshalling 
from xml to model
      */
     public static <T extends NamedNode> T createModelFromXml(CamelContext 
context, String xml, Class<T> type) throws JAXBException {
-        return modelToXml(context, xml, type);
+        return modelToXml(context, null, xml, type);
     }
 
     /**
@@ -129,7 +133,7 @@ public final class ModelHelper {
      * @throws javax.xml.bind.JAXBException is thrown if error unmarshalling 
from xml to model
      */
     public static <T extends NamedNode> T createModelFromXml(CamelContext 
context, InputStream stream, Class<T> type) throws JAXBException {
-        return modelToXml(context, stream, type);
+        return modelToXml(context, stream, null, type);
     }
 
     /**
@@ -142,8 +146,10 @@ public final class ModelHelper {
     public static RoutesDefinition loadRoutesDefinition(CamelContext context, 
InputStream inputStream) throws Exception {
         JAXBContext jaxbContext = getJAXBContext(context);
 
+        XmlConverter xmlConverter = newXmlConverter(context);
+        Document dom = xmlConverter.toDOMDocument(inputStream, null);
+
         Map<String, String> namespaces = new LinkedHashMap<>();
-        Document dom = 
context.getTypeConverter().mandatoryConvertTo(Document.class, inputStream);
         extractNamespaces(dom, namespaces);
 
         Binder<Node> binder = jaxbContext.createBinder();
@@ -172,11 +178,25 @@ public final class ModelHelper {
         return answer;
     }
 
-    private static <T extends NamedNode> T modelToXml(CamelContext context, 
Object object, Class<T> type) throws JAXBException {
+    private static <T extends NamedNode> T modelToXml(CamelContext context, 
InputStream is, String xml, Class<T> type) throws JAXBException {
         JAXBContext jaxbContext = getJAXBContext(context);
 
+        XmlConverter xmlConverter = newXmlConverter(context);
+        Document dom = null;
+        try {
+            if (is != null) {
+                dom = xmlConverter.toDOMDocument(is, null);
+            } else if (xml != null) {
+                dom = xmlConverter.toDOMDocument(xml, null);
+            }
+        } catch (Exception e) {
+            throw new TypeConversionException(xml, Document.class, e);
+        }
+        if (dom == null) {
+            throw new IllegalArgumentException("InputStream and XML is both 
null");
+        }
+
         Map<String, String> namespaces = new LinkedHashMap<>();
-        Document dom = context.getTypeConverter().convertTo(Document.class, 
object);
         extractNamespaces(dom, namespaces);
 
         Binder<Node> binder = jaxbContext.createBinder();
@@ -284,4 +304,22 @@ public final class ModelHelper {
             }
         }
     }
+
+    /**
+     * Creates a new {@link XmlConverter}
+     *
+     * @param context CamelContext if provided
+     * @return a new XmlConverter instance
+     */
+    private static XmlConverter newXmlConverter(CamelContext context) {
+        XmlConverter xmlConverter;
+        if (context != null) {
+            TypeConverterRegistry registry = 
context.getTypeConverterRegistry();
+            xmlConverter = 
registry.getInjector().newInstance(XmlConverter.class);
+        } else {
+            xmlConverter = new XmlConverter();
+        }
+        return xmlConverter;
+    }
+
 }

Reply via email to