You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2014/12/02 19:44:34 UTC

[1/4] camel git commit: CAMEL-8099: Add support for using default values in Camel property placeholder.:

Repository: camel
Updated Branches:
  refs/heads/camel-2.14.x 0c55f5600 -> f7b4bedf2
  refs/heads/master 39ccf5d6d -> c4d6eb333


CAMEL-8099: Add support for using default values in Camel property placeholder.:


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

Branch: refs/heads/camel-2.14.x
Commit: f7b4bedf282905264399786027df09b9d329bd90
Parents: e8abd63
Author: Claus Ibsen <da...@apache.org>
Authored: Tue Dec 2 17:30:53 2014 +0100
Committer: Claus Ibsen <da...@apache.org>
Committed: Tue Dec 2 19:44:16 2014 +0100

----------------------------------------------------------------------
 .../simple/ast/SimpleFunctionExpression.java    | 12 +++++++-
 .../PropertiesComponentGetOrElseTest.java       | 29 ++++++++++++++++++++
 .../PropertiesComponentSimpleLanguageTest.java  |  6 ++--
 3 files changed, 43 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/f7b4bedf/camel-core/src/main/java/org/apache/camel/language/simple/ast/SimpleFunctionExpression.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/language/simple/ast/SimpleFunctionExpression.java b/camel-core/src/main/java/org/apache/camel/language/simple/ast/SimpleFunctionExpression.java
