You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by pg...@apache.org on 2019/03/29 14:14:29 UTC

svn commit: r1856541 - /ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/ftp/FtpServices.java

Author: pgil
Date: Fri Mar 29 14:14:29 2019
New Revision: 1856541

URL: http://svn.apache.org/viewvc?rev=1856541&view=rev
Log:
Fixed: Switch ftpNotificationEnabled test condition to respect configuration meaning
(OFBIZ-10245)

Modified:
    ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/ftp/FtpServices.java

Modified: ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/ftp/FtpServices.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/ftp/FtpServices.java?rev=1856541&r1=1856540&r2=1856541&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/ftp/FtpServices.java (original)
+++ ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/ftp/FtpServices.java Fri Mar 29 14:14:29 2019
@@ -75,7 +75,7 @@ public class FtpServices {
         boolean forceTransferControlSuccess = EntityUtilProperties.propertyValueEqualsIgnoreCase("ftp", "ftp.force.transfer.control", "Y", delegator);
         boolean ftpNotificationEnabled = EntityUtilProperties.propertyValueEqualsIgnoreCase("ftp", "ftp.notifications.enabled", "Y", delegator);
 
-        if (ftpNotificationEnabled) return ServiceUtil.returnSuccess();
+        if (!ftpNotificationEnabled) return ServiceUtil.returnSuccess();
 
         // for ECA communicationEvent process
         Map<String, Object> resultMap = ServiceUtil.returnSuccess();