the bug was not fixed in the snapshot of November 5th. Also the enterprise

bmomjian · bmomjian · commit 960c18618486 · 2000-11-10T22:06:26.000Z
edition of the driver did not compile. I have fixed both issues again. I have
attached the modified files to this email, maybe you can check them into the
repository. (Fixes are marked with //FIXME). Enterprise edition driver now
compiles and seems to work.

Jan Thomae
diff --git a/src/interfaces/jdbc/org/postgresql/PostgresqlDataSource.java b/src/interfaces/jdbc/org/postgresql/PostgresqlDataSource.java
@@ -40,7 +40,7 @@
  *
  * Copyright 1999 (C) Exoffice Technologies Inc. All Rights Reserved.
  *
- * $Id: PostgresqlDataSource.java,v 1.1 2000/10/12 08:55:24 peter Exp $
+ * $Id: PostgresqlDataSource.java,v 1.2 2000/11/10 22:06:26 momjian Exp $
  */
 
 
@@ -64,9 +64,12 @@
 import javax.naming.Name;
 import javax.naming.NamingException;
 import javax.naming.spi.ObjectFactory;
-import postgresql.util.PSQLException;
-import postgresql.xa.XADataSourceImpl;
-
+// FIXME
+//import postgresql.util.PSQLException;
+//import postgresql.xa.XADataSourceImpl;
+import org.postgresql.util.PSQLException;
+import org.postgresql.xa.XADataSourceImpl;
+//---------
 
 /**
  * Implements a JDBC 2.0 {@link javax.sql.DataSource} for the
@@ -191,8 +194,10 @@ public class PostgresqlDataSource
      * Each datasource maintains it's own driver, in case of
      * driver-specific setup (e.g. pools, log writer).
      */
-    private transient postgresql.Driver _driver;
-
+// FIXME
+//    private transient postgresql.Driver _driver;
+private transient org.postgresql.Driver _driver;
+//---------
 
 
 
@@ -223,8 +228,15 @@ public synchronized Connection getConnection( String user, String password )
 		// Constructs a driver for use just by this data source
 		// which will produce TwoPhaseConnection-s. This driver
 		// is not registered with the driver manager.
-		_driver = new postgresql.Driver();
-		_driver.setLogWriter( _logWriter );
+// FIXME
+//		_driver = new postgresql.Driver();
+_driver = new org.postgresql.Driver();
+//-----------
+
+//FIXME
+//		_driver.setLogWriter( _logWriter );
+// Method seems to be unavailable. Just commented it out.
+//----------
 	    } catch ( SQLException except ) {
 		if ( _logWriter != null )
 		    _logWriter.println( "DataSource: Failed to initialize JDBC driver: " + except );
@@ -260,7 +272,10 @@ public synchronized Connection getConnection( String user, String password )
 	// attempt or a failure.
 	try {
 	    conn = _driver.connect( url, info );
-	    if ( ! ( conn instanceof postgresql.jdbc2.Connection ) ) {
+// FIXME
+//	    if ( ! ( conn instanceof postgresql.jdbc2.Connection ) ) {
+if ( ! ( conn instanceof org.postgresql.jdbc2.Connection ) ) {
+//--------
 		if ( _logWriter != null )
 		    _logWriter.println( "DataSource: JDBC 1 connections not supported" );
 		throw new PSQLException( "postgresql.ds.onlyjdbc2" );
@@ -289,7 +304,10 @@ public synchronized void setLogWriter( PrintWriter writer )
 	// synchronizing.
 	if ( writer != null ) {
 	    if ( _driver != null )
-		_driver.setLogWriter( writer );
+// FIXME
+//		_driver.setLogWriter( writer );
+// Method seems to be unavailable. Commented it out.
+//----------
 	    _logWriter = writer;
 	}
     }
diff --git a/src/interfaces/jdbc/org/postgresql/jdbc2/ResultSet.java b/src/interfaces/jdbc/org/postgresql/jdbc2/ResultSet.java
@@ -468,9 +468,20 @@ public Timestamp getTimestamp(int columnIndex) throws SQLException
     // This works, but it's commented out because Michael Stephenson's
     // solution is better still:
     //SimpleDateFormat df = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
-    
-    // Michael Stephenson's solution:
+// Modification by Jan Thomae
+    String sub = s.substring(s.length() - 3, s.length()-2);
+    if (sub.equals("+") || sub.equals("-")) {
+            s = s.substring(0, s.length()-3) + "GMT"+ s.substring(s.length()-3, s.length())+":00";
+    }
+// -------
+       // Michael Stephenson's solution:
     SimpleDateFormat df = null;
+
+// Modification by Jan Thomae
+    if (s.length()>27) {
+    df = new SimpleDateFormat("yyyy-MM-dd HH:mm:sszzzzzzzzz");  
+    } else              
+// -------
     if (s.length()>21 && s.indexOf('.') != -1) {
 	df = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss.SSzzz");
     } else if (s.length()>19 && s.indexOf('.') == -1) {
@@ -839,14 +850,14 @@ public boolean absolute(int index) throws SQLException
 
 	//if index<0, count from the end of the result set, but check
 	//to be sure that it is not beyond the first index
-	if (index<0) {
+	if (index<0) 
 	    if (index>=-rows.size())
 		internalIndex=rows.size()+index;
 	    else {
 		beforeFirst();
 		return false;
 	    }
-	} else {
+		
 	//must be the case that index>0, 
 	//find the correct place, assuming that 
 	//the index is not too large
@@ -856,7 +867,6 @@ public boolean absolute(int index) throws SQLException
 	    afterLast();
 	    return false;
 	}
-	}
 
 	current_row=internalIndex;
 	this_row = (byte [][])rows.elementAt(internalIndex);
