svn commit: r725725 - /activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/component/file/strategy/FileProcessStrategyFactory.java

[da...@apache.org]

Author: davsclaus
Date: Thu Dec 11 08:09:19 2008
New Revision: 725725

URL: http://svn.apache.org/viewvc?rev=725725&view=rev
Log:
polished code

Modified:
    activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/component/file/strategy/FileProcessStrategyFactory.java

Modified: activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/component/file/strategy/FileProcessStrategyFactory.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/component/file/strategy/FileProcessStrategyFactory.java?rev=725725&r1=725724&r2=725725&view=diff
==============================================================================
--- activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/component/file/strategy/FileProcessStrategyFactory.java (original)
+++ activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/component/file/strategy/FileProcessStrategyFactory.java Thu Dec 11 08:09:19 2008
@@ -36,6 +36,7 @@
     public static FileProcessStrategy createFileProcessStrategy(Map<String, Object> params) {
 
         // We assume a value is present only if its value not null for String and 'true' for boolean
+        boolean isNoop = params.get("noop") != null;
         boolean isDelete = params.get("delete") != null;
         boolean isLock = params.get("lock") != null;
         String moveNamePrefix = (String) params.get("moveNamePrefix");
@@ -47,7 +48,7 @@
         boolean move = moveNamePrefix != null || moveNamePostfix != null;
         boolean preMove = preMoveNamePrefix != null || preMoveNamePostfix != null;
 
-        if (params.containsKey("noop")) {
+        if (isNoop) {
             return new NoOpFileProcessStrategy(isLock);
         } else if (move || preMove) {
             RenameFileProcessStrategy strategy = new RenameFileProcessStrategy(isLock);