To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=103365
                 Issue #|103365
                 Summary|value field of BOOLERR record should be just one byte
               Component|Spreadsheet
                 Version|OOo 3.0.1
                Platform|All
                     URL|https://issues.apache.org/bugzilla/show_bug.cgi?id=474
                        |79
              OS/Version|All
                  Status|UNCONFIRMED
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|DEFECT
                Priority|P5
            Subcomponent|save-export
             Assigned to|spreadsheet
             Reported by|josh_micich





------- Additional comments from [email protected] Mon Jul  6 20:44:10 
+0000 2009 -------
Currently Cacl writes two bytes for the value field when the record type is 
EXC_BOOLERR_BOOL.  Correct behaviour occurs when the record type is 
EXC_BOOLERR_ERROR - the value is written as a single byte (since mnErrCode is 
sal_uInt8)

Excel seems to handle this OK but POI (until the recent patch) does not.

This should be a one line fix to xetable.cxx:
http://svn.services.openoffice.org/ooo/trunk/sc/source/filter/excel/xetable.cxx

Index: xetable.cxx
===================================================================
--- xetable.cxx (revision 268869)
+++ xetable.cxx (working copy)
@@ -648,7 +648,7 @@

 void XclExpBooleanCell::WriteContents( XclExpStream& rStrm )
 {
-    rStrm << sal_uInt16( mbValue ? 1 : 0 ) << EXC_BOOLERR_BOOL;
+    rStrm << sal_uInt8( mbValue ? 1 : 0 ) << EXC_BOOLERR_BOOL;
 }

 // ----------------------------------------------------------------------------

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to