diff options
author | Eike Rathke <erack@redhat.com> | 2014-12-16 01:37:31 +0100 |
---|---|---|
committer | Andras Timar <andras.timar@collabora.com> | 2015-01-08 14:46:13 +0100 |
commit | 992673fb89b79a1b2abd28173b013ccbc3987364 (patch) | |
tree | b4d25e53ef927aa4dba6a89685b8d6a9c3e6e255 | |
parent | ba5d09072c4e9c103f6af3c44e689bc6c93e0477 (diff) |
fdo#79249 call formula compiler with error string
... instead of attempting to stringize a NaN coded error value.
Regression introduced with 30a20743ae17e6e02183a65603d38968253b3ffb
(cherry picked from commit 994607b55104b9ae4554554c13b001b8d5d513b6)
construct grouped ScFormulaCell with bDirty=true, fdo#79249 related
Noticed when loading
https://bugs.freedesktop.org/attachment.cgi?id=99844
with fdo#79249 fix where oox::xls::applyCellFormulas() groups the
consecutive =#N/A formulas. Only A1 result was displayed, other cells
were displayed empty.
(cherry picked from commit a1dc5e97da273bf35d58d54e625149022569a993)
correct error string, #N/A instead of #NA
(cherry picked from commit 758755e31b3d9e1ed2eab522d4794282178346ac)
implement an actually working setErrorCell() from BIFF error codes
(cherry picked from commit ca9a81b2ca858b82e863e1e6f917928916fea79e)
3a541f74d3d25e1515a1c6d47f02ec6a8e817c93
15019072b6e812b9ffe29d3ee6afacd9ab526948
67b8fc324779875ba14e2d69204c40fe27cc180e
Reviewed-on: https://gerrit.libreoffice.org/13492
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Conflicts:
sc/source/filter/oox/sheetdatabuffer.cxx
Change-Id: Ia7a8ca39938820ac75db169404446fa696c6ee1b
-rw-r--r-- | sc/source/filter/inc/unitconverter.hxx | 3 | ||||
-rw-r--r-- | sc/source/filter/oox/sheetdatabuffer.cxx | 11 | ||||
-rw-r--r-- | sc/source/filter/oox/unitconverter.cxx | 14 |
3 files changed, 18 insertions, 10 deletions
diff --git a/sc/source/filter/inc/unitconverter.hxx b/sc/source/filter/inc/unitconverter.hxx index 88921d1ceb5b..dbd642403ba1 100644 --- a/sc/source/filter/inc/unitconverter.hxx +++ b/sc/source/filter/inc/unitconverter.hxx @@ -88,6 +88,9 @@ public: /** Returns a BIFF error code from the passed error string. */ sal_uInt8 calcBiffErrorCode( const OUString& rErrorCode ) const; + /** Returns an error string from the passed BIFF error code. */ + const OUString& calcErrorString( sal_uInt8 nErrorCode ) const; + private: /** Adds an error code to the internal maps. */ void addErrorCode( sal_uInt8 nErrorCode, const OUString& rErrorCode ); diff --git a/sc/source/filter/oox/sheetdatabuffer.cxx b/sc/source/filter/oox/sheetdatabuffer.cxx index 98cf68480316..11c9c7f960a2 100644 --- a/sc/source/filter/oox/sheetdatabuffer.cxx +++ b/sc/source/filter/oox/sheetdatabuffer.cxx @@ -216,16 +216,7 @@ void SheetDataBuffer::setErrorCell( const CellModel& rModel, const OUString& rEr void SheetDataBuffer::setErrorCell( const CellModel& rModel, sal_uInt8 nErrorCode ) { - assert(!"stringizing any NaN will only give 'nan'"); - /* FIXME: map nErrorCode to error string and call setErrorCell() above. */ - - OUStringBuffer aBuf; - aBuf.append('{'); - aBuf.append(BiffHelper::calcDoubleFromError(nErrorCode)); - aBuf.append('}'); - - getFormulaBuffer().setCellFormula(rModel.maCellAddr, aBuf.makeStringAndClear()); - setCellFormat( rModel ); + setErrorCell( rModel, getUnitConverter().calcErrorString( nErrorCode)); } void SheetDataBuffer::setDateCell( const CellModel& rModel, const OUString& rDateString ) diff --git a/sc/source/filter/oox/unitconverter.cxx b/sc/source/filter/oox/unitconverter.cxx index 108b248caa94..2eff67678064 100644 --- a/sc/source/filter/oox/unitconverter.cxx +++ b/sc/source/filter/oox/unitconverter.cxx @@ -226,6 +226,20 @@ sal_uInt8 UnitConverter::calcBiffErrorCode( const OUString& rErrorCode ) const return (aIt == maOoxErrCodes.end()) ? BIFF_ERR_NA : aIt->second; } +const OUString& UnitConverter::calcErrorString( sal_uInt8 nErrorCode ) const +{ + OoxErrorCodeMap::const_iterator iFail( maOoxErrCodes.end()); + for (OoxErrorCodeMap::const_iterator aIt( maOoxErrCodes.begin()); aIt != maOoxErrCodes.end(); ++aIt) + { + if (aIt->second == nErrorCode) + return aIt->first; + if (aIt->second == BIFF_ERR_NA) + iFail = aIt; + } + assert(iFail != maOoxErrCodes.end()); // BIFF_ERR_NA really should be in the map.. + return iFail->first; +} + void UnitConverter::addErrorCode( sal_uInt8 nErrorCode, const OUString& rErrorCode ) { maOoxErrCodes[ rErrorCode ] = nErrorCode; |