commit 318d384c596c1593667d6c936f55bf6ebd0c5db6
author Ouyang Leyan <ouyang.leyan@hotmail.com> Fri Jan 08 22:16:41 2021 +0100
committer Eike Rathke <erack@redhat.com> Tue Feb 02 01:05:03 2021 +0100
tree 12b5a0a0fa27cad159ca243259dc20c00ada01ef
parent 585b0a39d7170ef30d0a1d03601e008613df9807

tdf#120348: Avoid data loss when copying chart with error in formulas

During a chart copy in Calc, when skipping cells with ERR:502 in values,
the rest of the column was skipped as well, including valid values.

Change-Id: I2578dfdefe9d006de572f8faa3a8cc32e9f55010
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109025
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>

sc/source/core/data/column.cxx

diff --git a/sc/source/core/data/column.cxx b/sc/source/core/data/column.cxx
index fe968e8..7913523 100644
--- a/sc/source/core/data/column.cxx
+++ b/sc/source/core/data/column.cxx
@@ -1148,7 +1148,7 @@ void ScColumn::CopyStaticToDocument(

                    if (rFC.GetErrCode() != FormulaError::NONE)
                        // Skip cells with error.
                        break;
                        continue;

                    if (rFC.IsValue())
                        aDestPos.miCellPos = rDestCol.maCells.set(aDestPos.miCellPos, nRow, rFC.GetValue());