Author: ningjiang
Date: Wed Dec 28 07:50:16 2011
New Revision: 1225137

URL: http://svn.apache.org/viewvc?rev=1225137&view=rev
Log:
Merged revisions 1225077 via svnmerge from 
https://svn.apache.org/repos/asf/camel/trunk

........
  r1225077 | ningjiang | 2011-12-28 11:04:12 +0800 (Wed, 28 Dec 2011) | 1 line
  
   CAMEL-4834 camel properties component should support not specifying the file 
location
........

Added:
    
camel/branches/camel-2.8.x/components/camel-velocity/src/test/java/org/apache/camel/component/velocity/VelocityFileLetterWithPropertyTest.java
      - copied unchanged from r1225077, 
camel/trunk/components/camel-velocity/src/test/java/org/apache/camel/component/velocity/VelocityFileLetterWithPropertyTest.java
Modified:
    camel/branches/camel-2.8.x/   (props changed)
    
camel/branches/camel-2.8.x/camel-core/src/main/java/org/apache/camel/component/properties/PropertiesComponent.java

Propchange: camel/branches/camel-2.8.x/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Wed Dec 28 07:50:16 2011
@@ -1 +1 @@
-/camel/trunk:1212504,1215477,1221565,1224674
+/camel/trunk:1212504,1215477,1221565,1224674,1225077

Propchange: camel/branches/camel-2.8.x/
------------------------------------------------------------------------------
Binary property 'svnmerge-integrated' - no diff available.

Modified: 
camel/branches/camel-2.8.x/camel-core/src/main/java/org/apache/camel/component/properties/PropertiesComponent.java
URL: 
http://svn.apache.org/viewvc/camel/branches/camel-2.8.x/camel-core/src/main/java/org/apache/camel/component/properties/PropertiesComponent.java?rev=1225137&r1=1225136&r2=1225137&view=diff
==============================================================================
--- 
camel/branches/camel-2.8.x/camel-core/src/main/java/org/apache/camel/component/properties/PropertiesComponent.java
 (original)
+++ 
camel/branches/camel-2.8.x/camel-core/src/main/java/org/apache/camel/component/properties/PropertiesComponent.java
 Wed Dec 28 07:50:16 2011
@@ -83,19 +83,20 @@ public class PropertiesComponent extends
     }
 
     public String parseUri(String uri, String... paths) throws Exception {
-        ObjectHelper.notNull(paths, "paths");
-
-        // location may contain JVM system property or OS environment variables
-        // so we need to parse those
-        String[] locations = parseLocations(paths);
-
-        // check cache first
-        CacheKey key = new CacheKey(locations);
-        Properties prop = cache ? cacheMap.get(key) : null;
-        if (prop == null) {
-            prop = propertiesResolver.resolveProperties(getCamelContext(), 
locations);
-            if (cache) {
-                cacheMap.put(key, prop);
+        Properties prop = null;
+        if (paths != null) {
+            // location may contain JVM system property or OS environment 
variables
+            // so we need to parse those
+            String[] locations = parseLocations(paths);
+    
+            // check cache first
+            CacheKey key = new CacheKey(locations);
+            prop = cache ? cacheMap.get(key) : null;
+            if (prop == null) {
+                prop = propertiesResolver.resolveProperties(getCamelContext(), 
locations);
+                if (cache) {
+                    cacheMap.put(key, prop);
+                }
             }
         }
 


Reply via email to