Add code to load bug categories and patterns from the FindBugs messages.xml file.

[cfb.git] / prod / net / jaekl / cfb / db / Schema.java

diff --git a/prod/net/jaekl/cfb/db/Schema.java b/prod/net/jaekl/cfb/db/Schema.java
index bfcfdb8abf3bf98d866ad1290a4426573944aea3..f802b58ca3cf71f89f7b5ef6c8b0f57c95382c66 100644 (file)
--- a/prod/net/jaekl/cfb/db/Schema.java
+++ b/prod/net/jaekl/cfb/db/Schema.java
@@ -1,11 +1,14 @@
 package net.jaekl.cfb.db;
 
+// Copyright (C) 2015 Christian Jaekl
+
 import java.sql.Connection;
 import java.sql.DatabaseMetaData;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.util.ArrayList;
 import java.util.HashSet;
+import java.util.Locale;
 
 import net.jaekl.cfb.db.driver.DbDriver;
 
@@ -13,24 +16,31 @@ public class Schema {
        String m_name;
        DbDriver m_driver;
        ArrayList<Table> m_tables;
+       ArrayList<Sequence> m_sequences;
        
        public Schema(String name, DbDriver driver) {
                m_name = name;
                m_driver = driver;
                m_tables = new ArrayList<Table>();
+               m_sequences = new ArrayList<Sequence>();
        }
        
        public boolean ensureDbInitialized(Connection con) throws SQLException {
                assert(null != con);
-               boolean result = true;
                
                if (allTablesPresent(con)) {
                        return true;
                }
                
-               result = createAllTables(con);
+               if (!createAllTables(con)) {
+                       return false;
+               }
+               
+               if (!createAllSequences(con)) { 
+                       return false;
+               }
                
-               return result;
+               return true;
        }
        
        boolean allTablesPresent(Connection con) throws SQLException 
@@ -42,18 +52,20 @@ public class Schema {
                
                try (ResultSet rs = dbmd.getTables(null, null, null, new String[]{"TABLE"})) {
                        while (rs.next()) {
-                               extantTables.add(rs.getString(3));
+                               extantTables.add(rs.getString(3).toUpperCase(Locale.CANADA));
                        }
                }
                
                for (Table table : m_tables) {
-                       if ( ! extantTables.contains(table.getName()) ) {
+                       String name = table.getName().toUpperCase(Locale.CANADA);
+                       if ( ! extantTables.contains(name) ) {
                                // One or more tables missing
                                return false;
                        }
                }
                
                // We could be more thorough here, and check that the expected columns are in place.
+               // Also, eventually, some sort of DB schema versioning will be needed.
                
                return true;            
        }
@@ -67,6 +79,15 @@ public class Schema {
                return true;
        }
        
+       boolean createAllSequences(Connection con) throws SQLException {
+               for (Sequence seq : m_sequences) {
+                       if (!m_driver.createSequence(con, seq)) {
+                               return false;
+                       }
+               }
+               return true;
+       }
+       
        void addTable(Table table) {
                m_tables.add(table);
        }
@@ -82,5 +103,9 @@ public class Schema {
                for (Object[][] table : tables) {
                        addTable(Table.construct(table));
                }
-       }       
+       }
+       
+       void addSequence(Sequence seq) {
+               m_sequences.add(seq);
+       }
 }