sc/source/core/data/column.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit d03f68526a9fc6f6c26257d58166480397d285c8
Author:     Ouyang Leyan <ouyang.le...@hotmail.com>
AuthorDate: Fri Jan 8 22:16:41 2021 +0100
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Wed Feb 3 20:34:12 2021 +0100

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

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());
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to