Fixed logger usage: - From: if (logger.isTraceEnabled()) logger...

Jean-Noel Rouvignac

31.12.2014 adf6f5da58af4947ce7ba6f6ee0bcc19725c5a6c

Fixed logger usage:
- From:
    if (logger.isTraceEnabled())
        logger.traceException(e);
  To:
    logger.traceException(e);
- From:
    if (logger.isTraceEnabled())
    {
        logger.trace(message);
    }
  To:
    logger.trace(message);      

 opendj3-server-dev/src/server/org/opends/server/backends/TrustStoreBackend.java

@@ -557,10 +557,7 @@
    }
    catch (Exception e)
    {
      if (logger.isTraceEnabled())
      {
        logger.traceException(e);
      }

      LocalizableMessage message = ERR_TRUSTSTORE_CANNOT_RETRIEVE_CERT.get(
          certAlias, trustStoreFile, e.getMessage());
@@ -756,11 +753,7 @@
          }
          catch (Exception e)
          {
            if (logger.isTraceEnabled())
            {
              logger.traceException(e);
            }

            continue;
          }

@@ -768,7 +761,6 @@
          {
            searchOperation.returnEntry(certEntry, null);
          }

        }
      }
    }

 opendj3-server-dev/src/server/org/opends/server/backends/jeb/ID2Entry.java

@@ -245,13 +245,10 @@
      catch(IOException ioe)
      {
        // TODO: This should never happen with byte buffer.
        if(logger.isTraceEnabled())
        {
          logger.traceException(ioe);
        }
      }
    }
  }

  /**
   * Create a new ID2Entry object.

 opendj3-server-dev/src/server/org/opends/server/core/DirectoryServer.java

@@ -1056,13 +1056,9 @@
      }
      catch (Exception e)
      {
        if (logger.isTraceEnabled())
        {
          logger.traceException(e);
        }

        directoryServer.mBeanServer =
          MBeanServerFactory.newMBeanServer();
        directoryServer.mBeanServer = MBeanServerFactory.newMBeanServer();
      }
      directoryServer.mBeans = new ConcurrentHashMap<DN,JMXMBean>();
      registerAlertGenerator(directoryServer);

 opendj3-server-dev/src/server/org/opends/server/crypto/GetSymmetricKeyExtendedOperation.java

@@ -131,10 +131,7 @@
    }
    catch (ASN1Exception ae)
    {
      if (logger.isTraceEnabled())
      {
        logger.traceException(ae);
      }

      LocalizableMessage message = ERR_GET_SYMMETRIC_KEY_ASN1_DECODE_EXCEPTION.get(
           ae.getMessage());
@@ -143,10 +140,7 @@
    }
    catch (Exception e)
    {
      if (logger.isTraceEnabled())
      {
        logger.traceException(e);
      }

      operation.setResultCode(ResultCode.PROTOCOL_ERROR);


 opendj3-server-dev/src/server/org/opends/server/extensions/GSSAPISASLMechanismHandler.java

@@ -381,7 +381,6 @@
        saslContext = SASLContext.createSASLContext(saslProps, serverFQDN,
                                  SASL_MECHANISM_GSSAPI, identityMapper);
      } catch (SaslException ex) {
        if (logger.isTraceEnabled())
          logger.traceException(ex);
        LocalizableMessage msg;
        GSSException gex = (GSSException) ex.getCause();
@@ -404,7 +403,6 @@
    }
    catch (LoginException ex)
    {
      if (logger.isTraceEnabled())
        logger.traceException(ex);
      LocalizableMessage message = ERR_SASLGSSAPI_CANNOT_CREATE_LOGIN_CONTEXT
            .get(getExceptionMessage(ex));
@@ -489,11 +487,9 @@
    }
    catch (UnknownHostException ex)
    {
      if (logger.isTraceEnabled())
        logger.traceException(ex);
      LocalizableMessage message = ERR_SASL_CANNOT_GET_SERVER_FQDN.get(String
          .valueOf(configEntryDN), getExceptionMessage(ex));
      unacceptableReasons.add(message);
      unacceptableReasons.add(ERR_SASL_CANNOT_GET_SERVER_FQDN.get(
          configEntryDN, getExceptionMessage(ex)));
      isAcceptable = false;
    }

@@ -543,26 +539,18 @@
      this.configuration = newConfiguration;
    }
    catch (InitializationException ex) {
      if (logger.isTraceEnabled())
        logger.traceException(ex);
      LocalizableMessage message = ex.getMessageObject();
      messages.add(message);
      messages.add(ex.getMessageObject());
      clearProperties();
      resultCode = ResultCode.OTHER;
    } catch (UnknownHostException ex) {
      if (logger.isTraceEnabled())
        logger.traceException(ex);
        LocalizableMessage message = ERR_SASL_CANNOT_GET_SERVER_FQDN.get(String
          .valueOf(configEntryDN), getExceptionMessage(ex));
      messages.add(message);
      messages.add(ERR_SASL_CANNOT_GET_SERVER_FQDN.get(configEntryDN, getExceptionMessage(ex)));
      clearProperties();
      resultCode = ResultCode.OTHER;
    } catch (IOException ex) {
      if (logger.isTraceEnabled())
        logger.traceException(ex);
      LocalizableMessage message = ERR_SASLGSSAPI_CANNOT_CREATE_JAAS_CONFIG
        .get(getExceptionMessage(ex));
      messages.add(message);
      messages.add(ERR_SASLGSSAPI_CANNOT_CREATE_JAAS_CONFIG.get(getExceptionMessage(ex)));
      clearProperties();
      resultCode = ResultCode.OTHER;
    }

 opendj3-server-dev/src/server/org/opends/server/plugins/SambaPasswordPlugin.java

@@ -786,11 +786,8 @@
       * This should never happen as we are processing the post-operation which
       * succeeded so the entry has to exist if we have reached this point.
       */
      if (logger.isTraceEnabled())
      {
        logger.traceException(e);
      }
    }

    return PluginResult.PostOperation.continueOperationProcessing();
  }

 opendj3-server-dev/src/server/org/opends/server/replication/server/ReplicationServer.java

