[Jython-checkins] SF.net SVN: jython:[6937] trunk/jython

[<le...@us...>]

Revision: 6937
          http://jython.svn.sourceforge.net/jython/?rev=6937&view=rev
Author:   leosoto
Date:     2009-11-04 01:11:20 +0000 (Wed, 04 Nov 2009)

Log Message:
-----------
Fix for issue 1499: PostgreSQL data handler now return Decimals for NUMERIC columns

Modified Paths:
--------------
    trunk/jython/NEWS
    trunk/jython/src/com/ziclix/python/sql/handler/PostgresqlDataHandler.java

Modified: trunk/jython/NEWS
===================================================================
--- trunk/jython/NEWS	2009-10-31 01:48:29 UTC (rev 6936)
+++ trunk/jython/NEWS	2009-11-04 01:11:20 UTC (rev 6937)
@@ -8,6 +8,7 @@
     - [ 1493 ] tarfile.extractall() throws "AttributeError: 'module' object has no attribute 'chown'" when called by root
     - [ 1470 ] os.mkdir Errno difference from cpython
     - [ 1496 ] fix os.listdir errno for non-existing dirs
+    - [ 1499 ] PostgreSQL datahandler should return Decimals instead of floats for NUMERIC/DECIMAL columns
     - Fix runtime issues during exitfuncs triggered via SystemRestart (such as
       during Django or Pylons development mode reloading)
 

Modified: trunk/jython/src/com/ziclix/python/sql/handler/PostgresqlDataHandler.java
===================================================================
--- trunk/jython/src/com/ziclix/python/sql/handler/PostgresqlDataHandler.java	2009-10-31 01:48:29 UTC (rev 6936)
+++ trunk/jython/src/com/ziclix/python/sql/handler/PostgresqlDataHandler.java	2009-11-04 01:11:20 UTC (rev 6937)
@@ -61,7 +61,7 @@
       case Types.DECIMAL:
 
         BigDecimal bd = set.getBigDecimal(col);
-        obj = (bd == null) ? Py.None : Py.newFloat(bd.doubleValue());
+        obj = (bd == null) ? Py.None : Py.newDecimal(bd.toString());
         break;
 
       case Types.OTHER:
@@ -78,7 +78,6 @@
       default :
         obj = super.getPyObject(set, col, type);
     }
-
     return (set.wasNull() || (obj == null)) ? Py.None : obj;
   }
 
@@ -108,7 +107,7 @@
         } else {
           varchar = (String) object.__tojava__(String.class);
         }
-        
+
         stmt.setObject(index, varchar, type);
         break;
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.