[Libreoffice-commits] core.git: sc/source

[Julien Nabet (via logerrit)]

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

New commits:
commit bb3ea0ae51bb943f7bb3dca80eee153f9c55c03e
Author:     Julien Nabet <serval2412 at yahoo.fr>
AuthorDate: Thu Apr 16 12:47:46 2020 +0200
Commit:     Eike Rathke <erack at redhat.com>
CommitDate: Fri Apr 17 15:45:52 2020 +0200

    tdf#130583: fix Switch in Calc
    
    Don't set bFinished to true too soon if there are still some params to take into account
    
    Change-Id: Ie7bdd6f316c08342d379d55bf3853774dd738e7d
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92345
    Tested-by: Jenkins
    Reviewed-by: Eike Rathke <erack at redhat.com>

diff --git a/sc/source/core/tool/interpr8.cxx b/sc/source/core/tool/interpr8.cxx
index 1849ab13e6c9..ce978ea094a6 100644
--- a/sc/source/core/tool/interpr8.cxx
+++ b/sc/source/core/tool/interpr8.cxx
@@ -1974,7 +1974,7 @@ void ScInterpreter::ScSwitch_MS()
         else
             aStr = GetString();
         nParamCount--;
-        if ( nGlobalError != FormulaError::NONE || (( isValue && rtl::math::approxEqual( fRefVal, fVal ) ) ||
+        if ( (nGlobalError != FormulaError::NONE && nParamCount < 2) || (( isValue && rtl::math::approxEqual( fRefVal, fVal ) ) ||
              ( !isValue && aRefStr.getDataIgnoreCase() == aStr.getDataIgnoreCase() )) )
         {
             // TRUE