Changeset 22974 for main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/AbstractGS2Search.java

Timestamp:

2010-09-28T10:21:36+13:00 (14 years ago)

Author:

davidb

Message:

Code used to test 'coll_db == null' to determine if a database was opened correctly. Since this value is returned by a constructor, it is always non-null, even when it failed to open the database. The routine databaseOK() was added to core class, and is now used in these routines instead of testing for null

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/AbstractGS2Search.java (modified) (1 diff)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/AbstractGS2Search.java

@@ -110 +110 @@
     }
     coll_db = new SimpleCollectionDatabase(database_type);
-    if (coll_db == null) {
+    if (!coll_db.databaseOK()) {
       logger.error("Couldn't create the collection database of type "+database_type);
       return false;