[Squirrel-sql-commits] SF.net SVN: squirrel-sql:[6183] trunk/sql12

[<wi...@us...>]

Revision: 6183
          http://squirrel-sql.svn.sourceforge.net/squirrel-sql/?rev=6183&view=rev
Author:   wis775
Date:     2011-03-11 17:20:23 +0000 (Fri, 11 Mar 2011)

Log Message:
-----------
BugFix for 3175871: On export to Excel, some numeric values are not exported correctly. Numeric values was converted to float types. This truncated some values
 e.g   44764301 >> 4764300
In the fact that jxl.write.Number needs an Double as constructor argument, we  use now doubleValue()  instead of floatValue() for numeric types.

Modified Paths:
--------------
    trunk/sql12/doc/src/main/resources/changes.txt
    trunk/sql12/fw/src/main/java/net/sourceforge/squirrel_sql/fw/gui/action/TableExportCsvCommand.java

Modified: trunk/sql12/doc/src/main/resources/changes.txt
===================================================================
--- trunk/sql12/doc/src/main/resources/changes.txt	2011-03-10 23:01:35 UTC (rev 6182)
+++ trunk/sql12/doc/src/main/resources/changes.txt	2011-03-11 17:20:23 UTC (rev 6183)
@@ -25,6 +25,8 @@
 
 Bug-fixes:
 
+3175871: On export to Excel, some numeric values are not exported correctly.  
+
 SQL Scripts Plugin: Increased multi table script configuration dialog's size.
 
 3171762: Wrong syntax for DROP KEY (When qualify table name preference is selected this produced two schema

Modified: trunk/sql12/fw/src/main/java/net/sourceforge/squirrel_sql/fw/gui/action/TableExportCsvCommand.java
===================================================================
--- trunk/sql12/fw/src/main/java/net/sourceforge/squirrel_sql/fw/gui/action/TableExportCsvCommand.java	2011-03-10 23:01:35 UTC (rev 6182)
+++ trunk/sql12/fw/src/main/java/net/sourceforge/squirrel_sql/fw/gui/action/TableExportCsvCommand.java	2011-03-11 17:20:23 UTC (rev 6183)
@@ -458,26 +458,26 @@
             ret = new jxl.write.Boolean(colIdx, curRow, (Boolean) cellObj);
             break;
          case Types.INTEGER:
-            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).floatValue());
+            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).doubleValue());
             break;
          case Types.SMALLINT:
          case Types.TINYINT:
-            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).floatValue() ) ;
+            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).doubleValue() ) ;
             break;
          case Types.DECIMAL:
-            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).floatValue());
+            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).doubleValue());
             break;
          case Types.NUMERIC:
             ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).doubleValue());
             break;
          case Types.FLOAT:
-            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).floatValue());
+            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).doubleValue());
             break;
          case Types.DOUBLE:
-            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).floatValue());
+            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).doubleValue());
             break;
          case Types.REAL:
-            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).floatValue());
+            ret = new jxl.write.Number(colIdx, curRow, ((Number) cellObj).doubleValue());
             break;
          case Types.BIGINT:
             ret = new jxl.write.Number(colIdx, curRow, Long.parseLong(cellObj.toString()));


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