summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOuyang Leyan <ouyang.leyan@hotmail.com>2021-01-08 22:16:41 +0100
committerXisco Fauli <xiscofauli@libreoffice.org>2021-02-03 20:34:27 +0100
commita7997a44d20a78bc7aecbd0518528522cd041e16 (patch)
treeb2b55f8469933bf90cec5537ed289bced60c695a
parent5da235b0f2371b97cc719dc0e357834b2fb52ab9 (diff)
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>
-rw-r--r--sc/source/core/data/column.cxx2
1 files changed, 1 insertions, 1 deletions
diff --git a/sc/source/core/data/column.cxx b/sc/source/core/data/column.cxx
index 61011da9aeae..b20fbdb16f7b 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());