commit a7997a44d20a78bc7aecbd0518528522cd041e16
author Ouyang Leyan <ouyang.leyan@hotmail.com> Fri Jan 08 22:16:41 2021 +0100
committer Xisco Fauli <xiscofauli@libreoffice.org> Wed Feb 03 20:34:27 2021 +0100
tree b2b55f8469933bf90cec5537ed289bced60c695a
parent 5da235b0f2371b97cc719dc0e357834b2fb52ab9

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>
(cherry picked from commit 318d384c596c1593667d6c936f55bf6ebd0c5db6)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/110355
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>

sc/source/core/data/column.cxx

diff --git a/sc/source/core/data/column.cxx b/sc/source/core/data/column.cxx
index 61011da..b20fbdb 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());