@@ -376,7 +376,6 @@
    }
    catch (Exception e)
    {
      if (logger.isTraceEnabled())
        logger.traceException(e);
      close(session);
      close(socket);
@@ -691,11 +690,8 @@
    }
    catch (ChangelogException ignored)
    {
      if (logger.isTraceEnabled())
      {
        logger.traceException(ignored);
      }
    }

    // Remove this instance from the global instance list
    allInstances.remove(this);
@@ -777,7 +773,6 @@
      }
      catch (ChangelogException e)
      {
        if (logger.isTraceEnabled())
          logger.traceException(e);
        resultCode = ResultCode.OPERATIONS_ERROR;
      }
@@ -803,13 +798,11 @@
      }
      catch (IOException e)
      {
        if (logger.isTraceEnabled())
          logger.traceException(e);
        logger.error(ERR_COULD_NOT_CLOSE_THE_SOCKET.get(e));
      }
      catch (InterruptedException e)
      {
        if (logger.isTraceEnabled())
          logger.traceException(e);
        logger.error(ERR_COULD_NOT_STOP_LISTEN_THREAD.get(e));
      }

 opendj3-server-dev/src/server/org/opends/server/replication/server/changelog/je/JEChangeNumberIndexDB.java

@@ -497,7 +497,6 @@
      }
      catch (ChangelogException e)
      {
        if (logger.isTraceEnabled())
          logger.traceException(e);
      }
      return 0;

 opendj3-server-dev/src/server/org/opends/server/replication/server/changelog/je/JEChangelogDB.java

@@ -157,7 +157,6 @@
    }
    catch (Exception e)
    {
      if (logger.isTraceEnabled())
        logger.traceException(e);

      final LocalizableMessageBuilder mb = new LocalizableMessageBuilder();
@@ -317,7 +316,6 @@
    }
    catch (ChangelogException e)
    {
      if (logger.isTraceEnabled())
        logger.traceException(e);

      logger.error(ERR_COULD_NOT_READ_DB.get(this.dbDirectory.getAbsolutePath(),
@@ -462,8 +460,7 @@
          {
            firstException = e;
          }
          else if (logger.isTraceEnabled())
            logger.traceException(e);
          else logger.traceException(e);
        }

        cnIndexDB = null;
@@ -584,8 +581,7 @@
          {
            firstException = e;
          }
          else if (logger.isTraceEnabled())
            logger.traceException(e);
          else logger.traceException(e);
        }
      }
    }
