Author: hboutemy
Date: Sat Sep 20 21:00:25 2014
New Revision: 1626496

URL: http://svn.apache.org/r1626496
Log:
code simplification

Modified:
    
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/utils/DistributionManagementCheck.java

Modified: 
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/utils/DistributionManagementCheck.java
URL: 
http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/utils/DistributionManagementCheck.java?rev=1626496&r1=1626495&r2=1626496&view=diff
==============================================================================
--- 
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/utils/DistributionManagementCheck.java
 (original)
+++ 
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/utils/DistributionManagementCheck.java
 Sat Sep 20 21:00:25 2014
@@ -50,19 +50,11 @@ public class DistributionManagementCheck
                 throw new EnforcerRuleException( "You have defined a site in 
distributionManagement." );
             }
         }
-
     }
 
     private boolean hasRepository()
     {
-        if ( getDistributionManagement().getRepository() == null )
-        {
-            return false;
-        }
-        else
-        {
-            return true;
-        }
+        return getDistributionManagement().getRepository() != null;
     }
 
     public DistributionManagement getDistributionManagement()
@@ -77,38 +69,16 @@ public class DistributionManagementCheck
 
     private boolean hasSnapshotRepository()
     {
-        if ( getDistributionManagement().getSnapshotRepository() == null )
-        {
-            return false;
-        }
-        else
-        {
-            return true;
-        }
+        return getDistributionManagement().getSnapshotRepository() != null;
     }
 
     private boolean hasSite()
     {
-        if ( getDistributionManagement().getSite() == null )
-        {
-            return false;
-        }
-        else
-        {
-            return true;
-        }
+        return getDistributionManagement().getSite() != null;
     }
 
     private boolean hasDistributionManagement()
     {
-        if ( getDistributionManagement() == null )
-        {
-            return false;
-        }
-        else
-        {
-            return true;
-        }
+        return getDistributionManagement() != null;
     }
-
 }


Reply via email to