Fixing - squid:S1118:Utility classes should not have public

constructors.

Author: Faisal Hameed

activejdbc-instrumentation/src/main/java/org/javalite/instrumentation/JavaAgent.java

@@ -42,6 +42,10 @@ public class JavaAgent {
     private static final Set<ClassLoader> loaders = new HashSet<ClassLoader>();
     private static Method modelFound;
 
+    private JavaAgent() {
+        
+    }
+    
     @SuppressWarnings("unchecked")
     public static void premain(String args, java.lang.instrument.Instrumentation inst) {
 

activejdbc/src/main/java/org/javalite/activejdbc/Base.java

@@ -36,6 +36,10 @@
  */
 public class Base {
 
+    private Base() {
+        
+    }
+    
     /**
      * This method will open a connection defined in the file 'database.properties' located at
      * root of classpath. The connection picked from the file is defined by <code>ACTIVE_ENV</code>

activejdbc/src/main/java/org/javalite/activejdbc/ConnectionsAccess.java

@@ -30,6 +30,10 @@ public class ConnectionsAccess {
     private final static Logger logger = LoggerFactory.getLogger(ConnectionsAccess.class);
     private static final ThreadLocal<HashMap<String, Connection>> connectionsTL = new ThreadLocal<HashMap<String, Connection>>();
 
+    private ConnectionsAccess() {
+        
+    }
+    
     static Map<String, Connection> getConnectionMap(){
         if (connectionsTL.get() == null)
             connectionsTL.set(new HashMap<String, Connection>());

activejdbc/src/main/java/org/javalite/activejdbc/LogFilter.java

@@ -42,6 +42,10 @@ public class LogFilter {
             setLogExpression("a{10000000}");
         }
     }
+    
+    private LogFilter() {
+        
+    }
 
     public static void setLogExpression(String regexp){
         pattern = Pattern.compile(regexp, Pattern.CASE_INSENSITIVE);

activejdbc/src/main/java/org/javalite/activejdbc/Messages.java

@@ -34,6 +34,10 @@ public class Messages {
 
     private static final String BUNDLE = "activejdbc_messages";
 
+    private Messages() {
+        
+    }
+    
     /**
      * Looks for a localized property/message in <code>activejdbc_messages</code> bundle.
      *

activejdbc/src/main/java/org/javalite/activejdbc/ModelFinder.java

@@ -29,6 +29,10 @@ public class ModelFinder {
 
     private static final List<String> modelClassNames = new ArrayList<String>();
 
+    private ModelFinder() {
+        
+    }
+    
     protected static void findModels(String dbName) throws IOException, ClassNotFoundException {
         //this is for static instrumentation. In case of dynamic, the  modelClassNames will already be filled.
         List<String> models = Registry.instance().getConfiguration().getModelNames(dbName);

db-migrator/src/main/java/org/javalite/db_migrator/DbUtils.java

@@ -24,6 +24,10 @@ public class DbUtils {
     private static final String DB2_FRAGMENT = ":db2:";
     private static final String ORACLE_FRAGMENT = ":oracle:";
 
+    private DbUtils() {
+        
+    }
+    
     /**
      * Given a jdbc url, this tries to determine the target database and returns the driver class name as a string.
      *

javalite-common/src/main/java/org/javalite/common/Inflector.java

@@ -89,6 +89,10 @@ public class Inflector {
 
         uncountables = Arrays.asList("equipment", "information", "rice", "money", "species", "series", "fish", "sheep");
     }
+    
+    private Inflector() {
+        
+    }
 
     public static void addPlural(String rule, String replacement){
         plurals.add(0, new String[]{rule, replacement});

javalite-common/src/main/java/org/javalite/http/Http.java

@@ -37,8 +37,10 @@ public class Http {
      * Read timeout in milliseconds. Set this value to what you like to override default.
      */
     public static int READ_TIMEOUT = 5000;
-
-
+    
+    private Http() {
+        
+    }
 
     /**
      * Executes a POST request.

javalite-common/src/main/java/org/javalite/test/SystemStreamUtil.java

@@ -15,6 +15,10 @@ public class SystemStreamUtil {
     private static ByteArrayOutputStream outStream;
     private static ByteArrayOutputStream errorStream;
 
+    private SystemStreamUtil() {
+        
+    }
+    
     /**
      * Replaces <code>System.out</code> with internal buffer. All calls such as <code>System.out.print...</code>
      * will go to this buffer and not to STDIO