index 4eedcba..d5b648a 100644
--- a/camel-core/src/main/java/org/apache/camel/language/simple/ast/SimpleFunctionExpression.java
+++ b/camel-core/src/main/java/org/apache/camel/language/simple/ast/SimpleFunctionExpression.java
@@ -157,7 +157,17 @@ public class SimpleFunctionExpression extends LiteralExpression {
         if (remainder != null) {
             String[] parts = remainder.split(":");
             if (parts.length > 2) {
-                throw new SimpleParserException("Valid syntax: ${properties:[locations]:key} was: " + function, token.getIndex());
+                throw new SimpleParserException("Valid syntax: ${properties:key[:default]} was: " + function, token.getIndex());
+            }
+            return ExpressionBuilder.propertiesComponentExpression(remainder, null);
+        }
+
+        // properties-location: prefix
+        remainder = ifStartsWithReturnRemainder("properties-location:", function);
+        if (remainder != null) {
+            String[] parts = remainder.split(":");
+            if (parts.length > 2) {
+                throw new SimpleParserException("Valid syntax: ${properties-location:location:key[:default]} was: " + function, token.getIndex());
             }
 
             String locations = null;

http://git-wip-us.apache.org/repos/asf/camel/blob/f7b4bedf/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
----------------------------------------------------------------------
diff --git a/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
index e5e5d12..46db7df 100644
--- a/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
+++ b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
@@ -62,6 +62,35 @@ public class PropertiesComponentGetOrElseTest extends ContextTestSupport {
         assertMockEndpointsSatisfied();
     }
 
+    public void testPropertiesComponentSimpleLanguage() throws Exception {
+        context.addRoutes(new RouteBuilder() {
+            @Override
+            public void configure() throws Exception {
+                from("direct:start")
+                        .transform().simple("Hi ${body} do you think ${properties:cool.name} rocks?");
+            }
+        });
+        context.start();
+
+        String reply = template.requestBody("direct:start", "Claus", String.class);
+        assertEquals("Hi Claus do you think Camel rocks?", reply);
+    }
+
+    public void testPropertiesComponentSimpleLanguageUsingDefaultValue() throws Exception {
+        context.addRoutes(new RouteBuilder() {
+            @Override
+            public void configure() throws Exception {
+                from("direct:start")
+                        .transform().simple("Hi ${body} do you think ${properties:unknown:Beer} rocks?");
+            }
+        });
+        context.start();
+
+        String reply = template.requestBody("direct:start", "Claus", String.class);
+        assertEquals("Hi Claus do you think Beer rocks?", reply);
+    }
+
+
     @Override
     protected CamelContext createCamelContext() throws Exception {
         CamelContext context = super.createCamelContext();

http://git-wip-us.apache.org/repos/asf/camel/blob/f7b4bedf/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentSimpleLanguageTest.java
----------------------------------------------------------------------
diff --git a/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentSimpleLanguageTest.java b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentSimpleLanguageTest.java
index 860a1ed..80ed133 100644
--- a/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentSimpleLanguageTest.java
+++ b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentSimpleLanguageTest.java
@@ -65,7 +65,7 @@ public class PropertiesComponentSimpleLanguageTest extends ContextTestSupport {
             @Override
             public void configure() throws Exception {
                 from("direct:start")
-                    .transform().simple("Hi ${body}. ${properties:org/apache/camel/component/properties/bar.properties:bar.quote}.");
+                    .transform().simple("Hi ${body}. ${properties-location:org/apache/camel/component/properties/bar.properties:bar.quote}.");
             }
         });
         context.start();
@@ -80,7 +80,7 @@ public class PropertiesComponentSimpleLanguageTest extends ContextTestSupport {
             @Override
             public void configure() throws Exception {
                 from("direct:start")
-                    .transform().simple("Hi ${body}. ${properties:org/apache/camel/component/properties/bar.properties:bar.quote}.");
+                    .transform().simple("Hi ${body}. ${properties-location:org/apache/camel/component/properties/bar.properties:bar.quote}.");
             }
         });
         context.start();
@@ -95,7 +95,7 @@ public class PropertiesComponentSimpleLanguageTest extends ContextTestSupport {
             @Override
             public void configure() throws Exception {
                 from("direct:start")
-                    .transform().simple("Hi ${body}. ${properties:org/apache/camel/component/properties/bar.properties,"
+                    .transform().simple("Hi ${body}. ${properties-location:org/apache/camel/component/properties/bar.properties,"
                         + "org/apache/camel/component/properties/cheese.properties:cheese.quote}.");
             }
         });


[2/4] camel git commit: CAMEL-8099: Add support for using default values in Camel property placeholder.

Posted by da...@apache.org.
CAMEL-8099: Add support for using default values in Camel property placeholder.


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

Branch: refs/heads/camel-2.14.x
Commit: e8abd636f12c423e9f6f1ad319084aca0e254570
Parents: 0c55f56
Author: Claus Ibsen <da...@apache.org>
Authored: Tue Dec 2 17:03:19 2014 +0100
Committer: Claus Ibsen <da...@apache.org>
Committed: Tue Dec 2 19:44:16 2014 +0100

----------------------------------------------------------------------
 .../properties/DefaultPropertiesParser.java     | 15 ++++
 .../PropertiesComponentGetOrElseTest.java       | 72 ++++++++++++++++++++
 2 files changed, 87 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/e8abd636/camel-core/src/main/java/org/apache/camel/component/properties/DefaultPropertiesParser.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/component/properties/DefaultPropertiesParser.java b/camel-core/src/main/java/org/apache/camel/component/properties/DefaultPropertiesParser.java
index 16ef52d..6a5b80f 100644
--- a/camel-core/src/main/java/org/apache/camel/component/properties/DefaultPropertiesParser.java
+++ b/camel-core/src/main/java/org/apache/camel/component/properties/DefaultPropertiesParser.java
@@ -22,6 +22,7 @@ import java.util.Set;
 
 import static java.lang.String.format;
 
+import org.apache.camel.util.ObjectHelper;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -32,6 +33,8 @@ import org.slf4j.LoggerFactory;
 public class DefaultPropertiesParser implements AugmentedPropertyNameAwarePropertiesParser {
     protected final Logger log = LoggerFactory.getLogger(getClass());
 
+    private static final String GET_OR_ELSE_TOKEN = ":";
+
     @Override
     public String parseUri(String text, Properties properties, String prefixToken, String suffixToken) throws IllegalArgumentException {
         return parseUri(text, properties, prefixToken, suffixToken, null, null, false);
@@ -189,6 +192,13 @@ public class DefaultPropertiesParser implements AugmentedPropertyNameAwareProper
          * @return Value of the property with the given key
          */
         private String getPropertyValue(String key, String input) {
+            // they key may have a get or else expression
+            String defaultValue = null;
+            if (key.contains(GET_OR_ELSE_TOKEN)) {
+                defaultValue = ObjectHelper.after(key, GET_OR_ELSE_TOKEN);
+                key = ObjectHelper.before(key, GET_OR_ELSE_TOKEN);
+            }
+
             String augmentedKey = getAugmentedKey(key);
             boolean shouldFallback = fallbackToUnaugmentedProperty && !key.equals(augmentedKey);
 
@@ -198,6 +208,11 @@ public class DefaultPropertiesParser implements AugmentedPropertyNameAwareProper
                 value = doGetPropertyValue(key);
             }
 
+            if (value == null && defaultValue != null) {
+                log.debug("Property with key [{}] not found, using default value: {}", augmentedKey, defaultValue);
+                value = defaultValue;
+            }
+
             if (value == null) {
                 StringBuilder esb = new StringBuilder();
                 esb.append("Property with key [").append(augmentedKey).append("] ");

http://git-wip-us.apache.org/repos/asf/camel/blob/e8abd636/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
----------------------------------------------------------------------
diff --git a/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
new file mode 100644
index 0000000..e5e5d12
--- /dev/null
+++ b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
@@ -0,0 +1,72 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.properties;
+
+import org.apache.camel.CamelContext;
+import org.apache.camel.ContextTestSupport;
+import org.apache.camel.builder.RouteBuilder;
+
+/**
+ * @version 
+ */
+public class PropertiesComponentGetOrElseTest extends ContextTestSupport {
+
+    @Override
+    public boolean isUseRouteBuilder() {
+        return false;
+    }
+
+    public void testPropertiesComponentFoundKey() throws Exception {
+        context.addRoutes(new RouteBuilder() {
+            @Override
+            public void configure() throws Exception {
+                from("direct:start").to("properties:{{cool.end:mock:wrong}}");
+            }
+        });
+        context.start();
+
+        getMockEndpoint("mock:result").expectedMessageCount(1);
+
+        template.sendBody("direct:start", "Hello World");
+
+        assertMockEndpointsSatisfied();
+    }
+
+    public void testPropertiesComponentUseDefaultValue() throws Exception {
+        context.addRoutes(new RouteBuilder() {
+            @Override
+            public void configure() throws Exception {
+                from("direct:start").to("properties:{{unknown:mock:result}}");
+            }
+        });
+        context.start();
+
+        getMockEndpoint("mock:result").expectedMessageCount(1);
+
+        template.sendBody("direct:start", "Hello World");
+
+        assertMockEndpointsSatisfied();
+    }
+
+    @Override
+    protected CamelContext createCamelContext() throws Exception {
+        CamelContext context = super.createCamelContext();
+        context.addComponent("properties", new PropertiesComponent("classpath:org/apache/camel/component/properties/myproperties.properties"));
+        return context;
+    }
+
+}


[3/4] camel git commit: CAMEL-8099: Add support for using default values in Camel property placeholder.

Posted by da...@apache.org.
CAMEL-8099: Add support for using default values in Camel property placeholder.


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

Branch: refs/heads/master
Commit: 703edaddf00dcc38a15fd3547cbd13f79056c8fa
Parents: 39ccf5d
Author: Claus Ibsen <da...@apache.org>
Authored: Tue Dec 2 17:03:19 2014 +0100
Committer: Claus Ibsen <da...@apache.org>
Committed: Tue Dec 2 19:44:25 2014 +0100

----------------------------------------------------------------------
 .../properties/DefaultPropertiesParser.java     | 15 ++++
 .../PropertiesComponentGetOrElseTest.java       | 72 ++++++++++++++++++++
 2 files changed, 87 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/703edadd/camel-core/src/main/java/org/apache/camel/component/properties/DefaultPropertiesParser.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/component/properties/DefaultPropertiesParser.java b/camel-core/src/main/java/org/apache/camel/component/properties/DefaultPropertiesParser.java
index 16ef52d..6a5b80f 100644
--- a/camel-core/src/main/java/org/apache/camel/component/properties/DefaultPropertiesParser.java
+++ b/camel-core/src/main/java/org/apache/camel/component/properties/DefaultPropertiesParser.java
@@ -22,6 +22,7 @@ import java.util.Set;
 
 import static java.lang.String.format;
 
+import org.apache.camel.util.ObjectHelper;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -32,6 +33,8 @@ import org.slf4j.LoggerFactory;
 public class DefaultPropertiesParser implements AugmentedPropertyNameAwarePropertiesParser {
     protected final Logger log = LoggerFactory.getLogger(getClass());
 
+    private static final String GET_OR_ELSE_TOKEN = ":";
+
     @Override
     public String parseUri(String text, Properties properties, String prefixToken, String suffixToken) throws IllegalArgumentException {
         return parseUri(text, properties, prefixToken, suffixToken, null, null, false);
@@ -189,6 +192,13 @@ public class DefaultPropertiesParser implements AugmentedPropertyNameAwareProper
          * @return Value of the property with the given key
          */
         private String getPropertyValue(String key, String input) {
+            // they key may have a get or else expression
+            String defaultValue = null;
+            if (key.contains(GET_OR_ELSE_TOKEN)) {
+                defaultValue = ObjectHelper.after(key, GET_OR_ELSE_TOKEN);
+                key = ObjectHelper.before(key, GET_OR_ELSE_TOKEN);
+            }
+
             String augmentedKey = getAugmentedKey(key);
             boolean shouldFallback = fallbackToUnaugmentedProperty && !key.equals(augmentedKey);
 
@@ -198,6 +208,11 @@ public class DefaultPropertiesParser implements AugmentedPropertyNameAwareProper
                 value = doGetPropertyValue(key);
             }
 
+            if (value == null && defaultValue != null) {
+                log.debug("Property with key [{}] not found, using default value: {}", augmentedKey, defaultValue);
+                value = defaultValue;
+            }
+
             if (value == null) {
                 StringBuilder esb = new StringBuilder();
                 esb.append("Property with key [").append(augmentedKey).append("] ");

http://git-wip-us.apache.org/repos/asf/camel/blob/703edadd/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
----------------------------------------------------------------------
diff --git a/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
new file mode 100644
index 0000000..e5e5d12
--- /dev/null
+++ b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
@@ -0,0 +1,72 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.properties;
+
+import org.apache.camel.CamelContext;
+import org.apache.camel.ContextTestSupport;
+import org.apache.camel.builder.RouteBuilder;
+
+/**
+ * @version 
+ */
+public class PropertiesComponentGetOrElseTest extends ContextTestSupport {
+
+    @Override
+    public boolean isUseRouteBuilder() {
+        return false;
+    }
+
+    public void testPropertiesComponentFoundKey() throws Exception {
+        context.addRoutes(new RouteBuilder() {
+            @Override
+            public void configure() throws Exception {
+                from("direct:start").to("properties:{{cool.end:mock:wrong}}");
+            }
+        });
+        context.start();
+
+        getMockEndpoint("mock:result").expectedMessageCount(1);
+
+        template.sendBody("direct:start", "Hello World");
+
+        assertMockEndpointsSatisfied();
+    }
+
+    public void testPropertiesComponentUseDefaultValue() throws Exception {
+        context.addRoutes(new RouteBuilder() {
+            @Override
+            public void configure() throws Exception {
+                from("direct:start").to("properties:{{unknown:mock:result}}");
+            }
+        });
+        context.start();
+
+        getMockEndpoint("mock:result").expectedMessageCount(1);
+
+        template.sendBody("direct:start", "Hello World");
+
+        assertMockEndpointsSatisfied();
+    }
+
+    @Override
+    protected CamelContext createCamelContext() throws Exception {
+        CamelContext context = super.createCamelContext();
+        context.addComponent("properties", new PropertiesComponent("classpath:org/apache/camel/component/properties/myproperties.properties"));
+        return context;
+    }
+
+}


[4/4] camel git commit: CAMEL-8099: Add support for using default values in Camel property placeholder.:

Posted by da...@apache.org.
CAMEL-8099: Add support for using default values in Camel property placeholder.:


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

Branch: refs/heads/master
Commit: c4d6eb33364ed951aee446bd25d9551060f43540
Parents: 703edad
Author: Claus Ibsen <da...@apache.org>
Authored: Tue Dec 2 17:30:53 2014 +0100
Committer: Claus Ibsen <da...@apache.org>
Committed: Tue Dec 2 19:44:25 2014 +0100

----------------------------------------------------------------------
 .../simple/ast/SimpleFunctionExpression.java    | 12 +++++++-
 .../PropertiesComponentGetOrElseTest.java       | 29 ++++++++++++++++++++
 .../PropertiesComponentSimpleLanguageTest.java  |  6 ++--
 3 files changed, 43 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/c4d6eb33/camel-core/src/main/java/org/apache/camel/language/simple/ast/SimpleFunctionExpression.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/language/simple/ast/SimpleFunctionExpression.java b/camel-core/src/main/java/org/apache/camel/language/simple/ast/SimpleFunctionExpression.java
index 4eedcba..d5b648a 100644
--- a/camel-core/src/main/java/org/apache/camel/language/simple/ast/SimpleFunctionExpression.java
+++ b/camel-core/src/main/java/org/apache/camel/language/simple/ast/SimpleFunctionExpression.java
@@ -157,7 +157,17 @@ public class SimpleFunctionExpression extends LiteralExpression {
         if (remainder != null) {
             String[] parts = remainder.split(":");
             if (parts.length > 2) {
-                throw new SimpleParserException("Valid syntax: ${properties:[locations]:key} was: " + function, token.getIndex());
+                throw new SimpleParserException("Valid syntax: ${properties:key[:default]} was: " + function, token.getIndex());
+            }
+            return ExpressionBuilder.propertiesComponentExpression(remainder, null);
+        }
+
+        // properties-location: prefix
+        remainder = ifStartsWithReturnRemainder("properties-location:", function);
+        if (remainder != null) {
+            String[] parts = remainder.split(":");
+            if (parts.length > 2) {
+                throw new SimpleParserException("Valid syntax: ${properties-location:location:key[:default]} was: " + function, token.getIndex());
             }
 
             String locations = null;

http://git-wip-us.apache.org/repos/asf/camel/blob/c4d6eb33/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
----------------------------------------------------------------------
diff --git a/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
index e5e5d12..46db7df 100644
--- a/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
+++ b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentGetOrElseTest.java
@@ -62,6 +62,35 @@ public class PropertiesComponentGetOrElseTest extends ContextTestSupport {
         assertMockEndpointsSatisfied();
     }
 
+    public void testPropertiesComponentSimpleLanguage() throws Exception {
+        context.addRoutes(new RouteBuilder() {
+            @Override
+            public void configure() throws Exception {
+                from("direct:start")
+                        .transform().simple("Hi ${body} do you think ${properties:cool.name} rocks?");
+            }
+        });
+        context.start();
+
+        String reply = template.requestBody("direct:start", "Claus", String.class);
+        assertEquals("Hi Claus do you think Camel rocks?", reply);
+    }
+
+    public void testPropertiesComponentSimpleLanguageUsingDefaultValue() throws Exception {
+        context.addRoutes(new RouteBuilder() {
+            @Override
+            public void configure() throws Exception {
+                from("direct:start")
+                        .transform().simple("Hi ${body} do you think ${properties:unknown:Beer} rocks?");
+            }
+        });
+        context.start();
+
+        String reply = template.requestBody("direct:start", "Claus", String.class);
+        assertEquals("Hi Claus do you think Beer rocks?", reply);
+    }
+
+
     @Override
     protected CamelContext createCamelContext() throws Exception {
         CamelContext context = super.createCamelContext();

http://git-wip-us.apache.org/repos/asf/camel/blob/c4d6eb33/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentSimpleLanguageTest.java
----------------------------------------------------------------------
diff --git a/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentSimpleLanguageTest.java b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentSimpleLanguageTest.java
index 860a1ed..80ed133 100644
--- a/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentSimpleLanguageTest.java
+++ b/camel-core/src/test/java/org/apache/camel/component/properties/PropertiesComponentSimpleLanguageTest.java
@@ -65,7 +65,7 @@ public class PropertiesComponentSimpleLanguageTest extends ContextTestSupport {
             @Override
             public void configure() throws Exception {
                 from("direct:start")
-                    .transform().simple("Hi ${body}. ${properties:org/apache/camel/component/properties/bar.properties:bar.quote}.");
+                    .transform().simple("Hi ${body}. ${properties-location:org/apache/camel/component/properties/bar.properties:bar.quote}.");
             }
         });
         context.start();
@@ -80,7 +80,7 @@ public class PropertiesComponentSimpleLanguageTest extends ContextTestSupport {
             @Override
             public void configure() throws Exception {
                 from("direct:start")
-                    .transform().simple("Hi ${body}. ${properties:org/apache/camel/component/properties/bar.properties:bar.quote}.");
+                    .transform().simple("Hi ${body}. ${properties-location:org/apache/camel/component/properties/bar.properties:bar.quote}.");
             }
         });
         context.start();
@@ -95,7 +95,7 @@ public class PropertiesComponentSimpleLanguageTest extends ContextTestSupport {
             @Override
             public void configure() throws Exception {
                 from("direct:start")
-                    .transform().simple("Hi ${body}. ${properties:org/apache/camel/component/properties/bar.properties,"
+                    .transform().simple("Hi ${body}. ${properties-location:org/apache/camel/component/properties/bar.properties,"
                         + "org/apache/camel/component/properties/cheese.properties:cheese.quote}.");
             }
         });