[exo-jcr-commits] exo-jcr SVN: r5364 - in jcr/trunk: exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc and 4 other directories.

[do-not-reply at jboss.org]

Author: andrew.plotnikov
Date: 2011-12-27 05:07:13 -0500 (Tue, 27 Dec 2011)
New Revision: 5364

Modified:
   jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/core/version/ItemDataMergeVisitor.java
   jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/JDBCStorageConnection.java
   jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/update/StorageUpdateManager.java
   jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/PendingChangesLog.java
   jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/replication/Packet.java
   jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/replication/test/ReplicationTestService.java
Log:
EXOJCR-1687: Fixed new sonar violations in project

Modified: jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/core/version/ItemDataMergeVisitor.java
===================================================================
--- jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/core/version/ItemDataMergeVisitor.java	2011-12-27 10:06:03 UTC (rev 5363)
+++ jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/core/version/ItemDataMergeVisitor.java	2011-12-27 10:07:13 UTC (rev 5364)
@@ -88,7 +88,7 @@
 
       private final QPath path;
 
-      private VersionableState(QPath path, int result)
+      private VersionableState(QPath path, int result) //NOSONAR
       {
          this.path = path;
          this.result = result;
@@ -156,7 +156,7 @@
       }
    }
 
-   private class VersionableStateComparator implements Comparator<VersionableState>
+   private class VersionableStateComparator implements Comparator<VersionableState> //NOSONAR
    {
       public int compare(VersionableState nc1, VersionableState nc2)
       {

Modified: jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/JDBCStorageConnection.java
===================================================================
--- jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/JDBCStorageConnection.java	2011-12-27 10:06:03 UTC (rev 5363)
+++ jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/JDBCStorageConnection.java	2011-12-27 10:07:13 UTC (rev 5364)
@@ -237,12 +237,6 @@
       this.dbConnection = dbConnection;
       this.readOnly = readOnly;
 
-      // Fix for Sybase jConnect JDBC driver bug.
-      // Which throws SQLException(JZ016: The AutoCommit option is already set to
-      // false)
-      // if conn.setAutoCommit(false) called twise or more times with value
-      // 'false'.
-      // TODO remove workaround for Sybase, jconn 6.05 Build 26564
       if (!readOnly && dbConnection.getAutoCommit())
       {
          dbConnection.setAutoCommit(false);

Modified: jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/update/StorageUpdateManager.java
===================================================================
--- jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/update/StorageUpdateManager.java	2011-12-27 10:06:03 UTC (rev 5363)
+++ jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/update/StorageUpdateManager.java	2011-12-27 10:07:13 UTC (rev 5364)
@@ -219,7 +219,7 @@
       }
    }
 
-   private class Updater100 extends Updater
+   private class Updater100 extends Updater //NOSONAR
    {
 
       @Override
@@ -230,7 +230,7 @@
       }
    }
 
-   private class Updater101 extends Updater
+   private class Updater101 extends Updater //NOSONAR
    {
 
       @Override

Modified: jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/PendingChangesLog.java
===================================================================
--- jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/PendingChangesLog.java	2011-12-27 10:06:03 UTC (rev 5363)
+++ jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/PendingChangesLog.java	2011-12-27 10:07:13 UTC (rev 5364)
@@ -77,7 +77,7 @@
       /**
        * Empty constructor.
        */
-      private Type()
+      private Type() //NOSONAR
       {
       }
    }

Modified: jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/replication/Packet.java
===================================================================
--- jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/replication/Packet.java	2011-12-27 10:06:03 UTC (rev 5363)
+++ jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/replication/Packet.java	2011-12-27 10:07:13 UTC (rev 5364)
@@ -205,7 +205,7 @@
       /**
        * Private PacketType constructor.
        */
-      private PacketType()
+      private PacketType() //NOSONAR
       {
       }
    }

Modified: jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/replication/test/ReplicationTestService.java
===================================================================
--- jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/replication/test/ReplicationTestService.java	2011-12-27 10:06:03 UTC (rev 5363)
+++ jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/replication/test/ReplicationTestService.java	2011-12-27 10:07:13 UTC (rev 5364)
@@ -206,7 +206,7 @@
          /**
           * OperationType constructor.
           */
-         private OperationType()
+         private OperationType() //NOSONAR
          {
 
          }
@@ -215,7 +215,7 @@
       /**
        * Constants constructor.
        */
-      private Constants()
+      private Constants() //NOSONAR
       {
       }
    }