From d03f68526a9fc6f6c26257d58166480397d285c8 Mon Sep 17 00:00:00 2001 From: Ouyang Leyan Date: Fri, 8 Jan 2021 22:16:41 +0100 Subject: 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 (cherry picked from commit 318d384c596c1593667d6c936f55bf6ebd0c5db6) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/110354 Reviewed-by: Xisco Fauli --- sc/source/core/data/column.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'sc') diff --git a/sc/source/core/data/column.cxx b/sc/source/core/data/column.cxx index fe968e85a23d..791352304533 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()); -- cgit