directory-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From elecha...@apache.org
Subject svn commit: r1771556 - in /directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger: LdapOperation.java StoredProcedureParameter.java TriggerSpecification.java
Date Sun, 27 Nov 2016 10:41:47 GMT
Author: elecharny
Date: Sun Nov 27 10:41:47 2016
New Revision: 1771556

URL: http://svn.apache.org/viewvc?rev=1771556&view=rev
Log:
Fixed SonarLint warnings

Modified:
    directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/LdapOperation.java
    directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/StoredProcedureParameter.java
    directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/TriggerSpecification.java

Modified: directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/LdapOperation.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/LdapOperation.java?rev=1771556&r1=1771555&r2=1771556&view=diff
==============================================================================
--- directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/LdapOperation.java
(original)
+++ directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/LdapOperation.java
Sun Nov 27 10:41:47 2016
@@ -66,6 +66,10 @@ public enum LdapOperation
     }
 
 
+    /**
+     * {@inheritDoc}
+     */
+    @Override
     public String toString()
     {
         return name;

Modified: directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/StoredProcedureParameter.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/StoredProcedureParameter.java?rev=1771556&r1=1771555&r2=1771556&view=diff
==============================================================================
--- directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/StoredProcedureParameter.java
(original)
+++ directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/StoredProcedureParameter.java
Sun Nov 27 10:41:47 2016
@@ -55,6 +55,7 @@ public abstract class StoredProcedurePar
         }
 
 
+        @Override
         public String toString()
         {
             return name + " \"" + ctxName.getName() + "\"";
@@ -96,6 +97,7 @@ public abstract class StoredProcedurePar
     }
 
 
+    @Override
     public String toString()
     {
         return name;
@@ -106,6 +108,7 @@ public abstract class StoredProcedurePar
      * @see java.lang.Object#hashCode()
      * @return the instance's hash code 
      */
+    @Override
     public int hashCode()
     {
         int h = 37;
@@ -119,6 +122,7 @@ public abstract class StoredProcedurePar
     /**
      * @see java.lang.Object#equals(java.lang.Object)
      */
+    @Override
     public boolean equals( Object obj )
     {
         if ( this == obj )

Modified: directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/TriggerSpecification.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/TriggerSpecification.java?rev=1771556&r1=1771555&r2=1771556&view=diff
==============================================================================
--- directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/TriggerSpecification.java
(original)
+++ directory/shared/trunk/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/TriggerSpecification.java
Sun Nov 27 10:41:47 2016
@@ -52,16 +52,17 @@ public class TriggerSpecification
     public TriggerSpecification( LdapOperation ldapOperation, ActionTime actionTime, List<SPSpec>
spSpecs )
     {
         super();
-        if ( ldapOperation == null
-            || actionTime == null
-            || spSpecs == null )
+        
+        if ( ( ldapOperation == null ) || ( actionTime == null ) || ( spSpecs == null ) )
         {
             throw new NullArgumentException( I18n.err( I18n.ERR_04331 ) );
         }
-        if ( spSpecs.size() == 0 )
+        
+        if ( spSpecs.isEmpty() )
         {
             throw new IllegalArgumentException( I18n.err( I18n.ERR_04332 ) );
         }
+        
         this.ldapOperation = ldapOperation;
         this.actionTime = actionTime;
         this.spSpecs = spSpecs;



Mime
View raw message