diff options
| author | Eike Rathke <erack@redhat.com> | 2014-02-04 13:17:31 +0100 |
|---|---|---|
| committer | Michael Meeks <michael.meeks@collabora.com> | 2014-02-04 12:35:48 +0000 |
| commit | 9cb9d32c7a64bf5789eabe47d901237d78b44a4d (patch) | |
| tree | a6d5721564030dd4fc2b565383b28470850cbff8 | |
| parent | 3ab6b03413803c9512a7dd02d42542372f8e573e (diff) | |
resolved fdo#74493 do not trust aCell.maBaseCell
Regression introduced with 70c61a549184cf8611f7890ac31b0ba4c460cdec on
4-1 and 68a753b6a36560eb83ba6ebe4fe9897a8759d265 on 4-1-5
Apparently aCell.maBaseCell does not always follow the current cell
position, so do not trust it. Always explicitly obtain the correct cell.
Change-Id: Ib88de44a326be75c13d37538b8ae56d265b60ee5
Reviewed-on: https://gerrit.libreoffice.org/7844
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>
| -rw-r--r-- | sc/source/filter/xml/xmlexprt.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sc/source/filter/xml/xmlexprt.cxx b/sc/source/filter/xml/xmlexprt.cxx index fe0a33bc8dd5..09dc864ec3fa 100644 --- a/sc/source/filter/xml/xmlexprt.cxx +++ b/sc/source/filter/xml/xmlexprt.cxx @@ -2883,8 +2883,7 @@ void ScXMLExport::WriteCell(ScMyCell& aCell, sal_Int32 nEqualCellCount) case table::CellContentType_TEXT : { OUString sFormula(lcl_GetRawString(pDoc, aCellPos)); - if (aCell.maBaseCell.isEmpty()) - aCell.maBaseCell.assign( *pDoc, aCellPos); + aCell.maBaseCell.assign( *pDoc, aCellPos); OUString sCellString = aCell.maBaseCell.getString(pDoc); GetNumberFormatAttributesExportHelper()->SetNumberFormatAttributes( sCellString, sFormula, true, true); |