@@ -601,8 +597,7 @@
      {
        firstException = e;
      }
      else if (logger.isTraceEnabled())
        logger.traceException(e);
      else logger.traceException(e);
    }

    if (firstException != null)
@@ -699,7 +694,6 @@
        }
        catch (Exception e)
        {
          if (logger.isTraceEnabled())
            logger.traceException(e);
          logger.error(ERR_CHANGENUMBER_DATABASE.get(e.getLocalizedMessage()));
        }

 opendj3-server-dev/src/server/org/opends/server/replication/service/ReplicationBroker.java

@@ -1232,12 +1232,9 @@
          logger.error(errorMessage);
        }

        if (logger.isTraceEnabled())
        {
          logger.trace(errorMessage);
        }
      }
    }
    return setConnectedRS(ConnectedRS.noConnectedRS());
  }


 opendj3-server-dev/src/server/org/opends/server/schema/CertificateExactMatchingRule.java

@@ -193,11 +193,7 @@
    catch (CertificateException ce)
    {
      // There seems to be a problem while parsing the certificate.
      LocalizableMessage message = WARN_CERTIFICATE_MATCH_PARSE_ERROR.get(ce.getMessage());
      if (logger.isTraceEnabled())
      {
         logger.trace(message);
      }
      logger.trace(WARN_CERTIFICATE_MATCH_PARSE_ERROR.get(ce.getMessage()));

      // return the raw bytes as a fall back
      return value.toByteString();

 opendj3-server-dev/src/server/org/opends/server/types/FilePermission.java

@@ -264,18 +264,12 @@
    }
    catch (NoSuchMethodException e)
    {
      if (logger.isTraceEnabled())
      {
        logger.traceException(e);
      }
    }
    catch (ClassNotFoundException e)
    {
      if (logger.isTraceEnabled())
      {
        logger.traceException(e);
      }
    }
    finally
    {
      // Clean up if we only had partial success.

 opendj3-server-dev/src/server/org/opends/server/util/LDIFReader.java

@@ -1322,7 +1322,6 @@
        }
        rejectWriter.newLine();
      } catch (IOException ex) {
        if (logger.isTraceEnabled())
          logger.traceException(ex);
      }
    }

 opendj3-server-dev/src/server/org/opends/server/workflowelement/externalchangelog/ECLSearchOperation.java

@@ -241,7 +241,6 @@
      }
      catch (DirectoryException de)
      {
        if (logger.isTraceEnabled())
          logger.traceException(de);
        setResponseData(de);
        break searchProcessing;
@@ -254,7 +253,6 @@
      }
      catch (DirectoryException de)
      {
        if (logger.isTraceEnabled())
          logger.traceException(de);
        setResponseData(de);
        break searchProcessing;
@@ -296,9 +294,7 @@
      }
      catch (DirectoryException de)
      {
        if (logger.isTraceEnabled())
          logger.traceException(de);

        setResponseData(de);

        if (persistentSearch != null)
@@ -320,7 +316,6 @@
      }
      catch (Exception e)
      {
        if (logger.isTraceEnabled())
          logger.traceException(e);

        setResultCode(DirectoryServer.getServerErrorResultCode());

 opendj3-server-dev/src/server/org/opends/server/workflowelement/localbackend/LocalBackendSearchOperation.java

@@ -272,11 +272,7 @@
    }
    catch (DirectoryException de)
    {
      if (logger.isTraceEnabled())
      {
        logger.traceException(de);
      }

      setResponseData(de);

      if (persistentSearch != null)

 opendj3-server-dev/tests/unit-tests-testng/src/server/org/opends/server/replication/InitOnLineTest.java

@@ -114,11 +114,8 @@
  private void log(String s)
  {
    logger.error(LocalizableMessage.raw("InitOnLineTests/" + s));
    if (logger.isTraceEnabled())
    {
      logger.trace(s);
    }
  }

  /**
   * Set up the environment for performing the tests in this Class.

 opendj3-server-dev/tests/unit-tests-testng/src/server/org/opends/server/replication/ReSyncTest.java

@@ -54,11 +54,8 @@
  private void debugInfo(String s)
  {
    logger.error(LocalizableMessage.raw(s));
    if (logger.isTraceEnabled())
    {
      logger.trace(s);
    }
  }

  private static final String EXAMPLE_DN = "dc=example,dc=com";
  private File reSyncTempDir;