diff --git a/src/interfaces/jdbc/org/postgresql/jdbc1/ResultSet.java b/src/interfaces/jdbc/org/postgresql/jdbc1/ResultSet.java index aae8cf6ff26bd183d0542f9b6a301757ce5ad9b9..98af07b0b63bd001e0e1c3f5fd5ddd0d8ca007eb 100644 --- a/src/interfaces/jdbc/org/postgresql/jdbc1/ResultSet.java +++ b/src/interfaces/jdbc/org/postgresql/jdbc1/ResultSet.java @@ -472,13 +472,35 @@ public class ResultSet extends org.postgresql.ResultSet implements java.sql.Resu //so this code strips off timezone info and adds on the GMT+/-... //as well as adds a third digit for partial seconds if necessary StringBuffer strBuf = new StringBuffer(s); + + //we are looking to see if the backend has appended on a timezone. + //currently postgresql will return +/-HH:MM or +/-HH for timezone offset + //(i.e. -06, or +06:30, note the expectation of the leading zero for the + //hours, and the use of the : for delimiter between hours and minutes) + //if the backend ISO format changes in the future this code will + //need to be changed as well char sub = strBuf.charAt(strBuf.length()-3); if (sub == '+' || sub == '-') { strBuf.setLength(strBuf.length()-3); if (subsecond) { - strBuf = strBuf.append('0').append("GMT").append(s.substring(s.length()-3, s.length())).append(":00"); + strBuf.append('0').append("GMT").append(s.substring(s.length()-3, s.length())).append(":00"); + } else { + strBuf.append("GMT").append(s.substring(s.length()-3, s.length())).append(":00"); + } + } else if (sub == ':') { + //we may have found timezone info of format +/-HH:MM, or there is no + //timezone info at all and this is the : preceding the seconds + char sub2 = strBuf.charAt(strBuf.length()-5); + if (sub2 == '+' || sub2 == '-') { + //we have found timezone info of format +/-HH:MM + strBuf.setLength(strBuf.length()-5); + if (subsecond) { + strBuf.append('0').append("GMT").append(s.substring(s.length()-5)); } else { - strBuf = strBuf.append("GMT").append(s.substring(s.length()-3, s.length())).append(":00"); + strBuf.append("GMT").append(s.substring(s.length()-5)); + } + } else if (subsecond) { + strBuf.append('0'); } } else if (subsecond) { strBuf = strBuf.append('0'); diff --git a/src/interfaces/jdbc/org/postgresql/jdbc2/ResultSet.java b/src/interfaces/jdbc/org/postgresql/jdbc2/ResultSet.java index 7aebd950b0012e64a78b53c67f4d4fabf9ae66b4..2928ab969b3383086da17789aad3dcf4daea82d8 100644 --- a/src/interfaces/jdbc/org/postgresql/jdbc2/ResultSet.java +++ b/src/interfaces/jdbc/org/postgresql/jdbc2/ResultSet.java @@ -484,14 +484,36 @@ public class ResultSet extends org.postgresql.ResultSet implements java.sql.Resu sbuf.setLength(0); sbuf.append(s); + //we are looking to see if the backend has appended on a timezone. + //currently postgresql will return +/-HH:MM or +/-HH for timezone offset + //(i.e. -06, or +06:30, note the expectation of the leading zero for the + //hours, and the use of the : for delimiter between hours and minutes) + //if the backend ISO format changes in the future this code will + //need to be changed as well char sub = sbuf.charAt(sbuf.length()-3); if (sub == '+' || sub == '-') { + //we have found timezone info of format +/-HH sbuf.setLength(sbuf.length()-3); if (subsecond) { sbuf.append('0').append("GMT").append(s.substring(s.length()-3)).append(":00"); } else { sbuf.append("GMT").append(s.substring(s.length()-3)).append(":00"); } + } else if (sub == ':') { + //we may have found timezone info of format +/-HH:MM, or there is no + //timezone info at all and this is the : preceding the seconds + char sub2 = sbuf.charAt(sbuf.length()-5); + if (sub2 == '+' || sub2 == '-') { + //we have found timezone info of format +/-HH:MM + sbuf.setLength(sbuf.length()-5); + if (subsecond) { + sbuf.append('0').append("GMT").append(s.substring(s.length()-5)); + } else { + sbuf.append("GMT").append(s.substring(s.length()-5)); + } + } else if (subsecond) { + sbuf.append('0'); + } } else if (subsecond) { sbuf.append('0'); }